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

[PATCH v3 0/2] format-patch: teach format.notes config option

From
Denton Liu <liu.denton@gmail.com>
Date
May 10, 2019, 18:37 UTC
Message-ID
<cover.1557513353.git.liu.denton@gmail.com>
In-Reply-To
<cover.1557327652.git.liu.denton@gmail.com>
Hi Beat, thanks for catching the style errors. This version fixes those.
Changes since v2:
* Fixed if-else code style
* Fixed typoed errors in 2/2 log message
Changes since v1:
* Made format.notes accept a notes ref instead of a boolean
Denton Liu (2):
  git-format-patch.txt: document --no-notes option
  format-patch: teach format.notes config option
 Documentation/config/format.txt    | 13 ++++++
 Documentation/git-format-patch.txt |  7 ++-
 builtin/log.c                      | 18 +++++++-
 t/t4014-format-patch.sh            | 70 ++++++++++++++++++++++++++++++
 4 files changed, 106 insertions(+), 2 deletions(-)
Range-diff against v2:
1:  4c3535f25b = 1:  4c3535f25b git-format-patch.txt: document --no-notes option
2:  fe674bf63e ! 2:  df864c4adf format-patch: teach format.notes config option
    @@ -8,8 +8,8 @@
         that they may forget to include it and generate a patch series without
         notes.
     
    -    Teach git-format-patch the `format.notes` config option its value is a
    -    notes ref that will be automatically be appended. The special value of
    +    Teach git-format-patch the `format.notes` config option. Its value is a
    +    notes ref that will be automatically appended. The special value of
         "standard" can be used to specify the standard notes. This option is
         overridable with the `--no-notes` option in case a user wishes not to
         append notes.
    @@ -71,9 +71,9 @@
     +		struct strbuf buf = STRBUF_INIT;
     +
     +		rev->show_notes = 1;
    -+		if (!strcmp(value, "standard"))
    ++		if (!strcmp(value, "standard")) {
     +			rev->notes_opt.use_default_notes = 1;
    -+		else {
    ++		} else {
     +			strbuf_addstr(&buf, value);
     +			expand_notes_ref(&buf);
     +			string_list_append(&rev->notes_opt.extra_notes_refs,
-- 
2.21.0.1049.geb646f7864
Previous: Beat BolliNext: Denton Liu
Message 11 of 15 in “format-patch: teach format.notes config option”
  1. 0/2 format-patch: teach format.notes config optionDenton Liu, Apr 27, 2019
  2. 1/2 git-format-patch.txt: document --no-notes optionDenton Liu, Apr 27, 2019
  3. 2/2 format-patch: teach format.notes config optionDenton Liu, Apr 27, 2019
  4. Junio C HamanoMay 7, 2019
  5. 0/2 format-patch: teach format.notes config optionDenton Liu, May 8, 2019
  6. 1/2 git-format-patch.txt: document --no-notes optionDenton Liu, May 8, 2019
  7. 2/2 format-patch: teach format.notes config optionDenton Liu, May 8, 2019
  8. Beat BolliMay 8, 2019
  9. Denton LiuMay 8, 2019
  10. Beat BolliMay 8, 2019
  11. 0/2 format-patch: teach format.notes config optionDenton Liu, May 10, 2019
  12. 1/2 git-format-patch.txt: document --no-notes optionDenton Liu, May 10, 2019
  13. 2/2 format-patch: teach format.notes config optionDenton Liu, May 10, 2019
  14. Junio C HamanoMay 13, 2019
  15. Denton LiuMay 14, 2019

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.