Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-25 Thread Koen Kooi

Op 4 jul. 2014, om 11:08 heeft Jack Mitchell m...@communistcode.co.uk het 
volgende geschreven:

 On 16/06/14 11:52, Ross Burton wrote:
 OpenSSL has license complications and GnuTLS is preferred, so although the
 license complications don't impact wget use GnuTLS for consistency.
 
 Also add a recommendation on ca-certificates so that https: URLs work.
 
 Signed-off-by: Ross Burton ross.bur...@intel.com
 ---
 meta/recipes-extended/wget/wget.inc |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)
 
 diff --git a/meta/recipes-extended/wget/wget.inc 
 b/meta/recipes-extended/wget/wget.inc
 index a778bca..642d502 100644
 --- a/meta/recipes-extended/wget/wget.inc
 +++ b/meta/recipes-extended/wget/wget.inc
 @@ -3,15 +3,16 @@ HOMEPAGE = https://www.gnu.org/software/wget/;
 SECTION = console/network
 LICENSE = GPLv3
 LIC_FILES_CHKSUM = file://COPYING;md5=d32239bcb673463ab874e80d47fae504
 -DEPENDS = openssl zlib libpcre
 +DEPENDS = gnutls zlib libpcre
 
 INC_PR = r16
 
 inherit autotools gettext texinfo update-alternatives
 
 -EXTRA_OECONF = --enable-ipv6 --with-libssl-prefix=${STAGING_DIR_HOST} \
 ---with-ssl=openssl --disable-rpath --disable-iri \
 +EXTRA_OECONF = --enable-ipv6 --with-ssl=gnutls --disable-rpath 
 --disable-iri \
 ac_cv_header_uuid_uuid_h=no
 
 ALTERNATIVE_${PN} = wget
 ALTERNATIVE_PRIORITY = 100
 +
 +RRECOMMENDS_${PN} += ca-certificates
 
 
 Ross,
 
 I've just pulled this in today after being on holiday for a while and it
 causes a breakage on my system.
 
 | checking for libgnutls... no
 | configure: error: --with-ssl=gnutls was given, but GNUTLS is not
 available.
 | Configure failed. The contents of all config.log files follows to aid
 debugging
 
 log: http://ix.io/dfO
 
 I would have expected gnutls to have been pulled in through the depends
 but it obviously hasn't, or the system hasn't managed to find it...

FWIW, I'm seeing the same problem.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-25 Thread Khem Raj
On Mon, Jun 16, 2014 at 3:52 AM, Ross Burton ross.bur...@intel.com wrote:
 OpenSSL has license complications and GnuTLS is preferred, so although the
 license complications don't impact wget use GnuTLS for consistency.

 Also add a recommendation on ca-certificates so that https: URLs work.

 Signed-off-by: Ross Burton ross.bur...@intel.com
 ---
  meta/recipes-extended/wget/wget.inc |7 ---
  1 file changed, 4 insertions(+), 3 deletions(-)

 diff --git a/meta/recipes-extended/wget/wget.inc 
 b/meta/recipes-extended/wget/wget.inc
 index a778bca..642d502 100644
 --- a/meta/recipes-extended/wget/wget.inc
 +++ b/meta/recipes-extended/wget/wget.inc
 @@ -3,15 +3,16 @@ HOMEPAGE = https://www.gnu.org/software/wget/;
  SECTION = console/network
  LICENSE = GPLv3
  LIC_FILES_CHKSUM = file://COPYING;md5=d32239bcb673463ab874e80d47fae504
 -DEPENDS = openssl zlib libpcre
 +DEPENDS = gnutls zlib libpcre

it would be nice if this was either virtual/ssh or some sort of
PACKAGECONFIG so folks who
have ssl on the system dont need to pull gnutls as well.


  INC_PR = r16

  inherit autotools gettext texinfo update-alternatives

 -EXTRA_OECONF = --enable-ipv6 --with-libssl-prefix=${STAGING_DIR_HOST} \
 ---with-ssl=openssl --disable-rpath --disable-iri \
 +EXTRA_OECONF = --enable-ipv6 --with-ssl=gnutls --disable-rpath 
 --disable-iri \
  ac_cv_header_uuid_uuid_h=no

  ALTERNATIVE_${PN} = wget
  ALTERNATIVE_PRIORITY = 100
 +
 +RRECOMMENDS_${PN} += ca-certificates
 --
 1.7.10.4

 --
 ___
 Openembedded-core mailing list
 Openembedded-core@lists.openembedded.org
 http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-25 Thread Burton, Ross
On 25 July 2014 17:22, Khem Raj raj.k...@gmail.com wrote:
 it would be nice if this was either virtual/ssh or some sort of
 PACKAGECONFIG so folks who
 have ssl on the system dont need to pull gnutls as well.

