All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.