* [PATCH] spaceman: Add missing link to libuuid @ 2017-09-07 2:19 Khem Raj 2017-09-07 2:40 ` Dave Chinner 0 siblings, 1 reply; 11+ messages in thread From: Khem Raj @ 2017-09-07 2:19 UTC (permalink / raw) To: linux-xfs; +Cc: Khem Raj When uuid is enabled then we see linking failures with gold linker e.g. libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) Signed-off-by: Khem Raj <raj.khem@gmail.com> --- spaceman/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spaceman/Makefile b/spaceman/Makefile index 95ec3c0..060e0af 100644 --- a/spaceman/Makefile +++ b/spaceman/Makefile @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman HFILES = init.h space.h CFILES = init.c file.c prealloc.c trim.c -LLDLIBS = $(LIBXCMD) +LLDLIBS = $(LIBXCMD) $(LIBUUID) LTDEPENDENCIES = $(LIBXCMD) LLDFLAGS = -static -- 2.14.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-07 2:19 [PATCH] spaceman: Add missing link to libuuid Khem Raj @ 2017-09-07 2:40 ` Dave Chinner 2017-09-07 4:11 ` Khem Raj 2017-09-08 18:24 ` Khem Raj 0 siblings, 2 replies; 11+ messages in thread From: Dave Chinner @ 2017-09-07 2:40 UTC (permalink / raw) To: Khem Raj; +Cc: linux-xfs On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: > When uuid is enabled then we see linking failures with gold linker e.g. > > libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' > libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' > clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) That's libxfs, not xfs_spaceman.... > > Signed-off-by: Khem Raj <raj.khem@gmail.com> > --- > spaceman/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/spaceman/Makefile b/spaceman/Makefile > index 95ec3c0..060e0af 100644 > --- a/spaceman/Makefile > +++ b/spaceman/Makefile > @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman > HFILES = init.h space.h > CFILES = init.c file.c prealloc.c trim.c > > -LLDLIBS = $(LIBXCMD) > +LLDLIBS = $(LIBXCMD) $(LIBUUID) And spaceman doesn't link against libxfs, nor does libxcmd or spaceman use uuids so I don't see why this fixes a problem with libxfs... Cheers, Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-07 2:40 ` Dave Chinner @ 2017-09-07 4:11 ` Khem Raj 2017-09-07 6:59 ` Dave Chinner 2017-09-08 18:24 ` Khem Raj 1 sibling, 1 reply; 11+ messages in thread From: Khem Raj @ 2017-09-07 4:11 UTC (permalink / raw) To: Dave Chinner; +Cc: linux-xfs On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: > On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >> When uuid is enabled then we see linking failures with gold linker e.g. >> >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) > > That's libxfs, not xfs_spaceman.... yes its libxfs but its linking spaceman, I chopped the error logs too much I guess. > >> >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> --- >> spaceman/Makefile | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/spaceman/Makefile b/spaceman/Makefile >> index 95ec3c0..060e0af 100644 >> --- a/spaceman/Makefile >> +++ b/spaceman/Makefile >> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >> HFILES = init.h space.h >> CFILES = init.c file.c prealloc.c trim.c >> >> -LLDLIBS = $(LIBXCMD) >> +LLDLIBS = $(LIBXCMD) $(LIBUUID) > > And spaceman doesn't link against libxfs, nor does libxcmd or > spaceman use uuids so I don't see why this fixes a problem with > libxfs... I was just following what other binaries were doing, otherwise I agree libuuid should appear in DT_NEEDED flags in libxfs.so so linkers can pull it in automatically. try using LDFLAGS = "-fuse-ld=gold" if you want to reproduce the problem on your end. > > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-07 4:11 ` Khem Raj @ 2017-09-07 6:59 ` Dave Chinner 0 siblings, 0 replies; 11+ messages in thread From: Dave Chinner @ 2017-09-07 6:59 UTC (permalink / raw) To: Khem Raj; +Cc: linux-xfs On Wed, Sep 06, 2017 at 09:11:11PM -0700, Khem Raj wrote: > On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: > > On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: > >> When uuid is enabled then we see linking failures with gold linker e.g. > >> > >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' > >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' > >> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) > > > > That's libxfs, not xfs_spaceman.... > > yes its libxfs but its linking spaceman, I chopped the error logs too > much I guess. xfs_spaceman does not link against libuuid: $ ldd spaceman/xfs_spaceman linux-vdso.so.1 (0x00007ffca4f68000) libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fa36f358000) /lib64/ld-linux-x86-64.so.2 (0x00007fa36f6f7000) Nor libxfs - if it linked against libxfs, it would have to pull in 4 other libraries, including libuuid: $ ldd repair/xfs_repair linux-vdso.so.1 (0x00007ffd96bb4000) libuuid.so.1 => /lib/x86_64-linux-gnu/libuuid.so.1 (0x00007f5da2d4f000) librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007f5da2b47000) libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f5da292a000) libblkid.so.1 => /lib/x86_64-linux-gnu/libblkid.so.1 (0x00007f5da26e4000) libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f5da2345000) /lib64/ld-linux-x86-64.so.2 (0x00007f5da2f54000) So whatever build error you are having has nothing to do with xfs_spaceman. Have you installed all the necessary libraries? > >> Signed-off-by: Khem Raj <raj.khem@gmail.com> > >> --- > >> spaceman/Makefile | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/spaceman/Makefile b/spaceman/Makefile > >> index 95ec3c0..060e0af 100644 > >> --- a/spaceman/Makefile > >> +++ b/spaceman/Makefile > >> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman > >> HFILES = init.h space.h > >> CFILES = init.c file.c prealloc.c trim.c > >> > >> -LLDLIBS = $(LIBXCMD) > >> +LLDLIBS = $(LIBXCMD) $(LIBUUID) > > > > And spaceman doesn't link against libxfs, nor does libxcmd or > > spaceman use uuids so I don't see why this fixes a problem with > > libxfs... > > I was just following what other binaries were doing, otherwise I agree > libuuid should appear in DT_NEEDED flags in libxfs.so so linkers can > pull it in automatically. > > try using > > LDFLAGS = "-fuse-ld=gold" if you want to reproduce the problem on your end. $ gold -V GNU gold (GNU Binutils for Debian 2.29) 1.14 $ make distclean .... $ LDFLAGS="-fuse-ld=gold" make -j8 Q= .... [LD] xfs_spaceman /bin/bash ../libtool --quiet --tag=CC --mode=link gcc -o xfs_spaceman -fuse-ld=gold -fuse-ld=gold -fuse-ld=gold -fuse-ld=gold -static init.o file.o prealloc.o trim.o freesp.o ../libxcmd/libxcmd.la ..... $ ldd spaceman/xfs_spaceman linux-vdso.so.1 (0x00007ffc943e4000) libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f2def2e2000) /lib64/ld-linux-x86-64.so.2 (0x00007f2def681000) TOT builds just fine here on debian unstable using the gold linker, and again this shows that spaceman does not link against libuuid. Cheers, Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-07 2:40 ` Dave Chinner 2017-09-07 4:11 ` Khem Raj @ 2017-09-08 18:24 ` Khem Raj 2017-09-08 19:15 ` Eric Sandeen 2017-09-08 22:36 ` Dave Chinner 1 sibling, 2 replies; 11+ messages in thread From: Khem Raj @ 2017-09-08 18:24 UTC (permalink / raw) To: Dave Chinner; +Cc: linux-xfs Hi Dave On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: > On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >> When uuid is enabled then we see linking failures with gold linker e.g. >> >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) > > That's libxfs, not xfs_spaceman.... > >> >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> --- >> spaceman/Makefile | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/spaceman/Makefile b/spaceman/Makefile >> index 95ec3c0..060e0af 100644 >> --- a/spaceman/Makefile >> +++ b/spaceman/Makefile >> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >> HFILES = init.h space.h >> CFILES = init.c file.c prealloc.c trim.c >> >> -LLDLIBS = $(LIBXCMD) >> +LLDLIBS = $(LIBXCMD) $(LIBUUID) > > And spaceman doesn't link against libxfs, nor does libxcmd or > spaceman use uuids so I don't see why this fixes a problem with > libxfs... > I am building with --disable-static, I wonder if thats not a supported configuration for xfsprogs > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-08 18:24 ` Khem Raj @ 2017-09-08 19:15 ` Eric Sandeen 2017-09-08 20:38 ` Khem Raj 2017-09-08 22:36 ` Dave Chinner 1 sibling, 1 reply; 11+ messages in thread From: Eric Sandeen @ 2017-09-08 19:15 UTC (permalink / raw) To: Khem Raj, Dave Chinner; +Cc: linux-xfs On 9/8/17 1:24 PM, Khem Raj wrote: > Hi Dave > > On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: >> On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >>> When uuid is enabled then we see linking failures with gold linker e.g. >>> >>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >>> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) >> >> That's libxfs, not xfs_spaceman.... >> >>> >>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>> --- >>> spaceman/Makefile | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/spaceman/Makefile b/spaceman/Makefile >>> index 95ec3c0..060e0af 100644 >>> --- a/spaceman/Makefile >>> +++ b/spaceman/Makefile >>> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >>> HFILES = init.h space.h >>> CFILES = init.c file.c prealloc.c trim.c >>> >>> -LLDLIBS = $(LIBXCMD) >>> +LLDLIBS = $(LIBXCMD) $(LIBUUID) >> >> And spaceman doesn't link against libxfs, nor does libxcmd or >> spaceman use uuids so I don't see why this fixes a problem with >> libxfs... >> > > I am building with --disable-static, I wonder if thats not a supported > configuration > for xfsprogs Still doesn't make spaceman link against libxfs or use uuids ;) I guess I'd suggest starting w/ x86 and a standard toolchain, make --disable-static work for you there (I see that it does need some fixups), then try things w/ the alternate toolchain and see if more work is needed. -Eric ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-08 19:15 ` Eric Sandeen @ 2017-09-08 20:38 ` Khem Raj 0 siblings, 0 replies; 11+ messages in thread From: Khem Raj @ 2017-09-08 20:38 UTC (permalink / raw) To: Eric Sandeen; +Cc: Dave Chinner, linux-xfs On Fri, Sep 8, 2017 at 12:15 PM, Eric Sandeen <sandeen@sandeen.net> wrote: > On 9/8/17 1:24 PM, Khem Raj wrote: >> Hi Dave >> >> On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: >>> On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >>>> When uuid is enabled then we see linking failures with gold linker e.g. >>>> >>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >>>> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) >>> >>> That's libxfs, not xfs_spaceman.... >>> >>>> >>>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>> --- >>>> spaceman/Makefile | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/spaceman/Makefile b/spaceman/Makefile >>>> index 95ec3c0..060e0af 100644 >>>> --- a/spaceman/Makefile >>>> +++ b/spaceman/Makefile >>>> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >>>> HFILES = init.h space.h >>>> CFILES = init.c file.c prealloc.c trim.c >>>> >>>> -LLDLIBS = $(LIBXCMD) >>>> +LLDLIBS = $(LIBXCMD) $(LIBUUID) >>> >>> And spaceman doesn't link against libxfs, nor does libxcmd or >>> spaceman use uuids so I don't see why this fixes a problem with >>> libxfs... >>> >> >> I am building with --disable-static, I wonder if thats not a supported >> configuration >> for xfsprogs > > Still doesn't make spaceman link against libxfs or use uuids ;) > > I guess I'd suggest starting w/ x86 and a standard toolchain, make > --disable-static work for you there (I see that it does need some > fixups), then try things w/ the alternate toolchain and see > if more work is needed. I do see indirect dependency though libxcmd needs platform_findsizes symbol from libxfs I agree, there is some work needed to get it going with --disable-static > > -Eric ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-08 18:24 ` Khem Raj 2017-09-08 19:15 ` Eric Sandeen @ 2017-09-08 22:36 ` Dave Chinner 2017-09-08 23:22 ` Khem Raj 1 sibling, 1 reply; 11+ messages in thread From: Dave Chinner @ 2017-09-08 22:36 UTC (permalink / raw) To: Khem Raj; +Cc: linux-xfs On Fri, Sep 08, 2017 at 11:24:16AM -0700, Khem Raj wrote: > Hi Dave > > On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: > > On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: > >> When uuid is enabled then we see linking failures with gold linker e.g. > >> > >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' > >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' > >> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) > > > > That's libxfs, not xfs_spaceman.... > > > >> > >> Signed-off-by: Khem Raj <raj.khem@gmail.com> > >> --- > >> spaceman/Makefile | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/spaceman/Makefile b/spaceman/Makefile > >> index 95ec3c0..060e0af 100644 > >> --- a/spaceman/Makefile > >> +++ b/spaceman/Makefile > >> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman > >> HFILES = init.h space.h > >> CFILES = init.c file.c prealloc.c trim.c > >> > >> -LLDLIBS = $(LIBXCMD) > >> +LLDLIBS = $(LIBXCMD) $(LIBUUID) > > > > And spaceman doesn't link against libxfs, nor does libxcmd or > > spaceman use uuids so I don't see why this fixes a problem with > > libxfs... > > > > I am building with --disable-static, I wonder if thats not a supported > configuration > for xfsprogs For what reason? xfsprogs links it's internal libraries statically while it links to system libraries dynamically. We do not expose internal libraries like libxfs and libxcmd outside the build environment, and so dynamic linking of them doesn't make any sense because they won't be installed on any system you ship the binaries to.... Cheers, Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-08 22:36 ` Dave Chinner @ 2017-09-08 23:22 ` Khem Raj 2017-09-09 0:33 ` Eric Sandeen 0 siblings, 1 reply; 11+ messages in thread From: Khem Raj @ 2017-09-08 23:22 UTC (permalink / raw) To: Dave Chinner; +Cc: linux-xfs On Fri, Sep 8, 2017 at 3:36 PM, Dave Chinner <david@fromorbit.com> wrote: > On Fri, Sep 08, 2017 at 11:24:16AM -0700, Khem Raj wrote: >> Hi Dave >> >> On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: >> > On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >> >> When uuid is enabled then we see linking failures with gold linker e.g. >> >> >> >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >> >> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >> >> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) >> > >> > That's libxfs, not xfs_spaceman.... >> > >> >> >> >> Signed-off-by: Khem Raj <raj.khem@gmail.com> >> >> --- >> >> spaceman/Makefile | 2 +- >> >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> >> >> diff --git a/spaceman/Makefile b/spaceman/Makefile >> >> index 95ec3c0..060e0af 100644 >> >> --- a/spaceman/Makefile >> >> +++ b/spaceman/Makefile >> >> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >> >> HFILES = init.h space.h >> >> CFILES = init.c file.c prealloc.c trim.c >> >> >> >> -LLDLIBS = $(LIBXCMD) >> >> +LLDLIBS = $(LIBXCMD) $(LIBUUID) >> > >> > And spaceman doesn't link against libxfs, nor does libxcmd or >> > spaceman use uuids so I don't see why this fixes a problem with >> > libxfs... >> > >> >> I am building with --disable-static, I wonder if thats not a supported >> configuration >> for xfsprogs > > For what reason? > this is the distro policy under which I am trying to compile. No other reason. > xfsprogs links it's internal libraries statically while it links to > system libraries dynamically. We do not expose internal libraries > like libxfs and libxcmd outside the build environment, and so > dynamic linking of them doesn't make any sense because they won't be > installed on any system you ship the binaries to.... > Thats what I needed to know. I will remove --disable-static and both patches I sent I no more needed. > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-08 23:22 ` Khem Raj @ 2017-09-09 0:33 ` Eric Sandeen 2017-09-09 0:36 ` Khem Raj 0 siblings, 1 reply; 11+ messages in thread From: Eric Sandeen @ 2017-09-09 0:33 UTC (permalink / raw) To: Khem Raj, Dave Chinner; +Cc: linux-xfs On 9/8/17 6:22 PM, Khem Raj wrote: > On Fri, Sep 8, 2017 at 3:36 PM, Dave Chinner <david@fromorbit.com> wrote: >> On Fri, Sep 08, 2017 at 11:24:16AM -0700, Khem Raj wrote: >>> Hi Dave >>> >>> On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: >>>> On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >>>>> When uuid is enabled then we see linking failures with gold linker e.g. >>>>> >>>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >>>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >>>>> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) >>>> >>>> That's libxfs, not xfs_spaceman.... >>>> >>>>> >>>>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>>> --- >>>>> spaceman/Makefile | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> diff --git a/spaceman/Makefile b/spaceman/Makefile >>>>> index 95ec3c0..060e0af 100644 >>>>> --- a/spaceman/Makefile >>>>> +++ b/spaceman/Makefile >>>>> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >>>>> HFILES = init.h space.h >>>>> CFILES = init.c file.c prealloc.c trim.c >>>>> >>>>> -LLDLIBS = $(LIBXCMD) >>>>> +LLDLIBS = $(LIBXCMD) $(LIBUUID) >>>> >>>> And spaceman doesn't link against libxfs, nor does libxcmd or >>>> spaceman use uuids so I don't see why this fixes a problem with >>>> libxfs... >>>> >>> >>> I am building with --disable-static, I wonder if thats not a supported >>> configuration >>> for xfsprogs >> >> For what reason? >> > > this is the distro policy under which I am trying to compile. No other reason. which distro has this policy? There are certainly policies against static linking with /external/ libraries, because if libfoo has an exploit, you'd have to find every static linker and fix it, rather than just libfoo. But I can't imagine a policy that says a project cannot statically link with its own /internal/ libraries which are never distributed separately. -Eric ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] spaceman: Add missing link to libuuid 2017-09-09 0:33 ` Eric Sandeen @ 2017-09-09 0:36 ` Khem Raj 0 siblings, 0 replies; 11+ messages in thread From: Khem Raj @ 2017-09-09 0:36 UTC (permalink / raw) To: Eric Sandeen; +Cc: Dave Chinner, linux-xfs On Fri, Sep 8, 2017 at 5:33 PM, Eric Sandeen <sandeen@sandeen.net> wrote: > On 9/8/17 6:22 PM, Khem Raj wrote: >> On Fri, Sep 8, 2017 at 3:36 PM, Dave Chinner <david@fromorbit.com> wrote: >>> On Fri, Sep 08, 2017 at 11:24:16AM -0700, Khem Raj wrote: >>>> Hi Dave >>>> >>>> On Wed, Sep 6, 2017 at 7:40 PM, Dave Chinner <david@fromorbit.com> wrote: >>>>> On Wed, Sep 06, 2017 at 07:19:00PM -0700, Khem Raj wrote: >>>>>> When uuid is enabled then we see linking failures with gold linker e.g. >>>>>> >>>>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_compare' >>>>>> libxfs/.libs/libxfs.so: error: undefined reference to 'uuid_copy' >>>>>> clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation) >>>>> >>>>> That's libxfs, not xfs_spaceman.... >>>>> >>>>>> >>>>>> Signed-off-by: Khem Raj <raj.khem@gmail.com> >>>>>> --- >>>>>> spaceman/Makefile | 2 +- >>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/spaceman/Makefile b/spaceman/Makefile >>>>>> index 95ec3c0..060e0af 100644 >>>>>> --- a/spaceman/Makefile >>>>>> +++ b/spaceman/Makefile >>>>>> @@ -9,7 +9,7 @@ LTCOMMAND = xfs_spaceman >>>>>> HFILES = init.h space.h >>>>>> CFILES = init.c file.c prealloc.c trim.c >>>>>> >>>>>> -LLDLIBS = $(LIBXCMD) >>>>>> +LLDLIBS = $(LIBXCMD) $(LIBUUID) >>>>> >>>>> And spaceman doesn't link against libxfs, nor does libxcmd or >>>>> spaceman use uuids so I don't see why this fixes a problem with >>>>> libxfs... >>>>> >>>> >>>> I am building with --disable-static, I wonder if thats not a supported >>>> configuration >>>> for xfsprogs >>> >>> For what reason? >>> >> >> this is the distro policy under which I am trying to compile. No other reason. > > which distro has this policy? Custom distro based on yocto > > There are certainly policies against static linking with /external/ > libraries, because if libfoo has an exploit, you'd have to find > every static linker and fix it, rather than just libfoo. > > But I can't imagine a policy that says a project cannot statically > link with its own /internal/ libraries which are never distributed > separately. > I think you are right, in this case we should just make --disable-static to be no op or unsupported option in configure > -Eric > ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-09-09 0:37 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-09-07 2:19 [PATCH] spaceman: Add missing link to libuuid Khem Raj 2017-09-07 2:40 ` Dave Chinner 2017-09-07 4:11 ` Khem Raj 2017-09-07 6:59 ` Dave Chinner 2017-09-08 18:24 ` Khem Raj 2017-09-08 19:15 ` Eric Sandeen 2017-09-08 20:38 ` Khem Raj 2017-09-08 22:36 ` Dave Chinner 2017-09-08 23:22 ` Khem Raj 2017-09-09 0:33 ` Eric Sandeen 2017-09-09 0:36 ` Khem Raj
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox