{"thread":{"id":"18670","subject":"[PATCH] Add configuration variable for sign-off to format-patch","startedAt":"2009-03-31T18:50:19Z","lastAt":"2009-04-07T07:23:27Z","messageCount":13,"participants":["Heiko Voigt","Andreas Ericsson","Jeff King","Sverre Rabbelier","Junio C Hamano","Matthieu Moy"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"110049","messageId":"20090331185018.GD72569@macbook.lan","threadId":"18670","inReplyTo":null,"subject":"[PATCH] Add configuration variable for sign-off to format-patch","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2009-03-31T18:50:19Z","receivedAt":"2009-03-31T18:50:19Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"If you regularly create patches which require a Signed-off: line you may\nwant to make it your default to add that line. It also helps you not to forget\nto add the -s/--signoff switch.\n\nSigned-off-by: Heiko Voigt <hvoigt@hvoigt.net>\n---\n\nBecause it happened to me twice already that I forgot to add the switch.\n\n Documentation/config.txt           |    4 ++++\n Documentation/git-format-patch.txt |    1 +\n builtin-log.c                      |   23 ++++++++++++++++-------\n 3 files changed, 21 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex ad22cb8..ed85536 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -715,6 +715,10 @@ format.thread::\n \tA true boolean value is the same as `shallow`, and a false\n \tvalue disables threading.\n \n+format.signoff::\n+\tThis lets you enable the -s/--signoff option of format-patch by\n+\tdefault. It takes a boolean value.\n+\n gc.aggressiveWindow::\n \tThe window size parameter used in the delta compression\n \talgorithm used by 'git-gc --aggressive'.  This defaults\ndiff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt\nindex c2eb5fa..c25ea10 100644\n--- a/Documentation/git-format-patch.txt\n+++ b/Documentation/git-format-patch.txt\n@@ -197,6 +197,7 @@ more than one.\n \tnumbered = auto\n \tcc = <email>\n \tattach [ = mime-boundary-string ]\n+\tsignoff = true\n ------------\n \n \ndiff --git a/builtin-log.c b/builtin-log.c\nindex c7a5772..d77b7fb 100644\n--- a/builtin-log.c\n+++ b/builtin-log.c\n@@ -465,6 +465,7 @@ static void add_header(const char *value)\n #define THREAD_SHALLOW 1\n #define THREAD_DEEP 2\n static int thread = 0;\n+static int do_signoff = 0;\n \n static int git_format_config(const char *var, const char *value, void *cb)\n {\n@@ -514,6 +515,10 @@ static int git_format_config(const char *var, const char *value, void *cb)\n \t\tthread = git_config_bool(var, value) && THREAD_SHALLOW;\n \t\treturn 0;\n \t}\n+\tif (!strcmp(var, \"format.signoff\")) {\n+\t\tdo_signoff = git_config_bool(var, value);\n+\t\treturn 0;\n+\t}\n \n \treturn git_log_config(var, value, cb);\n }\n@@ -865,13 +870,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)\n \t\t}\n \t\telse if (!strcmp(argv[i], \"--signoff\") ||\n \t\t\t !strcmp(argv[i], \"-s\")) {\n-\t\t\tconst char *committer;\n-\t\t\tconst char *endpos;\n-\t\t\tcommitter = git_committer_info(IDENT_ERROR_ON_NO_NAME);\n-\t\t\tendpos = strchr(committer, '>');\n-\t\t\tif (!endpos)\n-\t\t\t\tdie(\"bogus committer info %s\", committer);\n-\t\t\tadd_signoff = xmemdupz(committer, endpos - committer + 1);\n+\t\t\tdo_signoff = 1;\n \t\t}\n \t\telse if (!strcmp(argv[i], \"--attach\")) {\n \t\t\trev.mime_boundary = git_version_string;\n@@ -923,6 +922,16 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)\n \t}\n \targc = j;\n \n+\tif(do_signoff) {\n+\t\tconst char *committer;\n+\t\tconst char *endpos;\n+\t\tcommitter = git_committer_info(IDENT_ERROR_ON_NO_NAME);\n+\t\tendpos = strchr(committer, '>');\n+\t\tif (!endpos)\n+\t\t\tdie(\"bogus committer info %s\", committer);\n+\t\tadd_signoff = xmemdupz(committer, endpos - committer + 1);\n+\t}\n+\n \tfor (i = 0; i < extra_hdr_nr; i++) {\n \t\tstrbuf_addstr(&buf, extra_hdr[i]);\n \t\tstrbuf_addch(&buf, '\\n');\n-- \n1.6.2.1.424.g0b27.dirty\n"},{"id":"110054","messageId":"49D27359.1040703@op5.se","threadId":"18670","inReplyTo":"20090331185018.GD72569@macbook.lan","subject":"Re: [PATCH] Add configuration variable for sign-off to format-patch","fromName":"Andreas Ericsson","fromEmail":"ae@op5.se","sentAt":"2009-03-31T19:47:37Z","receivedAt":"2009-03-31T19:47:37Z","isPatch":true,"sender":{"key":"ae@op5.se","avatar":"https://gravatar.com/avatar/426e89595c75a8f5252dd0c989e5fabe5bcac616e68557427ad9aef6b0ca342a?d=mp&s=160"},"body":"Heiko Voigt wrote:\n> If you regularly create patches which require a Signed-off: line you may\n> want to make it your default to add that line. It also helps you not to forget\n> to add the -s/--signoff switch.\n> \n> Signed-off-by: Heiko Voigt <hvoigt@hvoigt.net>\n> ---\n> \n> Because it happened to me twice already that I forgot to add the switch.\n> \n>  Documentation/config.txt           |    4 ++++\n>  Documentation/git-format-patch.txt |    1 +\n>  builtin-log.c                      |   23 ++++++++++++++++-------\n>  3 files changed, 21 insertions(+), 7 deletions(-)\n> \n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index ad22cb8..ed85536 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -715,6 +715,10 @@ format.thread::\n>  \tA true boolean value is the same as `shallow`, and a false\n>  \tvalue disables threading.\n>  \n> +format.signoff::\n> +\tThis lets you enable the -s/--signoff option of format-patch by\n> +\tdefault. It takes a boolean value.\n> +\n\nCan we please make it \"formatpatch.signoff\" or some such instead? Just\nplain \"format\" is a bit too generic for my taste.\n\n-- \nAndreas Ericsson                   andreas.ericsson@op5.se\nOP5 AB                             www.op5.se\nTel: +46 8-230225                  Fax: +46 8-230231\n\nConsidering the successes of the wars on alcohol, poverty, drugs and\nterror, I think we should give some serious thought to declaring war\non peace.\n"},{"id":"110056","messageId":"20090331200457.GA23879@coredump.intra.peff.net","threadId":"18670","inReplyTo":"20090331185018.GD72569@macbook.lan","subject":"Re: [PATCH] Add configuration variable for sign-off to format-patch","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-03-31T20:04:57Z","receivedAt":"2009-03-31T20:04:57Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Mar 31, 2009 at 08:50:19PM +0200, Heiko Voigt wrote:\n\n> If you regularly create patches which require a Signed-off: line you may\n> want to make it your default to add that line. It also helps you not to forget\n> to add the -s/--signoff switch.\n\nI personally have no problem with such an option, but it has been argued\nagainst before. E.g.:\n\n  http://article.gmane.org/gmane.comp.version-control.git/32522\n  http://article.gmane.org/gmane.comp.version-control.git/51780\n\nThere is also some discussion here indicating that it might be accepted:\n\n  http://article.gmane.org/gmane.comp.version-control.git/103939\n\n-Peff\n"},{"id":"110065","messageId":"20090331204338.GA88381@macbook.lan","threadId":"18670","inReplyTo":"20090331200457.GA23879@coredump.intra.peff.net","subject":"Re: [PATCH] Add configuration variable for sign-off to format-patch","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2009-03-31T20:43:48Z","receivedAt":"2009-03-31T20:43:48Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Tue, Mar 31, 2009 at 04:04:57PM -0400, Jeff King wrote:\n> On Tue, Mar 31, 2009 at 08:50:19PM +0200, Heiko Voigt wrote:\n> \n> > If you regularly create patches which require a Signed-off: line you may\n> > want to make it your default to add that line. It also helps you not to forget\n> > to add the -s/--signoff switch.\n> \n> I personally have no problem with such an option, but it has been argued\n> against before. E.g.:\n> \n>   http://article.gmane.org/gmane.comp.version-control.git/32522\n>   http://article.gmane.org/gmane.comp.version-control.git/51780\n> \n> There is also some discussion here indicating that it might be accepted:\n> \n>   http://article.gmane.org/gmane.comp.version-control.git/103939\n\nI see, it is of course true that you should not just sign off\neverything without thinking about it.\n\nHowever I always read through my messages before sending them and it is\nway easier to delete that line than typing/copying it.\n\nI suppose if A changes a patch which originated from B he also needs to\nbe nice and delete that the sign-off line from B (at least initially).\n\nI don't know but if I am preparing to send a patch and *see* my\nsigned-off line in the message I am conscious about it! I see this more\nas another customization. Some people tend to forget to add the\nsigned-off and others might tend to forget to delete it. It should be on\nyou to find out what type you are.\n\ncheers Heiko\n"},{"id":"110118","messageId":"20090401102610.GC26181@coredump.intra.peff.net","threadId":"18670","inReplyTo":"20090331204338.GA88381@macbook.lan","subject":"Re: [PATCH] Add configuration variable for sign-off to format-patch","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-04-01T10:26:10Z","receivedAt":"2009-04-01T10:26:10Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Mar 31, 2009 at 10:43:48PM +0200, Heiko Voigt wrote:\n\n> I see, it is of course true that you should not just sign off\n> everything without thinking about it.\n> \n> However I always read through my messages before sending them and it is\n> way easier to delete that line than typing/copying it.\n\nI agree. Personally, the concept of writing something that I _couldn't_\nsign off on is so foreign to me that I would be very conscious of the\nfact, and it would be simple to remove it during the final proof-read of\nthe patch that I do.\n\nI think to satisfy everyone, though, you need some extra text in the\ndocumentation indicating that signing off is supposed to be conscious,\nand you should understand the implications of setting the variable.\n\n-Peff\n"},{"id":"110154","messageId":"20090401175153.GA90421@macbook.lan","threadId":"18670","inReplyTo":"20090401102610.GC26181@coredump.intra.peff.net","subject":"[PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2009-04-01T17:51:54Z","receivedAt":"2009-04-01T17:51:54Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"If you regularly create patches which require a Signed-off: line you may\nwant to make it your default to add that line. It also helps you not to forget\nto add the -s/--signoff switch.\n\nSigned-off-by: Heiko Voigt <hvoigt@hvoigt.net>\n---\nThis adds a note about the purpose of the Signed-off-by: line to the\nDocumentation this is the interdiff:\n\n diff --git a/Documentation/config.txt b/Documentation/config.txt\n index 9ccc28c..27cb7f1 100644\n --- a/Documentation/config.txt\n +++ b/Documentation/config.txt\n @@ -717,7 +717,10 @@ format.thread::\n  \n  format.signoff::\n      A boolean value which lets you enable the `-s/--signoff` option of\n -    format-patch by default.\n +    format-patch by default. *Note:* Adding the Signed-off-by: line to a\n +    patch should be a conscious act and means that you certify you have\n +    the rights to submit this work under the same open source license.\n +    Please see the 'SubmittingPatches' document for further discussion.\n  \n  gc.aggressiveWindow::\n  \tThe window size parameter used in the delta compression\n\n Documentation/config.txt           |    7 +++++++\n Documentation/git-format-patch.txt |    1 +\n builtin-log.c                      |   23 ++++++++++++++++-------\n 3 files changed, 24 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex ad22cb8..27cb7f1 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -715,6 +715,13 @@ format.thread::\n \tA true boolean value is the same as `shallow`, and a false\n \tvalue disables threading.\n \n+format.signoff::\n+    A boolean value which lets you enable the `-s/--signoff` option of\n+    format-patch by default. *Note:* Adding the Signed-off-by: line to a\n+    patch should be a conscious act and means that you certify you have\n+    the rights to submit this work under the same open source license.\n+    Please see the 'SubmittingPatches' document for further discussion.\n+\n gc.aggressiveWindow::\n \tThe window size parameter used in the delta compression\n \talgorithm used by 'git-gc --aggressive'.  This defaults\ndiff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt\nindex c2eb5fa..c25ea10 100644\n--- a/Documentation/git-format-patch.txt\n+++ b/Documentation/git-format-patch.txt\n@@ -197,6 +197,7 @@ more than one.\n \tnumbered = auto\n \tcc = <email>\n \tattach [ = mime-boundary-string ]\n+\tsignoff = true\n ------------\n \n \ndiff --git a/builtin-log.c b/builtin-log.c\nindex c7a5772..d77b7fb 100644\n--- a/builtin-log.c\n+++ b/builtin-log.c\n@@ -465,6 +465,7 @@ static void add_header(const char *value)\n #define THREAD_SHALLOW 1\n #define THREAD_DEEP 2\n static int thread = 0;\n+static int do_signoff = 0;\n \n static int git_format_config(const char *var, const char *value, void *cb)\n {\n@@ -514,6 +515,10 @@ static int git_format_config(const char *var, const char *value, void *cb)\n \t\tthread = git_config_bool(var, value) && THREAD_SHALLOW;\n \t\treturn 0;\n \t}\n+\tif (!strcmp(var, \"format.signoff\")) {\n+\t\tdo_signoff = git_config_bool(var, value);\n+\t\treturn 0;\n+\t}\n \n \treturn git_log_config(var, value, cb);\n }\n@@ -865,13 +870,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)\n \t\t}\n \t\telse if (!strcmp(argv[i], \"--signoff\") ||\n \t\t\t !strcmp(argv[i], \"-s\")) {\n-\t\t\tconst char *committer;\n-\t\t\tconst char *endpos;\n-\t\t\tcommitter = git_committer_info(IDENT_ERROR_ON_NO_NAME);\n-\t\t\tendpos = strchr(committer, '>');\n-\t\t\tif (!endpos)\n-\t\t\t\tdie(\"bogus committer info %s\", committer);\n-\t\t\tadd_signoff = xmemdupz(committer, endpos - committer + 1);\n+\t\t\tdo_signoff = 1;\n \t\t}\n \t\telse if (!strcmp(argv[i], \"--attach\")) {\n \t\t\trev.mime_boundary = git_version_string;\n@@ -923,6 +922,16 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)\n \t}\n \targc = j;\n \n+\tif(do_signoff) {\n+\t\tconst char *committer;\n+\t\tconst char *endpos;\n+\t\tcommitter = git_committer_info(IDENT_ERROR_ON_NO_NAME);\n+\t\tendpos = strchr(committer, '>');\n+\t\tif (!endpos)\n+\t\t\tdie(\"bogus committer info %s\", committer);\n+\t\tadd_signoff = xmemdupz(committer, endpos - committer + 1);\n+\t}\n+\n \tfor (i = 0; i < extra_hdr_nr; i++) {\n \t\tstrbuf_addstr(&buf, extra_hdr[i]);\n \t\tstrbuf_addch(&buf, '\\n');\n-- \n1.6.2.1.424.g0b27.dirty\n"},{"id":"110155","messageId":"20090401175355.GB90421@macbook.lan","threadId":"18670","inReplyTo":"49D27359.1040703@op5.se","subject":"Re: [PATCH] Add configuration variable for sign-off to format-patch","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2009-04-01T17:53:55Z","receivedAt":"2009-04-01T17:53:55Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Tue, Mar 31, 2009 at 09:47:37PM +0200, Andreas Ericsson wrote:\n> Can we please make it \"formatpatch.signoff\" or some such instead? Just\n> plain \"format\" is a bit too generic for my taste.\n\nBut all options for format-patch are already beginning with format. So\nprobably leave it like this for consistency reasons.\n\ncheers Heiko\n"},{"id":"110156","messageId":"fabb9a1e0904011055u67b3ebfbuae15c8e23d7e5ed0@mail.gmail.com","threadId":"18670","inReplyTo":"20090401175153.GA90421@macbook.lan","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-04-01T17:55:43Z","receivedAt":"2009-04-01T17:55:43Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Wed, Apr 1, 2009 at 19:51, Heiko Voigt <hvoigt@hvoigt.net> wrote:\n>  format.signoff::\n>      A boolean value which lets you enable the `-s/--signoff` option of\n>  -    format-patch by default.\n>  +    format-patch by default. *Note:* Adding the Signed-off-by: line to a\n>  +    patch should be a conscious act and means that you certify you have\n>  +    the rights to submit this work under the same open source license.\n>  +    Please see the 'SubmittingPatches' document for further discussion.\n\nWhich brings us to the question... what about projects that do not\nhave such a file?\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"110524","messageId":"7veiw69p26.fsf@gitster.siamese.dyndns.org","threadId":"18670","inReplyTo":"20090401175153.GA90421@macbook.lan","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-04-06T08:14:25Z","receivedAt":"2009-04-06T08:14:25Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Heiko Voigt <hvoigt@hvoigt.net> writes:\n\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index ad22cb8..27cb7f1 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -715,6 +715,13 @@ format.thread::\n>  \tA true boolean value is the same as `shallow`, and a false\n>  \tvalue disables threading.\n>  \n> +format.signoff::\n> +    A boolean value which lets you enable the `-s/--signoff` option of\n> +    format-patch by default. *Note:* Adding the Signed-off-by: line to a\n> +    patch should be a conscious act and means that you certify you have\n> +    the rights to submit this work under the same open source license.\n> +    Please see the 'SubmittingPatches' document for further discussion.\n\nI have a mixed feeling about this description.  The existing description\non the --signoff option merely talks about what it does, leaving what it\nmeans, and it is quite deliberate.  If your project uses S-o-b, it may be\nuseful.  If yours doesn't, you simply just don't use it.  It does not\nmatter to _us_ as the document writer what that line means to your\nproject.\n\nWe do want to make the reader think twice iff S-o-b is used in the\nreader's project with the same meaning as it means in git and the Linux\nkernel project, which is what the description you added is about.  But\nshould we just assume if anybody uses S-o-b convention in their project\nthey must give it the same meaning as we give it?\n\nThe patch looks straightforward enough, and the wording we can update if\nsomebody can come up with a better one, so I'll apply the patch to\n'master' and we will go from there.\n\nThanks.\n"},{"id":"110554","messageId":"49D9E040.40007@op5.se","threadId":"18670","inReplyTo":"7veiw69p26.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Andreas Ericsson","fromEmail":"exon@op5.com","sentAt":"2009-04-06T10:58:08Z","receivedAt":"2009-04-06T10:58:08Z","isPatch":true,"sender":{"key":"exon@op5.com","avatar":null},"body":"Junio C Hamano wrote:\n> Heiko Voigt <hvoigt@hvoigt.net> writes:\n> \n>> diff --git a/Documentation/config.txt b/Documentation/config.txt\n>> index ad22cb8..27cb7f1 100644\n>> --- a/Documentation/config.txt\n>> +++ b/Documentation/config.txt\n>> @@ -715,6 +715,13 @@ format.thread::\n>>  \tA true boolean value is the same as `shallow`, and a false\n>>  \tvalue disables threading.\n>>  \n>> +format.signoff::\n>> +    A boolean value which lets you enable the `-s/--signoff` option of\n>> +    format-patch by default. *Note:* Adding the Signed-off-by: line to a\n>> +    patch should be a conscious act and means that you certify you have\n>> +    the rights to submit this work under the same open source license.\n>> +    Please see the 'SubmittingPatches' document for further discussion.\n> \n> I have a mixed feeling about this description.  The existing description\n> on the --signoff option merely talks about what it does, leaving what it\n> means, and it is quite deliberate.  If your project uses S-o-b, it may be\n> useful.  If yours doesn't, you simply just don't use it.  It does not\n> matter to _us_ as the document writer what that line means to your\n> project.\n> \n> We do want to make the reader think twice iff S-o-b is used in the\n> reader's project with the same meaning as it means in git and the Linux\n> kernel project, which is what the description you added is about.  But\n> should we just assume if anybody uses S-o-b convention in their project\n> they must give it the same meaning as we give it?\n> \n> The patch looks straightforward enough, and the wording we can update if\n> somebody can come up with a better one, so I'll apply the patch to\n> 'master' and we will go from there.\n> \n\nHow about:\n\"Some projects (notably the Linux kernel and git itself) put special\nmeaning in a 'Signed-off-by' line while other's dont. Please refer\nto your project's documentation for appropriate behaviour.\"\n\nOptionally with the following amendment:\n\n\"In general, you should refrain from signing off on a patch containing\nmaterial that you're not sure can be legally spread under the project's\nlicense.\"\n\nI'll whip up a patch if someone cares enough about it to say \"+1\" to\neither proposal, or make a better one.\n\n-- \nAndreas Ericsson                   andreas.ericsson@op5.se\nOP5 AB                             www.op5.se\nTel: +46 8-230225                  Fax: +46 8-230231\n\nConsidering the successes of the wars on alcohol, poverty, drugs and\nterror, I think we should give some serious thought to declaring war\non peace.\n"},{"id":"110556","messageId":"vpqvdpi5807.fsf@bauges.imag.fr","threadId":"18670","inReplyTo":"49D9E040.40007@op5.se","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2009-04-06T11:36:24Z","receivedAt":"2009-04-06T11:36:24Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"Andreas Ericsson <exon@op5.com> writes:\n\n> How about:\n> \"Some projects (notably the Linux kernel and git itself) put special\n> meaning in a 'Signed-off-by' line while other's dont. Please refer\n> to your project's documentation for appropriate behaviour.\"\n>\n> Optionally with the following amendment:\n>\n> \"In general, you should refrain from signing off on a patch containing\n> material that you're not sure can be legally spread under the project's\n> license.\"\n\nTo me, this is at least an improvement over refering to\nSubmittingPatches (which is targeted to Git contributors) in the\ndocumentation (targeted to Git users), yes.\n\nNot sure whether these two sentences should come in the documentation\nfor the config option or the command-line switch.\n\n-- \nMatthieu\n"},{"id":"110611","messageId":"20090406180925.GA466@macbook.lan","threadId":"18670","inReplyTo":"vpqvdpi5807.fsf@bauges.imag.fr","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2009-04-06T18:09:25Z","receivedAt":"2009-04-06T18:09:25Z","isPatch":true,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Mon, Apr 06, 2009 at 01:36:24PM +0200, Matthieu Moy wrote:\n> Andreas Ericsson <exon@op5.com> writes:\n> \n> > How about:\n> > \"Some projects (notably the Linux kernel and git itself) put special\n> > meaning in a 'Signed-off-by' line while other's dont. Please refer\n> > to your project's documentation for appropriate behaviour.\"\n> >\n> > Optionally with the following amendment:\n> >\n> > \"In general, you should refrain from signing off on a patch containing\n> > material that you're not sure can be legally spread under the project's\n> > license.\"\n> \n> To me, this is at least an improvement over refering to\n> SubmittingPatches (which is targeted to Git contributors) in the\n> documentation (targeted to Git users), yes.\n\nTo me too. When writing I was probably too focused on Git.\n\n> Not sure whether these two sentences should come in the documentation\n> for the config option or the command-line switch.\n\nI think you should be able to get to this note from all locations where\nthis option is provided (+ the commit option) so maybe a link ?\n\ncheers Heiko\n"},{"id":"110664","messageId":"20090407072327.GA5239@coredump.intra.peff.net","threadId":"18670","inReplyTo":"49D9E040.40007@op5.se","subject":"Re: [PATCH v2] Add configuration variable for sign-off to format-patch","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-04-07T07:23:27Z","receivedAt":"2009-04-07T07:23:27Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Apr 06, 2009 at 12:58:08PM +0200, Andreas Ericsson wrote:\n\n> How about:\n> \"Some projects (notably the Linux kernel and git itself) put special\n> meaning in a 'Signed-off-by' line while other's dont. Please refer\n> to your project's documentation for appropriate behaviour.\"\n\nI think that is fine, with some typo fixups:\n\n  s/other's/others/\n  s/dont/don't/\n\nAnd of course being American, I would spell it behavior. :)\n\n> Optionally with the following amendment:\n>\n> \"In general, you should refrain from signing off on a patch containing\n> material that you're not sure can be legally spread under the project's\n> license.\"\n\nI don't think that makes sense. That is what signing-off is about for\nthe kernel and for git, but I don't think there is any reason another\nproject might not want to use the convenient signoff options to mean\nsomething totally unrelated (like we use Acked-by).\n\n-Peff\n"}]}