Thread (5 messages) flat view 5 messages, 3 authors, 2021-10-21

Re: git format-patch --signoff

From: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Date: 2021-10-21 23:37:21

On Thu, Oct 21, 2021 at 03:34:39PM -0700, Junio C Hamano wrote:
Michał Mirosław [off-list ref] writes:
quoted
I just noticed that `git format-patch --signoff` adds the 'Signed-off-by'
line even if the exact same line is already present in the commit message.
Could this be avoided in the tool?

git version 2.30.2

Best Regards
Michał Mirosław
The rule should be "avoid adding the same sign-off as the one at the
end".  In other words, as a record of the flow of patch custody,
[...]
This test hasn't changed since it was written in Feb 2013, and I
think 2.30.2 is recent enough to conform to the rule to pass this
test.
The test indeed works correctly, and I couldn't reproduce the effect
on a fresh git repo. I finally took a look with hexdump on the output:
the culprit was a UTF-8 non-breaking space -- indistinguishable on
a terminal from a normal space. I'm not sure what to think about this...
Sorry for the noise.

Best Regards
Michał Mirosław
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help