[PATCH 2/2] dim: Fix outlook_author variable.
Rodrigo Vivi
rodrigo.vivi at intel.com
Thu Feb 15 19:14:10 UTC 2018
The current version author_outlook ends up like
"First, Last," instead of "First Last" as it should be.
Honestly I'm so glad that this never worked. So I had a chance
to contact authors before merging the patch and fix the
outlook + patchwork mess.
But anyways, if we decide to stay with this logic let's at least
fix it.
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dim b/dim
index 4493b48b1494..ed8b9b03c42a 100755
--- a/dim
+++ b/dim
@@ -732,7 +732,7 @@ function checkpatch_commit_push
author=$(git show -s $sha1 --format="format:%an")
committer=$(git show -s $sha1 --format="format:%cn")
# outlook mangles mails into "Last, First"
- author_outlook=$(git show -s $sha1 --format="format:%an" | sed -e 's/\([^ ]*\) \(.*\)/\2, \1/')
+ author_outlook=$(git show -s $sha1 --format="format:%an" | sed -e 's/\([^ ]*\) \(.*\)/\2 \1/;s/,//')
# check for author sign-off
if ! git show -s $sha1 | grep -qi "S.*-by:.*\\($author\\|$author_outlook\\)" ; then
--
2.13.6
More information about the dim-tools
mailing list