| Message ID | 20181221060256.4892-1-gavin.hu@arm.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Thomas Monjalon |
| Headers | show |
| Series | [v3] devtools: fix wrong headline lowercase for arm | expand |
| Context | Check | Description |
|---|---|---|
| ci/checkpatch | success | coding style OK |
| ci/Intel-compilation | fail | Compilation issues |
| ci/mellanox-Performance-Testing | success | Performance Testing PASS |
| ci/intel-Performance-Testing | success | Performance Testing PASS |
On Fri, 2018-12-21 at 14:02 +0800, Gavin Hu wrote: > External Email > > ------------------------------------------------------------------- > --- > 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> Looks good to me. Acked-by: Jerin Jacob <jerinj@marvell.com> > --- > devtools/check-git-log.sh | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/devtools/check-git-log.sh b/devtools/check-git-log.sh > index 85d67fb9a..d39064f9d 100755 > --- a/devtools/check-git-log.sh > +++ b/devtools/check-git-log.sh > @@ -90,9 +90,10 @@ bad=$(echo "$headlines" | grep -E --color=always \ > -e ':.*\<[hsf]w\>' \ > -e ':.*\<l[234]\>' \ > -e ':.*\<api\>' \ > - -e ':.*\<arm\>' \ > - -e ':.*\<armv7\>' \ > - -e ':.*\<armv8\>' \ > + -e ':.*\<ARM\>' \ > + -e ':.*\<(Aarch64|AArch64|AARCH64|Aarch32|AArch32|AARCH32)\>' \ > + -e ':.*\<(Armv7|ARMv7|ArmV7|armV7|ARMV7)\>' \ > + -e ':.*\<(Armv8|ARMv8|ArmV8|armV8|ARMV8)\>' \ > -e ':.*\<crc\>' \ > -e ':.*\<dma\>' \ > -e ':.*\<eeprom\>' \
27/12/2018 13:34, Jerin Jacob Kollanukkaran: > On Fri, 2018-12-21 at 14:02 +0800, Gavin Hu wrote: > > 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> > > Looks good to me. > > Acked-by: Jerin Jacob <jerinj@marvell.com> Applied, thanks
diff --git a/devtools/check-git-log.sh b/devtools/check-git-log.sh index 85d67fb9a..d39064f9d 100755 --- a/devtools/check-git-log.sh +++ b/devtools/check-git-log.sh @@ -90,9 +90,10 @@ bad=$(echo "$headlines" | grep -E --color=always \ -e ':.*\<[hsf]w\>' \ -e ':.*\<l[234]\>' \ -e ':.*\<api\>' \ - -e ':.*\<arm\>' \ - -e ':.*\<armv7\>' \ - -e ':.*\<armv8\>' \ + -e ':.*\<ARM\>' \ + -e ':.*\<(Aarch64|AArch64|AARCH64|Aarch32|AArch32|AARCH32)\>' \ + -e ':.*\<(Armv7|ARMv7|ArmV7|armV7|ARMV7)\>' \ + -e ':.*\<(Armv8|ARMv8|ArmV8|armV8|ARMV8)\>' \ -e ':.*\<crc\>' \ -e ':.*\<dma\>' \ -e ':.*\<eeprom\>' \
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 | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)