b4: Should send-series-to supress send-auto-to-cmd?

b4: Should send-series-to supress send-auto-to-cmd?

[Jiaxun Yang]

Hi all,

This is a question (or possibly bug) raised from QEMU's .b4-config [1].

In QEMU's workflow, we have a fixed To point to the main mailing list, while
Cc is gathered from get_maintainers.pl.

However, at least in b4 0.14.2 specifying send-series-to won't suppress send-auto-to-cmd.
In such .b4-config configuration:

```
[b4]
    send-series-to = qemu-devel@nongnu.org
    send-auto-cc-cmd = scripts/get_maintainer.pl --noroles --norolestats --nogit --nogit-fallback
```

b4 prep -c will still cause default send-auto-to-cmd to run
```
$ b4 prep -c
Will collect To: addresses using get_maintainer.pl
Will collect Cc: addresses using get_maintainer.pl
Collecting To/Cc addresses
CRITICAL: Running /[...]/qemu/scripts/get_maintainer.pl --nogit --nogit-fallback --nogit-chief-penguins --norolestats --nol failed:
Unknown option: nogit-chief-penguins
/[...]/qemu/scripts/get_maintainer.pl: invalid argument - use --help if necessary
```

My current workaround is to specify a dummy send-auto-to-cmd (i.e. echo).

I wonder if it is an expected behaviour?

Thanks

[1]: https://lore.kernel.org/qemu-devel/20241223-swift-of-splendid-protection-b5dcac@lemur/
- Jiaxun

Re: b4: Should send-series-to supress send-auto-to-cmd?

[Konstantin Ryabitsev]

On Tue, Dec 24, 2024 at 01:06:27PM +0000, Jiaxun Yang wrote:
> Hi all,
> 
> This is a question (or possibly bug) raised from QEMU's .b4-config [1].
> 
> In QEMU's workflow, we have a fixed To point to the main mailing list, while
> Cc is gathered from get_maintainers.pl.
> 
> However, at least in b4 0.14.2 specifying send-series-to won't suppress send-auto-to-cmd.
> In such .b4-config configuration:
> 
> ```
> [b4]
>     send-series-to = qemu-devel@nongnu.org
>     send-auto-cc-cmd = scripts/get_maintainer.pl --noroles --norolestats --nogit --nogit-fallback
> ```
> 
> b4 prep -c will still cause default send-auto-to-cmd to run
> ```
> $ b4 prep -c
> Will collect To: addresses using get_maintainer.pl
> Will collect Cc: addresses using get_maintainer.pl
> Collecting To/Cc addresses
> CRITICAL: Running /[...]/qemu/scripts/get_maintainer.pl --nogit --nogit-fallback --nogit-chief-penguins --norolestats --nol failed:
> Unknown option: nogit-chief-penguins
> /[...]/qemu/scripts/get_maintainer.pl: invalid argument - use --help if necessary
> ```
> 
> My current workaround is to specify a dummy send-auto-to-cmd (i.e. echo).
> 
> I wonder if it is an expected behaviour?

It's not, it's just an unexpected condition. I'll figure out the right way to
go about it.

bugspray tag me

Re: b4: Should send-series-to supress send-auto-to-cmd?

[Bugspray Bot]

Hello:

This conversation is now tracked by Kernel.org Bugzilla:
https://bugzilla.kernel.org/show_bug.cgi?id=219633

There is no need to do anything else, just keep talking.
-- 
Deet-doot-dot, I am a bot.
Kernel.org Bugzilla (bugspray 0.1-dev)