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

Re: [PATCH 8/8] Docs: config: send-email config options included

From
Jeff King <peff@peff.net>
Date
Sep 28, 2008, 03:24 UTC
Message-ID
<20080928032407.GC15696@sigill.intra.peff.net>
In-Reply-To
<1222564196-84202-8-git-send-email-mfwitten@mit.edu>
On Sat, Sep 27, 2008 at 08:09:56PM -0500, Michael Witten wrote:
> I just copied what is in the send-email man text.

NAK, this makes things much worse by creating two places which must be updated with any changes. Shouldn't doing it the right way be as simple as something like the patch below?

Though it looks like imap.* simply says "See git-imap-send(1)" in config.txt, which would also be OK by me (and makes sense especially if we drop most of the sendemail.* description in favor of just mentioning the config option near the matching command-line option).

---
 Documentation/config.txt           |    2 +
 Documentation/git-send-email.txt   |   48 +-----------------------------------
 Documentation/sendemail-config.txt |   47 +++++++++++++++++++++++++++++++++++
 3 files changed, 50 insertions(+), 47 deletions(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index c806749..0bb2621 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -1173,6 +1173,8 @@ receive.denyNonFastForwards::
 	even if that push is forced. This configuration variable is
 	set when initializing a shared repository.
 
+include::sendemail-config.txt[]
+
 transfer.unpackLimit::
 	When `fetch.unpackLimit` or `receive.unpackLimit` are
 	not set, the value of this variable is used instead.
diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 3c3e1b0..465cf56 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -195,53 +195,7 @@ The --to option must be repeated for each user you want on the to list.
 
 CONFIGURATION
 -------------
-sendemail.identity::
-	The default configuration identity. When specified,
-	'sendemail.<identity>.<item>' will have higher precedence than
-	'sendemail.<item>'. This is useful to declare multiple SMTP
-	identities and to hoist sensitive authentication information
-	out of the repository and into the global configuration file.
-
-sendemail.aliasesfile::
-	To avoid typing long email addresses, point this to one or more
-	email aliases files.  You must also supply 'sendemail.aliasfiletype'.
-
-sendemail.aliasfiletype::
-	Format of the file(s) specified in sendemail.aliasesfile. Must be
-	one of 'mutt', 'mailrc', 'pine', or 'gnus'.
-
-sendemail.to::
-	Email address (or alias) to always send to.
-
-sendemail.cccmd::
-	Command to execute to generate per patch file specific "Cc:"s.
-
-sendemail.bcc::
-	Email address (or alias) to always bcc.
-
-sendemail.chainreplyto::
-	Boolean value specifying the default to the '--chain_reply_to'
-	parameter.
-
-sendemail.smtpserver::
-	Default SMTP server to use.
-
-sendemail.smtpserverport::
-	Default SMTP server port to use.
-
-sendemail.smtpuser::
-	Default SMTP-AUTH username.
-
-sendemail.smtppass::
-	Default SMTP-AUTH password.
-
-sendemail.smtpencryption::
-	Default encryption method.  Use 'ssl' for SSL (and specify an
-	appropriate port), or 'tls' for TLS.  Takes precedence over
-	'smtpssl' if both are specified.
-
-sendemail.smtpssl::
-	Legacy boolean that sets 'smtpencryption=ssl' if enabled.
+include::sendemail-config.txt[]
 
 Author
 ------
diff --git a/Documentation/sendemail-config.txt b/Documentation/sendemail-config.txt
new file mode 100644
index 0000000..7409534
--- /dev/null
+++ b/Documentation/sendemail-config.txt
@@ -0,0 +1,47 @@
+sendemail.identity::
+	The default configuration identity. When specified,
+	'sendemail.<identity>.<item>' will have higher precedence than
+	'sendemail.<item>'. This is useful to declare multiple SMTP
+	identities and to hoist sensitive authentication information
+	out of the repository and into the global configuration file.
+
+sendemail.aliasesfile::
+	To avoid typing long email addresses, point this to one or more
+	email aliases files.  You must also supply 'sendemail.aliasfiletype'.
+
+sendemail.aliasfiletype::
+	Format of the file(s) specified in sendemail.aliasesfile. Must be
+	one of 'mutt', 'mailrc', 'pine', or 'gnus'.
+
+sendemail.to::
+	Email address (or alias) to always send to.
+
+sendemail.cccmd::
+	Command to execute to generate per patch file specific "Cc:"s.
+
+sendemail.bcc::
+	Email address (or alias) to always bcc.
+
+sendemail.chainreplyto::
+	Boolean value specifying the default to the '--chain_reply_to'
+	parameter.
+
+sendemail.smtpserver::
+	Default SMTP server to use.
+
+sendemail.smtpserverport::
+	Default SMTP server port to use.
+
+sendemail.smtpuser::
+	Default SMTP-AUTH username.
+
+sendemail.smtppass::
+	Default SMTP-AUTH password.
+
+sendemail.smtpencryption::
+	Default encryption method.  Use 'ssl' for SSL (and specify an
+	appropriate port), or 'tls' for TLS.  Takes precedence over
+	'smtpssl' if both are specified.
+
+sendemail.smtpssl::
+	Legacy boolean that sets 'smtpencryption=ssl' if enabled.
Previous: Michael WittenNext: Michael Witten
Message 9 of 45 in “Docs: send-email's usage text and man page mention same options”
  1. 1/8 Docs: send-email's usage text and man page mention same optionsMichael Witten, Sep 28, 2008
  2. 2/8 Docs: send-email usage text much sexierMichael Witten, Sep 28, 2008
  3. 3/8 Docs: send-email: Man page option orderingMichael Witten, Sep 28, 2008
  4. 4/8 send-email: change --no-validate to boolean --[no-]validateMichael Witten, Sep 28, 2008
  5. 5/8 Docs: send-email: --chain_reply_to -> --[no-]chain-reply-toMichael Witten, Sep 28, 2008
  6. 6/8 Docs: Arranged config options in man pageMichael Witten, Sep 28, 2008
  7. 7/8 Docs: send-email: Added all config variables to man endMichael Witten, Sep 28, 2008
  8. 8/8 Docs: config: send-email config options includedMichael Witten, Sep 28, 2008
  9. Jeff KingSep 28, 2008
  10. Michael WittenSep 28, 2008
  11. Jeff KingSep 28, 2008
  12. Jeff KingSep 28, 2008
  13. Michael WittenSep 28, 2008
  14. Jeff KingSep 28, 2008
  15. Jeff KingSep 28, 2008
  16. Michael WittenSep 28, 2008
  17. bash completion: Add --[no-]-validate to "git send-email"Teemu Likonen, Sep 28, 2008
  18. Teemu LikonenSep 28, 2008
  19. Jakub NarebskiSep 28, 2008
  20. Michael WittenSep 28, 2008
  21. Jakub NarebskiSep 29, 2008
  22. Jeff KingSep 29, 2008
  23. 6/9 Docs: send-email: Remove unnecessary config variable descriptionMichael Witten, Sep 29, 2008
  24. 7/9 send-email: Completely replace --signed-off-cc with --signed-off-by-ccMichael Witten, Sep 29, 2008
  25. 8/9 Docs: send-email: Create logical groupings for --help textMichael Witten, Sep 29, 2008
  26. 9/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 29, 2008
  27. Jeff KingSep 29, 2008
  28. Miklos VajnaSep 28, 2008
  29. Michael WittenSep 28, 2008
  30. Jeff KingSep 29, 2008
  31. 8/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 29, 2008
  32. 9/9 send-email: signedoffcc -> signedoffbycc, but handle bothMichael Witten, Sep 29, 2008
  33. Jeff KingSep 29, 2008
  34. 1/9 Docs: send-email's usage text and man page mention same optionsMichael Witten, Sep 30, 2008
  35. 2/9 Docs: send-email usage text much sexierMichael Witten, Sep 30, 2008
  36. 3/9 Docs: send-email: Man page option orderingMichael Witten, Sep 30, 2008
  37. 4/9 send-email: change --no-validate to boolean --[no-]validateMichael Witten, Sep 30, 2008
  38. 5/9 Docs: send-email: --chain_reply_to -> --[no-]chain-reply-toMichael Witten, Sep 30, 2008
  39. 6/9 Docs: send-email: Remove unnecessary config variable descriptionMichael Witten, Sep 30, 2008
  40. 7/9 Docs: send-email: Create logical groupings for --help textMichael Witten, Sep 30, 2008
  41. 8/9 Docs: send-email: Create logical groupings for man textMichael Witten, Sep 30, 2008
  42. 9/9 send-email: signedoffcc -> signedoffbycc, but handle bothMichael Witten, Sep 30, 2008
  43. Jeff KingOct 1, 2008
  44. Michael WittenOct 1, 2008
  45. Shawn O. PearceOct 1, 2008

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.