threads / patch / 12192

patch, 4 partsformat-patch header improvements, --cc

Subject: [PATCH 0/4] format-patch header improvements, --cc

## tl;dr

One message between Feb 19, 2008 and Feb 19, 2008. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Daniel Barkalow· Feb 19, 2008, 07:40 UTC · lore
This series improves the handling of extra headers in format-patch:
 - extra headers should get tested in the test suite
 - it shouldn't be necessary to have \n in the config file in order to 
   have multiple extra headers. (It's actually challenging to test having 
   newlines in values in the config file, let alone set things up that way 
   for an end user...)
 - multiple "to:" headers, or multiple "cc:" headers should be combined by 
   the program, without requiring the user to do it.

Finally, the series adds a --cc=<email> option, which can be used to CC people who are reviewing this particular series and may not be interested in your patches in general otherwise.

This series is based on the cover-letter series, but the changes are actually independant; they are just adjacent to each other, and generate a lot of conflicts if applied separately and merged.

I messed up some extra whitespace initially, corrected it in the patches, but didn't regenerate the cover letter, so the diffstat will be off by two blank lines not actually added.

Daniel Barkalow (4):
      Add tests for extra headers in format-patch
      Fix format.headers not ending with a newline
      Combine To: and Cc: headers
      Support a --cc=<email> option in format-patch
 Documentation/git-format-patch.txt |    5 ++
 builtin-log.c                      |   80 +++++++++++++++++++++++++++++++----
 t/t4014-format-patch.sh            |   42 +++++++++++++++++++
 3 files changed, 117 insertions(+), 10 deletions(-)

← back to recent threads