b4 send needs a subject in the cover

b4 send needs a subject in the cover

[Mark Brown]

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

If I've got a patch series with a single patch in it then there's no
need for a cover letter and if it's the first version no need for a
changelog so I'd expect to just delete all the text in the b4 cover
other than the recipients.  However if I do that then what seems to end
up happening is that the first recipient gets deleted, I'm guessing
parsed as a subject which then doesn't get used due to this being a
single message.

I can see why this happens but it's a bit of a gotcha - printing the
subject might help make it clearer what's going on in this case?

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

Re: b4 send needs a subject in the cover

[Mark Brown]

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

On Wed, Dec 07, 2022 at 11:02:41PM +0000, Mark Brown wrote:
> If I've got a patch series with a single patch in it then there's no
> need for a cover letter and if it's the first version no need for a
> changelog so I'd expect to just delete all the text in the b4 cover
> other than the recipients.  However if I do that then what seems to end
> up happening is that the first recipient gets deleted, I'm guessing
> parsed as a subject which then doesn't get used due to this being a
> single message.
> 
> I can see why this happens but it's a bit of a gotcha - printing the
> subject might help make it clearer what's going on in this case?

...and further if you look at

   https://lore.kernel.org/r/20221207-arm64-mvfr-fix-v1-0-0c739eb77bea@kernel.org

you'll see that the to and Cc lines end up in the sent patch which isn't
entirely what would be expected.

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

Re: b4 send needs a subject in the cover

[Konstantin Ryabitsev]

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

On Thu, Dec 08, 2022 at 12:39:01AM +0000, Mark Brown wrote:
> On Wed, Dec 07, 2022 at 11:02:41PM +0000, Mark Brown wrote:
> > If I've got a patch series with a single patch in it then there's no
> > need for a cover letter and if it's the first version no need for a
> > changelog so I'd expect to just delete all the text in the b4 cover
> > other than the recipients.  However if I do that then what seems to end
> > up happening is that the first recipient gets deleted, I'm guessing
> > parsed as a subject which then doesn't get used due to this being a
> > single message.
> > 
> > I can see why this happens but it's a bit of a gotcha - printing the
> > subject might help make it clearer what's going on in this case?
> 
> ...and further if you look at
> 
>    https://lore.kernel.org/r/20221207-arm64-mvfr-fix-v1-0-0c739eb77bea@kernel.org
> 
> you'll see that the to and Cc lines end up in the sent patch which isn't
> entirely what would be expected.

I believe the latest master should fix both of these problems.

-K

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