From: Beat Bolli <bbolli@ewanet•ch>
To: Junio C Hamano <gitster@pobox•com>
Cc: git@vger•kernel.org, Eric Raible <raible@nextest•com>
Subject: Re: [PATCH] cherry-pick -x: always insert an empty line
Date: Thu, 23 Feb 2012 09:39:13 +0100 [thread overview]
Message-ID: <4F45FB31.4000808@ewanet.ch> (raw)
In-Reply-To: <7vhaynp605.fsf@alter.siamese.dyndns.org>
[-- Attachment #1: Type: text/plain, Size: 2390 bytes --]
On 2012-02-19 09:44, Junio C Hamano wrote:
> Beat Bolli <bbolli@ewanet•ch> writes:
>
>> When cherry-picking a commit that has only a summary, the -x option
>> creates an invalid commit message because it puts the hash of the commit
>> being picked on the second line which should be left empty.
>>
>> This patch fixes this buglet by always inserting an empty line before
>> the added line.
>>
>> Aside from that, even with a non-trivial commit the generated note
>> "(cherry picked from commit 555c9864971744abb558796aea28e12a1ac20839)"
>> seems abrupt when appended directly.
>>
>> Cc: Eric Raible <raible@nextest•com>
>> Signed-off-by: Beat Bolli <bbolli@ewanet•ch>
>> ---
>> sequencer.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/sequencer.c b/sequencer.c
>> index 5fcbcb8..63fd589 100644
>> --- a/sequencer.c
>> +++ b/sequencer.c
>> @@ -382,7 +382,7 @@ static int do_pick_commit(struct commit *commit, struct replay_opts *opts)
>> }
>>
>> if (opts->record_origin) {
>> - strbuf_addstr(&msgbuf, "(cherry picked from commit ");
>> + strbuf_addstr(&msgbuf, "\n(cherry picked from commit ");
>
> Doesn't this need to be conditional?
>
> If we cherry pick your commit somewhere else with "-x -s", the resulting
> commit log message would end like this:
>
> Aside from that, even with a non-trivial commit the generated note
> "(cherry picked from commit 555c9864971744abb558796aea28e12a1ac20839)"
> seems abrupt when appended directly.
>
> Cc: Eric Raible <raible@nextest•com>
> Signed-off-by: Beat Bolli <bbolli@ewanet•ch>
>
> (cherry picked from commit 555c9864971744abb558796aea28e12a1ac20839)
> Signed-off-by: Frotz Xyzzy <nitfol@example•xz>
>
> which clearly is worse-looking with the extra LF you added in this patch.
What should the condition then be? What is the canonical format for
cherry-picks with Signed-off-by: and Cc: footer lines?
>
>> strbuf_addstr(&msgbuf, sha1_to_hex(commit->object.sha1));
>> strbuf_addstr(&msgbuf, ")\n");
>> }
--
mail: echo '<bNbolOli@ewaSPnetAM•ch>' | tr -d '[A-S]'
pgp: 0x506A903A; 49D5 794A EA77 F907 764F D89E 304B 93CF 506A 903A
gsm: 4.7.7.6.0.7.7.9.7.1.4.e164.arpa
icbm: 47.0452 N, 7.2715 E
http: www.drbeat.li
"It takes love over gold, and mind over matter" -- Dire Straits
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 260 bytes --]
next prev parent reply other threads:[~2012-02-23 8:47 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-18 21:14 [PATCH] cherry-pick -x: always insert an empty line Beat Bolli
2012-02-19 8:44 ` Junio C Hamano
2012-02-23 8:39 ` Beat Bolli [this message]
2012-02-23 9:38 ` Sebastian Schuberth
2012-02-23 8:56 ` Sebastian Schuberth
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=4F45FB31.4000808@ewanet.ch \
--to=bbolli@ewanet$(echo .)ch \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=raible@nextest$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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