Thread (1 message) 1 message, 1 author, 2025-03-19

Re: [PATCH] format-patch: use raw format for notes

From: Junio C Hamano <hidden>
Date: 2025-03-19 00:52:54

Tuomas Ahola [off-list ref] writes:
From: Junio C Hamano <redacted>
Subject: Re: [PATCH] format-patch: use raw format for notes
Date: Tue, 18 Mar 2025 14:17:32 -0700
quoted
[--] more importantly, the contents of the note loses its crucial
leading spaces that makes sure that any random lines in the note
that happen to begin with "diff", "---", etc. are not mistaken as
the beginning of the first patch.
Thanks for quick response. That was indeed a compelling point.
quoted
So, no, this change is not a good thing to do, at least in its
current form.  Besides, unconditional change like this will break
existing users.
I see that similar patch was proposed in 2017. I should have searched
more thoroughly, I guess.
Heh, your archive spelunking skills are far superiour than mine, it
seems.  And in

https://lore.kernel.org/git/xmqqingw8ppj.fsf@gitster.mtv.corp.google.com/ (local)

I see that I said exactly the same thing to exactly the same patch.

It is not to say that I've been a good person to be very consistent
(I do not have to be---over the years I can hear more opinions from
others that may sway how I think about the same issue), but says
that there aren't new arguments to sway the old decision in the past
7.5 years.

And exactly the same way as back then, I am open to a valid argument
to add such an output as an optional feature if there is a good use
case for it.

Thanks.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help