# [PATCH] git-commit: Having $GIT_COMMITTER_NAME implies -s

3 messages from 2005-11-16 to 2005-11-17. Participants: Andreas Ericsson, Junio C Hamano, H. Peter Anvin.
Thread: https://gitlist.dev/t/2560

## Andreas Ericsson, 2005-11-16 23:38

Subject: [PATCH] git-commit: Having $GIT_COMMITTER_NAME implies -s
Message-ID: <20051116233829.0B3C55BA81@nox.op5.se>
URL: https://gitlist.dev/e/20051116233829.0B3C55BA81%40nox.op5.se

```

I've been trying, in vain, to tell my colleagues to use the -s flag.
With this patch I can at least fix up their .profile's myself.

Signed-off-by: Andreas Ericsson <ae@op5.se>

---

 git-commit.sh |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

applies-to: a2fe76123d6f5b835d3312a70a02ed4d07e25f8e
d32c78570e25a9718721937b1ee9051e950ad1fd
diff --git a/git-commit.sh b/git-commit.sh
index 41955e8..ff9502c 100755
--- a/git-commit.sh
+++ b/git-commit.sh
@@ -134,8 +134,7 @@ then
 	cat "$GIT_DIR/MERGE_MSG"
 fi | git-stripspace >"$GIT_DIR"/COMMIT_EDITMSG
 
-case "$signoff" in
-t)
+if [ "$signoff" = t -o "$GIT_COMMITTER_NAME" ]; then
 	{
 		echo
 		git-var GIT_COMMITTER_IDENT | sed -e '
@@ -143,8 +142,7 @@ t)
 			s/^/Signed-off-by: /
 		'
 	} >>"$GIT_DIR"/COMMIT_EDITMSG
-	;;
-esac
+fi
 
 if [ -f "$GIT_DIR/MERGE_HEAD" ]; then
 	echo "#"
---
0.99.9.GIT

```

## Junio C Hamano, 2005-11-17 04:28

Subject: Re: [PATCH] git-commit: Having $GIT_COMMITTER_NAME implies -s
Message-ID: <7vbr0j3nku.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vbr0j3nku.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <20051116233829.0B3C55BA81@nox.op5.se>

```
I am not so sure about this.  A Signed-off-by line used in the
kernel and git projects has a specific meaning; you've read
SubmittingPatches in either projects, have'nt you?

I do work on a machine from which I make commits to both git
project and my day-job work project.  I do not necessarily want
to add Signed-off-by line in commits I make for the latter.

```

## H. Peter Anvin, 2005-11-17 05:27

Subject: Re: [PATCH] git-commit: Having $GIT_COMMITTER_NAME implies -s
Message-ID: <437C14DA.8000905@zytor.com>
URL: https://gitlist.dev/e/437C14DA.8000905%40zytor.com
In-Reply-To: <7vbr0j3nku.fsf@assigned-by-dhcp.cox.net>

```
Junio C Hamano wrote:
> I am not so sure about this.  A Signed-off-by line used in the
> kernel and git projects has a specific meaning; you've read
> SubmittingPatches in either projects, have'nt you?
> 
> I do work on a machine from which I make commits to both git
> project and my day-job work project.  I do not necessarily want
> to add Signed-off-by line in commits I make for the latter.
> 

More importantly, it's vital that a Signed-off-by: line is always added 
with explicit awareness of the signer.

	-hpa

```