summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@arm.com>2024-02-10 16:12:34 +0000
committerSteve Sakoman <steve@sakoman.com>2024-02-28 03:32:09 -1000
commitf517fe8929ee256521e025bd40c006a35bd97660 (patch)
tree613523b722b8bac789c25c0365bcd5f9c50e8a5c
parent32e41c230484586e6b8e51ca1d8ec876fd787842 (diff)
downloadpoky-f517fe8929ee256521e025bd40c006a35bd97660.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: 01440b4968ded30c1970c335fe1598b684527831) 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> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com> Signed-off-by: Steve Sakoman <steve@sakoman.com>
-rw-r--r--meta/recipes-support/curl/curl_7.82.0.bb4
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/recipes-support/curl/curl_7.82.0.bb b/meta/recipes-support/curl/curl_7.82.0.bb
index de69d3d53b..383cf415d9 100644
--- a/meta/recipes-support/curl/curl_7.82.0.bb
+++ b/meta/recipes-support/curl/curl_7.82.0.bb
@@ -79,6 +79,8 @@ PACKAGECONFIG:class-nativesdk = "ipv6 openssl proxy random threaded-resolver ver
79PACKAGECONFIG[ares] = "--enable-ares,--disable-ares,c-ares,,,threaded-resolver" 79PACKAGECONFIG[ares] = "--enable-ares,--disable-ares,c-ares,,,threaded-resolver"
80PACKAGECONFIG[brotli] = "--with-brotli,--without-brotli,brotli" 80PACKAGECONFIG[brotli] = "--with-brotli,--without-brotli,brotli"
81PACKAGECONFIG[builtinmanual] = "--enable-manual,--disable-manual" 81PACKAGECONFIG[builtinmanual] = "--enable-manual,--disable-manual"
82# Don't use this in production
83PACKAGECONFIG[debug] = "--enable-debug,--disable-debug"
82PACKAGECONFIG[dict] = "--enable-dict,--disable-dict," 84PACKAGECONFIG[dict] = "--enable-dict,--disable-dict,"
83PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls" 85PACKAGECONFIG[gnutls] = "--with-gnutls,--without-gnutls,gnutls"
84PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher," 86PACKAGECONFIG[gopher] = "--enable-gopher,--disable-gopher,"
@@ -115,9 +117,7 @@ EXTRA_OECONF = " \
115 --enable-crypto-auth \ 117 --enable-crypto-auth \
116 --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \ 118 --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt \
117 --without-libpsl \ 119 --without-libpsl \
118 --enable-debug \
119 --enable-optimize \ 120 --enable-optimize \
120 --disable-curldebug \
121 ${@'--without-ssl' if (bb.utils.filter('PACKAGECONFIG', 'gnutls mbedtls nss openssl', d) == '') else ''} \ 121 ${@'--without-ssl' if (bb.utils.filter('PACKAGECONFIG', 'gnutls mbedtls nss openssl', d) == '') else ''} \
122" 122"
123 123