From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 80DCC158094 for ; Sat, 3 Sep 2022 00:34:21 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 1CCCCE095A; Sat, 3 Sep 2022 00:34:20 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 06495E095A for ; Sat, 3 Sep 2022 00:34:20 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id C51C43410CB for ; Sat, 3 Sep 2022 00:34:18 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 57947526 for ; Sat, 3 Sep 2022 00:34:17 +0000 (UTC) From: "Sam James" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Sam James" Message-ID: <1662165244.bd4d42f83c774c36bf879a5b7ec89d373546743e.sam@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: net-misc/curl/ X-VCS-Repository: repo/gentoo X-VCS-Files: net-misc/curl/curl-7.85.0-r1.ebuild net-misc/curl/curl-7.85.0.ebuild X-VCS-Directories: net-misc/curl/ X-VCS-Committer: sam X-VCS-Committer-Name: Sam James X-VCS-Revision: bd4d42f83c774c36bf879a5b7ec89d373546743e X-VCS-Branch: master Date: Sat, 3 Sep 2022 00:34:17 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 93e658cc-c215-4989-b76b-f26ebbabbf96 X-Archives-Hash: 36ba6a5c488a471536eedc7879f23824 commit: bd4d42f83c774c36bf879a5b7ec89d373546743e Author: Sam James gentoo org> AuthorDate: Thu Sep 1 21:42:26 2022 +0000 Commit: Sam James gentoo org> CommitDate: Sat Sep 3 00:34:04 2022 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bd4d42f8 net-misc/curl: drop USE=threads USE=threads is a terrible USE flag (not the fault of net-misc/curl or its maintainer) and we're looking to get rid of it entirely. This is motivated further by the fact users have USE=threads in make.conf seeking performance improvements and such (which is understandable) but as per below, it can lead to conflicting and inconsistent effects. (In particular, anyone with USE="threads" in make.conf (don't do this for the reasons below!) will end up with a REQUIRED_USE issue between the now-default USE=adns and USE=threads). There are a few possibilities with threads support: 1. Upstreams always have threads support (hence USE=threads is only affecting a "handful" of packages anyway, not every single package, as many will just always be using pthreads and there's no way of disabling it); 2. The "threads" toggle means enabling some experimental or legacy implementation (think e.g. dev-lang/perl, which uses USE=ithreads for this, as it breaks ABI) (also applies to net-misc/curl given USE=threads doesn't help with the adns functionality many applications desire, see linked bug); 3. Not having "threads" on enables some experimental or legacy implementation again (this *again* applies to net-misc/curl given USE="-adns -threads" leaves us with a dodgy implementation as well). Commit 60b7aaa7a70149751446237d7207ff1ca20c256e enabled adns by default (IUSE=+adns). This commit drops USE=threads entirely and makes the default pthreads with adns opt-in with IUSE=+adns still. Now, the ebuild previously had REQUIRED_USE="threads? ( !adns )". I can't see a reason for this other than making the previous situation "meaningful", i.e. allowing people to decisively choose which is used at runtime, as while curl can build with both threads and adns, I think adns ends up preferred at runtime. With this change and given all of the above, we actually want adns to take priority if it's set, as threads will just be the default, so drop the REQUIRED_USE. Bug: https://bugs.gentoo.org/699632 See: 60b7aaa7a70149751446237d7207ff1ca20c256e Signed-off-by: Sam James gentoo.org> Closes: https://github.com/gentoo/gentoo/pull/27111 Signed-off-by: Sam James gentoo.org> net-misc/curl/{curl-7.85.0.ebuild => curl-7.85.0-r1.ebuild} | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/net-misc/curl/curl-7.85.0.ebuild b/net-misc/curl/curl-7.85.0-r1.ebuild similarity index 96% rename from net-misc/curl/curl-7.85.0.ebuild rename to net-misc/curl/curl-7.85.0-r1.ebuild index 15c6bef88efc..dfa9c49ff480 100644 --- a/net-misc/curl/curl-7.85.0.ebuild +++ b/net-misc/curl/curl-7.85.0-r1.ebuild @@ -13,15 +13,13 @@ SRC_URI="https://curl.haxx.se/download/${P}.tar.xz LICENSE="curl" SLOT="0" KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~loong ~m68k ~mips ~ppc ~ppc64 ~riscv ~s390 ~sparc ~x86 ~x64-cygwin ~amd64-linux ~x86-linux ~ppc-macos ~x64-macos ~sparc-solaris ~sparc64-solaris ~x64-solaris ~x86-solaris" -IUSE="+adns alt-svc brotli +ftp gnutls gopher hsts +http2 idn +imap ipv6 kerberos ldap mbedtls nss +openssl +pop3 +progress-meter rtmp samba +smtp ssh ssl sslv3 static-libs test telnet +tftp threads zstd" +IUSE="+adns alt-svc brotli +ftp gnutls gopher hsts +http2 idn +imap ipv6 kerberos ldap mbedtls nss +openssl +pop3 +progress-meter rtmp samba +smtp ssh ssl sslv3 static-libs test telnet +tftp zstd" IUSE+=" curl_ssl_gnutls curl_ssl_mbedtls curl_ssl_nss +curl_ssl_openssl" IUSE+=" nghttp3 quiche" VERIFY_SIG_OPENPGP_KEY_PATH="${BROOT}"/usr/share/openpgp-keys/danielstenberg.asc -# c-ares must be disabled for threads -# only one default ssl provider can be enabled +# Only one default ssl provider can be enabled REQUIRED_USE=" - threads? ( !adns ) ssl? ( ^^ ( curl_ssl_gnutls @@ -202,8 +200,8 @@ multilib_src_configure() { --enable-proxy --disable-sspi $(use_enable static-libs static) - $(use_enable threads threaded-resolver) - $(use_enable threads pthreads) + --enable-pthreads + --enable-threaded-resolver --disable-versioned-symbols --without-amissl --without-bearssl