[PATCH] Add --sign option to git-format-patch-script

2005-08-12 Thread Johannes Schindelin
This adds the option --sign to git-format-patch-script which adds
a Signed-off-by: line automatically.

Signed-off-by: Johannes Schindelin [EMAIL PROTECTED]

---

 git-format-patch-script |   12 +++-
 1 files changed, 11 insertions(+), 1 deletions(-)

a72d370d43c397238a26c1c7f260a3fd6401f4e6
diff --git a/git-format-patch-script b/git-format-patch-script
--- a/git-format-patch-script
+++ b/git-format-patch-script
@@ -6,7 +6,7 @@
 . git-sh-setup-script || die Not a git archive.
 
 usage () {
-echo 2 usage: $0' [-n] [-o dir] [--mbox] [--check] [-diff 
options...] upstream [ our-head ]
+echo 2 usage: $0' [-n] [-o dir] [--mbox] [--check] [--sign] [-diff 
options...] upstream [ our-head ]
 
 Prepare each commit with its patch since our-head forked from upstream,
 one file per patch, for e-mail submission.  Each output file is
@@ -46,6 +46,8 @@ do
 date=t author=t mbox=t ;;
 -n|--n|--nu|--num|--numb|--numbe|--number|--numbere|--numbered)
 numbered=t ;;
+-s|--s|--si|--sig|--sign)
+signoff=t ;;
 -o=*|--o=*|--ou=*|--out=*|--outp=*|--outpu=*|--output=*|--output-=*|\
 --output-d=*|--output-di=*|--output-dir=*|--output-dire=*|\
 --output-direc=*|--output-direct=*|--output-directo=*|\
@@ -174,6 +176,14 @@ Date: '$ad
b body'
 
sed -ne $mailScript $commsg
+
+   test $signoff = t  {
+   offsigner=`git-var GIT_COMMITTER_IDENT | sed -e 's/.*//'`
+   echo
+   echo Signed-off-by: $offsigner
+   echo
+   }
+
echo '---'
echo
git-diff-tree -p $diff_opts $commit | git-apply --stat --summary
-
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] Add --sign option to git-format-patch-script

2005-08-12 Thread Junio C Hamano
Good intentions, but I'd rather see these S-O-B lines in the
actual commit objects.  Giving format-patch this option would
discourage people to do so.  Maybe a patch to git commit would
be more appropriate, methinks.


-
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] Add --sign option to git-format-patch-script

2005-08-12 Thread Johannes Schindelin
Hi,

On Fri, 12 Aug 2005, Junio C Hamano wrote:

 Good intentions, but I'd rather see these S-O-B lines in the
 actual commit objects.  Giving format-patch this option would
 discourage people to do so.  Maybe a patch to git commit would
 be more appropriate, methinks.

Maybe in addition to this one? I meant that option to be for on-passers of 
commits they are not author of.

Ciao,
Dscho

-
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html