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

[StGit PATCH v2 0/6] add support for git send-email

From
ACAlex Chiang <achiang@hp.com>
Date
Dec 2, 2009, 00:46 UTC
Message-ID
<20091202003503.7737.51579.stgit@bob.kio>

This is v2 of the series that starts teaching stg mail how to call git send-email.

I made all the changes that Karl recommended, and made sure to run the t1900-mail.sh test at every point in the series (passes successfully).

I also experimented with adding another test case for --git mode, basically duplicating t1900-mail.sh, and then adding the --git argument wherever it made sense.

However, that resulted in failure of the last 3 test cases, which is due to the fact that we no longer parse To/Cc/Bcc command line args in --git mode, and the resulting mbox file was missing the expected recipient addresses.

I played around with that for a while, thinking that I could use git send-email --dry-run to do something equivalent, but then realized that git send-email's run-run mode is definitely not analogous to stg mail's --mbox mode.

The upshot is that in stg mail, --git and --mbox don't interact well, and the resulting mbox file will lack the recipients. This might be fixed in the future if we teach git send-email how to generate mbox files, but then we introduce a versioning problem.

So let's just accept this wart for now, and say, if you want an mbox file generated, don't use --git. That seems reasonable to me.

This mail was sent with the following command line:
	./stg mail --git -a -e --auto -v v2 --prefix=StGit
	--to=catalin.marinas@gmail.com --cc=git

Note that the --cc= contains an alias for the git mailing list that I defined in my ~/.mutt.aliases file (and specified in .gitconfig -> sendemail.aliasesfile and sendemail.aliasfiletype.

Thanks, /ac

---
Alex Chiang (6):
      stg mail: Refactor __send_message and friends
      stg mail: reorder __build_[message|cover] parameters
      stg mail: make __send_message do more
      stg mail: factor out __update_header
      stg mail: add basic support for git send-email
      stg mail: don't parse To/Cc/Bcc in --git mode
 stgit/commands/mail.py |  196 +++++++++++++++++++++++++++---------------------
 1 files changed, 112 insertions(+), 84 deletions(-)
Next: Alex Chiang
Message 1 of 19 in “add support for git send-email”
  1. 0/6 add support for git send-emailAlex Chiang, Dec 2, 2009
  2. 1/6 stg mail: Refactor __send_message and friendsAlex Chiang, Dec 2, 2009
  3. Karl WibergDec 2, 2009
  4. Alex ChiangDec 3, 2009
  5. 1/6 stg mail: Refactor __send_message and friendsAlex Chiang, Dec 3, 2009
  6. 2/6 stg mail: reorder __build_[message|cover] parametersAlex Chiang, Dec 2, 2009
  7. 3/6 stg mail: make __send_message do moreAlex Chiang, Dec 2, 2009
  8. Karl WibergDec 2, 2009
  9. Alex ChiangDec 3, 2009
  10. Karl WibergDec 4, 2009
  11. 4/6 stg mail: factor out __update_headerAlex Chiang, Dec 2, 2009
  12. 5/6 stg mail: add basic support for git send-emailAlex Chiang, Dec 2, 2009
  13. 6/6 stg mail: don't parse To/Cc/Bcc in --git modeAlex Chiang, Dec 2, 2009
  14. Karl WibergDec 2, 2009
  15. Alex ChiangDec 3, 2009
  16. Karl WibergDec 2, 2009
  17. Catalin MarinasDec 2, 2009
  18. Catalin MarinasDec 6, 2009
  19. Karl WibergDec 7, 2009

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.