diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-09-13 16:51:25 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-09-13 10:39:04 -0700 |
commit | 7ce3dcd5335981d6d339bd22592ef9fa72808692 (patch) | |
tree | a1ad7c2195746f9037488a7c21a9b3196b63b173 /versioncmp.c | |
parent | Makefile: drop support for curl < 7.9.8 (again) (diff) | |
download | tgif-7ce3dcd5335981d6d339bd22592ef9fa72808692.tar.xz |
http: drop support for curl < 7.18.0 (again)
In 644de29e220 (http: drop support for curl < 7.19.4, 2021-07-30) we
dropped support for curl < 7.19.4, so we can drop support for this
non-obvious dependency on curl < 7.18.0.
It's non-obvious because in curl's hex version notation 0x071800 is
version 7.24.0, *not* 7.18.0, so at a glance this patch looks
incorrect.
But it's correct, because the existing version check being removed
here is wrong. The check guards use of the following curl defines:
CURLPROXY_SOCKS4 7.10
CURLPROXY_SOCKS4A 7.18.0
CURLPROXY_SOCKS5 7.10
CURLPROXY_SOCKS5_HOSTNAME 7.18.0
I.e. the oldest version that has these is in fact 7.18.0, not
7.24.0. That we were checking 7.24.0 is just an mistake in
6d7afe07f29 (remote-http(s): support SOCKS proxies, 2015-10-26),
i.e. its author confusing base 10 and base 16.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'versioncmp.c')
0 files changed, 0 insertions, 0 deletions