From: Junio C Hamano <gitster@pobox.com>
Date: Sun, 10 Jul 2022 16:39:18 GMT
Subject: Re: [PATCH v2] sha256: add support for Nettle
Message-ID: <xmqqwnckrkg9.fsf@gitster.g>
In-Reply-To: <220710.86mtdh81ty.gmgdl@evledraar.gmail.com>

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:

> This just carries forward existing technical debt, but it's unfortunate
> that we don't catch OPENSSL_SHA256 overridding NETTLE_SHA256, and error
> if both are defined.

You are mistaken, unless I am ;-)  

Allowing users to list whatever is available, instead of requiring
all users to choose only one, is a deliberate feature in the
arrangement, so it is not unfortunate and it would be breaking
end-user expectation if we gave an error when more than one is
given (and it would be easier to write and maintain autoconf rules
for the feature---we do not want to have two places that makes
decisions on precedence).