summaryrefslogtreecommitdiffstats
path: root/meta/recipes-support/curl
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@arm.com>2023-01-11 14:17:53 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2023-01-12 23:08:59 +0000
commit0cd69599e33c7dc80826542440ff696e4f47db14 (patch)
tree1ac61a49a76db84d289f9420911cd1434650abd1 /meta/recipes-support/curl
parent3cef25ba0f4f396b399744649128eb766f753293 (diff)
downloadpoky-0cd69599e33c7dc80826542440ff696e4f47db14.tar.gz
curl: don't enable debug builds
In oe-core 27824261 --enable-debug was added to the configure arguments to turn on debugging symbols. However, enabling debug mode does more than turn on debugging symbols and introduces some codepaths that can be controlled with environment variables. Bluntly, the curl maintainer says that --enable-debug should not be used in production: https://curl.se/mail/lib-2023-01/0039.html I did a build and verified that the curl-dbg package doesn't massively shrink, so the debug symbols are still being built. Remove the debug options and hide them behind a PACKAGECONFIG, with a comment that it should not be used in production. (From OE-Core rev: 3ed3b2ffeeaa5d888f77dd30cb8cd81d3275398a) Signed-off-by: Ross Burton <ross.burton@arm.com> Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-support/curl')
-rw-r--r--meta/recipes-support/curl/curl_7.87.0.bb4
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/recipes-support/curl/curl_7.87.0.bb b/meta/recipes-support/curl/curl_7.87.0.bb
index a3ad66f1aa..9956a28422 100644
--- a/meta/recipes-support/curl/curl_7.87.0.bb
+++ b/meta/recipes-support/curl/curl_7.87.0.bb
@@ -32,6 +32,8 @@ PACKAGECONFIG:class-nativesdk = "ipv6 openssl proxy random threaded-resolver ver
32PACKAGECONFIG[ares] = "--enable-ares,--disable-ares,c-ares,,,threaded-resolver" 32PACKAGECONFIG[ares] = "--enable-ares,--disable-ares,c-ares,,,threaded-resolver"
33PACKAGECONFIG[brotli] = "--with-brotli,--without-brotli,brotli" 33PACKAGECONFIG[brotli] = "--with-brotli,--without-brotli,brotli"
34PACKAGECONFIG[builtinmanual] = "--enable-manual,--disable-manual" 34PACKAGECONFIG[builtinmanual] = "--enable-manual,--disable-manual"
35# Don't use this in production
36PACKAGECONFIG[debug] = "--enable-debug,--disable-debug"
35PACKAGECONFIG[dict] = "--enable-dict,--disable-dict," 37PACKAGECONFIG[dict] = "--enable-dict,--disable-dict,"
36PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls" 38PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls"
37PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher," 39PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher,"
@@ -68,9 +70,7 @@ EXTRA_OECONF = " \
68 --enable-crypto-auth \ 70 --enable-crypto-auth \
69 --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \ 71 --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \
70 --without-libpsl \ 72 --without-libpsl \
71 --enable-debug \
72 --enable-optimize \ 73 --enable-optimize \
73 --disable-curldebug \
74 ${@'--without-ssl' if (bb.utils.filter('PACKAGECONFIG', 'gnutls mbedtls nss openssl', d) == '') else ''} \ 74 ${@'--without-ssl' if (bb.utils.filter('PACKAGECONFIG', 'gnutls mbedtls nss openssl', d) == '') else ''} \
75" 75"
76 76