+1 2016-06-08 9:41 GMT+08:00 Shuai Lin <[email protected]>:
> Oops, I forgot to put the aufs tests in the block guarded by the __liunx__ > macro, sorry for that. > > I've posted a fix here: https://reviews.apache.org/r/48388/ > > @jieyu please take a look. > > On Wed, Jun 8, 2016 at 9:13 AM, Jay JN Guo <[email protected]> wrote: > > > > > Hi, > > > > I'm getting this error when compiling latest code on OSX: > > > > In file included > > from ../../src/tests/containerizer/provisioner_backend_tests.cpp:17: > > In file included > > from ../../3rdparty/libprocess/include/process/gtest.hpp:16: > > In file included > > from ../3rdparty/gmock-1.7.0/gtest/include/gtest/gtest.h:58: > > > > > ../3rdparty/gmock-1.7.0/gtest/include/gtest/internal/gtest-internal.h:454:39: > > error: cannot initialize return object of type 'testing::Test *' with an > > rvalue of type > > 'mesos::internal::tests::AufsBackendTest_ROOT_AUFS_AufsBackend_Test > > *' > > virtual Test* CreateTest() { return new TestClass; } > > ^~~ > > ../../src/tests/containerizer/provisioner_backend_tests.cpp:208:1: note: > in > > instantiation of member function > > > > > > > 'testing::internal::TestFactoryImpl<mesos::internal::tests::AufsBackendTest_ROOT_AUFS_AufsBackend_Test>::CreateTest' > > requested here > > TEST_F(AufsBackendTest, ROOT_AUFS_AufsBackend) > > ^ > > ../3rdparty/gmock-1.7.0/gtest/include/gtest/gtest.h:2272:3: note: > expanded > > from macro 'TEST_F' > > GTEST_TEST_(test_fixture, test_name, test_fixture, \ > > ^ > > > > > ../3rdparty/gmock-1.7.0/gtest/include/gtest/internal/gtest-internal.h:1154:13: > > note: expanded from macro 'GTEST_TEST_' > > new ::testing::internal::TestFactoryImpl<\ > > ^ > > 5 errors generated. > > make[1]: *** > [tests/containerizer/mesos_tests-provisioner_backend_tests.o] > > Error 1 > > make[1]: *** Waiting for unfinished jobs.... > > > > > > > > I guess it has something to do with these commits? > > > > commit e5358ed1c132923d5fa357d1e337e037d1f29c8a > > Author: Shuai Lin <[email protected]> > > Date: Mon Jun 6 18:05:15 2016 -0700 > > > > Added aufs provisioning backend. > > > > Review: https://reviews.apache.org/r/47396/ > > > > commit ca09304040dee306b6ae2b081297e694f2f55599 > > Author: Shuai Lin <[email protected]> > > Date: Mon Jun 6 18:05:11 2016 -0700 > > > > Added fs::aufs::supported() function. > > > > Review: https://reviews.apache.org/r/48300/ > > > > > > AFAIK, aufs is not available on OSX and should we have _LINUX_ > preprocessor > > for this? > > > > cheers, > > /J > > > -- Deshi Xiao Twitter: xds2000 E-mail: xiaods(AT)gmail.com
