Skip to contentSkip to conversation

gnutls: fix link-time build failures

PATCH Superseded Gustavo Zacarias 3 participants 4 messages Oct 24, 2012 – Oct 27, 2012

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 gnutls: fix link-time build failures

Fixes
http://autobuild.buildroot.net/results/23da17446266bdd49ea0da0c1892d66d3ac6607f/build-end.log
And may the <BEEP> rest in peace at last...

---
 package/gnutls/gnutls.mk |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
package/gnutls/gnutls.mk +2 -1
index e889821..1ef6e3f 100644
@@ -10,8 +10,9 @@ GNUTLS_SITE = $(BR2_GNU_MIRROR)/gnutls
 GNUTLS_LICENSE = GPLv3+ LGPLv2.1+
 GNUTLS_LICENSE_FILES = COPYING lib/COPYING
 GNUTLS_DEPENDENCIES = host-pkg-config libgcrypt $(if $(BR2_PACKAGE_ZLIB),zlib)
+GNUTLS_CONF_ENV = acl_cv_rpath=no
 GNUTLS_CONF_OPT = --with-libgcrypt --without-libgcrypt-prefix \
-		--without-p11-kit
+		--without-p11-kit --disable-rpath
 GNUTLS_INSTALL_STAGING = YES
 
 $(eval $(autotools-package))
Signature · 1 line
1.7.8.6
#1 lore raw

PATCH v2 1/1 gnutls: fix link-time build failures

Quoted 8 lines · Gustavo Zacarias

Fixes
http://autobuild.buildroot.net/results/23da17446266bdd49ea0da0c1892d66d3ac6607f/build-end.log
And may the <BEEP> rest in peace at last...

---
 package/gnutls/gnutls.mk |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH v2 1/1 gnutls: fix link-time build failures

In reply to Gustavo Zacarias
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gustavo" == Gustavo Zacarias <[email protected]> writes:

Gustavo> Fixes
Gustavo> http://autobuild.buildroot.net/results/23da17446266bdd49ea0da0c1892d66d3ac6607f/build-end.log
Gustavo> And may the <BEEP> rest in peace at last...

Gustavo> Signed-off-by: Gustavo Zacarias <[email protected]> Gustavo> ---

 Gustavo>  package/gnutls/gnutls.mk |    3 ++-

Gustavo> 1 files changed, 2 insertions(+), 1 deletions(-)

Gustavo> diff --git a/package/gnutls/gnutls.mk b/package/gnutls/gnutls.mk Gustavo> index e889821..1ef6e3f 100644
Gustavo> --- a/package/gnutls/gnutls.mk
Gustavo> +++ b/package/gnutls/gnutls.mk
Gustavo> @@ -10,8 +10,9 @@ GNUTLS_SITE = $(BR2_GNU_MIRROR)/gnutls

 Gustavo>  GNUTLS_LICENSE = GPLv3+ LGPLv2.1+
 Gustavo>  GNUTLS_LICENSE_FILES = COPYING lib/COPYING
 Gustavo>  GNUTLS_DEPENDENCIES = host-pkg-config libgcrypt $(if $(BR2_PACKAGE_ZLIB),zlib)

Gustavo> +GNUTLS_CONF_ENV = acl_cv_rpath=no
Gustavo> GNUTLS_CONF_OPT = --with-libgcrypt --without-libgcrypt-prefix \
Gustavo> - --without-p11-kit
Gustavo> + --without-p11-kit --disable-rpath

It's a bit odd that you need both the variable and the configure options, but OK - You presumably do.

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard
#3 lore raw

PATCH v2 1/1 gnutls: fix link-time build failures

Quoted 4 lines · Peter Korsgaard

It's a bit odd that you need both the variable and the configure options, but OK - You presumably do.

Committed, thanks.

--disable-rpath surely isn't enough, we've tested it with Thomas. Looking at the complete source they're each referenced in different inclusive and exclusive parts so i'd rather stay on the safe side with both. I didn't test with just the configure env variable but looking at configure they're both doing their separate assignments and i think we don't want this to rear it's ugly head again :)
Regards.