git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] git-send-email: die if sendmail.* config is set

From
Jeff King <peff@peff.net>
Date
Jul 20, 2020, 17:33 UTC
Message-ID
<20200720173336.GC2045458@coredump.intra.peff.net>
In-Reply-To
<20200718202142.111015-1-sir@cmpwn.com>
On Sat, Jul 18, 2020 at 04:21:42PM -0400, Drew DeVault wrote:
> I've seen several people mis-configure git send-email on their first
> attempt because they set the sendmail.* config options - not
> sendemail.*. This patch detects this mistake and bails out with a
> friendly warning.

This basically claims the "sendmail.*" namespace for send-email. Not strictly, but if we're going to warn about anything set in it, it effectively shuts out other uses.

I'm OK with that, but if we're going to do so, should we perhaps just say "sendmail.* is an alias for sendemail.*"? Then rather than getting a warning, this mistake would Just Work.

-Peff
Previous: Junio C HamanoNext: Drew DeVault
Message 4 of 5 in “git-send-email: die if sendmail.* config is set”
  1. git-send-email: die if sendmail.* config is setDrew DeVault, Jul 18, 2020
  2. Junio C HamanoJul 18, 2020
  3. Junio C HamanoJul 19, 2020
  4. Jeff KingJul 20, 2020
  5. Drew DeVaultJul 20, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.