[lttng-dev] [PATCH] Fix linker library order

Alexandre Montplaisir alexmonthy at voxpopuli.im
Mon May 13 17:57:28 EDT 2013


On 13-05-11 10:59 PM, Mathieu Desnoyers wrote:
> * Francis Giraldeau (francis.giraldeau at gmail.com) wrote:
>> Libraries must be specified after the binary target.
> merged, thanks!
>
> Mathieu

This seems to have broken the daily PPA packages:
https://launchpadlibrarian.net/139804087/buildlog_ubuntu-saucy-amd64.lttng-ust_2.2~pre-0%2Bbzr1574%2Bpack37%2B201305131247~saucy1_FAILEDTOBUILD.txt.gz

That package has no particular patches. If I build it locally from
source it builds fine, but if I build the package locally (with
dpkg-buildpackage), I get the same error. It could be related to the
--as-needed problem we've had before...

Cheers,
Alex

>
>> Signed-off-by: Francis Giraldeau <francis.giraldeau at gmail.com>
>> ---
>>  doc/examples/demo/Makefile | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/doc/examples/demo/Makefile b/doc/examples/demo/Makefile
>> index 7ae6547..bbc8ccb 100644
>> --- a/doc/examples/demo/Makefile
>> +++ b/doc/examples/demo/Makefile
>> @@ -55,7 +55,7 @@ demo.o: demo.c
>>  	$(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ $<
>>  
>>  demo: demo.o
>> -	$(CC) -o $@ $(LIBS) $^
>> +	$(CC) -o $@ $^ $(LIBS)
>>  
>>  .PHONY: clean
>>  clean:
>> -- 
>> 1.8.1.2
>>
>>
>> _______________________________________________
>> lttng-dev mailing list
>> lttng-dev at lists.lttng.org
>> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev




More information about the lttng-dev mailing list