{"thread":{"id":"29943","subject":"[PATCH 1/2] Documentation/diff-options: fix typo in --submodule text","startedAt":"2012-03-13T19:00:50Z","lastAt":"2012-03-14T18:47:55Z","messageCount":5,"participants":["Tim Henigan","Junio C Hamano","Jens Lehmann"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"186892","messageId":"1331665251-11147-1-git-send-email-tim.henigan@gmail.com","threadId":"29943","inReplyTo":null,"subject":"[PATCH 1/2] Documentation/diff-options: fix typo in --submodule text","fromName":"Tim Henigan","fromEmail":"tim.henigan@gmail.com","sentAt":"2012-03-13T19:00:50Z","receivedAt":"2012-03-13T19:00:50Z","isPatch":true,"sender":{"key":"tim.henigan@gmail.com","avatar":"https://avatars.githubusercontent.com/u/42022?v=4"},"body":"Signed-off-by: Tim Henigan \n---\n\nResent due to typo in developer list email address.\n\n Documentation/diff-options.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\nindex 7d4566f..f44f3fc 100644\n--- a/Documentation/diff-options.txt\n+++ b/Documentation/diff-options.txt\n@@ -165,7 +165,7 @@ any of those replacements occurred.\n \tof the `--diff-filter` option on what the status letters mean.\n \n --submodule[=]::\n-\tChose the output format for submodule differences. can be one of\n+\tChoose the output format for submodule differences. can be one of\n \t'short' and 'log'. 'short' just shows pairs of commit names, this format\n \tis used when this option is not given. 'log' is the default value for this\n \toption and lists the commits in that commit range like the 'summary'\n-- \n1.7.10.rc0\n"},{"id":"186893","messageId":"1331665251-11147-2-git-send-email-tim.henigan@gmail.com","threadId":"29943","inReplyTo":"1331665251-11147-1-git-send-email-tim.henigan@gmail.com","subject":"[PATCH 2/2] Documentation/diff-options: reword description of --submodule option","fromName":"Tim Henigan","fromEmail":"tim.henigan@gmail.com","sentAt":"2012-03-13T19:00:51Z","receivedAt":"2012-03-13T19:00:51Z","isPatch":true,"sender":{"key":"tim.henigan@gmail.com","avatar":"https://avatars.githubusercontent.com/u/42022?v=4"},"body":"The previous description was a bit confusing. This rewrite makes it\neasier to understand.\n\nIt should be noted that even though 'log' is listed as the default\nformat, it is not used when the '--submodule' option is missing. This\nmeans that the output of:\n 'git diff' matches 'git diff --submodule=short'\n 'git diff --submodule' matches 'git diff --submodule=log'\n\nSigned-off-by: Tim Henigan \n---\n\nResent due to a typo in the developer list email address.\n\nI will understand if this change is rejected, but I found the current\ndescription difficult to follow the first couple times I read it.\n\n Documentation/diff-options.txt | 9 ++++-----\n 1 file changed, 4 insertions(+), 5 deletions(-)\n\ndiff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\nindex f44f3fc..0e1c753 100644\n--- a/Documentation/diff-options.txt\n+++ b/Documentation/diff-options.txt\n@@ -165,11 +165,10 @@ any of those replacements occurred.\n \tof the `--diff-filter` option on what the status letters mean.\n \n --submodule[=]::\n-\tChoose the output format for submodule differences. can be one of\n-\t'short' and 'log'. 'short' just shows pairs of commit names, this format\n-\tis used when this option is not given. 'log' is the default value for this\n-\toption and lists the commits in that commit range like the 'summary'\n-\toption of linkgit:git-submodule[1] does.\n+\tChoose the output format for submodule differences. can be one\n+\tof 'short' or 'log'. The default value is 'log'.\n+\t'log' lists commits in the commit range like linkgit:git-submodule[1] `--summary`.\n+\t'short' shows only the SHA1 of the source and destination.\n \n --color[=]::\n \tShow colored diff.\n-- \n1.7.10.rc0\n"},{"id":"186898","messageId":"7vipi8uv12.fsf@alter.siamese.dyndns.org","threadId":"29943","inReplyTo":"1331665251-11147-2-git-send-email-tim.henigan@gmail.com","subject":"Re: [PATCH 2/2] Documentation/diff-options: reword description of --submodule option","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-13T20:04:57Z","receivedAt":"2012-03-13T20:04:57Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Tim Henigan writes:\n\n> The previous description was a bit confusing. This rewrite makes it\n> easier to understand.\n\nI can understand the confusion. We are trying to say too many things at\nthe same time:\n\n * There are two formats, log and short.\n\n * The log format is like \"git submodule --summary A..B\" for the range.\n\n * The short format just shows A and B for the range.\n\n * The log format is obtained by giving --submodule or --submodule=log.\n\n * The short format is obtained by not giving this option at all, or\n giving --submodule=short.\n\n> diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt\n> index f44f3fc..0e1c753 100644\n> --- a/Documentation/diff-options.txt\n> +++ b/Documentation/diff-options.txt\n> @@ -165,11 +165,10 @@ any of those replacements occurred.\n> \tof the `--diff-filter` option on what the status letters mean.\n> \n> --submodule[=]::\n> -\tChoose the output format for submodule differences. can be one of\n> -\t'short' and 'log'. 'short' just shows pairs of commit names, this format\n> -\tis used when this option is not given. 'log' is the default value for this\n> -\toption and lists the commits in that commit range like the 'summary'\n> -\toption of linkgit:git-submodule[1] does.\n> +\tChoose the output format for submodule differences. can be one\n> +\tof 'short' or 'log'. The default value is 'log'.\n> +\t'log' lists commits in the commit range like linkgit:git-submodule[1] `--summary`.\n> +\t'short' shows only the SHA1 of the source and destination.\n\nDoes it risk being unclear to say src/dst here without defining, or is it\nsufficiently clear from the context (the previous line says \"range\")?\n\nYou lost \"not giving this option at all is the same as --submodule=short\".\n\nHere is my attempt, but I do not think I particularly did a good job at\nthis.\n\n Specify how differences in submodules are shown. A `--submodule`\n or `--submodule=log` option uses the long format, which lists the\n commits in the range like linkgit:git-submodule[1] `--summary` does.\n\tOmitting the `--submodule` option, or a `--submodule=short` option,\n\tuses the short format, which just shows the names of the commits\n\tat the beginning and the end of the range.\n"},{"id":"186900","messageId":"CAFouetgSw7MMzHWOJox6DiPmBWD=6j4mST1KFZYsZyLmKHRrEQ@mail.gmail.com","threadId":"29943","inReplyTo":"7vipi8uv12.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 2/2] Documentation/diff-options: reword description of --submodule option","fromName":"Tim Henigan","fromEmail":"tim.henigan@gmail.com","sentAt":"2012-03-13T20:36:48Z","receivedAt":"2012-03-13T20:36:48Z","isPatch":true,"sender":{"key":"tim.henigan@gmail.com","avatar":"https://avatars.githubusercontent.com/u/42022?v=4"},"body":">>  --submodule[=]::\n>> -     Choose the output format for submodule differences. can be one of\n>> -     'short' and 'log'. 'short' just shows pairs of commit names, this format\n>> -     is used when this option is not given. 'log' is the default value for this\n>> -     option and lists the commits in that commit range like the 'summary'\n>> -     option of linkgit:git-submodule[1] does.\n>> +     Choose the output format for submodule differences. can be one\n>> +     of 'short' or 'log'. The default value is 'log'.\n>> +     'log' lists commits in the commit range like linkgit:git-submodule[1] `--summary`.\n>> +     'short' shows only the SHA1 of the source and destination.\n>\n> Does it risk being unclear to say src/dst here without defining, or is it\n> sufficiently clear from the context (the previous line says \"range\")?\n\nI chose those words based on descriptions in some of the other diff\noptions (see `changes` and --src-prefix in the same file. That being\nsaid, the description you gave is more clear.\n\n\n> You lost \"not giving this option at all is the same as --submodule=short\".\n\nI removed that part on purpose. I wasn't sure that it needs to be\ndocumented here since it is about how the 'diff' command normally\nbehaves rather than about the '--submodule' option itself.\n\n\n> Here is my attempt, but I do not think I particularly did a good job at\n> this.\n>\n>        Specify how differences in submodules are shown.  A `--submodule`\n>        or `--submodule=log` option uses the long format, which lists the\n>        commits in the range like linkgit:git-submodule[1] `--summary` does.\n>        Omitting the `--submodule` option, or a `--submodule=short` option,\n>        uses the short format, which just shows the names of the commits\n>        at the beginning and the end of the range.\n\nSome further minor changes to your version:\n\n    Specify how differences in submodules are shown.  When `--submodule`\n    or `--submodule=log` is given, the 'log' format is used. This format lists\n    the commits in the range like linkgit:git-submodule[1] `--summary` does.\n    Omitting the `--submodule` option or specifying `--submodule=short`,\n    uses the 'short' format. This format just shows the names of the commits\n    at the beginning and end of the range.\n"},{"id":"186978","messageId":"4F60E7DB.3030808@web.de","threadId":"29943","inReplyTo":"CAFouetgSw7MMzHWOJox6DiPmBWD=6j4mST1KFZYsZyLmKHRrEQ@mail.gmail.com","subject":"Re: [PATCH 2/2] Documentation/diff-options: reword description of --submodule option","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2012-03-14T18:47:55Z","receivedAt":"2012-03-14T18:47:55Z","isPatch":true,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Am 13.03.2012 21:36, schrieb Tim Henigan:\n> Some further minor changes to your version:\n> \n> Specify how differences in submodules are shown. When `--submodule`\n> or `--submodule=log` is given, the 'log' format is used. This format lists\n> the commits in the range like linkgit:git-submodule[1] `--summary` does.\n\nNit:\n the commits in the range like linkgit:git-submodule[1] `summary` does.\n\n(`summary` is a command of \"git submodule\", not an option)\n\n> Omitting the `--submodule` option or specifying `--submodule=short`,\n> uses the 'short' format. This format just shows the names of the commits\n> at the beginning and end of the range.\n\nBut apart from that I really like it.\n"}]}