Re: b4: use new trailers in 'b4 send --resend'

[Luca Ceresoli <luca.ceresoli@bootlin.com>]

Hello Konstantin,

On Fri, 26 Apr 2024 14:04:49 -0400
Konstantin Ryabitsev <konstantin@linuxfoundation.org> wrote:

> On Fri, Apr 26, 2024 at 06:44:25PM GMT, Luca Ceresoli wrote:
> > > It's not really necessary to resend a series just for trailer 
> > > changes anyway. The maintainer will most likely use b4 to retrieve 
> > > it anyway, and that will pull in any new trailers.  
> > 
> > I cannot recall exactly when it did happen and provide a link right now,
> > but I clearly remember having been asked by some maintainer to do
> > exactly this, instead of pinging.
> > 
> > The rationale is that sending a new version just to add review tags was
> > considered bad because one can expect the patches or commit description
> > to have changed somewhat, which it hasn't.  
> 
> Yes, to be clear -- I wasn't suggesting that you send a new revision 
> just to add trailers either. We're both aligned here.
> 
> > OTOH resending without new trailers can lead to applying a patch
> > without those tags (not recognizing reviewers time), or even worse to
> > discard a patch because it does not show the needed tags.  
> 
> I expect this is one of those "every subsystem does things slightly 
> differently" situation. If the maintainer uses b4, this entire step is 
> unnecessary, because any missing trailers will be applied anyway when 
> they run "b4 am".

Ah, interesting feature. I had no idea about it as I'm using b4 for
sending patches only. Thanks for the clarification!

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com