[Buildroot] [PATCH v2 1/2] package/harfbuzz: disable test programs
Romain Naour
romain.naour at openwide.fr
Mon May 18 21:40:02 UTC 2015
Hi Thomas,
Le 18/05/2015 23:03, Thomas Petazzoni a écrit :
> Dear Romain Naour,
>
> On Tue, 12 May 2015 23:51:23 +0200, Romain Naour wrote:
>> Workaround a build issue by disabling all test
>> binaries since it forget to link with -lstd++.
>>
>> Add a new configure option to disable test programs
>> in a upstreamable way.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/210/2107f9dfb39eeb6559fb4271c7af8b39aef521ca/
>>
>> Signed-off-by: Romain Naour <romain.naour at openwide.fr>
>
> I was almost going to apply this one, but I'm not entirely happy in
> fact. I really works around the real problem, and does not fix it:
> imagine a real program now links statically to harfbuzz using harfbuzz
> pkg-config file. This pkg-config file does not advertise the dependency
> on libstdc++ in its Libs.private, so the link would also fail, just
> like it currently fails for the test programs.
-lstdc++ is only needed by libharfbuzz-icu.la which use the icu package.
So maybe -lstdc++ is missing in Libs.private from icu-uc.pc ?
>
> Unfortunately, fixing the .pc file would not fix the internal harfbuzz
> test programs, since they don't use pkg-config to get the link flags.
>
> To be honest, I am not sure what is the correct automake way of fixing
> this problem. Maybe:
>
> -LDADD = $(top_builddir)/src/libharfbuzz.la $(GLIB_LIBS)
> +LDADD = $(top_builddir)/src/libharfbuzz.la $(GLIB_LIBS) -lstdc++
>
> in test/api/Makefile.am ?
You would said ;)
test_unicode_LDADD += $(top_builddir)/src/libharfbuzz-icu.la -lstdc++
With that fixed, Harbuzz build fine.
Thanks for the review.
Best regards,
Romain
>
> Best regards,
>
> Thomas
>
More information about the buildroot
mailing list