]> git.ipfire.org Git - thirdparty/git.git/commitdiff
INSTALL: bump libcurl version to 7.21.3
authorJeff King <peff@peff.net>
Tue, 2 Apr 2024 20:06:00 +0000 (16:06 -0400)
committerJohannes Schindelin <johannes.schindelin@gmx.de>
Wed, 10 Apr 2024 17:24:48 +0000 (19:24 +0200)
Our documentation claims we support curl versions back to 7.19.5. But we
can no longer compile with that version since adding an unconditional
use of CURLOPT_RESOLVE in 511cfd3bff (http: add custom hostname to IP
address resolutions, 2022-05-16). That feature wasn't added to libcurl
until 7.21.3.

We could add #ifdefs to make this work back to 7.19.5. But given that
nobody noticed the compilation failure in the intervening two years, it
makes more sense to bump the version in the documentation to 7.21.3
(which is itself over 13 years old).

We could perhaps go forward even more (which would let us drop some
cruft from git-curl-compat.h), but this should be an obviously safe
jump, and we can move forward later.

Note that user-visible syntax for CURLOPT_RESOLVE has grown new features
in subsequent curl versions. Our documentation mentions "+" and "-"
entries, which require more recent versions than 7.21.3. We could
perhaps clarify that in our docs, but it's probably not worth cluttering
them with restrictions of ancient curl versions.

This is a backport of c28ee09503 (INSTALL: bump libcurl version to
7.21.3, 2024-04-02) into the `maint-2.39` branch.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
INSTALL

diff --git a/INSTALL b/INSTALL
index 8dd577e1020bcca6351abde0d876cc2dfa11493c..113920ff0612e6b7630bfa9eeed57b3a2d856d28 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -144,7 +144,7 @@ Issues of note:
          not need that functionality, use NO_CURL to build without
          it.
 
-         Git requires version "7.19.5" or later of "libcurl" to build
+         Git requires version "7.21.3" or later of "libcurl" to build
          without NO_CURL. This version requirement may be bumped in
          the future.