[Intel-gfx] [PATCH] dim: Accept author x signed-off based on email, but warn.

[Rodrigo Vivi]

It seems Patchwork or SMTP servers are messing some patches
and changing the original git's author name on git per "Last, First".
So we end up with a mismatch were signed-off uses one name format
and author is using another format.

So, let's check for email addresses instead.

However let's continue to WARN so commiters can take action on it
before pushing patch upstream with incorrect "Last, First" name.

Cc: Jani Nikula <jani.nikula at intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
---
 dim | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/dim b/dim
index dbaeb1ec944d..13cc5f00209d 100755
--- a/dim
+++ b/dim
@@ -690,11 +690,16 @@ function checkpatch_commit_push
 
 	# use real names for people with many different email addresses
 	author=$(git show -s $sha1 --format="format:%an")
+	author_email=$(git show -s $sha1 --format="format:%ae")
 	committer=$(git show -s $sha1 --format="format:%cn")
 
 	# check for author sign-off
 	if ! git show -s $sha1 | grep -qi "S.*-by:.*$author"  ; then
-		warn_or_fail "$sha1 is lacking author of sign-off"
+		if git show -s $sha1 | grep -qi "S.*-by:.*$author_email"  ; then
+			echoerr "WARNING: Author name mismatch. Patchwork or SMTP messing it up. Consider fixing it before pushing it."
+		else
+			warn_or_fail "$sha1 is lacking author of sign-off"
+		fi
 	fi
 
 	# check for committer sign-off
-- 
2.13.5