[OE-core] [PATCH] curl: Modify PACKAGECONFIG to enable openssl

[Li, Xin]

Hi, Anders

Thanks for your replay.

> Subject: Re: [OE-core] [PATCH] curl: Modify PACKAGECONFIG to enable openssl
> 
> * Li xin <lixin.fnst at cn.fujitsu.com> [150612 05:41]:
> 
> > Enabled openssl defalutly to use https, just like Ubuntu and Red Hat do.
> 
> > Signed-off-by: Li Xin <lixin.fnst at cn.fujitsu.com>
> > ---
> 
> > -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