Thread (14 messages) flat view 14 messages, 3 authors, 30m ago

Re: [PATCH 3/3] format-patch: learn --[no-]range-diff-notes

From: Junio C Hamano <hidden>
Date: 2026-08-28 00:31:12

"Kristoffer Haugsbakk" [off-list ref] writes:
quoted
Hmph, this is a bit too complex for me.  When I say

    $ git format-patch --no-notes --range-diff-notes ...

I would expect that individual patches would not get notes, but the
range-diff will include them in the comparison.  But if
--range-diff-notes just falls back to default (i.e., inherit what
patches use), would I see the notes used in the range-diff?
You will not get patch notes and not get
range diff notes. That --range-diff-notes
told it to use the patch notes which you
just turned off/emptied the list.

Code-wise, the list of notes is cleared so you
you would have to change the --notes implementation
if you want to keep a sort of shadow list
of not-patch-notes-but-RD-notes.
IOW, the design of how these options interact does not support the
usecase I gave?
And another problem, or fact, is that format-patch
does not show notes by default. So what should
--RD-notes show? The default notes?
I do not know.  My preference actually is not to introuce a new
option whose interaction with the existing --notes option cannot be
defined in simple terms.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help