threads / patch / 22190

patchAdd `commit.signoff` configuration variable.

Subject: [PATCH] Add `commit.signoff` configuration variable.

## tl;dr

3 messages between Jan 13, 2010 and Jan 13, 2010. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Steven Drake· Jan 13, 2010, 03:36 UTC · lore
Signed-off-by: Steven Drake <sdrake@xnet.co.nz>
---
 Documentation/config.txt     |    4 ++++
 Documentation/git-commit.txt |    2 +-
 builtin-commit.c             |    4 ++++
 3 files changed, 9 insertions(+), 1 deletions(-)
Show changes to 3 files +9 −1

Documentation/config.txt, Documentation/git-commit.txt, builtin-commit.c

diff --git a/Documentation/config.txt b/Documentation/config.txt
index 23a965e..dd261cf 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -712,6 +712,10 @@ color.ui::
 	terminal. When more specific variables of color.* are set, they always
 	take precedence over this setting. Defaults to false.
 
+commit.signoff::
+	Add Signed-off-by line by the committer at the end of the commit
+	log message.
+
 commit.template::
 	Specify a file to use as the template for new commit messages.
 	"{tilde}/" is expanded to the value of `$HOME` and "{tilde}user/" to the
diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt
index c97c151..5a977b6 100644
--- a/Documentation/git-commit.txt
+++ b/Documentation/git-commit.txt
@@ -114,7 +114,7 @@ OPTIONS
 -s::
 --signoff::
 	Add Signed-off-by line by the committer at the end of the commit
-	log message.
+	log message. This overrides the `commit.signoff` configuration variable.
 
 -n::
 --no-verify::
diff --git a/builtin-commit.c b/builtin-commit.c
index 3dfcd77..db90e7a 100644
--- a/builtin-commit.c
+++ b/builtin-commit.c
@@ -1093,6 +1093,10 @@ static int git_commit_config(const char *k, const char *v, void *cb)
 
 	if (!strcmp(k, "commit.template"))
 		return git_config_pathname(&template_file, k, v);
+	if (!strcmp(k, "commit.signoff")) {
+		signoff = git_config_bool(k, v);
+		return 0;
+	}
 
 	return git_status_config(k, v, s);
 }
-- 
1.6.4
Steven Drake· Jan 13, 2010, 04:36 UTC · re: Steven Drake · lore

Re: [PATCH] Add `commit.signoff` configuration variable.

On Wed, 13 Jan 2010, Steven Drake wrote:
> Signed-off-by: Steven Drake <sdrake@xnet.co.nz>

I ment to make a note with that patch that the commit message Signed-off-by was added using using the `commit.signoff` feature.

-- 
Steven
  1: Linux - will work for fish.
  2: The Linux penguin - looks stuffed to the brim with herring.
  ( make your own conclusions )
Junio C Hamano· Jan 13, 2010, 08:30 UTC · re: Steven Drake · lore

Re: [PATCH] Add `commit.signoff` configuration variable.

You need to defend this one better, in fact a *lot* better, than other ones. See

  http://thread.gmane.org/gmane.comp.version-control.git/32503/focus=32522
  http://thread.gmane.org/gmane.comp.version-control.git/103939/focus=104207

for starters. Your justifications should address all the points raised in previous threads.

← back to recent threads