On Mon, Sep 29, 2025 at 10:32:38AM -0400, Konstantin Ryabitsev wrote: > On Mon, Sep 29, 2025 at 08:15:07AM -0600, Tom Rini wrote: > > > So it looks like the default command used by b4 for the Cc: section is > > > "scripts/get_maintainer.pl --nogit --nogit-fallback --nogit-chief-penguins > > > --norolestats --nom" which causes recent signers to not be included due to > > > the '--nogit' argument. > > > > > > I have added the following to my git config file. > > > > > > [b4] > > > send-auto-cc-cmd = scripts/get_maintainer.pl --nogit-fallback > > > --nogit-chief-penguins --norolestats --nom > > > > > > This will mean that from now on recent signers will be added to the Cc: > > > list. I hope that this is what is expected. > > > > > > I will resend this patch with the expanded Cc: list. > > > > A while back someone submitted a .b4.conf (or .b4.config? I forget...) > > .b4-config > > > patch, but at the time it wasn't setting anything non-standard I > > thought. However, is the above something that can be configured there? > > If so, that would be good to put in a project-wide b4 config file, and > > if it's not it'd be a good feature request for upstream. > > Yeah, we try to walk a fine balance between being newbie-friendly but also > allowing power users to tweak things. The decision to add --nogit by default > was because for most subsystems this returns a list of CC's that is way too > huge and actually trips up some MTAs to refuse delivery with "too many > recipients." > > The biggest downside of providing settings in the toplevel .b4-config is that > it can't currently be done per-subsystem, so projects like the kernel cannot > carry a .b4-config without it clashing between what different subsystems want > to set as defaults. I've not yet come up with a good way to solve this.
Ah, thanks for explaining. Yes, there are some challenges with co-opting kernel tooling for our own needs :) In another part of the thread Andrew just now pointed out that https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/ does handle this for us. -- Tom
signature.asc
Description: PGP signature
