Guix warns as follows when curl/fixed is exported via define-public.

guix repl: warning: ambiguous package specification `curl'
guix repl: warning: choosing curl@8.20.0 from gnu/packages/curl.scm:69:2

* gnu/packages/curl.scm (curl/fixed): Unexport.

Merges: https://codeberg.org/guix/guix/pulls/10629
Reviewed-by: bdunahu <bdunahu@operationnull.com>
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/curl.scm (trurl): Fix build.
[source]<patches>: Add patches.
* gnu/packages/patches/trurl-fix-tests-for-newer-libcurl.patch,
gnu/packages/patches/trurl-mark-additional-tests-case-sensitive.patch: New
files.
* gnu/local.mk (dist_patch_DATA): Register new files.

Change-Id: I63ef3ad2d70effe21febcfdfc99c238196d741d0

* gnu/packages/curl.scm (gnurl): Delete variable.

curl 8.20 has a bug which causes high cpu usage
when built with `enable-threaded-resolver`.

* gnu/packages/patches/curl-fix-wakeup-consumption.patch: New file.
* gnu/local.mk (dist_patch_DATA): Register it.
* gnu/packages/curl.scm (curl/fixed): New variable.
  (curl)[replacement]: Use it here.

Fixes: https://codeberg.org/guix/guix/issues/10284
Merges: https://codeberg.org/guix/guix/pulls/10330
Reviewed-by: Yelninei <yelninei@tutamail.com>
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

These tests fail with the newer nghttp2 because of the failure
to set SO_REUSEPORT causes the server to exit.

* gnu/packages/curl.scm (curl)[arguments]<#:phases>{skip-tests/hurd}:
  Skip tests 1541, 1700, 1701, 1702, 2402, 2403 and 2404.

Merges: https://codeberg.org/guix/guix/pulls/10095
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/patches/curl-use-ssl-cert-env.patch: Refresh patch.
* gnu/packages/patches/curl-CVE-2024-8096.patch: Delete patch.
* gnu/local.mk (dist_patch_DATA): Deregister it.
* gnu/packages/curl.scm (curl): Update to 8.20.
  [origin]: Remove curl-CVE-2024-8096.patch.
  [native-inputs]: Add openssl.
  [#:phases]: Add 'sanitize-libcurl.pc phase.

Merges: https://codeberg.org/guix/guix/pulls/7919
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/curl.scm (curl)[arguments]<#:phases>
  {'check}: Do not replace.
  {'patch-runtests}: New phase.
  {'skip-tests}: Add phase for arm targets.
  {'skip-tests/hurd}: Add phase for hurd targets.

Co-authored-by: Yelninei <yelninei@tutamail.com>
Merges: https://codeberg.org/guix/guix/pulls/7919
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/curl.scm (curlie)[home-page]: Fix URL.

Merges: guix/guix!9502
Signed-off-by: Sharlatan Hellseher <sharlatanus@gmail.com>

This commit was made by running this command:

  sed -e's/modify-inputs (package-\([a-z-]*\)inputs [a-zA-Z0-9-]\+)/modify-inputs \1inputs/g' -i gnu/packages/*.scm

… and then reverting individual hunks where the change would trigger unbound
variable warnings or other issues (such as ‘native-inputs’ is bound in the
body of the ‘inputs’ field, but it refers to the ‘native-inputs’ thunk defined
just above).

Change-Id: I6d94819f2809313fa1fbefc61897502ee7d66fab

This commit was made by running this command:

  sed -e's/substitute-keyword-arguments (package-arguments [a-zA-Z0-9-]\+)/substitute-keyword-arguments arguments/g' -i gnu/packages/*.scm

… and then:

  1. reverting changes from ‘gnu/packages/rust.scm’ and
     ‘gnu/packages/java.scm’ since they would incur derivation changes and/or
     breakage;
  2. reverting the change for ‘gcc-final’ in ‘gnu/packages/commencement.scm’;
  3. reverting the change for ‘onnx-optimizer’, ‘openquest’, and ‘certbot’,
     which use ‘substitute-keyword-arguments’ for arguments that are not
     inherited (and thus ‘arguments’ would be unbound);
  4. reverting the change for ‘insight-toolkit-legacy’ and ‘wine64-staging’
     which make bogus assumptions about inherited arguments.

Change-Id: I122a7cf517b6b63cae38944b5d33ade4b1f5a89c