diff mbox series

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

Message ID 20250213-update-release-v4-9-c06883a8bbd6@kernel.org (mailing list archive)
State New
Headers show
Series Update release.sh | expand

Commit Message

Andrey Albershteyn Feb. 13, 2025, 8:14 p.m. UTC
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(+)

Comments

Darrick J. Wong Feb. 13, 2025, 9:45 p.m. UTC | #1
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:

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 Albershteyn Feb. 13, 2025, 10:27 p.m. UTC | #2
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 <>

> 
> 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
> > 
> > 
>
Darrick J. Wong Feb. 13, 2025, 10:47 p.m. UTC | #3
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
> 
>
diff mbox series

Patch

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 }
 
 		// {
 			if (date_seen == 0)