From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Intel-gfx@lists.freedesktop.org
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: [PATCH v2] dim: Automatically sign-off patches from others
Date: Tue, 26 Jan 2016 10:22:42 +0000 [thread overview]
Message-ID: <1453803762-28734-1-git-send-email-tvrtko.ursulin@linux.intel.com> (raw)
In-Reply-To: <20160125184352.GU11240@phenom.ffwll.local>
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
It is very easy to forget that commiters should sign-off
patches not authored by themselves.
v2: Missed to make sob variable local and align with dim
coding style. (Daniel Vetter)
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
---
...utomatically-sign-off-patches-from-others.patch | 34 ++++++++++++++++++++++
dim | 9 +++++-
2 files changed, 42 insertions(+), 1 deletion(-)
create mode 100644 0001-dim-Automatically-sign-off-patches-from-others.patch
diff --git a/0001-dim-Automatically-sign-off-patches-from-others.patch b/0001-dim-Automatically-sign-off-patches-from-others.patch
new file mode 100644
index 000000000000..2e9e4bfb39be
--- /dev/null
+++ b/0001-dim-Automatically-sign-off-patches-from-others.patch
@@ -0,0 +1,34 @@
+From d14e55a4fb687a1237b1251bcc3d5cc29e1a7f5e Mon Sep 17 00:00:00 2001
+From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
+Date: Mon, 25 Jan 2016 17:23:34 +0000
+Subject: [PATCH] dim: Automatically sign-off patches from others
+
+It is very easy to forget that commiters should sign-off
+patches not authored by themselves.
+
+Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
+Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
+---
+ dim | 6 +++++-
+ 1 file changed, 5 insertions(+), 1 deletion(-)
+
+diff --git a/dim b/dim
+index c749cebd1187..ff5d89ca5df8 100755
+--- a/dim
++++ b/dim
+@@ -380,7 +380,11 @@ function dim_apply
+
+ dim_shell_checkpatch "cat $file"
+
+- cat $file | git am -3 "$@"
++ local commiter_email=$(git config --get user.email)
++ local patch_from=$(grep "From:" "$file" | head -1)
++ [[ "$patch_from" == *"$commiter_email"* ]] || sob=-s
++
++ cat $file | git am -3 $sob "$@"
+
+ if [ -n "$message_id" ]; then
+ commit_add_tag "Link" "http://patchwork.freedesktop.org/patch/msgid/$message_id"
+--
+1.9.1
+
diff --git a/dim b/dim
index c749cebd1187..7d1d7b1952f8 100755
--- a/dim
+++ b/dim
@@ -380,7 +380,14 @@ function dim_apply
dim_shell_checkpatch "cat $file"
- cat $file | git am -3 "$@"
+ local commiter_email=$(git config --get user.email)
+ local patch_from=$(grep "From:" "$file" | head -1)
+ local sob
+ if [[ "$patch_from" != *"$commiter_email"* ]] ; then
+ sob=-s
+ fi
+
+ cat $file | git am -3 $sob "$@"
if [ -n "$message_id" ]; then
commit_add_tag "Link" "http://patchwork.freedesktop.org/patch/msgid/$message_id"
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2016-01-26 10:22 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-25 17:27 [PATCH] dim: Automatically sign-off patches from others Tvrtko Ursulin
2016-01-25 18:43 ` Daniel Vetter
2016-01-26 10:22 ` Tvrtko Ursulin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1453803762-28734-1-git-send-email-tvrtko.ursulin@linux.intel.com \
--to=tvrtko.ursulin@linux.intel.com \
--cc=Intel-gfx@lists.freedesktop.org \
--cc=daniel.vetter@ffwll.ch \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox