* [dpdk-dev] [PATCH] devtools: update headline prefix for drivers
@ 2017-01-26 14:07 Ferruh Yigit
2017-01-29 14:42 ` Thomas Monjalon
0 siblings, 1 reply; 2+ messages in thread
From: Ferruh Yigit @ 2017-01-26 14:07 UTC (permalink / raw)
To: dev; +Cc: Ferruh Yigit, Thomas Monjalon, Pablo de Lara
For the patches that touch multiple drivers in same driver group,
script forces headline prefix start with drv group.
Like for net/a net/b net/c, patch title should be "net: x y z"
Update rule to let "driver" prefix in headline,
for above sample patch title becomes: "drivers/net: x y z"
This prevents patch confused with library with same name.
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
devtools/check-git-log.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/devtools/check-git-log.sh b/devtools/check-git-log.sh
index 62b5f43..9ccef66 100755
--- a/devtools/check-git-log.sh
+++ b/devtools/check-git-log.sh
@@ -87,7 +87,7 @@ bad=$(for commit in $commits ; do
if [ $(echo "$drvgrp" | wc -l) -gt 1 ] ; then
echo "$headline" | grep -v '^drivers:'
elif [ $(echo "$drv" | wc -l) -gt 1 ] ; then
- echo "$headline" | grep -v "^$drvgrp"
+ echo "$headline" | grep -v "^drivers/$drvgrp"
else
echo "$headline" | grep -v "^$drv"
fi
--
2.9.3
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dpdk-dev] [PATCH] devtools: update headline prefix for drivers
2017-01-26 14:07 [dpdk-dev] [PATCH] devtools: update headline prefix for drivers Ferruh Yigit
@ 2017-01-29 14:42 ` Thomas Monjalon
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2017-01-29 14:42 UTC (permalink / raw)
To: Ferruh Yigit; +Cc: dev, Pablo de Lara
2017-01-26 14:07, Ferruh Yigit:
> For the patches that touch multiple drivers in same driver group,
> script forces headline prefix start with drv group.
>
> Like for net/a net/b net/c, patch title should be "net: x y z"
>
> Update rule to let "driver" prefix in headline,
> for above sample patch title becomes: "drivers/net: x y z"
>
> This prevents patch confused with library with same name.
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Applied, thanks
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-01-29 14:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-26 14:07 [dpdk-dev] [PATCH] devtools: update headline prefix for drivers Ferruh Yigit
2017-01-29 14:42 ` Thomas Monjalon
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).