There's API differences so it would have to be a PACKAGECONFIG but
agreed.  For what it's worth I'm planning on doing a PACKAGECONFIG for
xserver as it supports three crypto libraries for SHA and pulling in
gnutls just for that is madness.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-25 Thread Khem Raj
On Thu, Jul 17, 2014 at 5:48 AM, Richard Purdie
richard.pur...@linuxfoundation.org wrote:
 So why on earth is it looking at /usr/lib/libgnutls.so?

 Total guess by try adding EXTRA_OECONF += --without-libgnutls-prefix

The problem is due to use of

AC_LIB_HAVE_LINKFLAGS([gnutls] since we use DESTDIR while cross
compiling, so above option should have worked I dont know why it did
not but you could try

--with-libgnutls-prefix=target-sysroot

if that doesnt work either then may be replace use of
AC_LIB_HAVE_LINKFLAGS with AC_CHECK_LIB
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-25 Thread Richard Purdie
On Fri, 2014-07-25 at 09:40 -0700, Khem Raj wrote:
 On Thu, Jul 17, 2014 at 5:48 AM, Richard Purdie
 richard.pur...@linuxfoundation.org wrote:
  So why on earth is it looking at /usr/lib/libgnutls.so?
 
  Total guess by try adding EXTRA_OECONF += --without-libgnutls-prefix
 
 The problem is due to use of
 
 AC_LIB_HAVE_LINKFLAGS([gnutls] since we use DESTDIR while cross
 compiling, so above option should have worked I dont know why it did
 not but you could try
 
 --with-libgnutls-prefix=target-sysroot
 
 if that doesnt work either then may be replace use of
 AC_LIB_HAVE_LINKFLAGS with AC_CHECK_LIB

I looked at the generated files and the --without option just stops it
looking in the wrong places. It seems counter-intuitive compared to how
these options usually work but does seem to fix the problem...

Cheers,

Richard


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Jack Mitchell
So this error is still blocking my master build, any ideas?

On 04/07/14 10:08, Jack Mitchell wrote:
 Ross,

 I've just pulled this in today after being on holiday for a while and it
 causes a breakage on my system.

 | checking for libgnutls... no
 | configure: error: --with-ssl=gnutls was given, but GNUTLS is not
 available.
 | Configure failed. The contents of all config.log files follows to aid
 debugging

 log: http://ix.io/dfO

 I would have expected gnutls to have been pulled in through the depends
 but it obviously hasn't, or the system hasn't managed to find it...

 Any ideas?

 Cheers,

 On 16/06/14 11:52, Ross Burton wrote:
 OpenSSL has license complications and GnuTLS is preferred, so although the
 license complications don't impact wget use GnuTLS for consistency.

 Also add a recommendation on ca-certificates so that https: URLs work.

 Signed-off-by: Ross Burton ross.bur...@intel.com
 ---
  meta/recipes-extended/wget/wget.inc |7 ---
  1 file changed, 4 insertions(+), 3 deletions(-)

 diff --git a/meta/recipes-extended/wget/wget.inc 
 b/meta/recipes-extended/wget/wget.inc
 index a778bca..642d502 100644
 --- a/meta/recipes-extended/wget/wget.inc
 +++ b/meta/recipes-extended/wget/wget.inc
 @@ -3,15 +3,16 @@ HOMEPAGE = https://www.gnu.org/software/wget/;
  SECTION = console/network
  LICENSE = GPLv3
  LIC_FILES_CHKSUM = file://COPYING;md5=d32239bcb673463ab874e80d47fae504
 -DEPENDS = openssl zlib libpcre
 +DEPENDS = gnutls zlib libpcre
  
  INC_PR = r16
  
  inherit autotools gettext texinfo update-alternatives
  
 -EXTRA_OECONF = --enable-ipv6 --with-libssl-prefix=${STAGING_DIR_HOST} \
 ---with-ssl=openssl --disable-rpath --disable-iri \
 +EXTRA_OECONF = --enable-ipv6 --with-ssl=gnutls --disable-rpath 
 --disable-iri \
  ac_cv_header_uuid_uuid_h=no
  
  ALTERNATIVE_${PN} = wget
  ALTERNATIVE_PRIORITY = 100
 +
 +RRECOMMENDS_${PN} += ca-certificates

 
 


-- 
  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  Cambridgeshire, UK
  http://www.embed.me.uk
-- 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Richard Purdie
On Thu, 2014-07-17 at 12:06 +0100, Jack Mitchell wrote:
 So this error is still blocking my master build, any ideas?

Share the config.log so we can see how the configure test is failing? Is
gnutls actually built and in the sysroot or not?

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Jack Mitchell
config.log: http://ix.io/dr3

and yes, gnutls is being built, or at least it's in the work dir.

[jack@jackArch gnutls]$ pwd
/home/jack/Work/oe-core.git/test-build/tmp-eglibc/work/core2-32-oe-linux/gnutls
[jack@jackArch gnutls]$ ls
2.12.23-r8.4  3.2.13-r0  3.2.15-r0  3.3.5-r0
[jack@jackArch gnutls]$

On 17/07/14 12:33, Richard Purdie wrote:
 On Thu, 2014-07-17 at 12:06 +0100, Jack Mitchell wrote:
 So this error is still blocking my master build, any ideas?
 Share the config.log so we can see how the configure test is failing? Is
 gnutls actually built and in the sysroot or not?

 Cheers,

 Richard



-- 
  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  Cambridgeshire, UK
  http://www.embed.me.uk
-- 

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Jack Mitchell
Wait, that config.log is bad. It's from my build with the patch
reverted. Hold up I'll get the proper one out.

On 17/07/14 12:39, Jack Mitchell wrote:
 config.log: http://ix.io/dr3
 
 and yes, gnutls is being built, or at least it's in the work dir.
 
 [jack@jackArch gnutls]$ pwd
 /home/jack/Work/oe-core.git/test-build/tmp-eglibc/work/core2-32-oe-linux/gnutls
 [jack@jackArch gnutls]$ ls
 2.12.23-r8.4  3.2.13-r0  3.2.15-r0  3.3.5-r0
 [jack@jackArch gnutls]$
 
 On 17/07/14 12:33, Richard Purdie wrote:
 On Thu, 2014-07-17 at 12:06 +0100, Jack Mitchell wrote:
 So this error is still blocking my master build, any ideas?
 Share the config.log so we can see how the configure test is failing? Is
 gnutls actually built and in the sysroot or not?

 Cheers,

 Richard

 
 


-- 
  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  Cambridgeshire, UK
  http://www.embed.me.uk
-- 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Jack Mitchell
Failed with gnutls config.log: http://ix.io/dr4

On 17/07/14 12:39, Jack Mitchell wrote:
 config.log: http://ix.io/dr3
 
 and yes, gnutls is being built, or at least it's in the work dir.
 
 [jack@jackArch gnutls]$ pwd
 /home/jack/Work/oe-core.git/test-build/tmp-eglibc/work/core2-32-oe-linux/gnutls
 [jack@jackArch gnutls]$ ls
 2.12.23-r8.4  3.2.13-r0  3.2.15-r0  3.3.5-r0
 [jack@jackArch gnutls]$
 
 On 17/07/14 12:33, Richard Purdie wrote:
 On Thu, 2014-07-17 at 12:06 +0100, Jack Mitchell wrote:
 So this error is still blocking my master build, any ideas?
 Share the config.log so we can see how the configure test is failing? Is
 gnutls actually built and in the sysroot or not?

 Cheers,

 Richard

 
 


-- 
  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  Cambridgeshire, UK
  http://www.embed.me.uk
-- 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Richard Purdie
On Thu, 2014-07-17 at 12:46 +0100, Jack Mitchell wrote:
 Failed with gnutls config.log: http://ix.io/dr4

That is helpful:

configure:35748: checking for libgnutls
configure:35772: i586-oe-linux-gcc  -m32 -march=core2 -mtune=core2 -msse3 
-mfpmath=sse 
--sysroot=/home/jack/Work/oe-core.git/test-build/tmp-eglibc/sysroots/cbnlcomexpress
 -o conftest  -O2 -pipe -g -feliminate-unused-debug-types  -Wl,-O1 
-Wl,--hash-style=gnu -Wl,--as-needed conftest.c /usr/lib/libgnutls.so -lz  5
/usr/lib/libgnutls.so: error adding symbols: File in wrong format
collect2: error: ld returned 1 exit status
configure:35772: $? = 1

So why on earth is it looking at /usr/lib/libgnutls.so?

Cheers,

Richard



-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-17 Thread Richard Purdie
On Thu, 2014-07-17 at 13:39 +0100, Richard Purdie wrote:
 On Thu, 2014-07-17 at 12:46 +0100, Jack Mitchell wrote:
  Failed with gnutls config.log: http://ix.io/dr4
 
 That is helpful:
 
 configure:35748: checking for libgnutls
 configure:35772: i586-oe-linux-gcc  -m32 -march=core2 -mtune=core2 -msse3 
 -mfpmath=sse 
 --sysroot=/home/jack/Work/oe-core.git/test-build/tmp-eglibc/sysroots/cbnlcomexpress
  -o conftest  -O2 -pipe -g -feliminate-unused-debug-types  -Wl,-O1 
 -Wl,--hash-style=gnu -Wl,--as-needed conftest.c /usr/lib/libgnutls.so -lz  5
 /usr/lib/libgnutls.so: error adding symbols: File in wrong format
 collect2: error: ld returned 1 exit status
 configure:35772: $? = 1
 
 So why on earth is it looking at /usr/lib/libgnutls.so?

Total guess by try adding EXTRA_OECONF += --without-libgnutls-prefix

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-07-04 Thread Jack Mitchell
On 16/06/14 11:52, Ross Burton wrote:
 OpenSSL has license complications and GnuTLS is preferred, so although the
 license complications don't impact wget use GnuTLS for consistency.
 
 Also add a recommendation on ca-certificates so that https: URLs work.
 
 Signed-off-by: Ross Burton ross.bur...@intel.com
 ---
  meta/recipes-extended/wget/wget.inc |7 ---
  1 file changed, 4 insertions(+), 3 deletions(-)
 
 diff --git a/meta/recipes-extended/wget/wget.inc 
 b/meta/recipes-extended/wget/wget.inc
 index a778bca..642d502 100644
 --- a/meta/recipes-extended/wget/wget.inc
 +++ b/meta/recipes-extended/wget/wget.inc
 @@ -3,15 +3,16 @@ HOMEPAGE = https://www.gnu.org/software/wget/;
  SECTION = console/network
  LICENSE = GPLv3
  LIC_FILES_CHKSUM = file://COPYING;md5=d32239bcb673463ab874e80d47fae504
 -DEPENDS = openssl zlib libpcre
 +DEPENDS = gnutls zlib libpcre
  
  INC_PR = r16
  
  inherit autotools gettext texinfo update-alternatives
  
 -EXTRA_OECONF = --enable-ipv6 --with-libssl-prefix=${STAGING_DIR_HOST} \
 ---with-ssl=openssl --disable-rpath --disable-iri \
 +EXTRA_OECONF = --enable-ipv6 --with-ssl=gnutls --disable-rpath 
 --disable-iri \
  ac_cv_header_uuid_uuid_h=no
  
  ALTERNATIVE_${PN} = wget
  ALTERNATIVE_PRIORITY = 100
 +
 +RRECOMMENDS_${PN} += ca-certificates
 

Ross,

I've just pulled this in today after being on holiday for a while and it
causes a breakage on my system.

| checking for libgnutls... no
| configure: error: --with-ssl=gnutls was given, but GNUTLS is not
available.
| Configure failed. The contents of all config.log files follows to aid
debugging

log: http://ix.io/dfO

I would have expected gnutls to have been pulled in through the depends
but it obviously hasn't, or the system hasn't managed to find it...

Any ideas?

Cheers,

-- 
  Jack Mitchell (j...@embed.me.uk)
  Embedded Systems Engineer
  Cambridgeshire, UK
  http://www.embed.me.uk
-- 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] wget: use GnuTLS instead of OpenSSL

2014-06-16 Thread Ross Burton
OpenSSL has license complications and GnuTLS is preferred, so although the
license complications don't impact wget use GnuTLS for consistency.

Also add a recommendation on ca-certificates so that https: URLs work.

Signed-off-by: Ross Burton ross.bur...@intel.com
---
 meta/recipes-extended/wget/wget.inc |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-extended/wget/wget.inc 
b/meta/recipes-extended/wget/wget.inc
index a778bca..642d502 100644
--- a/meta/recipes-extended/wget/wget.inc
+++ b/meta/recipes-extended/wget/wget.inc
@@ -3,15 +3,16 @@ HOMEPAGE = https://www.gnu.org/software/wget/;
 SECTION = console/network
 LICENSE = GPLv3
 LIC_FILES_CHKSUM = file://COPYING;md5=d32239bcb673463ab874e80d47fae504
-DEPENDS = openssl zlib libpcre
+DEPENDS = gnutls zlib libpcre
 
 INC_PR = r16
 
 inherit autotools gettext texinfo update-alternatives
 
-EXTRA_OECONF = --enable-ipv6 --with-libssl-prefix=${STAGING_DIR_HOST} \
---with-ssl=openssl --disable-rpath --disable-iri \
+EXTRA_OECONF = --enable-ipv6 --with-ssl=gnutls --disable-rpath --disable-iri \
 ac_cv_header_uuid_uuid_h=no
 
 ALTERNATIVE_${PN} = wget
 ALTERNATIVE_PRIORITY = 100
+
+RRECOMMENDS_${PN} += ca-certificates
-- 
1.7.10.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core