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

[Mark Kirkwood <mark.kirkwood@catalyst.net.nz>]

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