Thread (1 message) 1 message, 1 author, 2016-06-15

Re: Minimum libCurl version for git

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:46:27

Mike Ralphson [off-list ref] writes:
2009/3/20 Junio C Hamano [off-list ref]:
quoted
We can have these two macro definitions on our side

       #if curl older than 7.10.8
       #define CURLINFO_RESPONSE_CODE CURLINFO_HTTP_CODE
       #endif

       #if curl older than 7.9.7
       #define CURLOPT_READDATA CURLOPT_INFILE
       #endif

for backward compatibility, while writing our code to the recent API by
using CURLINFO_RESPONSE_CODE and CURLOPT_READDATA, and people with older
curl would not have to suffer a bit.
See? That's why they pay you the big maintainer-bucks... 8-)
The big maintainer-buck is called zero cents.  I am only paid with the
freedom to spend 20% of my day-job time on git [*1*].

In any case, "Write to the latest API, support older platforms with
backward compatibility wrapper as necessary" is a good practice employed
by many successful projects, including the kernel, and I think it would
apply here nicely.
quoted
Mike, I'd say we declare 7.9.3 as the floor and go from there.  That's
your #3, I think.
Short patch series to follow, though maybe not today.
Thanks.

[Footnote]

*1* ... which is still generous of my employer and NEC, given the current
economic climate, but I wouldn't exactly call that "big bucks" ;-).
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help