Review Request 37791: Removed provisioner_backend_tests.cpp from build in OSX
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/ --- Review request for mesos, Adam B and Till Toenshoff. Bugs: MESOS-3316 https://issues.apache.org/jira/browse/MESOS-3316 Repository: mesos Description --- The commit 0b6dfac introduced the file `provisioner_backend_test.cpp`. Such file tests linux only features but was included in the OSX build to which resulted in compilation errors. This patch addresses that by moving the file to the `OS_LINUX` section in the `src/Makefile.am` file. Since the file is only built in linux now, it removes the `ifdef` guards in the file which were useless to begin with. Diffs - src/Makefile.am 7b620ff66856b3f0adac121b3297d55ed71a3d99 src/tests/containerizer/provisioner_backend_tests.cpp 096bd4d4bbd602913796c5e542e033bdd9e04964 Diff: https://reviews.apache.org/r/37791/diff/ Testing --- make check Thanks, Alexander Rojas
Re: Review Request 37791: Removed provisioner_backend_tests.cpp from build in OSX
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/#review96515 --- Seems another fix got committed already, want to discard this one? - Till Toenshoff On Aug. 26, 2015, 8:06 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/ --- (Updated Aug. 26, 2015, 8:06 a.m.) Review request for mesos, Adam B and Till Toenshoff. Bugs: MESOS-3316 https://issues.apache.org/jira/browse/MESOS-3316 Repository: mesos Description --- The commit 0b6dfac introduced the file `provisioner_backend_test.cpp`. Such file tests linux only features but was included in the OSX build to which resulted in compilation errors. This patch addresses that by moving the file to the `OS_LINUX` section in the `src/Makefile.am` file. Since the file is only built in linux now, it removes the `ifdef` guards in the file which were useless to begin with. Diffs - src/Makefile.am 7b620ff66856b3f0adac121b3297d55ed71a3d99 src/tests/containerizer/provisioner_backend_tests.cpp 096bd4d4bbd602913796c5e542e033bdd9e04964 Diff: https://reviews.apache.org/r/37791/diff/ Testing --- # Both linux and OSX. make check Thanks, Alexander Rojas
Re: Review Request 37791: Removed provisioner_backend_tests.cpp from build in OSX
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/#review96512 --- Ship it! LGTM. I could fix the alphabetization when committing. But maybe Yan and Jie want to take a look? src/Makefile.am (lines 1682 - 1685) https://reviews.apache.org/r/37791/#comment151940 Please alphabetize - Adam B On Aug. 26, 2015, 1:06 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/ --- (Updated Aug. 26, 2015, 1:06 a.m.) Review request for mesos, Adam B and Till Toenshoff. Bugs: MESOS-3316 https://issues.apache.org/jira/browse/MESOS-3316 Repository: mesos Description --- The commit 0b6dfac introduced the file `provisioner_backend_test.cpp`. Such file tests linux only features but was included in the OSX build to which resulted in compilation errors. This patch addresses that by moving the file to the `OS_LINUX` section in the `src/Makefile.am` file. Since the file is only built in linux now, it removes the `ifdef` guards in the file which were useless to begin with. Diffs - src/Makefile.am 7b620ff66856b3f0adac121b3297d55ed71a3d99 src/tests/containerizer/provisioner_backend_tests.cpp 096bd4d4bbd602913796c5e542e033bdd9e04964 Diff: https://reviews.apache.org/r/37791/diff/ Testing --- # Both linux and OSX. make check Thanks, Alexander Rojas
Re: Review Request 37791: Removed provisioner_backend_tests.cpp from build in OSX
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37791/ --- (Updated Aug. 26, 2015, 10:06 a.m.) Review request for mesos, Adam B and Till Toenshoff. Bugs: MESOS-3316 https://issues.apache.org/jira/browse/MESOS-3316 Repository: mesos Description --- The commit 0b6dfac introduced the file `provisioner_backend_test.cpp`. Such file tests linux only features but was included in the OSX build to which resulted in compilation errors. This patch addresses that by moving the file to the `OS_LINUX` section in the `src/Makefile.am` file. Since the file is only built in linux now, it removes the `ifdef` guards in the file which were useless to begin with. Diffs - src/Makefile.am 7b620ff66856b3f0adac121b3297d55ed71a3d99 src/tests/containerizer/provisioner_backend_tests.cpp 096bd4d4bbd602913796c5e542e033bdd9e04964 Diff: https://reviews.apache.org/r/37791/diff/ Testing (updated) --- # Both linux and windows. make check Thanks, Alexander Rojas