{"thread":{"id":"38610","subject":"[PATCH 2/2] Documentation/git-am.txt: mention mailinfo.scissor config variable","startedAt":"2015-02-20T10:09:06Z","lastAt":"2015-02-20T19:32:21Z","messageCount":9,"participants":["Matthieu Moy","Jeff King","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"256397","messageId":"1424426947-26441-1-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":null,"subject":"[PATCH 1/2] Documentation/config.txt: document mailinfo.scissor","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T10:09:06Z","receivedAt":"2015-02-20T10:09:06Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"The variale was documented in git-mailinfo.txt, but not in config.txt.\nThe detailed documentation is still the one of --scissors in\ngit-mailinfo.txt, but we give enough information here to let the user\nunderstand what it is about, and to make it easy to find it (e.g.\nsearching \">8\" and \"8<\" finds it).\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\n Documentation/config.txt | 7 +++++++\n 1 file changed, 7 insertions(+)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex ae6791d..4a2a078 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1740,6 +1740,13 @@ log.mailmap::\n \tIf true, makes linkgit:git-log[1], linkgit:git-show[1], and\n \tlinkgit:git-whatchanged[1] assume `--use-mailmap`.\n \n+mailinfo.scissor::\n+\tIf true, makes linkgit:git-mailinfo[1] (and therefore\n+\tlinkgit:git-am[1]) act by default as if the --scissors option\n+\twas provided on the command-line. When active, this features\n+\tremoves everything from the message body before a scissors\n+\tline (i.e. consisting mainly of \">8\", \"8<\" and \"-\").\n+\n mailmap.file::\n \tThe location of an augmenting mailmap file. The default\n \tmailmap, located in the root of the repository, is loaded\n-- \n2.3.0.157.g79e124b\n"},{"id":"256396","messageId":"1424426947-26441-2-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":"1424426947-26441-1-git-send-email-Matthieu.Moy@imag.fr","subject":"[PATCH 2/2] Documentation/git-am.txt: mention mailinfo.scissor config variable","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T10:09:07Z","receivedAt":"2015-02-20T10:09:07Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"It was already documented, but the user had to follow the link to\ngit-mailinfo.txt to find it.\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\nThis one may be more controversial. I think it makes sense to apply\nit, but I won't fight for it if you think it adds too much noise to\nthe doc.\n\n Documentation/git-am.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex f4eea28..1128570 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -52,7 +52,8 @@ OPTIONS\n -c::\n --scissors::\n \tRemove everything in body before a scissors line (see\n-\tlinkgit:git-mailinfo[1]).\n+\tlinkgit:git-mailinfo[1]). Can be activated by default using\n+\tthe `mailinfo.scissor` configuration variable.\n \n --no-scissors::\n \tIgnore scissors lines (see linkgit:git-mailinfo[1]).\n-- \n2.3.0.157.g79e124b\n"},{"id":"256399","messageId":"20150220101723.GA29980@peff.net","threadId":"38610","inReplyTo":"1424426947-26441-1-git-send-email-Matthieu.Moy@imag.fr","subject":"Re: [PATCH 1/2] Documentation/config.txt: document mailinfo.scissor","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-02-20T10:17:24Z","receivedAt":"2015-02-20T10:17:24Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Feb 20, 2015 at 11:09:06AM +0100, Matthieu Moy wrote:\n\n> The variale was documented in git-mailinfo.txt, but not in config.txt.\n\nMissing 'b' in variable.\n\n> +mailinfo.scissor::\n\ns/scissor/&s/\n\n-Peff\n"},{"id":"256400","messageId":"20150220101952.GB29980@peff.net","threadId":"38610","inReplyTo":"1424426947-26441-2-git-send-email-Matthieu.Moy@imag.fr","subject":"Re: [PATCH 2/2] Documentation/git-am.txt: mention mailinfo.scissor config variable","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-02-20T10:19:53Z","receivedAt":"2015-02-20T10:19:53Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Feb 20, 2015 at 11:09:07AM +0100, Matthieu Moy wrote:\n\n> It was already documented, but the user had to follow the link to\n> git-mailinfo.txt to find it.\n> \n> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n> ---\n> This one may be more controversial. I think it makes sense to apply\n> it, but I won't fight for it if you think it adds too much noise to\n> the doc.\n\nI recently looked up the option to reply to your other mail, knowing it\nexisted but forgetting the exact name. It took me several minutes to\nfind it. I'm in favor of this.\n\n>  --scissors::\n>  \tRemove everything in body before a scissors line (see\n> -\tlinkgit:git-mailinfo[1]).\n> +\tlinkgit:git-mailinfo[1]). Can be activated by default using\n> +\tthe `mailinfo.scissor` configuration variable.\n\nAs in the other patch, \"scissors\", with a trailing \"s\".\n\n-Peff\n"},{"id":"256403","messageId":"1424428143-641-1-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":"1424426947-26441-1-git-send-email-Matthieu.Moy@imag.fr","subject":"[PATCH v2 1/2] Documentation/config.txt: document mailinfo.scissor","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T10:29:02Z","receivedAt":"2015-02-20T10:29:02Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"The variable was documented in git-mailinfo.txt, but not in config.txt.\nThe detailed documentation is still the one of --scissors in\ngit-mailinfo.txt, but we give enough information here to let the user\nunderstand what it is about, and to make it easy to find it (e.g.\nsearching \">8\" and \"8<\" finds it).\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\nChanges since v1 for both patches : typos noted by Peff.\n\n Documentation/config.txt | 7 +++++++\n 1 file changed, 7 insertions(+)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex ae6791d..15f8695 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1740,6 +1740,13 @@ log.mailmap::\n \tIf true, makes linkgit:git-log[1], linkgit:git-show[1], and\n \tlinkgit:git-whatchanged[1] assume `--use-mailmap`.\n \n+mailinfo.scissors::\n+\tIf true, makes linkgit:git-mailinfo[1] (and therefore\n+\tlinkgit:git-am[1]) act by default as if the --scissors option\n+\twas provided on the command-line. When active, this features\n+\tremoves everything from the message body before a scissors\n+\tline (i.e. consisting mainly of \">8\", \"8<\" and \"-\").\n+\n mailmap.file::\n \tThe location of an augmenting mailmap file. The default\n \tmailmap, located in the root of the repository, is loaded\n-- \n2.3.0.157.g79e124b\n"},{"id":"256402","messageId":"1424428143-641-2-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":"1424428143-641-1-git-send-email-Matthieu.Moy@imag.fr","subject":"[PATCH v2 2/2] Documentation/git-am.txt: mention mailinfo.scissors config variable","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T10:29:03Z","receivedAt":"2015-02-20T10:29:03Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"It was already documented, but the user had to follow the link to\ngit-mailinfo.txt to find it.\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\n Documentation/git-am.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex f4eea28..0d8ba48 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -52,7 +52,8 @@ OPTIONS\n -c::\n --scissors::\n \tRemove everything in body before a scissors line (see\n-\tlinkgit:git-mailinfo[1]).\n+\tlinkgit:git-mailinfo[1]). Can be activated by default using\n+\tthe `mailinfo.scissors` configuration variable.\n \n --no-scissors::\n \tIgnore scissors lines (see linkgit:git-mailinfo[1]).\n-- \n2.3.0.157.g79e124b\n"},{"id":"256424","messageId":"CAPig+cToo3+JKAuxiCraPWD61eHw5W+xY=8H1UuWiy-8RtEAPQ@mail.gmail.com","threadId":"38610","inReplyTo":"1424428143-641-1-git-send-email-Matthieu.Moy@imag.fr","subject":"Re: [PATCH v2 1/2] Documentation/config.txt: document mailinfo.scissor","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-02-20T17:11:48Z","receivedAt":"2015-02-20T17:11:48Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Fri, Feb 20, 2015 at 5:29 AM, Matthieu Moy <Matthieu.Moy@imag.fr> wrote:\n> Subject: Documentation/config.txt: document mailinfo.scissor\n>\n> The variable was documented in git-mailinfo.txt, but not in config.txt.\n> The detailed documentation is still the one of --scissors in\n> git-mailinfo.txt, but we give enough information here to let the user\n> understand what it is about, and to make it easy to find it (e.g.\n> searching \">8\" and \"8<\" finds it).\n>\n> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n> ---\n> Changes since v1 for both patches : typos noted by Peff.\n\nThe same typo (s/scissor/scissors/) exists in the subjects.\n\n>  Documentation/config.txt | 7 +++++++\n>  1 file changed, 7 insertions(+)\n>\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index ae6791d..15f8695 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -1740,6 +1740,13 @@ log.mailmap::\n>         If true, makes linkgit:git-log[1], linkgit:git-show[1], and\n>         linkgit:git-whatchanged[1] assume `--use-mailmap`.\n>\n> +mailinfo.scissors::\n> +       If true, makes linkgit:git-mailinfo[1] (and therefore\n> +       linkgit:git-am[1]) act by default as if the --scissors option\n> +       was provided on the command-line. When active, this features\n> +       removes everything from the message body before a scissors\n> +       line (i.e. consisting mainly of \">8\", \"8<\" and \"-\").\n> +\n>  mailmap.file::\n>         The location of an augmenting mailmap file. The default\n>         mailmap, located in the root of the repository, is loaded\n> --\n> 2.3.0.157.g79e124b\n"},{"id":"256433","messageId":"1424460741-25533-1-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":"1424426947-26441-1-git-send-email-Matthieu.Moy@imag.fr","subject":"[PATCH v3 1/2] Documentation/config.txt: document mailinfo.scissors","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T19:32:20Z","receivedAt":"2015-02-20T19:32:20Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"The variable was documented in git-mailinfo.txt, but not in config.txt.\nThe detailed documentation is still the one of --scissors in\ngit-mailinfo.txt, but we give enough information here to let the user\nunderstand what it is about, and to make it easy to find it (e.g.\nsearching \">8\" and \"8<\" finds it).\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\nHopefully, I'd have fixed all intances of my typo now :-\\.\n\nSorry for the noise.\n\n Documentation/config.txt | 7 +++++++\n 1 file changed, 7 insertions(+)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex ae6791d..15f8695 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1740,6 +1740,13 @@ log.mailmap::\n \tIf true, makes linkgit:git-log[1], linkgit:git-show[1], and\n \tlinkgit:git-whatchanged[1] assume `--use-mailmap`.\n \n+mailinfo.scissors::\n+\tIf true, makes linkgit:git-mailinfo[1] (and therefore\n+\tlinkgit:git-am[1]) act by default as if the --scissors option\n+\twas provided on the command-line. When active, this features\n+\tremoves everything from the message body before a scissors\n+\tline (i.e. consisting mainly of \">8\", \"8<\" and \"-\").\n+\n mailmap.file::\n \tThe location of an augmenting mailmap file. The default\n \tmailmap, located in the root of the repository, is loaded\n-- \n2.3.0.157.g79e124b\n"},{"id":"256432","messageId":"1424460741-25533-2-git-send-email-Matthieu.Moy@imag.fr","threadId":"38610","inReplyTo":"1424460741-25533-1-git-send-email-Matthieu.Moy@imag.fr","subject":"[PATCH v3 2/2] Documentation/git-am.txt: mention mailinfo.scissors config variable","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@imag.fr","sentAt":"2015-02-20T19:32:21Z","receivedAt":"2015-02-20T19:32:21Z","isPatch":true,"sender":{"key":"git@matthieu-moy.fr","avatar":"https://avatars.githubusercontent.com/u/14709?v=4"},"body":"It was already documented, but the user had to follow the link to\ngit-mailinfo.txt to find it.\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\n---\n Documentation/git-am.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex f4eea28..0d8ba48 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -52,7 +52,8 @@ OPTIONS\n -c::\n --scissors::\n \tRemove everything in body before a scissors line (see\n-\tlinkgit:git-mailinfo[1]).\n+\tlinkgit:git-mailinfo[1]). Can be activated by default using\n+\tthe `mailinfo.scissors` configuration variable.\n \n --no-scissors::\n \tIgnore scissors lines (see linkgit:git-mailinfo[1]).\n-- \n2.3.0.157.g79e124b\n"}]}