b4 needs-editing check vs single patch serieses

b4 needs-editing check vs single patch serieses

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 465 bytes --]

Hi,

The needs-editing flag in b4 prep --show-info was intended to support
scripting which wraps b4, allowing it to see if b4 would refuse to send.
However as implemented the check does not take account of the logic b4
has which doesn't require a cover letter for a single patch series, it
will just report if EDITME is present in the cover.  Wrappers could do
their own version of the single patch check but it would be good if this
were done in b4.

Thanks,
Mark

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: b4 needs-editing check vs single patch serieses

[Konstantin Ryabitsev]

On Mon, Apr 08, 2024 at 07:51:36PM +0100, Mark Brown wrote:
> The needs-editing flag in b4 prep --show-info was intended to support
> scripting which wraps b4, allowing it to see if b4 would refuse to send.
> However as implemented the check does not take account of the logic b4
> has which doesn't require a cover letter for a single patch series, it
> will just report if EDITME is present in the cover.  Wrappers could do
> their own version of the single patch check but it would be good if this
> were done in b4.

bugbot assign to me

-K

Re: b4 needs-editing check vs single patch serieses

[Kernel.org Bugbot]

Hello:

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

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

Re: b4 needs-editing check vs single patch serieses

[Kernel.org Bugbot]

Konstantin Ryabitsev writes in commit 11c5cc46b7f4bb10c134e4ba05e984e8ab13ed6f:

ez: improve needs-editing logic for single-patch series

When sending single-patch series, the cover letter does not need to be
edited even if it contains EDITME.

Reported-by: Mark Brown <broonie@kernel.org>
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218706
Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>

(via https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=11c5cc46b7f4)
-- 
Deet-doot-dot, I am a bot.
Kernel.org Bugzilla (peebz 0.1)