Re: [PATCH v4 09/10] libxfs-apply: drop Cc: to stable release list

["Darrick J. Wong" <djwong@kernel.org>]

On Thu, Feb 13, 2025 at 11:27:06PM +0100, Andrey Albershteyn wrote:
> On 2025-02-13 13:45:41, Darrick J. Wong wrote:
> > On Thu, Feb 13, 2025 at 09:14:31PM +0100, Andrey Albershteyn wrote:
> > > These Cc: tags are intended for kernel commits which need to be
> > > backported to stable kernels. Maintainers of stable kernel aren't
> > > interested in xfsprogs syncs.
> > > 
> > > Signed-off-by: Andrey Albershteyn <aalbersh@kernel.org>
> > > ---
> > >  tools/libxfs-apply | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/tools/libxfs-apply b/tools/libxfs-apply
> > > index 097a695f942bb832c2fb1456a0fd8c28c025d1a6..e9672e572d23af296dccfe6499eda9b909f44afd 100755
> > > --- a/tools/libxfs-apply
> > > +++ b/tools/libxfs-apply
> > > @@ -254,6 +254,7 @@ fixup_header_format()
> > >  		}
> > >  		/^Date:/ { date_seen=1; next }
> > >  		/^difflib/ { next }
> > > +		/[Cc]{2}: <?stable@vger.kernel.org>?.*/ { next }
> > 
> > You might want to ignore the angle brackets, because some people do:
> 
> The ? does that :) One or zero of <>

Aha, I missed that!  Thanks for the correction.

Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>

--D



> > 
> > Cc: stable@vger.kernel.org
> > 
> > which is valid rfc822 even if SubmittingPatches says not to do that.
> > Annoyingly, other parts of the documentation lay that out as an example.
> > 
> > 		/[Cc]{2}:.*stable@vger.kernel.org/ { next }
> > 
> > <shrug>
> > 
> > --D
> > 
> > >  
> > >  		// {
> > >  			if (date_seen == 0)
> > > 
> > > -- 
> > > 2.47.2
> > > 
> > > 
> > 
> 
> -- 
> - Andrey
> 
>