From: Junio C Hamano <gitster@pobox.com>
To: "brian m. carlson" <sandals@crustytoothpaste.net>
Cc: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
git@vger.kernel.org, "Ramsay Jones" <ramsay@ramsayjones.plus.com>,
"Jeff King" <peff@peff.net>
Subject: Re: [PATCH v2] sha256: add support for Nettle
Date: Sun, 10 Jul 2022 13:37:57 -0700 [thread overview]
Message-ID: <xmqqa69gr9ei.fsf@gitster.g> (raw)
In-Reply-To: <Yssyt40xrxjtaL6m@tapette.crustytoothpaste.net> (brian m. carlson's message of "Sun, 10 Jul 2022 20:12:39 +0000")
"brian m. carlson" <sandals@crustytoothpaste.net> writes:
> Yeah, I think given the fact that many folks use autoconf, it's
> beneficial to allow multiple options and just choose the one we like the
> most.
Not that we any of *_SHA256 set in the configure script ;-) It still
is worth noting that the Makefile is prepared to deal with configure
scripts written in the most simple way, i.e. test for any backend
independent of other possible backends.
There can be at least three possible arrangements, and I think the
current one is the most sensible.
(1) We can allow multiple to be set and let Makefile define
precedence. "If you do not choose any, then we fall back to
..." will fall out as a natural consequence.
If somebody wants to write autoconf support, they do not have
to reinvent the precedence order. They can just check for the
presence of individual libraries independently from each other.
This is what we have today.
(2) We can allow multiple to be set and let Makefile define
precedence. But a misguided autoconf author can invent their
own precedence order that may not be compatible with what the
Makefile has.
This allows the autoconf author weird satisifaction for being
different from what Makefile already decided for the sake of
being different.
This would be unfortunate, and we should watch out when we
review patches to add autoconf support for *_SHA256 variables.
(3) We can allow only one to be set and otherwise error out.
If somebody wants to write autoconf support, they have to come
up with the precedence order, or support the "--with[out]-X"
thing to customize, in order to ensure that the configure
script picks exactly one.
Because not everybody has nor need to have enough knowledge to
choose one among the multiple choices available to them, this
is hostile to those who do not use configure.
Not an improvement over what we have today.
next prev parent reply other threads:[~2022-07-10 20:38 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-05 23:05 [PATCH] sha256: add support for Nettle brian m. carlson
2022-07-06 1:00 ` Ramsay Jones
2022-07-06 1:08 ` brian m. carlson
2022-07-06 8:23 ` Jeff King
2022-07-06 8:45 ` Ævar Arnfjörð Bjarmason
2022-07-06 9:23 ` Jeff King
2022-07-06 10:39 ` brian m. carlson
2022-07-06 17:49 ` Junio C Hamano
2022-07-06 23:05 ` Junio C Hamano
2022-07-07 6:43 ` Junio C Hamano
2022-07-07 16:18 ` Junio C Hamano
2022-07-10 13:29 ` [PATCH v2] " brian m. carlson
2022-07-10 14:41 ` Ævar Arnfjörð Bjarmason
2022-07-10 16:39 ` Junio C Hamano
2022-07-10 20:12 ` brian m. carlson
2022-07-10 20:37 ` Junio C Hamano [this message]
2022-08-31 5:34 ` Reza Mahdi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqa69gr9ei.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
--cc=ramsay@ramsayjones.plus.com \
--cc=sandals@crustytoothpaste.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).