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

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

From
Jan “Khardix” Staněk <khardix@gmail.com>
Date
Feb 16, 2021, 22:31 UTC
Message-ID
<CAG+K25NhVdM6f2sXG3bDODH2ZEBXyfPuoXYtHoeC1X2YFOwnRQ@mail.gmail.com>
In-Reply-To
<YCwXf1CA0Xyw/B88@coredump.intra.peff.net>
On 2021-02-16, Jeff King wrote:
> … It matches alias config, credential config, and perhaps others
> as "treat this as a shell command".

I will have to take closer look at the '!' prefix in other git parts, but I like that approach.

> (I'd prefer to just have $smtp_program or similar as that signal
> instead, but it doesn't seem worth trying to retrofit it now).

That would also be my preference, but it would be a larger change and I'm not currently confident enough to go this route.

From the points and ideas raised (and btw, thanks a lot for those), the easiest one seems to be treating $smtp_server as path options, the cleanest one a new $smtp_program options, and the middle ground being adding support for '!' prefix. Unless someone beats me to it, I will try to throw together a patch, aiming for now for the '!' prefix and falling back to path option – as this is potentially my first contribution here, I'll see how it goes. -- Jan Staněk – Khardix

Previous: Jeff King
Message 13 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.