Dan Williams wrote: > While the Link: tag is disruptive to some top-level maintainer workflows > [1], it is also useful to a significant number of developers and subsystem > maintainers. > > It is also the case that dynamic patch-id lookup [2] is an incomplete > replacement for having the submission Link: trailer readily available. > Specifically, navigating to a patch on gitweb or displaying the patch in > the local developer tree it is convenient to have the metadata inline. > > A method to have that metadata available without polluting upstream is to > keep git notes locally. > > Add a new option to shazam that annotates newly applied commits with the > Link: trailer of the submission. Honor the b4.linkmask option to use the > preferred namespace (patch.msgid.link) for these links. > > Note: Claude Sonnet 4 was used to help early drafts of this patch, but all > submitted lines are authored by me or copied from other parts of b4. > > Link: > http://lore.kernel.org/CAHk-=whP2zoFm+-EmgQ69-00cxM5jgoEGWyAYVQ8bQYFbb2j=q...@mail.gmail.com > [1] > Signed-off-by: Dan Williams <[email protected]> > --- > src/b4/command.py | 2 ++ > src/b4/mbox.py | 58 +++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 60 insertions(+) [..] > diff --git a/src/b4/mbox.py b/src/b4/mbox.py > index 8810ddd71b21..9479b8995019 100644 > --- a/src/b4/mbox.py > +++ b/src/b4/mbox.py [..] > @@ -547,6 +552,59 @@ def thanks_record_am(lser: b4.LoreSeries, cherrypick: > Optional[List[int]]) -> No [..] > +def shazam_notes(gitdir: Optional[str], lser: 'b4.LoreSeries', branch: str) > -> None: > + """Match commits to LoreMessages using git patch-id and emit debug info > for later git notes processing."""
Whoops, this stale comment is from a work-in-progress debug build.
