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

Re: git-send-email: smtpserver in $HOME

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 16, 2021, 19:16 UTC
Message-ID
<xmqqy2fnq0vx.fsf@gitster.c.googlers.com>
In-Reply-To
<YCwXf1CA0Xyw/B88@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
> Sure, but you cannot say "does it refer to a local file" for a
> non-absolute path.
Hmph, why not?  I would expect that this would work as a valid way
	$ git send-email --smtp-server=./my-phoney-smtp

to test a server substitute (perhaps for testing). The only reason why it does not is because file_name_is_absolute() check would not like it.

> And that is the source of the problem, IMHO: there is
> no way to signal "this is a command I expect to be executed" except by
> using an absolute path.
Yes.
> Or do you mean that we should see if $smtp_server exists in the PATH,
> and if so prefer it over a network hostname?

We can certainly go in that direction, too. A new --smtp-program option would be a cleaner way to solve it, though, as you said elsewhere.

Previous: Jeff KingNext: Jeff King
Message 11 of 13 in “git-send-email: smtpserver in $HOME”
  1. Jan “Khardix” StaněkFeb 15, 2021
  2. Junio C HamanoFeb 16, 2021
  3. Jan “Khardix” StaněkFeb 16, 2021
  4. Junio C HamanoFeb 16, 2021
  5. Jan “Khardix” StaněkFeb 16, 2021
  6. Chris TorekFeb 18, 2021
  7. Andreas SchwabFeb 18, 2021
  8. Jeff KingFeb 16, 2021
  9. Junio C HamanoFeb 16, 2021
  10. Jeff KingFeb 16, 2021
  11. Junio C HamanoFeb 16, 2021
  12. Jeff KingFeb 16, 2021
  13. Jan “Khardix” StaněkFeb 16, 2021

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.