[Buildroot] [PATCH 1/1] Point libxslt to the correct location of libxml2
Ferdinand van Aartsen
ferdinand at ombud.nl
Fri Sep 21 12:53:52 UTC 2018
Thomas Petazzoni schreef op 2018-09-21 13:43:
> Hello,
>
> On Fri, 21 Sep 2018 12:19:14 +0200, Ferdinand van Aartsen wrote:
>
>> Currently libxslt configure finds native /usr/bin/xml2-config.
>>
>> Signed-off-by: Ferdinand van Aartsen <ferdinand at ombud.nl>
>
> Thanks for this contribution! The commit title should always follow the
> format:
>
> <package>: <description>
>
> so, something like:
>
> libxslt: point to the correct location of xml2-config
>
Yes, got it right for my second patch, where I then proceeded to forget
to add a deleted file to the commit. Ah, some days... But, I'll get
there.
>> diff --git a/package/libxslt/libxslt.mk b/package/libxslt/libxslt.mk
>> index 972d5b80d5..fb5d03ab44 100644
>> --- a/package/libxslt/libxslt.mk
>> +++ b/package/libxslt/libxslt.mk
>> @@ -13,7 +13,8 @@ LIBXSLT_LICENSE_FILES = COPYING
>> LIBXSLT_CONF_OPTS = \
>> --with-gnu-ld \
>> --without-debug \
>> - --without-python
>> + --without-python \
>> + --with-libxml-prefix=$(STAGING_DIR)/usr
>> LIBXSLT_CONFIG_SCRIPTS = xslt-config
>> LIBXSLT_DEPENDENCIES = host-pkgconf libxml2
>>
>> @@ -25,7 +26,11 @@ else
>> LIBXSLT_CONF_OPTS += --without-crypto
>> endif
>>
>> -HOST_LIBXSLT_CONF_OPTS = --without-debug --without-python
>> --without-crypto
>> +HOST_LIBXSLT_CONF_OPTS = \
>> + --without-debug \
>> + --without-python \
>> + --without-crypto \
>> + --with-libxml-prefix=$(HOST_DIR)/usr
>
> This should be just $(HOST_DIR), and not $(HOST_DIR)/usr.
>
> Those two minor comments can be fixed when applying the patch, no need
> to resend just for that.
>
> Thanks!
>
> Thomas
I copied behaviour I saw in other .mk files. But I double-checked and
saw that host-libxslt configure detects the right xml2-config without
--with-libxml-prefix anyway. So the host part can be left as it was.
Ferdinand
More information about the buildroot
mailing list