diff mbox

[dpdk-dev] devtools: update headline prefix for drivers

Message ID 20170126140744.21626-2-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel compilation success Compilation OK

Commit Message

Ferruh Yigit Jan. 26, 2017, 2:07 p.m. UTC 
  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(-)

Comments

Thomas Monjalon Jan. 29, 2017, 2:42 p.m. UTC | #1 
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
diff mbox

Patch

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