DPDK CI discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@redhat.com>
To: thomas@monjalon.net, alialnu@nvidia.com
Cc: ci@dpdk.org
Subject: [PATCH v2] tools: accept patch with subject starting with RFC
Date: Tue, 12 Apr 2022 11:12:07 +0200	[thread overview]
Message-ID: <20220412091207.24005-1-david.marchand@redhat.com> (raw)
In-Reply-To: <20220412085417.15884-1-david.marchand@redhat.com>

If a mail does not contain a X-Mailer: header, only a subject matching
[PATCH qualifies this mail as being a patch.
Extend this filter and accept [RFC PATCH too.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
Changes since v1:
- factorize matching with a regexp,

---
 tools/filter-patch-email.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/filter-patch-email.sh b/tools/filter-patch-email.sh
index 9ac8221..6593403 100755
--- a/tools/filter-patch-email.sh
+++ b/tools/filter-patch-email.sh
@@ -43,11 +43,11 @@ parse ()
 		set -- $line
 		if ! $content ; then
 			[ "$1" != 'X-Mailer:' -o "$2" != 'git-send-email' ] || gitsend=true
-			if echo "$line" | grep -qa '^Subject:.*\[PATCH' ; then
+			if echo "$line" | grep -qa '^Subject:.*PATCH' ; then
 				subject=$(echo "$line" | sed 's,^Subject:[[:space:]]*,,')
 				while [ -n "$subject" ] ; do
 					echo "$subject" | grep -q '^\[' || break
-					if echo "$subject" | grep -q '^\[PATCH' ; then
+					if echo "$subject" | grep -qE '^\[(RFC |)PATCH' ; then
 						patchsubject=true
 						break
 					fi
-- 
2.23.0


  reply	other threads:[~2022-04-12  9:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-12  8:54 [PATCH] " David Marchand
2022-04-12  9:12 ` David Marchand [this message]
2022-04-12  9:21   ` [PATCH v2] " Ali Alnubani
2022-04-12 13:53     ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220412091207.24005-1-david.marchand@redhat.com \
    --to=david.marchand@redhat.com \
    --cc=alialnu@nvidia.com \
    --cc=ci@dpdk.org \
    --cc=thomas@monjalon.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).