patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH v1] devtools: fix wrong headline lowercase for arm
@ 2018-12-20 15:52 Gavin Hu
  2018-12-20 15:57 ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Gavin Hu @ 2018-12-20 15:52 UTC (permalink / raw)
  To: dev
  Cc: thomas, bruce.richardson, ferruh.yigit, Honnappa.Nagarahalli, nd,
	Gavin Hu, stable

Change to the new wording according to Arm corp:
https://mails.dpdk.org/archives/dev/2018-October/115174.html
otherwise, "Wrong headline lowercase" may be falsely reported.

Fixes: f146ada21d1e ("scripts: add more git log checks")
Cc: stable@dpdk.org

Signed-off-by: Gavin Hu <gavin.hu@arm.com>
---
 devtools/check-git-log.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/devtools/check-git-log.sh b/devtools/check-git-log.sh
index 85d67fb9a..f8b2bec83 100755
--- a/devtools/check-git-log.sh
+++ b/devtools/check-git-log.sh
@@ -90,9 +90,9 @@ bad=$(echo "$headlines" | grep -E --color=always \
 	-e ':.*\<[hsf]w\>' \
 	-e ':.*\<l[234]\>' \
 	-e ':.*\<api\>' \
-	-e ':.*\<arm\>' \
-	-e ':.*\<armv7\>' \
-	-e ':.*\<armv8\>' \
+	-e ':.*\<(Arm|ARM)\>' \
+	-e ':.*\<(Armv7|ARMv7|ArmV7|armV7|ARMV7)\>' \
+	-e ':.*\<(Armv8|ARMv8|ArmV8|armV8|ARMV8)\>' \
 	-e ':.*\<crc\>' \
 	-e ':.*\<dma\>' \
 	-e ':.*\<eeprom\>' \
-- 
2.11.0

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [PATCH v1] devtools: fix wrong headline lowercase for arm
  2018-12-20 15:52 [dpdk-stable] [PATCH v1] devtools: fix wrong headline lowercase for arm Gavin Hu
@ 2018-12-20 15:57 ` Thomas Monjalon
  2018-12-20 16:01   ` Gavin Hu (Arm Technology China)
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2018-12-20 15:57 UTC (permalink / raw)
  To: Gavin Hu
  Cc: dev, bruce.richardson, ferruh.yigit, Honnappa.Nagarahalli, nd, stable

20/12/2018 16:52, Gavin Hu:
> Change to the new wording according to Arm corp:
> https://mails.dpdk.org/archives/dev/2018-October/115174.html
> otherwise, "Wrong headline lowercase" may be falsely reported.
> 
> Fixes: f146ada21d1e ("scripts: add more git log checks")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Gavin Hu <gavin.hu@arm.com>
> ---
> --- a/devtools/check-git-log.sh
> +++ b/devtools/check-git-log.sh
> -	-e ':.*\<arm\>' \
> -	-e ':.*\<armv7\>' \
> -	-e ':.*\<armv8\>' \
> +	-e ':.*\<(Arm|ARM)\>' \

Are you saying that we should not write "Arm"?
I thought it was the right spelling when talking about the company.

> +	-e ':.*\<(Armv7|ARMv7|ArmV7|armV7|ARMV7)\>' \
> +	-e ':.*\<(Armv8|ARMv8|ArmV8|armV8|ARMV8)\>' \

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [PATCH v1] devtools: fix wrong headline lowercase for arm
  2018-12-20 15:57 ` Thomas Monjalon
@ 2018-12-20 16:01   ` Gavin Hu (Arm Technology China)
  0 siblings, 0 replies; 3+ messages in thread
From: Gavin Hu (Arm Technology China) @ 2018-12-20 16:01 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: dev, bruce.richardson, ferruh.yigit, Honnappa Nagarahalli, nd,
	stable, nd



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Thursday, December 20, 2018 11:57 PM
> To: Gavin Hu (Arm Technology China) <Gavin.Hu@arm.com>
> Cc: dev@dpdk.org; bruce.richardson@intel.com; ferruh.yigit@intel.com;
> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; nd
> <nd@arm.com>; stable@dpdk.org
> Subject: Re: [PATCH v1] devtools: fix wrong headline lowercase for arm
> 
> 20/12/2018 16:52, Gavin Hu:
> > Change to the new wording according to Arm corp:
> > https://mails.dpdk.org/archives/dev/2018-October/115174.html
> > otherwise, "Wrong headline lowercase" may be falsely reported.
> >
> > Fixes: f146ada21d1e ("scripts: add more git log checks")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Gavin Hu <gavin.hu@arm.com>
> > ---
> > --- a/devtools/check-git-log.sh
> > +++ b/devtools/check-git-log.sh
> > -	-e ':.*\<arm\>' \
> > -	-e ':.*\<armv7\>' \
> > -	-e ':.*\<armv8\>' \
> > +	-e ':.*\<(Arm|ARM)\>' \
> 
> Are you saying that we should not write "Arm"?
> I thought it was the right spelling when talking about the company.

Not sure if Arm, as a company name will appear on the patch headline.
Anyway I will remove Arm here in V2.

> 
> > +	-e ':.*\<(Armv7|ARMv7|ArmV7|armV7|ARMV7)\>' \
> > +	-e ':.*\<(Armv8|ARMv8|ArmV8|armV8|ARMV8)\>' \
> 
> 
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-12-20 16:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-20 15:52 [dpdk-stable] [PATCH v1] devtools: fix wrong headline lowercase for arm Gavin Hu
2018-12-20 15:57 ` Thomas Monjalon
2018-12-20 16:01   ` Gavin Hu (Arm Technology China)

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).