From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Wed, 16 Mar 2022 14:47:17 GMT
Subject: Re: [PATCH 2/2] git-curl-compat.h: addition of all symbols defined by curl
Message-ID: <220316.86czimdkb0.gmgdl@evledraar.gmail.com>
In-Reply-To: <20220316140106.14678-3-gitter.spiros@gmail.com>


On Wed, Mar 16 2022, Elia Pinto wrote:

Per the comment on v1 I really think we should not do this...

> This file was produced from a modified version of symbols.pl
> (https://github.com/curl/curl/blob/master/docs/libcurl/symbols.pl) and
> by manually adding the previous comments describing the dates of release
> of some curl versions not currently reported in the symbols-in-versions.
>
> To do this the symbols are listed in the order defined in the file
> symbols-in-versions rather than as they were previously inserted based
> on release dates.
>
> Most of these symbols are not used by git today. However, inserting
> them all starting from an automatic tool makes it largely unnecessary
> to update this file and therefore reduces the possibility
> of introducing possible errors in the future.
>
> Helped-by: Junio C Hamano <gitster@pobox.com>
> Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>

But more generally, re your reply on v1:

    In the end I did not understand if you think it is worthwhile that i
    do a reroll of  the patch[...]

Better than a re-roll is replying to outstanding concerns about your
patches. I.e. this, which still applies here
https://lore.kernel.org/git/220315.86pmmndmre.gmgdl@evledraar.gmail.com/