Re: [PATCH b4] review: allow a custom Message-Id command for TUI replies
Konstantin Ryabitsev <[email protected]> Tue, 9 Jun 2026 21:11:01 -0400
| Newsgroups | org.kernel.linux.tools |
|---|---|
| Message-ID | <20260609-loutish-busy-curassow-33ac37@meerkat> |
On Fri, May 22, 2026 at 12:57:56PM +0200, Christian Brauner wrote: > The review TUI composes replies and sends them directly over SMTP via > send_mail(), bypassing git-send-email and therefore any sendemail-validate > hook that would normally rewrite the Message-Id. Review replies thus always > got a built-in <...@b4> id. > > Add an opt-in b4.review-msgid-cmd option: when set, run it and use its > stdout as the Message-Id (angle brackets optional), falling back to the > built-in id when unset or on failure. Wire it into both the review-reply > builder and LoreMessage.make_reply() so follow-up replies are covered too. > > Signed-off-by: Christian Brauner (Amutable) <[email protected]> > --- > I love b4 review and I'm in the process of making it my default review > tool. 2026 and kernel review is officially better than Github. Didn't > expect to see that day tbh. Oh, amazing, thank you so much for saying that. :) I'm not sure I'd go as far as being better than GH -- there are still lots of sharp edges, but I'm glad it's starting to shape up a little. > All my reviews come with specific message ids and I would like to be > able to continue using them so I would appreciate the ability to define > a message-id generation command in the [b4] .gitconfig section. This patch won't quite cover every possible case -- e.g. it will miss the thank-you notes that are still sent with the same code path as "b4 ty". Let me think of a more global way to plug this in that won't clash with message-id's that are generated for series with "b4 prep/send". Thanks, -K