]> code.ossystems Code Review - openembedded-core.git/commit
curl: DEPENDS on libidn2 (not libidn)
authorAndré Draszik <andre.draszik@jci.com>
Thu, 5 Apr 2018 11:08:58 +0000 (12:08 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 6 Apr 2018 10:41:01 +0000 (11:41 +0100)
commit705eaea991622bdbb2ee83eefa8df8e665e3efe4
tree1fde02aa52264204a06dd2a74c3d185a15177f7c
parenta2cbb5dc4c6191ca343dc4211abde693dc1a579a
curl: DEPENDS on libidn2 (not libidn)

Since v7.51.0, libidn2 is the only available option, libidn
support was dropped.
The configure option was renamed as of v7.53.0

Therefore, curl unconditionally tries to build against libidn2,
which in particular is a problem for curl-native, as that might
or might not build against the build-machine's libidn2 now,
which furthermore causes problems when trying to share sstate
between multiple build machines.

We therefore see the following in the config log:
    ...
    checking whether to build with libidn2... (assumed) yes
    ...
    checking for libidn2 options with pkg-config... no
    configure: IDN_LIBS: "-lidn2"
    configure: IDN_LDFLAGS: ""
    configure: IDN_CPPFLAGS: ""
    configure: IDN_DIR: ""
    checking if idn2_lookup_ul can be linked... yes
    checking idn2.h usability... yes
    checking idn2.h presence... yes
    checking for idn2.h... yes
    ...
      IDN support:      enabled (libidn2)
    ...
even though this recipe tries to disable that.

While libidn2 isn't available in OE, this change at least:
* prevents curl-native to silently build against libidn2 if
  that is installed on build machine, even if not requested
* alerts people who use the PACKAGECONFIG option that it's
  not actually doing what they intend to do

Signed-off-by: André Draszik <andre.draszik@jci.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-support/curl/curl_7.58.0.bb