On Wed, Nov 23, 2016 at 01:49:53PM -0500, Mike Gilbert wrote: > On Wed, Nov 23, 2016 at 1:43 PM, Mike Gilbert <floppymaster@xxxxxxxxx> wrote: > > On Wed, Nov 23, 2016 at 4:45 AM, David Sterba <dsterba@xxxxxxx> wrote: > >> On Tue, Nov 22, 2016 at 01:49:15PM -0500, Mike Gilbert wrote: > >>> Hi, > >>> > >>> I received a bug report about a build failure in a package (snapper) > >>> that utilizes libbtrfs. > >>> > >>> https://bugs.gentoo.org/show_bug.cgi?id=600078 > >>> > >>> To summarize the issue, the package has a configure test that executes > >>> the following: > >>> > >>> configure:16618: x86_64-pc-linux-gnu-gcc -o conftest -O2 -pipe > >>> -march=amdfam10 -frecord-gcc-switches -std=c99 -Wall -Wextra > >>> -Wformat=2 -Wmissing-prototypes -Wno-unused-parameter > >>> -DCONFDIR='"/etc/conf.d"' -Wl,-O1 -Wl,--as-needed conftest.c -lbtrfs > >>> >&5 > >>> /usr/lib/gcc/x86_64-pc-linux-gnu/5.4.0/../../../../lib64/libbtrfs.so: > >>> undefined reference to `lookup_path_rootid' > >>> collect2: error: ld returned 1 exit status > >>> configure:16618: $? = 1 > >>> > >>> It seems that the "lookup_path_rootid" symbol is not included in the library. > >>> > >>> Could someone look into this? > >> > >> Thanks for the report, bug reproduced here. My referential snapper build > >> does not fail so I did not know about that. I've now extended the > >> library test to cover the missing symbol. As the build does not fail > >> it's needed to actually run the built binary. > > > > That's very strange; I definitely get a build-time failure here. Maybe > > there's some difference in your toolchain. > > > > On current master: > > > > floppym@naomi btrfs-progs % make V=1 library-test > > [LD] library-test > > gcc -g -O1 -Wall -D_FORTIFY_SOURCE=2 -include config.h > > -DBTRFS_FLAT_INCLUDES -D_XOPEN_SOURCE=700 -fno-strict-aliasing -fPIC > > -I/home/floppym/src/btrfs-progs > > -I/home/floppym/src/btrfs-progs/kernel-lib -o library-test > > library-test.o -rdynamic -lbtrfs > > /usr/lib/gcc/x86_64-pc-linux-gnu/5.4.0/../../../../lib64/libbtrfs.so: > > undefined reference to `lookup_path_rootid' > > collect2: error: ld returned 1 exit status > > make: *** [Makefile:433: library-test] Error 1 > > Oh! Based on that command line, library-test is getting linked against > /usr/lib64/libbtrfs.so instead of the freshly built copy in $PWD. Oh, right, that's what we get when we don't use automake :) > You probably need to add -L. or -L$(TOPDIR) to that command in the Makefile. That was it, thank you! -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
