format-patch --cover-letter --in-reply-to
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest4 patches 1 replyIn review
- v14 patches 1 replySuperseded
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/4 | format-patch: threading test reactivationt/ | +103 | 0 replies | |
| 2/4 | format-patch: track several referencesbuiltin-log.c log-tree.c revision.h | +19 | 0 replies | |
| 3/4 | format-patch: thread as reply to cover letter even with in-reply-tobuiltin-log.c t/ | +21 | 0 replies | |
| 4/4 | format-patch: support deep threadingDocumentation/ Documentation/ builtin-log.c +1 more | +170 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [PATCH v2 next 0/4] format-patch --cover-letter --in-reply-to
Quoted 11 lines · Daniel Barkalow
Quoted 5 lines · Thomas Rast
Currently, format-patch can only track a single reference (the In-Reply-To:) for each mail. To ensure proper threading, we should list all known references for every mail.
Change the rev_info.ref_message_id field to a
string_listIt would be nice to change the field name to "
ref_message_ids", which is more likely to suggest that it's a list of ids, rather than a single id stored strangely.
Good point, I changed that.
Quoted 2 lines
Aside from that, parts 2-4 look good to me (I'm not up on reviewing shell code).
Thanks.
I also noticed that there is a textual conflict with 0db5260 (Enable setting attach as the default in .gitconfig for git-format-patch., 2009-02-12), which is already in 'next'. So I rebased the series to 'next'.
Thomas Rast (4):
format-patch: threading test reactivation
format-patch: track several references
format-patch: thread as reply to cover letter even with in-reply-to
format-patch: support deep threading
Documentation/config.txt | 10 ++ Documentation/git-format-patch.txt | 10 ++- builtin-log.c | 66 ++++++++-- log-tree.c | 11 +- revision.h | 2 +- t/t4014-format-patch.sh | 265 ++++++++++++++++++++++++++++++------ 6 files changed, 308 insertions(+), 56 deletions(-)