Re: [PATCH] nfsref: switch the way libraries are linked.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 12/11/18 2:12 PM, Chuck Lever wrote:
> 
>> On Dec 11, 2018, at 2:09 PM, Steve Dickson <steved@xxxxxxxxxx> wrote:
>>
>> Link the static libs before the dynamic libs allowing
>> the routines in the static libs to be defined
>> by the dynamic libs
> 
> Hi Steve, Why is this change necessary? Is there a bug report?
No... when I enabled junctions in the Fedora rawhide (F30) I got
a bunch of xml undefined routines 
   https://koji.fedoraproject.org/koji/getfile?taskID=31408933&volume=DEFAULT&name=build.log&offset=-4000

It compiled fine on f29 but not on rawhide

After asking if the XML api changed, people suggested I simply
change the order of how the static and dynamic libs are linked,
which worked

https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx/message/6UIAF47S73QFLCJDWEWXKQHZTOMB5JXQ/

https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx/message/VK2XPHV2MELV6IRKBG2ITI6GKUBMMRQW/

steved.

> 
> 
>> Signed-off-by: Steve Dickson <steved@xxxxxxxxxx>
>> ---
>> utils/nfsref/Makefile.am | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/utils/nfsref/Makefile.am b/utils/nfsref/Makefile.am
>> index 44edc83..2409dd0 100644
>> --- a/utils/nfsref/Makefile.am
>> +++ b/utils/nfsref/Makefile.am
>> @@ -27,9 +27,9 @@ noinst_HEADERS		= nfsref.h
>>
>> sbin_PROGRAMS		= nfsref
>> nfsref_SOURCES		= add.c lookup.c nfsref.c remove.c
>> -LDADD			= $(LIBXML2) $(LIBCAP) \
>> -			  ../../support/nfs/libnfs.la \
>> -			  ../../support/junction/libjunction.la
>> +LDADD			= ../../support/nfs/libnfs.la \
>> +			  ../../support/junction/libjunction.la \
>> +			  $(LIBXML2) $(LIBCAP)
>>
>> man8_MANS		= nfsref.man
>>
>> -- 
>> 2.19.2
>>
> 
> --
> Chuck Lever
> 
> 
> 



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux