* [Buildroot] [PATCH 1/1] package/xmlstarlet: fix static linking
@ 2014-08-18 21:35 Romain Naour
2014-08-21 13:26 ` Thomas Petazzoni
0 siblings, 1 reply; 3+ messages in thread
From: Romain Naour @ 2014-08-18 21:35 UTC (permalink / raw)
To: buildroot
The static linking with libxml2 and lib(e)xslt (--enable-static-libs)
need to be enabled when BR2_PREFER_STATIC_LIB is set.
Fixes:
http://autobuild.buildroot.net/results/0fd/0fd4a65ab98a4cf3fe893c29628608a71e96ecbc/build-end.log
Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
package/xmlstarlet/xmlstarlet.mk | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/package/xmlstarlet/xmlstarlet.mk b/package/xmlstarlet/xmlstarlet.mk
index 16f736e..7a8451f 100644
--- a/package/xmlstarlet/xmlstarlet.mk
+++ b/package/xmlstarlet/xmlstarlet.mk
@@ -12,9 +12,15 @@ XMLSTARLET_LICENSE_FILES = COPYING
XMLSTARLET_DEPENDENCIES += libxml2 libxslt \
$(if $(BR2_PACKAGE_LIBICONV),libiconv)
-XMLSTARLET_CONF_OPT += --disable-static-libs \
+XMLSTARLET_CONF_OPT += \
--with-libxml-prefix=${STAGING_DIR}/usr \
--with-libxslt-prefix=${STAGING_DIR}/usr \
--with-libiconv-prefix=${STAGING_DIR}/usr
+ifeq ($(BR2_PREFER_STATIC_LIB),y)
+XMLSTARLET_CONF_OPT += --enable-static-libs
+else
+XMLSTARLET_CONF_OPT += --disable-static-libs
+endif
+
$(eval $(autotools-package))
--
1.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/xmlstarlet: fix static linking
2014-08-18 21:35 [Buildroot] [PATCH 1/1] package/xmlstarlet: fix static linking Romain Naour
@ 2014-08-21 13:26 ` Thomas Petazzoni
2014-08-21 13:54 ` Romain Naour
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2014-08-21 13:26 UTC (permalink / raw)
To: buildroot
Dear Romain Naour,
On Mon, 18 Aug 2014 23:35:42 +0200, Romain Naour wrote:
> The static linking with libxml2 and lib(e)xslt (--enable-static-libs)
> need to be enabled when BR2_PREFER_STATIC_LIB is set.
>
> Fixes:
> http://autobuild.buildroot.net/results/0fd/0fd4a65ab98a4cf3fe893c29628608a71e96ecbc/build-end.log
>
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
> package/xmlstarlet/xmlstarlet.mk | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
Patch applied, but one comment below...
>
> diff --git a/package/xmlstarlet/xmlstarlet.mk b/package/xmlstarlet/xmlstarlet.mk
> index 16f736e..7a8451f 100644
> --- a/package/xmlstarlet/xmlstarlet.mk
> +++ b/package/xmlstarlet/xmlstarlet.mk
> @@ -12,9 +12,15 @@ XMLSTARLET_LICENSE_FILES = COPYING
> XMLSTARLET_DEPENDENCIES += libxml2 libxslt \
> $(if $(BR2_PACKAGE_LIBICONV),libiconv)
>
> -XMLSTARLET_CONF_OPT += --disable-static-libs \
> +XMLSTARLET_CONF_OPT += \
> --with-libxml-prefix=${STAGING_DIR}/usr \
> --with-libxslt-prefix=${STAGING_DIR}/usr \
> --with-libiconv-prefix=${STAGING_DIR}/usr
>
> +ifeq ($(BR2_PREFER_STATIC_LIB),y)
> +XMLSTARLET_CONF_OPT += --enable-static-libs
> +else
> +XMLSTARLET_CONF_OPT += --disable-static-libs
> +endif
Not introduced by your patch, but I don't understand why we have
--disable-static-libs in the else case here. When BR2_PREFER_STATIC_LIB
is disabled, we should normally build both the shared and static
variants of all libraries.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/xmlstarlet: fix static linking
2014-08-21 13:26 ` Thomas Petazzoni
@ 2014-08-21 13:54 ` Romain Naour
0 siblings, 0 replies; 3+ messages in thread
From: Romain Naour @ 2014-08-21 13:54 UTC (permalink / raw)
To: buildroot
Hi Thomas,
Le 21/08/2014 15:26, Thomas Petazzoni a ?crit :
> Dear Romain Naour,
>
> On Mon, 18 Aug 2014 23:35:42 +0200, Romain Naour wrote:
>> The static linking with libxml2 and lib(e)xslt (--enable-static-libs)
>> need to be enabled when BR2_PREFER_STATIC_LIB is set.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/0fd/0fd4a65ab98a4cf3fe893c29628608a71e96ecbc/build-end.log
>>
>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
>> ---
>> package/xmlstarlet/xmlstarlet.mk | 8 +++++++-
>> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> Patch applied, but one comment below...
>
>>
>> diff --git a/package/xmlstarlet/xmlstarlet.mk b/package/xmlstarlet/xmlstarlet.mk
>> index 16f736e..7a8451f 100644
>> --- a/package/xmlstarlet/xmlstarlet.mk
>> +++ b/package/xmlstarlet/xmlstarlet.mk
>> @@ -12,9 +12,15 @@ XMLSTARLET_LICENSE_FILES = COPYING
>> XMLSTARLET_DEPENDENCIES += libxml2 libxslt \
>> $(if $(BR2_PACKAGE_LIBICONV),libiconv)
>>
>> -XMLSTARLET_CONF_OPT += --disable-static-libs \
>> +XMLSTARLET_CONF_OPT += \
>> --with-libxml-prefix=${STAGING_DIR}/usr \
>> --with-libxslt-prefix=${STAGING_DIR}/usr \
>> --with-libiconv-prefix=${STAGING_DIR}/usr
>>
>> +ifeq ($(BR2_PREFER_STATIC_LIB),y)
>> +XMLSTARLET_CONF_OPT += --enable-static-libs
>> +else
>> +XMLSTARLET_CONF_OPT += --disable-static-libs
>> +endif
>
> Not introduced by your patch, but I don't understand why we have
> --disable-static-libs in the else case here. When BR2_PREFER_STATIC_LIB
> is disabled, we should normally build both the shared and static
> variants of all libraries.
>
I don't know either.
I wanted to keep the same behavior when BR2_PREFER_STATIC_LIB is enabled.
Best regards,
Romain
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-21 13:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-18 21:35 [Buildroot] [PATCH 1/1] package/xmlstarlet: fix static linking Romain Naour
2014-08-21 13:26 ` Thomas Petazzoni
2014-08-21 13:54 ` Romain Naour
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox