Ceph master - build broken unless --enable-debug specified

Ceph master - build broken unless --enable-debug specified

[Mark Kirkwood]

Hi,

Building without --enable-debug produces:

ceph_fuse.cc: In member function ‘virtual void* main(int, const char**, 
const char**)::RemountTest::entry()’:
ceph_fuse.cc:146:15: warning: ignoring return value of ‘int system(const 
char*)’, declared with attribute warn_unused_result [-Wunused-result]
     system(buf);
                ^
   CXX      ceph_osd.o
   CXX      ceph_mds.o
make[3]: *** No rule to make target '../src/gmock/lib/libgmock_main.la', 
needed by 'unittest_librbd'.  Stop.
make[3]: *** Waiting for unfinished jobs....
   CXX      test/erasure-code/ceph_erasure_code_non_regression.o
make[3]: Leaving directory '/home/markir/develop/c/ceph/src'
Makefile:20716: recipe for target 'all-recursive' failed
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory '/home/markir/develop/c/ceph/src'
Makefile:8977: recipe for target 'all' failed
make[1]: *** [all] Error 2
make[1]: Leaving directory '/home/markir/develop/c/ceph/src'
Makefile:467: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1


Adding in --enable-debug gives a successful build.

This is on Ubuntu 14.10 64 bit, and the build procedure is:

$ git pull
$ git submodule update --init
$ ./autogen.sh
$ ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var \
               [--with-debug \ ]
               --with-nss \
               --with-radosgw \
               --with-librocksdb-static=check \

$ make [ -j4 ]


Cheers

Mark
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Ceph master - build broken unless --enable-debug specified

[Gregory Farnum]

On Sat, Apr 11, 2015 at 8:42 PM, Mark Kirkwood
<mark.kirkwood@catalyst.net.nz> wrote:
> Hi,
>
> Building without --enable-debug produces:
>
> ceph_fuse.cc: In member function ‘virtual void* main(int, const char**,
> const char**)::RemountTest::entry()’:
> ceph_fuse.cc:146:15: warning: ignoring return value of ‘int system(const
> char*)’, declared with attribute warn_unused_result [-Wunused-result]
>     system(buf);
>                ^
>   CXX      ceph_osd.o
>   CXX      ceph_mds.o
> make[3]: *** No rule to make target '../src/gmock/lib/libgmock_main.la',
> needed by 'unittest_librbd'.  Stop.
> make[3]: *** Waiting for unfinished jobs....
>   CXX      test/erasure-code/ceph_erasure_code_non_regression.o
> make[3]: Leaving directory '/home/markir/develop/c/ceph/src'
> Makefile:20716: recipe for target 'all-recursive' failed
> make[2]: *** [all-recursive] Error 1
> make[2]: Leaving directory '/home/markir/develop/c/ceph/src'
> Makefile:8977: recipe for target 'all' failed
> make[1]: *** [all] Error 2
> make[1]: Leaving directory '/home/markir/develop/c/ceph/src'
> Makefile:467: recipe for target 'all-recursive' failed
> make: *** [all-recursive] Error 1
>
>
> Adding in --enable-debug gives a successful build.
>
> This is on Ubuntu 14.10 64 bit, and the build procedure is:
>
> $ git pull
> $ git submodule update --init
> $ ./autogen.sh
> $ ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var \
>               [--with-debug \ ]
>               --with-nss \
>               --with-radosgw \
>               --with-librocksdb-static=check \
>
> $ make [ -j4 ]

Yep, looks like the unittest_librbd binary is in the noinst_PROGRAMS
target (whatever that is) rather than the check_PROGRAMS target.
Changing that seems to work — I pushed a branch wip-nodebug-build
fixing it, but if you have your own fix a PR is welcome. If not I'll
make a PR in the next couple days.
-Greg
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: Ceph master - build broken unless --enable-debug specified

[Mark Kirkwood]

On 17/04/15 12:27, Gregory Farnum wrote:
> On Sat, Apr 11, 2015 at 8:42 PM, Mark Kirkwood
> <mark.kirkwood@catalyst.net.nz> wrote:
>> Hi,
>>
>> Building without --enable-debug produces:
>>
>> ceph_fuse.cc: In member function ‘virtual void* main(int, const char**,
>> const char**)::RemountTest::entry()’:
>> ceph_fuse.cc:146:15: warning: ignoring return value of ‘int system(const
>> char*)’, declared with attribute warn_unused_result [-Wunused-result]
>>      system(buf);
>>                 ^
>>    CXX      ceph_osd.o
>>    CXX      ceph_mds.o
>> make[3]: *** No rule to make target '../src/gmock/lib/libgmock_main.la',
>> needed by 'unittest_librbd'.  Stop.
>> make[3]: *** Waiting for unfinished jobs....
>>    CXX      test/erasure-code/ceph_erasure_code_non_regression.o
>> make[3]: Leaving directory '/home/markir/develop/c/ceph/src'
>> Makefile:20716: recipe for target 'all-recursive' failed
>> make[2]: *** [all-recursive] Error 1
>> make[2]: Leaving directory '/home/markir/develop/c/ceph/src'
>> Makefile:8977: recipe for target 'all' failed
>> make[1]: *** [all] Error 2
>> make[1]: Leaving directory '/home/markir/develop/c/ceph/src'
>> Makefile:467: recipe for target 'all-recursive' failed
>> make: *** [all-recursive] Error 1
>>
>>
>> Adding in --enable-debug gives a successful build.
>>
>> This is on Ubuntu 14.10 64 bit, and the build procedure is:
>>
>> $ git pull
>> $ git submodule update --init
>> $ ./autogen.sh
>> $ ./configure --prefix=/usr --sysconfdir=/etc --localstatedir=/var \
>>                [--with-debug \ ]
>>                --with-nss \
>>                --with-radosgw \
>>                --with-librocksdb-static=check \
>>
>> $ make [ -j4 ]
>
> Yep, looks like the unittest_librbd binary is in the noinst_PROGRAMS
> target (whatever that is) rather than the check_PROGRAMS target.
> Changing that seems to work — I pushed a branch wip-nodebug-build
> fixing it, but if you have your own fix a PR is welcome. If not I'll
> make a PR in the next couple days.

I had not looked very closely at what the exact problem was - your 
analysis looks good to me, I'll leave you to file a PR :-)

Cheers

Mark

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html