From: Junio C Hamano <gitster@pobox.com>
To: git@vger.kernel.org
Subject: [PATCH 0/6] Bring "format-patch --notes" closer to a real feature
Date: Wed, 17 Oct 2012 22:45:22 -0700 [thread overview]
Message-ID: <1350539128-21577-1-git-send-email-gitster@pobox.com> (raw)
This replaces the earlier "wip" with a real thing.
We never advertised the "--notes" option to format-patch (or
anything related to the pretty format options for that matter)
because the behaviour of these options was whatever they happened to
do, not what they were designed to do.
It had a few obvious glitches:
* The notes section was appended immediately after the log message,
and then the three-dash line was added. Such a supplimental
material should come after the three-dash line.
* The logic to append a new sign-off with "format-patch --signoff"
worked on the message after the note was added, which made the
detection of existing sign-off lines incorrect.
This updates the handling of "--notes" option to correct these, in
an attempt to bring it closer to a real feature.
Junio C Hamano (6):
pretty: remove reencode_commit_message()
format_note(): simplify API
pretty: prepare notes message at a centralized place
pretty_print_commit(): do not append notes message
format-patch: append --signature after notes
format-patch --notes: show notes after three-dashes
builtin/blame.c | 5 +++--
commit.h | 4 +---
log-tree.c | 32 ++++++++++++++++++++++++++++----
notes.c | 13 +++++++------
notes.h | 6 +-----
pretty.c | 22 ++++------------------
revision.c | 2 +-
revision.h | 1 +
t/t4014-format-patch.sh | 15 +++++++++++++--
9 files changed, 59 insertions(+), 41 deletions(-)
--
1.8.0.rc3.112.gdb88a5e
next reply other threads:[~2012-10-18 5:45 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-18 5:45 Junio C Hamano [this message]
2012-10-18 5:45 ` [PATCH 1/6] pretty: remove reencode_commit_message() Junio C Hamano
2012-10-18 5:45 ` [PATCH 2/6] format_note(): simplify API Junio C Hamano
2012-10-18 5:45 ` [PATCH 3/6] pretty: prepare notes message at a centralized place Junio C Hamano
2012-10-18 7:49 ` Jeff King
2012-10-18 9:17 ` Junio C Hamano
2012-10-18 9:18 ` Jeff King
2012-10-18 10:00 ` [PATCH] strbuf: always return a non-NULL value from strbuf_detach Jeff King
2012-10-18 5:45 ` [PATCH 4/6] pretty_print_commit(): do not append notes message Junio C Hamano
2012-10-18 5:45 ` [PATCH 5/6] format-patch: append --signature after notes Junio C Hamano
2012-10-18 5:45 ` [PATCH 6/6] format-patch --notes: show notes after three-dashes Junio C Hamano
2012-10-18 21:35 ` Philip Oakley
2012-10-18 22:08 ` Junio C Hamano
2012-10-19 20:06 ` Junio C Hamano
2012-10-21 21:33 ` Philip Oakley
2012-10-18 10:02 ` [PATCH 0/6] Bring "format-patch --notes" closer to a real feature Jeff King
2012-10-18 11:06 ` Nguyen Thai Ngoc Duy
2012-10-18 12:24 ` Michael J Gruber
2012-10-18 17:04 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1350539128-21577-1-git-send-email-gitster@pobox.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).