On 17/04/15 12:27, Gregory Farnum wrote:
On Sat, Apr 11, 2015 at 8:42 PM, Mark Kirkwood <mark.kirkwood@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
