Re: [PATCH b4] config: accept prep-pre-flight-checks from .b4-config
Konstantin Ryabitsev <[email protected]> Tue, 21 Jul 2026 16:38:45 -0400
| Newsgroups | org.kernel.linux.tools |
|---|---|
| Message-ID | <20260721-capable-beagle-of-experiment-1f2d8c@meerkat> |
On Fri, Jul 10, 2026 at 05:51:10PM +0200, Matthieu Baerts wrote: > > Add 'prep-pre-flight-checks' to the wtglobs list so it can be set > > from .b4-config, allowing projects to disable checks (e.g. > > needs-checking, needs-auto-to-cc) that are not relevant to their > > workflow and share that decision via version control. > > I don't remember if it was present in this list for security reasons -- > similar to *-check-cmd at some points [1] -- but I do remember a bug > report with the same patch [2] which has never been applied. > > On my side, I think it would be useful to be able to use > "disable-needs-auto-to-cc" on some specific dev tree of a > "sub-subsystem". On these dev branches, no need to add extra reviewers > from the "parent" subsystem. > > We could then have this config: > > [b4] > send-series-to = <dev ML> > prep-pre-flight-checks = disable-needs-auto-to-cc > send-prefixes = <specific dev prefix> > > [1] https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=a1360385 > [2] https://bugzilla.kernel.org/show_bug.cgi?id=219705 I think it mostly slipped through the cracks. I did want to work on this some more as a general feature, but there's no reason to hold back on this small change, which is clearly useful to projects. The only concern with these .b4-config flags is the potential for abuse if they are set to execute commands, but this doesn't fall under that category and therefore we can easily add it. Thanks! -K