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 <guojian...@cn.ibm.com> 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 <linshuai2...@gmail.com>
> 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 <linshuai2...@gmail.com>
> 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
>

Reply via email to