Hi, Anders

Thanks for your replay.

> Subject: Re: [OE-core] [PATCH] curl: Modify PACKAGECONFIG to enable openssl
> 
> * Li xin <[email protected]> [150612 05:41]:
> 
> > Enabled openssl defalutly to use https, just like Ubuntu and Red Hat do.
> 
> > Signed-off-by: Li Xin <[email protected]>
> > ---
> 
> > -PACKAGECONFIG ??= "${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6",
> "", d)} gnutls zlib"
> > +PACKAGECONFIG ??= "${@bb.utils.contains("DISTRO_FEATURES", "ipv6", "ipv6",
> "", d)} ssl zlib"
> 
> I think you need to motivate why you're changing from gnutls to ssl
> (openssl) here.
> 
> Not that I'm personally against it. I've occasionally done the same thing 
> locally. But
> this is a change of the defaults, and the default dependencies. Thus, it 
> needs to
> have a good motivation.

The motivation is as following:
Curl depends on gnutls and gnutls depends on gmp, but the LICENSE of gmp is 
LGPLv3.
If the users do not want to use the LGPLv3 recipes(the rpm of gmp is not be 
installed), there will be an error when using https.
Such as:
----------------
# curl -k https://192.168.246.1
curl: error while loading shared libraries: libgmp.so.10: cannot open shared 
object file: No such file or directory 
----------------
But after changing gnutls to ssl in PACKAGECONFIG, there will be no errors.

> 
> >  PACKAGECONFIG_class-native = "ipv6 ssl zlib"
> >  PACKAGECONFIG_class-nativesdk = "ipv6 ssl zlib"
> 
> > @@ -36,6 +36,7 @@ EXTRA_OECONF = "--without-libidn \
> >                  --disable-ldap \
> >                  --disable-ldaps \
> >
> > --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \
> > +                --with-libssh2 \
> 
> NACK.
> 
> We have a PACKAGECONFIG för libssh2, which defaults to off. Do /not/ enable it
> directly in EXTRA_OECONF. This will break builds (at least the determinism of
> builds).
> 
Yes.
Enabling -with-libssh2 in EXTRA_OECONF is not right.
My goal is to let sftp supported by curl, by adding libssh2 in PACKAGECONFIG.
But libssh2 is not in OE-core by default. 
And I think there may be someone else want to use libssh2.
So how about moving libssh2 recipe from Meta-oe to OE-core? :)

Thank you very much.

Regards,
Li

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to