{"thread":{"id":"48258","subject":"[PATCH 0/6] Some doc-fixes","startedAt":"2018-04-10T18:32:50Z","lastAt":"2018-05-04T18:47:33Z","messageCount":54,"participants":["Andreas Heiduk","Martin Ågren","Junio C Hamano","Eric Wong","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":6},"messages":[{"id":"344362","messageId":"20180410183224.10780-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":null,"subject":"[PATCH 0/6] Some doc-fixes","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:32:18Z","receivedAt":"2018-04-10T18:32:50Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"I'm flushing a queue of small fixes to the docs. Handling these\nindiviually is just to much hassle - at least I hope so :-)\n\nAndreas Heiduk (6):\n doc: fix formatting inconsistency in githooks.txt\n doc: align 'diff --no-index' in text with synopsis\n doc: clarify ignore rules for git ls-files\n doc: added '-d' and '-q' for 'git push'\n git-svn: commit-diff does not support --add-author-from\n doc: add note about shell quoting to revision.txt\n\n Documentation/git-diff.txt | 2 +-\n Documentation/git-ls-files.txt | 3 ++-\n Documentation/git-push.txt | 3 ++-\n Documentation/git-svn.txt | 2 +-\n Documentation/githooks.txt | 4 ++--\n Documentation/revisions.txt | 6 ++++++\n 6 files changed, 14 insertions(+), 6 deletions(-)\n\n-- \n2.16.2\n\n"},{"id":"344363","messageId":"20180410183224.10780-5-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 4/6] doc: added '-d' and '-q' for 'git push'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:32:22Z","receivedAt":"2018-04-10T18:33:30Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Add the missing `-o` shortcut for `--push-option` to the synposis.\nAdd the missing `-d` shortcut for `--delete` in the main section.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-push.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\nindex 5b08302fc2..f2bbda6e32 100644\n--- a/Documentation/git-push.txt\n+++ b/Documentation/git-push.txt\n@@ -11,7 +11,7 @@ SYNOPSIS\n [verse]\n 'git push' [--all | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=]\n \t [--repo=] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]\n-\t [-u | --set-upstream] [--push-option=]\n+\t [-u | --set-upstream] [-o | --push-option=]\n \t [--[no-]signed|--signed=(true|false|if-asked)]\n \t [--force-with-lease[=[:]]]\n \t [--no-verify] [ [...]]\n@@ -123,6 +123,7 @@ already exists on the remote side.\n \twill be tab-separated and sent to stdout instead of stderr. The full\n \tsymbolic names of the refs will be given.\n \n+-d::\n --delete::\n \tAll listed refs are deleted from the remote repository. This is\n \tthe same as prefixing all refs with a colon.\n-- \n2.16.2\n\n"},{"id":"344364","messageId":"20180410183224.10780-4-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 3/6] doc: clarify ignore rules for git ls-files","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:32:21Z","receivedAt":"2018-04-10T18:34:11Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Explain that `git ls-files --ignored` requires at least one\nof the `--exclude*` options to do its job.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-ls-files.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-ls-files.txt b/Documentation/git-ls-files.txt\nindex 3ac3e3a77d..f3474b2ede 100644\n--- a/Documentation/git-ls-files.txt\n+++ b/Documentation/git-ls-files.txt\n@@ -53,7 +53,8 @@ OPTIONS\n \tShow only ignored files in the output. When showing files in the\n \tindex, print only those matched by an exclude pattern. When\n \tshowing \"other\" files, show only those matched by an exclude\n-\tpattern.\n+\tpattern. Standard ignore rules are not automatically activated,\n+\ttherefore at least one of the `--exclude*` options is required.\n \n -s::\n --stage::\n-- \n2.16.2\n\n"},{"id":"344365","messageId":"20180410183224.10780-3-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:32:20Z","receivedAt":"2018-04-10T18:36:00Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Comparing\nThe two '' parameters are not optional but the option\n'--no-index' is. Also move the `--options` part to the same\nplace where the other variants show them.\n\nAll three items are already correct in the synopsis.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-diff.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex b0c1bb95c8..ee1c509bd3 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' --no-index [--options] [--] [...]::\n+'git diff' [--options] [--no-index] [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n-- \n2.16.2\n\n"},{"id":"344366","messageId":"20180410183224.10780-2-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:32:19Z","receivedAt":"2018-04-10T18:36:21Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"The section 'post-rewrite' in 'githooks.txt' renders only one command\nusing backticks (`git commit`) but the other commands using single quotes\n('git-rebase'). Align this formatting to use single quotes.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/githooks.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex f877f7b7cd..070e745b41 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -417,8 +417,8 @@ to abort.\n post-rewrite\n ~~~~~~~~~~~~\n \n-This hook is invoked by commands that rewrite commits (`git commit\n---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call\n+This hook is invoked by commands that rewrite commits ('git commit\n+--amend', 'git-rebase'; currently 'git-filter-branch' does 'not' call\n it!). Its first argument denotes the command it was invoked by:\n currently one of `amend` or `rebase`. Further command-dependent\n arguments may be passed in the future.\n-- \n2.16.2\n\n"},{"id":"344368","messageId":"20180410183901.11859-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 5/6] git-svn: commit-diff does not support --add-author-from","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:39:01Z","receivedAt":"2018-04-10T18:39:25Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Signed-off-by: Andreas Heiduk \n---\n Documentation/git-svn.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt\nindex 636e09048e..11aefadf7a 100644\n--- a/Documentation/git-svn.txt\n+++ b/Documentation/git-svn.txt\n@@ -700,7 +700,7 @@ creating the branch or tag.\n config key: svn.useLogAuthor\n \n --add-author-from::\n-\tWhen committing to svn from Git (as part of 'commit-diff', 'set-tree' or 'dcommit'\n+\tWhen committing to svn from Git (as part of 'set-tree' or 'dcommit'\n \toperations), if the existing log message doesn't already have a\n \t`From:` or `Signed-off-by:` line, append a `From:` line based on the\n \tGit commit's author string. If you use this, then `--use-log-author`\n-- \n2.16.2\n\n"},{"id":"344369","messageId":"20180410183939.12146-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH 6/6] doc: add note about shell quoting to revision.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T18:39:39Z","receivedAt":"2018-04-10T18:40:04Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Signed-off-by: Andreas Heiduk \n---\n Documentation/revisions.txt | 6 ++++++\n 1 file changed, 6 insertions(+)\n\ndiff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\nindex dfcc49c72c..c1d3a40a90 100644\n--- a/Documentation/revisions.txt\n+++ b/Documentation/revisions.txt\n@@ -7,6 +7,10 @@ syntax. Here are various ways to spell object names. The\n ones listed near the end of this list name trees and\n blobs contained in a commit.\n \n+NOTE: This document shows the \"raw\" syntax as seen by git. The shell\n+and other UIs might require additional quoting to protect special\n+characters and to avoid word splitting.\n+\n '', e.g. 'dae86e1950b1277e545cee180551750029cfe735', 'dae86e'::\n The full SHA-1 object name (40-byte hexadecimal string), or\n a leading substring that is unique within the repository.\n@@ -186,6 +190,8 @@ existing tag object.\n is matched. ':/!-foo' performs a negative match, while ':/!!foo' matches a\n literal '!' character, followed by 'foo'. Any other sequence beginning with\n ':/!' is reserved for now.\n+ Depending on the given text the shell's word splitting rules might\n+ require additional quoting.\n \n ':', e.g. 'HEAD:README', ':README', 'master:./README'::\n A suffix ':' followed by a path names the blob or tree\n-- \n2.16.2\n\n"},{"id":"344374","messageId":"CAN0heSquxLpMfrLQs2em1amLv77-Uywr+885Viuq4-7DiWYmAQ@mail.gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-2-asheiduk@gmail.com","subject":"Re: [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-10T19:13:30Z","receivedAt":"2018-04-10T19:13:36Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n> The section 'post-rewrite' in 'githooks.txt' renders only one command\n> using backticks (`git commit`) but the other commands using single quotes\n> ('git-rebase'). Align this formatting to use single quotes.\n>\n> Signed-off-by: Andreas Heiduk \n> ---\n> Documentation/githooks.txt | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n>\n> diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\n> index f877f7b7cd..070e745b41 100644\n> --- a/Documentation/githooks.txt\n> +++ b/Documentation/githooks.txt\n> @@ -417,8 +417,8 @@ to abort.\n> post-rewrite\n> ~~~~~~~~~~~~\n>\n> -This hook is invoked by commands that rewrite commits (`git commit\n> ---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call\n> +This hook is invoked by commands that rewrite commits ('git commit\n> +--amend', 'git-rebase'; currently 'git-filter-branch' does 'not' call\n> it!). Its first argument denotes the command it was invoked by:\n> currently one of `amend` or `rebase`. Further command-dependent\n> arguments may be passed in the future.\n\nHmm, I wonder if that is actually intentional. `git commit --amend`\ncould be run exactly like that and would do what this paragraph expects\nof it. The 'git-rebase' is a Git subcommand name, i.e., not some\ncopy-paste command-line ready for use. If it were something like `git\nrebase -i HEAD~5`, I would expect the backticks.\n\nA second discrepancy is the dash in \"git commit\" vs \"git-rebase\" and\n\"git-ls-remote\". That could perhaps be explained by the same reasoning.\n\nMartin\n"},{"id":"344375","messageId":"CAN0heSrGrtRi-xCsSEOrW0D+OvMHjbE_YEvFtQd9f+FH+h5iiQ@mail.gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-3-asheiduk@gmail.com","subject":"Re: [PATCH 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-10T19:14:54Z","receivedAt":"2018-04-10T19:14:59Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n> Comparing\n> The two '' parameters are not optional but the option\n> '--no-index' is. Also move the `--options` part to the same\n> place where the other variants show them.\n\nThat first line should probably not be there. The diff LGTM.\n\nMartin\n"},{"id":"344376","messageId":"CAN0heSrrPa5-ckjsonUh+3JpoAnaAT=efJT4Rp7uEn=UfWpC_g@mail.gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-5-asheiduk@gmail.com","subject":"Re: [PATCH 4/6] doc: added '-d' and '-q' for 'git push'","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-10T19:17:45Z","receivedAt":"2018-04-10T19:18:02Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n> Add the missing `-o` shortcut for `--push-option` to the synposis.\n> Add the missing `-d` shortcut for `--delete` in the main section.\n\ns/synposis/synopsis/\n\nThe subject of this patch says -q, which should be -o. The subject\ncould also be in imperative (\"doc: add ...\", or \"doc: add missing ...\").\nThe diff LGTM.\n\nMartin\n"},{"id":"344380","messageId":"4a6adc49-e96e-6066-9516-962ecf565c30@gmail.com","threadId":"48258","inReplyTo":"CAN0heSrGrtRi-xCsSEOrW0D+OvMHjbE_YEvFtQd9f+FH+h5iiQ@mail.gmail.com","subject":"Re: [PATCH 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T19:32:50Z","receivedAt":"2018-04-10T19:32:58Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 10.04.2018 um 21:14 schrieb Martin Ågren:\n> On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n>> Comparing\n> \n> That first line should probably not be there. The diff LGTM.\n> \n> Martin\n> \n\nACK, Thanks\n"},{"id":"344382","messageId":"85c6d713-7e36-6068-f6d3-9b9144a4958d@gmail.com","threadId":"48258","inReplyTo":"CAN0heSrrPa5-ckjsonUh+3JpoAnaAT=efJT4Rp7uEn=UfWpC_g@mail.gmail.com","subject":"Re: [PATCH 4/6] doc: added '-d' and '-q' for 'git push'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T19:38:13Z","receivedAt":"2018-04-10T19:38:20Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 10.04.2018 um 21:17 schrieb Martin Ågren:\n> On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n>> Add the missing `-o` shortcut for `--push-option` to the synposis.\n>> Add the missing `-d` shortcut for `--delete` in the main section.\n> \n> s/synposis/synopsis/\n> \n> The subject of this patch says -q, which should be -o. The subject\n> could also be in imperative (\"doc: add ...\", or \"doc: add missing ...\").\n> The diff LGTM.\n> \n> Martin\n> \n\nACK & Thanks,\n\nCan I add \"Reviewed-by: $YOU\" to this one and 2/6?\n\nAndreas\n"},{"id":"344383","messageId":"f45ef7cb-dc0c-8c63-1d2f-6ece0d0091cc@gmail.com","threadId":"48258","inReplyTo":"CAN0heSquxLpMfrLQs2em1amLv77-Uywr+885Viuq4-7DiWYmAQ@mail.gmail.com","subject":"Re: [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-10T20:04:35Z","receivedAt":"2018-04-10T20:04:49Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 10.04.2018 um 21:13 schrieb Martin Ågren:\n> On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n>> The section 'post-rewrite' in 'githooks.txt' renders only one command\n>> using backticks (`git commit`) but the other commands using single quotes\n>> ('git-rebase'). Align this formatting to use single quotes.\n>>\n>> Signed-off-by: Andreas Heiduk \n>> ---\n>> Documentation/githooks.txt | 4 ++--\n>> 1 file changed, 2 insertions(+), 2 deletions(-)\n>>\n>> diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\n>> index f877f7b7cd..070e745b41 100644\n>> --- a/Documentation/githooks.txt\n>> +++ b/Documentation/githooks.txt\n>> @@ -417,8 +417,8 @@ to abort.\n>> post-rewrite\n>> ~~~~~~~~~~~~\n>>\n>> -This hook is invoked by commands that rewrite commits (`git commit\n>> ---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call\n>> +This hook is invoked by commands that rewrite commits ('git commit\n>> +--amend', 'git-rebase'; currently 'git-filter-branch' does 'not' call\n>> it!). Its first argument denotes the command it was invoked by:\n>> currently one of `amend` or `rebase`. Further command-dependent\n>> arguments may be passed in the future.\n> \n> Hmm, I wonder if that is actually intentional. `git commit --amend`\n> could be run exactly like that and would do what this paragraph expects\n> of it. The 'git-rebase' is a Git subcommand name, i.e., not some\n> copy-paste command-line ready for use. If it were something like `git\n> rebase -i HEAD~5`, I would expect the backticks.\n\nThat page mostly uses single quotes and no dash ('git send-email')for\nformatting. Reading 'CodingGuidelines' my understanding is, that git\ncommands should be typeset with backticks, no dash (`git send-email`). \nSo 'git-rebase' (an similar) *should* be typeset as `git rebase`. But\ndoing so consistently would be a full-diff for this manual page.\n\nShould I do this?\n\n> \n> A second discrepancy is the dash in \"git commit\" vs \"git-rebase\" and\n> \"git-ls-remote\". That could perhaps be explained by the same reasoning.\n> \n> Martin\n> \n\n"},{"id":"344384","messageId":"CAN0heSqsBjLAgWwZPoJq1h2m4hGHm5hZRkpDYVEkuiFJEhOhEg@mail.gmail.com","threadId":"48258","inReplyTo":"85c6d713-7e36-6068-f6d3-9b9144a4958d@gmail.com","subject":"Re: [PATCH 4/6] doc: added '-d' and '-q' for 'git push'","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-10T20:05:21Z","receivedAt":"2018-04-10T20:05:30Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 10 April 2018 at 21:38, Andreas Heiduk wrote:\n> Can I add \"Reviewed-by: $YOU\" to this one and 2/6?\n\nSure!\n"},{"id":"344391","messageId":"CAN0heSojXxQZz50rwCJefkMpF4bPfpzt+=r9qcVoczU1+qN0Dg@mail.gmail.com","threadId":"48258","inReplyTo":"f45ef7cb-dc0c-8c63-1d2f-6ece0d0091cc@gmail.com","subject":"Re: [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-10T20:44:51Z","receivedAt":"2018-04-10T20:45:00Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 10 April 2018 at 22:04, Andreas Heiduk wrote:\n> Am 10.04.2018 um 21:13 schrieb Martin Ågren:\n>> On 10 April 2018 at 20:32, Andreas Heiduk wrote:\n>> Hmm, I wonder if that is actually intentional. `git commit --amend`\n>> could be run exactly like that and would do what this paragraph expects\n>> of it. The 'git-rebase' is a Git subcommand name, i.e., not some\n>> copy-paste command-line ready for use. If it were something like `git\n>> rebase -i HEAD~5`, I would expect the backticks.\n>\n> That page mostly uses single quotes and no dash ('git send-email')for\n> formatting. Reading 'CodingGuidelines' my understanding is, that git\n> commands should be typeset with backticks, no dash (`git send-email`).\n> So 'git-rebase' (an similar) *should* be typeset as `git rebase`. But\n> doing so consistently would be a full-diff for this manual page.\n>\n> Should I do this?\n\nYour reading seems correct, so I was wrong in my speculation. My guess\nis such a patch would be welcome. I checked a couple of man-pages and\nthis one seems particularly heavy on 'git foo' as opposed to `git foo`.\nI think that's a reason to fix it, not to leave it behind.\n\nMartin\n"},{"id":"344397","messageId":"xmqq7epe3gfe.fsf@gitster-ct.c.googlers.com","threadId":"48258","inReplyTo":"20180410183224.10780-3-asheiduk@gmail.com","subject":"Re: [PATCH 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-04-10T21:22:29Z","receivedAt":"2018-04-10T21:22:36Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Andreas Heiduk writes:\n\n> \n> -'git diff' --no-index [--options] [--] [...]::\n> +'git diff' [--options] [--no-index] [--] ::\n> \n> \tThis form is to compare the given two paths on the\n> \tfilesystem. You can omit the `--no-index` option when\n\nIt definitely is a good change to show two (and only two) on\nthe command line as non-optional arguments.\n\nI however find the change to mark that -\"-no-index\" is optional is\ninviting more confusion in the form presented in this patch. It is\noptional under specific conditions, and that is not conveyed with\nthese two path arguments named very genericly (as opposed to making\nit clear that they are paths that are not managed by Git) on the\nexample command line.\n\nI have a suspicion that it would be safer to have the description\nsay under what condition \"--no-index\" becomes optional (which our\ntext already does), without marking it as if it is always optional\nlike this patch does (i.e. do not lose [] around it from this line).\nI dunno.\n\n"},{"id":"344398","messageId":"xmqq37023gdx.fsf@gitster-ct.c.googlers.com","threadId":"48258","inReplyTo":"CAN0heSojXxQZz50rwCJefkMpF4bPfpzt+=r9qcVoczU1+qN0Dg@mail.gmail.com","subject":"Re: [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-04-10T21:23:22Z","receivedAt":"2018-04-10T21:23:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Martin Ågren writes:\n\n> Your reading seems correct, so I was wrong in my speculation. My guess\n> is such a patch would be welcome. I checked a couple of man-pages and\n> this one seems particularly heavy on 'git foo' as opposed to `git foo`.\n> I think that's a reason to fix it, not to leave it behind.\n\nSounds sensible. Hopefully there isn't a topic in flight that wants\nto change this file, so it may be a good time to do a wholesale\ncleanup of it.\n"},{"id":"344399","messageId":"xmqqy3hu21pq.fsf@gitster-ct.c.googlers.com","threadId":"48258","inReplyTo":"20180410183939.12146-1-asheiduk@gmail.com","subject":"Re: [PATCH 6/6] doc: add note about shell quoting to revision.txt","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-04-10T21:25:37Z","receivedAt":"2018-04-10T21:25:44Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Andreas Heiduk writes:\n\n> Signed-off-by: Andreas Heiduk \n> ---\n> Documentation/revisions.txt | 6 ++++++\n> 1 file changed, 6 insertions(+)\n>\n> diff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\n> index dfcc49c72c..c1d3a40a90 100644\n> --- a/Documentation/revisions.txt\n> +++ b/Documentation/revisions.txt\n> @@ -7,6 +7,10 @@ syntax. Here are various ways to spell object names. The\n> ones listed near the end of this list name trees and\n> blobs contained in a commit.\n> \n> +NOTE: This document shows the \"raw\" syntax as seen by git. The shell\n> +and other UIs might require additional quoting to protect special\n> +characters and to avoid word splitting.\n> +\n> '', e.g. 'dae86e1950b1277e545cee180551750029cfe735', 'dae86e'::\n> The full SHA-1 object name (40-byte hexadecimal string), or\n> a leading substring that is unique within the repository.\n> @@ -186,6 +190,8 @@ existing tag object.\n> is matched. ':/!-foo' performs a negative match, while ':/!!foo' matches a\n> literal '!' character, followed by 'foo'. Any other sequence beginning with\n> ':/!' is reserved for now.\n> + Depending on the given text the shell's word splitting rules might\n> + require additional quoting.\n> \n> ':', e.g. 'HEAD:README', ':README', 'master:./README'::\n> A suffix ':' followed by a path names the blob or tree\n\nI've seen this suggested before and thought it is a good idea. GOod\nto see it is finally happening ;-) Thanks.\n"},{"id":"344496","messageId":"b227d2b3-dfe8-09cf-2705-da70bfebd268@gmail.com","threadId":"48258","inReplyTo":"xmqq37023gdx.fsf@gitster-ct.c.googlers.com","subject":"Re: [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-11T21:05:18Z","receivedAt":"2018-04-11T21:05:29Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"So the following two fixups should cleanup that page considerably.\n\n"},{"id":"344497","messageId":"20180411210857.22598-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"b227d2b3-dfe8-09cf-2705-da70bfebd268@gmail.com","subject":"fixup! [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-11T21:08:56Z","receivedAt":"2018-04-11T21:09:30Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"- add linkgit: to callers of hooks\n- change 'git-foo' and similar to `git foo`\n- add some more `` for fsmonitor\n---\n Documentation/githooks.txt | 101 +++++++++++++++++++++++----------------------\n 1 file changed, 51 insertions(+), 50 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex 070e745b41..be31376767 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -31,7 +31,7 @@ Hooks can get their arguments via the environment, command-line\n arguments, and stdin. See the documentation for each hook below for\n details.\n \n-'git init' may copy hooks to the new repository, depending on its\n+`git init` may copy hooks to the new repository, depending on its\n configuration. See the \"TEMPLATE DIRECTORY\" section in\n linkgit:git-init[1] for details. When the rest of this document refers\n to \"default hooks\" it's talking about the default template shipped\n@@ -45,9 +45,9 @@ HOOKS\n applypatch-msg\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes a single\n+This hook is invoked by linkgit:git-am[1]. It takes a single\n parameter, the name of the file that holds the proposed commit\n-log message. Exiting with a non-zero status causes 'git am' to abort\n+log message. Exiting with a non-zero status causes `git am` to abort\n before applying the patch.\n \n The hook is allowed to edit the message file in place, and can\n@@ -61,7 +61,7 @@ The default 'applypatch-msg' hook, when enabled, runs the\n pre-applypatch\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter, and is\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter, and is\n invoked after the patch is applied, but before a commit is made.\n \n If it exits with non-zero status, then the working tree will not be\n@@ -76,7 +76,7 @@ The default 'pre-applypatch' hook, when enabled, runs the\n post-applypatch\n ~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter,\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter,\n and is invoked after the patch is applied and a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n@@ -85,24 +85,24 @@ the outcome of 'git am'.\n pre-commit\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit', and can be bypassed\n+This hook is invoked by linkgit:git-commit[1], and can be bypassed\n with the `--no-verify` option. It takes no parameters, and is\n invoked before obtaining the proposed commit log message and\n making a commit. Exiting with a non-zero status from this script\n-causes the 'git commit' command to abort before creating a commit.\n+causes the `git commit` command to abort before creating a commit.\n \n The default 'pre-commit' hook, when enabled, catches introduction\n of lines with trailing whitespaces and aborts the commit when\n such a line is found.\n \n-All the 'git commit' hooks are invoked with the environment\n+All the `git commit` hooks are invoked with the environment\n variable `GIT_EDITOR=:` if the command will not bring up an editor\n to modify the commit message.\n \n prepare-commit-msg\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git commit' right after preparing the\n+This hook is invoked by linkgit:git-commit[1] right after preparing the\n default log message, and before the editor is started.\n \n It takes one to three parameters. The first is the name of the file\n@@ -114,7 +114,7 @@ commit is a merge or a `.git/MERGE_MSG` file exists); `squash`\n (if a `.git/SQUASH_MSG` file exists); or `commit`, followed by\n a commit SHA-1 (if a `-c`, `-C` or `--amend` option was given).\n \n-If the exit status is non-zero, 'git commit' will abort.\n+If the exit status is non-zero, `git commit` will abort.\n \n The purpose of the hook is to edit the message file in place, and\n it is not suppressed by the `--no-verify` option. A non-zero exit\n@@ -127,7 +127,7 @@ help message found in the commented portion of the commit template.\n commit-msg\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit' and 'git merge', and can be\n+This hook is invoked by linkgit:git-commit[1] and linkgit:git-merge[1], and can be\n bypassed with the `--no-verify` option. It takes a single parameter,\n the name of the file that holds the proposed commit log message.\n Exiting with a non-zero status causes the command to abort.\n@@ -143,16 +143,16 @@ The default 'commit-msg' hook, when enabled, detects duplicate\n post-commit\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git commit'. It takes no parameters, and is\n+This hook is invoked by linkgit:git-commit[1]. It takes no parameters, and is\n invoked after a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git commit'.\n+the outcome of `git commit`.\n \n pre-rebase\n ~~~~~~~~~~\n \n-This hook is called by 'git rebase' and can be used to prevent a\n+This hook is called by linkgit:git-rebase[1] and can be used to prevent a\n branch from getting rebased. The hook may be called with one or\n two parameters. The first parameter is the upstream from which\n the series was forked. The second parameter is the branch being\n@@ -161,17 +161,17 @@ rebased, and is not set when rebasing the current branch.\n post-checkout\n ~~~~~~~~~~~~~\n \n-This hook is invoked when a 'git checkout' is run after having updated the\n+This hook is invoked when a linkgit:git-checkout[1] is run after having updated the\n worktree. The hook is given three parameters: the ref of the previous HEAD,\n the ref of the new HEAD (which may or may not have changed), and a flag\n indicating whether the checkout was a branch checkout (changing branches,\n flag=1) or a file checkout (retrieving a file from the index, flag=0).\n-This hook cannot affect the outcome of 'git checkout'.\n+This hook cannot affect the outcome of `git checkout`.\n \n-It is also run after 'git clone', unless the --no-checkout (-n) option is\n+It is also run after linkgit:git-clone[1], unless the `--no-checkout` (`-n`) option is\n used. The first parameter given to the hook is the null-ref, the second the\n-ref of the new HEAD and the flag is always 1. Likewise for 'git worktree add'\n-unless --no-checkout is used.\n+ref of the new HEAD and the flag is always 1. Likewise for `git worktree add`\n+unless `--no-checkout` is used.\n \n This hook can be used to perform repository validity checks, auto-display\n differences from the previous HEAD if different, or set working dir metadata\n@@ -180,10 +180,10 @@ properties.\n post-merge\n ~~~~~~~~~~\n \n-This hook is invoked by 'git merge', which happens when a 'git pull'\n+This hook is invoked by linkgit:git-merge[1], which happens when a `git pull`\n is done on a local repository. The hook takes a single parameter, a status\n flag specifying whether or not the merge being done was a squash merge.\n-This hook cannot affect the outcome of 'git merge' and is not executed,\n+This hook cannot affect the outcome of `git merge` and is not executed,\n if the merge failed due to conflicts.\n \n This hook can be used in conjunction with a corresponding pre-commit hook to\n@@ -194,7 +194,7 @@ for an example of how to do this.\n pre-push\n ~~~~~~~~\n \n-This hook is called by 'git push' and can be used to prevent a push from taking\n+This hook is called by linkgit:git-push[1] and can be used to prevent a push from taking\n place. The hook is called with two parameters which provide the name and\n location of the destination remote, if a named remote is not being used both\n values will be the same.\n@@ -216,7 +216,7 @@ SHA-1>` will be 40 `0`. If the local commit was specified by something other\n than a name which could be expanded (such as `HEAD~`, or a SHA-1) it will be\n supplied as it was originally given.\n \n-If this hook exits with a non-zero status, 'git push' will abort without\n+If this hook exits with a non-zero status, `git push` will abort without\n pushing anything. Information about why the push is rejected may be sent\n to the user by writing to standard error.\n \n@@ -224,8 +224,8 @@ to the user by writing to standard error.\n pre-receive\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before starting to update refs on the remote repository, the\n pre-receive hook is invoked. Its exit status determines the success\n or failure of the update.\n@@ -246,7 +246,7 @@ updated. If the hook exits with zero, updating of individual refs can\n still be prevented by the <> hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The number of push options given on the command line of\n@@ -265,8 +265,8 @@ linkgit:git-receive-pack[1] for some caveats.\n update\n ~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before updating the ref on the remote repository, the update hook\n is invoked. Its exit status determines the success or failure of\n the ref update.\n@@ -279,7 +279,7 @@ three parameters:\n - and the new object name to be stored in the ref.\n \n A zero exit from the update hook allows the ref to be updated.\n-Exiting with a non-zero status prevents 'git-receive-pack'\n+Exiting with a non-zero status prevents `git receive-pack`\n from updating that ref.\n \n This hook can be used to prevent 'forced' update on certain refs by\n@@ -299,7 +299,7 @@ membership. See linkgit:git-shell[1] for how you might use the login\n shell to restrict the user's access to only git commands.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'update' hook, when enabled--and with\n@@ -310,8 +310,8 @@ unannotated tags to be pushed.\n post-receive\n ~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -320,7 +320,7 @@ arguments, but gets the same information as the\n <>\n hook does on its standard input.\n \n-This hook does not affect the outcome of 'git-receive-pack', as it\n+This hook does not affect the outcome of `git receive-pack`, as it\n is called after the real work is done.\n \n This supersedes the <> hook in that it gets\n@@ -328,7 +328,7 @@ both old and new values of all the refs in addition to their\n names.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'post-receive' hook is empty, but there is\n@@ -349,8 +349,8 @@ will be set to zero, `GIT_PUSH_OPTION_COUNT=0`.\n post-update\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -358,7 +358,7 @@ It takes a variable number of parameters, each of which is the\n name of ref that was actually updated.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git-receive-pack'.\n+the outcome of `git receive-pack`.\n \n The 'post-update' hook can tell what are the heads that were pushed,\n but it does not know what their original and updated values are,\n@@ -368,20 +368,20 @@ updated values of the refs. You might consider it instead if you need\n them.\n \n When enabled, the default 'post-update' hook runs\n-'git update-server-info' to keep the information used by dumb\n+`git update-server-info` to keep the information used by dumb\n transports (e.g., HTTP) up to date. If you are publishing\n a Git repository that is accessible via HTTP, you should\n probably enable this hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n push-to-checkout\n ~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository, and when\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository, and when\n the push tries to update the branch that is currently checked out\n and the `receive.denyCurrentBranch` configuration variable is set to\n `updateInstead`. Such a push by default is refused if the working\n@@ -410,15 +410,16 @@ with the difference between the branches.\n pre-auto-gc\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git gc --auto'. It takes no parameter, and\n-exiting with non-zero status from this script causes the 'git gc --auto'\n+This hook is invoked by `git gc --auto` (see linkgit:git-gc[1]). It takes no parameter, and\n+exiting with non-zero status from this script causes the `git gc --auto`\n to abort.\n \n post-rewrite\n ~~~~~~~~~~~~\n \n-This hook is invoked by commands that rewrite commits ('git commit\n---amend', 'git-rebase'; currently 'git-filter-branch' does 'not' call\n+This hook is invoked by commands that rewrite commits\n+(linkgit:git-commit[1] when called with `--amend` and\n+linkgit:git-rebase[1]; currently `git filter-branch` does 'not' call\n it!). Its first argument denotes the command it was invoked by:\n currently one of `amend` or `rebase`. Further command-dependent\n arguments may be passed in the future.\n@@ -450,16 +451,16 @@ processed by rebase.\n sendemail-validate\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git send-email'. It takes a single parameter,\n+This hook is invoked by linkgit:git-send-email[1]. It takes a single parameter,\n the name of the file that holds the e-mail to be sent. Exiting with a\n-non-zero status causes 'git send-email' to abort before sending any\n+non-zero status causes `git send-email` to abort before sending any\n e-mails.\n \n fsmonitor-watchman\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked when the configuration option core.fsmonitor is\n-set to .git/hooks/fsmonitor-watchman. It takes two arguments, a version\n+This hook is invoked when the configuration option `core.fsmonitor` is\n+set to `.git/hooks/fsmonitor-watchman`. It takes two arguments, a version\n (currently 1) and the time in elapsed nanoseconds since midnight,\n January 1, 1970.\n \n@@ -478,7 +479,7 @@ directories are checked for untracked files based on the path names\n given.\n \n An optimized way to tell git \"all files have changed\" is to return\n-the filename '/'.\n+the filename `/`.\n \n The exit status determines whether git will use the data from the\n hook to limit its search. On error, it will fall back to verifying\n-- \n2.16.2\n\n"},{"id":"344498","messageId":"20180411210857.22598-2-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180411210857.22598-1-asheiduk@gmail.com","subject":"fixup! [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-11T21:08:57Z","receivedAt":"2018-04-11T21:09:38Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"- reflow some paragraphs\n---\n Documentation/githooks.txt | 14 +++++++-------\n 1 file changed, 7 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex be31376767..ab5ce80e13 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -194,10 +194,10 @@ for an example of how to do this.\n pre-push\n ~~~~~~~~\n \n-This hook is called by linkgit:git-push[1] and can be used to prevent a push from taking\n-place. The hook is called with two parameters which provide the name and\n-location of the destination remote, if a named remote is not being used both\n-values will be the same.\n+This hook is called by linkgit:git-push[1] and can be used to prevent\n+a push from taking place. The hook is called with two parameters\n+which provide the name and location of the destination remote, if a\n+named remote is not being used both values will be the same.\n \n Information about what is to be pushed is provided on the hook's standard\n input with lines of the form:\n@@ -410,9 +410,9 @@ with the difference between the branches.\n pre-auto-gc\n ~~~~~~~~~~~\n \n-This hook is invoked by `git gc --auto` (see linkgit:git-gc[1]). It takes no parameter, and\n-exiting with non-zero status from this script causes the `git gc --auto`\n-to abort.\n+This hook is invoked by `git gc --auto` (see linkgit:git-gc[1]). It\n+takes no parameter, and exiting with non-zero status from this script\n+causes the `git gc --auto` to abort.\n \n post-rewrite\n ~~~~~~~~~~~~\n-- \n2.16.2\n\n"},{"id":"344500","messageId":"20180411212109.24051-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"xmqq7epe3gfe.fsf@gitster-ct.c.googlers.com","subject":"fixup! [PATCH 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-11T21:21:09Z","receivedAt":"2018-04-11T21:21:34Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Signed-off-by: Andreas Heiduk \n---\n Documentation/git-diff.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex ee1c509bd3..6593b58299 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -13,7 +13,7 @@ SYNOPSIS\n 'git diff' [options] --cached [] [--] [...]\n 'git diff' [options] [--] [...]\n 'git diff' [options] \n-'git diff' [options] [--no-index] [--] \n+'git diff' [options] --no-index [--] \n \n DESCRIPTION\n -----------\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' [--options] [--no-index] [--] ::\n+'git diff' [--options] --no-index [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n-- \n2.16.2\n\n"},{"id":"344555","messageId":"CAN0heSpD+TurNsP9Xtwh2OwQaenSCJ2GdpDbZWoifPDBC+tZHg@mail.gmail.com","threadId":"48258","inReplyTo":"20180411210857.22598-2-asheiduk@gmail.com","subject":"Re: fixup! [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-12T19:36:14Z","receivedAt":"2018-04-12T19:36:20Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 11 April 2018 at 23:08, Andreas Heiduk wrote:\n> - reflow some paragraphs\n> ---\n> Documentation/githooks.txt | 14 +++++++-------\n> 1 file changed, 7 insertions(+), 7 deletions(-)\n\nI have reviewed the resulting githooks.txt. See the diff below for two\nmore instances that I found. For the second hunk, I have difficulties\nparsing that paragraph, but I still claim those should be backticks and\n*git* read-tree...\n\nMartin\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex ab5ce80e13..e3c283a174 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -80,7 +80,7 @@ This hook is invoked by linkgit:git-am[1]. It takes\nno parameter,\n and is invoked after the patch is applied and a commit is made.\n\n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git am'.\n+the outcome of `git am`.\n\n pre-commit\n ~~~~~~~~~~\n@@ -400,8 +400,8 @@ when the tip of the current branch is updated to\nthe new commit, and\n exit with a zero status.\n\n For example, the hook can simply run `git read-tree -u -m HEAD \"$1\"`\n-in order to emulate 'git fetch' that is run in the reverse direction\n-with `git push`, as the two-tree form of `read-tree -u -m` is\n+in order to emulate `git fetch` that is run in the reverse direction\n+with `git push`, as the two-tree form of `git read-tree -u -m` is\n essentially the same as `git checkout` that switches branches while\n keeping the local changes in the working tree that do not interfere\n with the difference between the branches.\n"},{"id":"344864","messageId":"20180417061814.GA19648@dcvr","threadId":"48258","inReplyTo":"20180410183901.11859-1-asheiduk@gmail.com","subject":"Re: [PATCH 5/6] git-svn: commit-diff does not support --add-author-from","fromName":"Eric Wong","fromEmail":"e@80x24.org","sentAt":"2018-04-17T06:18:14Z","receivedAt":"2018-04-17T06:18:19Z","isPatch":true,"sender":{"key":"e@80x24.org","avatar":null},"body":"Andreas Heiduk wrote:\n> Signed-off-by: Andreas Heiduk \n\nThanks.\nSigned-off-by: Eric Wong \n\nAnd pushed for Junio:\n\nThe following changes since commit fe0a9eaf31dd0c349ae4308498c33a5c3794b293:\n\n Merge branch 'svn/authors-prog-2' of git://bogomips.org/git-svn (2018-04-12 08:05:28 +0900)\n\nare available in the Git repository at:\n\n git://bogomips.org/git-svn.git svn/doc\n\nfor you to fetch changes up to ceb4d16198586f110aad5fbbd7eb68eec7a0c5cd:\n\n git-svn: commit-diff does not support --add-author-from (2018-04-16 21:24:30 +0000)\n\n----------------------------------------------------------------\nAndreas Heiduk (1):\n git-svn: commit-diff does not support --add-author-from\n\n Documentation/git-svn.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n"},{"id":"345952","messageId":"ddf24ddc-d4c0-96a6-ca85-c541df26c532@gmail.com","threadId":"48258","inReplyTo":"CAN0heSpD+TurNsP9Xtwh2OwQaenSCJ2GdpDbZWoifPDBC+tZHg@mail.gmail.com","subject":"Re: fixup! [PATCH 1/6] doc: fix formatting inconsistency in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T16:21:36Z","receivedAt":"2018-04-27T16:21:44Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 12.04.2018 um 21:36 schrieb Martin Ågren:\n> On 11 April 2018 at 23:08, Andreas Heiduk wrote:\n>> - reflow some paragraphs\n>> ---\n>> Documentation/githooks.txt | 14 +++++++-------\n>> 1 file changed, 7 insertions(+), 7 deletions(-)\n>\n> I have reviewed the resulting githooks.txt. See the diff below for two\n> more instances that I found. For the second hunk, I have difficulties\n> parsing that paragraph, but I still claim those should be backticks and\n> *git* read-tree...\n>\n> Martin\n\nI've added the fixes for the next reroll and put you into a Reviewed-by\nTrailer :-)\n\nAndreas\n"},{"id":"345953","messageId":"b06caf94-3c70-a2af-fc32-6f7db90ac4e3@gmail.com","threadId":"48258","inReplyTo":"20180417061814.GA19648@dcvr","subject":"Re: [PATCH 5/6] git-svn: commit-diff does not support --add-author-from","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T16:31:45Z","receivedAt":"2018-04-27T16:31:51Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 17.04.2018 um 08:18 schrieb Eric Wong:\n> Andreas Heiduk wrote:\n>> Signed-off-by: Andreas Heiduk \n> \n> Thanks.\n> Signed-off-by: Eric Wong \n> \n> And pushed for Junio:\n[...]\n\nI'd like to keep the patches together, so I borrow your 'Signed-off-By'\nfrom the commit and do a complete reroll of this mini-series.\n"},{"id":"345955","messageId":"20180427170440.30418-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH v2 0/6] Some doc-fixes","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:34Z","receivedAt":"2018-04-27T17:05:15Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"This reroll incorporates the comments of the first version, including a\n\"large scale\" rewrtie of githooks.txt. I'v added \"Reviewed-by\" and\n\"Signed-off-by\" as appropriate.\n\nAndreas Heiduk (6):\n doc: improve formatting in githooks.txt\n doc: align 'diff --no-index' in text with synopsis\n doc: clarify ignore rules for git ls-files\n doc: add '-d' and '-o' for 'git push'\n git-svn: remove ''--add-author-from' for 'commit-diff'\n doc: add note about shell quoting to revision.txt\n\n Documentation/git-diff.txt | 4 +-\n Documentation/git-ls-files.txt | 3 +-\n Documentation/git-push.txt | 3 +-\n Documentation/git-svn.txt | 2 +-\n Documentation/githooks.txt | 115 +++++++++++++++++++++--------------------\n Documentation/revisions.txt | 6 +++\n 6 files changed, 71 insertions(+), 62 deletions(-)\n\n-- \n2.16.2\n\n"},{"id":"345956","messageId":"20180427170440.30418-2-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 1/6] doc: improve formatting in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:35Z","receivedAt":"2018-04-27T17:05:26Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Typeset commands and similar things with as `git foo` instead of\n'git foo' or 'git-foo' and add linkgit to the commands which run\nthe hooks.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/githooks.txt | 115 +++++++++++++++++++++++----------------------\n 1 file changed, 58 insertions(+), 57 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex f877f7b7cd..e3c283a174 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -31,7 +31,7 @@ Hooks can get their arguments via the environment, command-line\n arguments, and stdin. See the documentation for each hook below for\n details.\n \n-'git init' may copy hooks to the new repository, depending on its\n+`git init` may copy hooks to the new repository, depending on its\n configuration. See the \"TEMPLATE DIRECTORY\" section in\n linkgit:git-init[1] for details. When the rest of this document refers\n to \"default hooks\" it's talking about the default template shipped\n@@ -45,9 +45,9 @@ HOOKS\n applypatch-msg\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes a single\n+This hook is invoked by linkgit:git-am[1]. It takes a single\n parameter, the name of the file that holds the proposed commit\n-log message. Exiting with a non-zero status causes 'git am' to abort\n+log message. Exiting with a non-zero status causes `git am` to abort\n before applying the patch.\n \n The hook is allowed to edit the message file in place, and can\n@@ -61,7 +61,7 @@ The default 'applypatch-msg' hook, when enabled, runs the\n pre-applypatch\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter, and is\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter, and is\n invoked after the patch is applied, but before a commit is made.\n \n If it exits with non-zero status, then the working tree will not be\n@@ -76,33 +76,33 @@ The default 'pre-applypatch' hook, when enabled, runs the\n post-applypatch\n ~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter,\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter,\n and is invoked after the patch is applied and a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git am'.\n+the outcome of `git am`.\n \n pre-commit\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit', and can be bypassed\n+This hook is invoked by linkgit:git-commit[1], and can be bypassed\n with the `--no-verify` option. It takes no parameters, and is\n invoked before obtaining the proposed commit log message and\n making a commit. Exiting with a non-zero status from this script\n-causes the 'git commit' command to abort before creating a commit.\n+causes the `git commit` command to abort before creating a commit.\n \n The default 'pre-commit' hook, when enabled, catches introduction\n of lines with trailing whitespaces and aborts the commit when\n such a line is found.\n \n-All the 'git commit' hooks are invoked with the environment\n+All the `git commit` hooks are invoked with the environment\n variable `GIT_EDITOR=:` if the command will not bring up an editor\n to modify the commit message.\n \n prepare-commit-msg\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git commit' right after preparing the\n+This hook is invoked by linkgit:git-commit[1] right after preparing the\n default log message, and before the editor is started.\n \n It takes one to three parameters. The first is the name of the file\n@@ -114,7 +114,7 @@ commit is a merge or a `.git/MERGE_MSG` file exists); `squash`\n (if a `.git/SQUASH_MSG` file exists); or `commit`, followed by\n a commit SHA-1 (if a `-c`, `-C` or `--amend` option was given).\n \n-If the exit status is non-zero, 'git commit' will abort.\n+If the exit status is non-zero, `git commit` will abort.\n \n The purpose of the hook is to edit the message file in place, and\n it is not suppressed by the `--no-verify` option. A non-zero exit\n@@ -127,7 +127,7 @@ help message found in the commented portion of the commit template.\n commit-msg\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit' and 'git merge', and can be\n+This hook is invoked by linkgit:git-commit[1] and linkgit:git-merge[1], and can be\n bypassed with the `--no-verify` option. It takes a single parameter,\n the name of the file that holds the proposed commit log message.\n Exiting with a non-zero status causes the command to abort.\n@@ -143,16 +143,16 @@ The default 'commit-msg' hook, when enabled, detects duplicate\n post-commit\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git commit'. It takes no parameters, and is\n+This hook is invoked by linkgit:git-commit[1]. It takes no parameters, and is\n invoked after a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git commit'.\n+the outcome of `git commit`.\n \n pre-rebase\n ~~~~~~~~~~\n \n-This hook is called by 'git rebase' and can be used to prevent a\n+This hook is called by linkgit:git-rebase[1] and can be used to prevent a\n branch from getting rebased. The hook may be called with one or\n two parameters. The first parameter is the upstream from which\n the series was forked. The second parameter is the branch being\n@@ -161,17 +161,17 @@ rebased, and is not set when rebasing the current branch.\n post-checkout\n ~~~~~~~~~~~~~\n \n-This hook is invoked when a 'git checkout' is run after having updated the\n+This hook is invoked when a linkgit:git-checkout[1] is run after having updated the\n worktree. The hook is given three parameters: the ref of the previous HEAD,\n the ref of the new HEAD (which may or may not have changed), and a flag\n indicating whether the checkout was a branch checkout (changing branches,\n flag=1) or a file checkout (retrieving a file from the index, flag=0).\n-This hook cannot affect the outcome of 'git checkout'.\n+This hook cannot affect the outcome of `git checkout`.\n \n-It is also run after 'git clone', unless the --no-checkout (-n) option is\n+It is also run after linkgit:git-clone[1], unless the `--no-checkout` (`-n`) option is\n used. The first parameter given to the hook is the null-ref, the second the\n-ref of the new HEAD and the flag is always 1. Likewise for 'git worktree add'\n-unless --no-checkout is used.\n+ref of the new HEAD and the flag is always 1. Likewise for `git worktree add`\n+unless `--no-checkout` is used.\n \n This hook can be used to perform repository validity checks, auto-display\n differences from the previous HEAD if different, or set working dir metadata\n@@ -180,10 +180,10 @@ properties.\n post-merge\n ~~~~~~~~~~\n \n-This hook is invoked by 'git merge', which happens when a 'git pull'\n+This hook is invoked by linkgit:git-merge[1], which happens when a `git pull`\n is done on a local repository. The hook takes a single parameter, a status\n flag specifying whether or not the merge being done was a squash merge.\n-This hook cannot affect the outcome of 'git merge' and is not executed,\n+This hook cannot affect the outcome of `git merge` and is not executed,\n if the merge failed due to conflicts.\n \n This hook can be used in conjunction with a corresponding pre-commit hook to\n@@ -194,10 +194,10 @@ for an example of how to do this.\n pre-push\n ~~~~~~~~\n \n-This hook is called by 'git push' and can be used to prevent a push from taking\n-place. The hook is called with two parameters which provide the name and\n-location of the destination remote, if a named remote is not being used both\n-values will be the same.\n+This hook is called by linkgit:git-push[1] and can be used to prevent\n+a push from taking place. The hook is called with two parameters\n+which provide the name and location of the destination remote, if a\n+named remote is not being used both values will be the same.\n \n Information about what is to be pushed is provided on the hook's standard\n input with lines of the form:\n@@ -216,7 +216,7 @@ SHA-1>` will be 40 `0`. If the local commit was specified by something other\n than a name which could be expanded (such as `HEAD~`, or a SHA-1) it will be\n supplied as it was originally given.\n \n-If this hook exits with a non-zero status, 'git push' will abort without\n+If this hook exits with a non-zero status, `git push` will abort without\n pushing anything. Information about why the push is rejected may be sent\n to the user by writing to standard error.\n \n@@ -224,8 +224,8 @@ to the user by writing to standard error.\n pre-receive\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before starting to update refs on the remote repository, the\n pre-receive hook is invoked. Its exit status determines the success\n or failure of the update.\n@@ -246,7 +246,7 @@ updated. If the hook exits with zero, updating of individual refs can\n still be prevented by the <> hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The number of push options given on the command line of\n@@ -265,8 +265,8 @@ linkgit:git-receive-pack[1] for some caveats.\n update\n ~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before updating the ref on the remote repository, the update hook\n is invoked. Its exit status determines the success or failure of\n the ref update.\n@@ -279,7 +279,7 @@ three parameters:\n - and the new object name to be stored in the ref.\n \n A zero exit from the update hook allows the ref to be updated.\n-Exiting with a non-zero status prevents 'git-receive-pack'\n+Exiting with a non-zero status prevents `git receive-pack`\n from updating that ref.\n \n This hook can be used to prevent 'forced' update on certain refs by\n@@ -299,7 +299,7 @@ membership. See linkgit:git-shell[1] for how you might use the login\n shell to restrict the user's access to only git commands.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'update' hook, when enabled--and with\n@@ -310,8 +310,8 @@ unannotated tags to be pushed.\n post-receive\n ~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -320,7 +320,7 @@ arguments, but gets the same information as the\n <>\n hook does on its standard input.\n \n-This hook does not affect the outcome of 'git-receive-pack', as it\n+This hook does not affect the outcome of `git receive-pack`, as it\n is called after the real work is done.\n \n This supersedes the <> hook in that it gets\n@@ -328,7 +328,7 @@ both old and new values of all the refs in addition to their\n names.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'post-receive' hook is empty, but there is\n@@ -349,8 +349,8 @@ will be set to zero, `GIT_PUSH_OPTION_COUNT=0`.\n post-update\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -358,7 +358,7 @@ It takes a variable number of parameters, each of which is the\n name of ref that was actually updated.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git-receive-pack'.\n+the outcome of `git receive-pack`.\n \n The 'post-update' hook can tell what are the heads that were pushed,\n but it does not know what their original and updated values are,\n@@ -368,20 +368,20 @@ updated values of the refs. You might consider it instead if you need\n them.\n \n When enabled, the default 'post-update' hook runs\n-'git update-server-info' to keep the information used by dumb\n+`git update-server-info` to keep the information used by dumb\n transports (e.g., HTTP) up to date. If you are publishing\n a Git repository that is accessible via HTTP, you should\n probably enable this hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n push-to-checkout\n ~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository, and when\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository, and when\n the push tries to update the branch that is currently checked out\n and the `receive.denyCurrentBranch` configuration variable is set to\n `updateInstead`. Such a push by default is refused if the working\n@@ -400,8 +400,8 @@ when the tip of the current branch is updated to the new commit, and\n exit with a zero status.\n \n For example, the hook can simply run `git read-tree -u -m HEAD \"$1\"`\n-in order to emulate 'git fetch' that is run in the reverse direction\n-with `git push`, as the two-tree form of `read-tree -u -m` is\n+in order to emulate `git fetch` that is run in the reverse direction\n+with `git push`, as the two-tree form of `git read-tree -u -m` is\n essentially the same as `git checkout` that switches branches while\n keeping the local changes in the working tree that do not interfere\n with the difference between the branches.\n@@ -410,15 +410,16 @@ with the difference between the branches.\n pre-auto-gc\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git gc --auto'. It takes no parameter, and\n-exiting with non-zero status from this script causes the 'git gc --auto'\n-to abort.\n+This hook is invoked by `git gc --auto` (see linkgit:git-gc[1]). It\n+takes no parameter, and exiting with non-zero status from this script\n+causes the `git gc --auto` to abort.\n \n post-rewrite\n ~~~~~~~~~~~~\n \n-This hook is invoked by commands that rewrite commits (`git commit\n---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call\n+This hook is invoked by commands that rewrite commits\n+(linkgit:git-commit[1] when called with `--amend` and\n+linkgit:git-rebase[1]; currently `git filter-branch` does 'not' call\n it!). Its first argument denotes the command it was invoked by:\n currently one of `amend` or `rebase`. Further command-dependent\n arguments may be passed in the future.\n@@ -450,16 +451,16 @@ processed by rebase.\n sendemail-validate\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git send-email'. It takes a single parameter,\n+This hook is invoked by linkgit:git-send-email[1]. It takes a single parameter,\n the name of the file that holds the e-mail to be sent. Exiting with a\n-non-zero status causes 'git send-email' to abort before sending any\n+non-zero status causes `git send-email` to abort before sending any\n e-mails.\n \n fsmonitor-watchman\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked when the configuration option core.fsmonitor is\n-set to .git/hooks/fsmonitor-watchman. It takes two arguments, a version\n+This hook is invoked when the configuration option `core.fsmonitor` is\n+set to `.git/hooks/fsmonitor-watchman`. It takes two arguments, a version\n (currently 1) and the time in elapsed nanoseconds since midnight,\n January 1, 1970.\n \n@@ -478,7 +479,7 @@ directories are checked for untracked files based on the path names\n given.\n \n An optimized way to tell git \"all files have changed\" is to return\n-the filename '/'.\n+the filename `/`.\n \n The exit status determines whether git will use the data from the\n hook to limit its search. On error, it will fall back to verifying\n-- \n2.16.2\n\n"},{"id":"345957","messageId":"20180427170440.30418-3-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:36Z","receivedAt":"2018-04-27T17:05:29Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"The two '' parameters are not optional but the option\n'--no-index' is. Also move the `--options` part to the same\nplace where the other variants show them.\n\nAll three items are already correct in the synopsis.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/git-diff.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex b0c1bb95c8..6593b58299 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -13,7 +13,7 @@ SYNOPSIS\n 'git diff' [options] --cached [] [--] [...]\n 'git diff' [options] [--] [...]\n 'git diff' [options] \n-'git diff' [options] [--no-index] [--] \n+'git diff' [options] --no-index [--] \n \n DESCRIPTION\n -----------\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' --no-index [--options] [--] [...]::\n+'git diff' [--options] --no-index [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n-- \n2.16.2\n\n"},{"id":"345958","messageId":"20180427170440.30418-5-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 4/6] doc: add '-d' and '-o' for 'git push'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:38Z","receivedAt":"2018-04-27T17:05:33Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Add the missing `-o` shortcut for `--push-option` to the synopsis.\nAdd the missing `-d` shortcut for `--delete` in the main section.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/git-push.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\nindex 5b08302fc2..f2bbda6e32 100644\n--- a/Documentation/git-push.txt\n+++ b/Documentation/git-push.txt\n@@ -11,7 +11,7 @@ SYNOPSIS\n [verse]\n 'git push' [--all | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=]\n \t [--repo=] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]\n-\t [-u | --set-upstream] [--push-option=]\n+\t [-u | --set-upstream] [-o | --push-option=]\n \t [--[no-]signed|--signed=(true|false|if-asked)]\n \t [--force-with-lease[=[:]]]\n \t [--no-verify] [ [...]]\n@@ -123,6 +123,7 @@ already exists on the remote side.\n \twill be tab-separated and sent to stdout instead of stderr. The full\n \tsymbolic names of the refs will be given.\n \n+-d::\n --delete::\n \tAll listed refs are deleted from the remote repository. This is\n \tthe same as prefixing all refs with a colon.\n-- \n2.16.2\n\n"},{"id":"345959","messageId":"20180427170440.30418-4-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 3/6] doc: clarify ignore rules for git ls-files","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:37Z","receivedAt":"2018-04-27T17:05:36Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Explain that `git ls-files --ignored` requires at least one\nof the `--exclude*` options to do its job.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-ls-files.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-ls-files.txt b/Documentation/git-ls-files.txt\nindex 3ac3e3a77d..f3474b2ede 100644\n--- a/Documentation/git-ls-files.txt\n+++ b/Documentation/git-ls-files.txt\n@@ -53,7 +53,8 @@ OPTIONS\n \tShow only ignored files in the output. When showing files in the\n \tindex, print only those matched by an exclude pattern. When\n \tshowing \"other\" files, show only those matched by an exclude\n-\tpattern.\n+\tpattern. Standard ignore rules are not automatically activated,\n+\ttherefore at least one of the `--exclude*` options is required.\n \n -s::\n --stage::\n-- \n2.16.2\n\n"},{"id":"345960","messageId":"20180427170440.30418-6-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 5/6] git-svn: remove ''--add-author-from' for 'commit-diff'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:39Z","receivedAt":"2018-04-27T17:05:39Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"The subcommand 'commit-diff' does not support the option\n'--add-author-from'.\n\nSigned-off-by: Andreas Heiduk \nSigned-off-by: Eric Wong \n---\n Documentation/git-svn.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt\nindex d59379ee23..e9615951d2 100644\n--- a/Documentation/git-svn.txt\n+++ b/Documentation/git-svn.txt\n@@ -707,7 +707,7 @@ creating the branch or tag.\n config key: svn.useLogAuthor\n \n --add-author-from::\n-\tWhen committing to svn from Git (as part of 'commit-diff', 'set-tree' or 'dcommit'\n+\tWhen committing to svn from Git (as part of 'set-tree' or 'dcommit'\n \toperations), if the existing log message doesn't already have a\n \t`From:` or `Signed-off-by:` line, append a `From:` line based on the\n \tGit commit's author string. If you use this, then `--use-log-author`\n-- \n2.16.2\n\n"},{"id":"345961","messageId":"20180427170440.30418-7-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v2 6/6] doc: add note about shell quoting to revision.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T17:04:40Z","receivedAt":"2018-04-27T17:05:41Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Signed-off-by: Andreas Heiduk \nReviewed-by: Junio C Hamano \n---\n Documentation/revisions.txt | 6 ++++++\n 1 file changed, 6 insertions(+)\n\ndiff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\nindex dfcc49c72c..c1d3a40a90 100644\n--- a/Documentation/revisions.txt\n+++ b/Documentation/revisions.txt\n@@ -7,6 +7,10 @@ syntax. Here are various ways to spell object names. The\n ones listed near the end of this list name trees and\n blobs contained in a commit.\n \n+NOTE: This document shows the \"raw\" syntax as seen by git. The shell\n+and other UIs might require additional quoting to protect special\n+characters and to avoid word splitting.\n+\n '', e.g. 'dae86e1950b1277e545cee180551750029cfe735', 'dae86e'::\n The full SHA-1 object name (40-byte hexadecimal string), or\n a leading substring that is unique within the repository.\n@@ -186,6 +190,8 @@ existing tag object.\n is matched. ':/!-foo' performs a negative match, while ':/!!foo' matches a\n literal '!' character, followed by 'foo'. Any other sequence beginning with\n ':/!' is reserved for now.\n+ Depending on the given text the shell's word splitting rules might\n+ require additional quoting.\n \n ':', e.g. 'HEAD:README', ':README', 'master:./README'::\n A suffix ':' followed by a path names the blob or tree\n-- \n2.16.2\n\n"},{"id":"345964","messageId":"CAN0heSpdc51+7-uWcLVXU72qLZgZApdME8ywBJaQQy4DiWHAfg@mail.gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-3-asheiduk@gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-27T17:18:23Z","receivedAt":"2018-04-27T17:18:29Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 27 April 2018 at 19:04, Andreas Heiduk wrote:\n> The two '' parameters are not optional but the option\n> '--no-index' is. Also move the `--options` part to the same\n> place where the other variants show them.\n\nShould this commit message be updated after the changes you did to\naddress Junio's comment? This text suggests you want to place --no-index\nin [] (and you did in v1) but you do not do that below.\n\n> All three items are already correct in the synopsis.\n\nSame here, now you actually do change things there.\n\n> Signed-off-by: Andreas Heiduk \n> Reviewed-by: Martin Ågren \n\nStrictly speaking, my Reviewed-by was on another patch. I do find this\none better though thanks to Junio's suggestion (except the mismatch with\nthe commit message).\n\nThanks for continuing with this series.\n\nMartin\n\n> ---\n> Documentation/git-diff.txt | 4 ++--\n> 1 file changed, 2 insertions(+), 2 deletions(-)\n>\n> diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\n> index b0c1bb95c8..6593b58299 100644\n> --- a/Documentation/git-diff.txt\n> +++ b/Documentation/git-diff.txt\n> @@ -13,7 +13,7 @@ SYNOPSIS\n> 'git diff' [options] --cached [] [--] [...]\n> 'git diff' [options] [--] [...]\n> 'git diff' [options] \n> -'git diff' [options] [--no-index] [--] \n> +'git diff' [options] --no-index [--] \n>\n> DESCRIPTION\n> -----------\n> @@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n> further add to the index but you still haven't. You can\n> stage these changes by using linkgit:git-add[1].\n>\n> -'git diff' --no-index [--options] [--] [...]::\n> +'git diff' [--options] --no-index [--] ::\n>\n> This form is to compare the given two paths on the\n> filesystem. You can omit the `--no-index` option when\n"},{"id":"345966","messageId":"CAPig+cTM1J35hvwYwj9BOkFxw0tGQJFZjPL_WnCa=wDxzYzj0g@mail.gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-3-asheiduk@gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2018-04-27T17:33:54Z","receivedAt":"2018-04-27T17:33:58Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n> The two '' parameters are not optional but the option\n> '--no-index' is. Also move the `--options` part to the same\n> place where the other variants show them.\n>\n> All three items are already correct in the synopsis.\n>\n> Signed-off-by: Andreas Heiduk \n> ---\n> diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\n> @@ -13,7 +13,7 @@ SYNOPSIS\n> -'git diff' [options] [--no-index] [--] \n> +'git diff' [options] --no-index [--] \n> @@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n> -'git diff' --no-index [--options] [--] [...]::\n> +'git diff' [--options] --no-index [--] ::\n\nNot a problem introduced by this patch, but shouldn't this say\n\"[options]\" rather than \"[--options]\"? Since the aim of this patch\nseries is to clean up botches and normalize documentation, perhaps it\ncould also fix this oddness(?).\n"},{"id":"345967","messageId":"CAPig+cRWn66C6wcmYv5G3qjfZAux6pVCp1H+w9c4+CzevfXMRg@mail.gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-7-asheiduk@gmail.com","subject":"Re: [PATCH v2 6/6] doc: add note about shell quoting to revision.txt","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2018-04-27T17:36:38Z","receivedAt":"2018-04-27T17:36:47Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n> Signed-off-by: Andreas Heiduk \n> Reviewed-by: Junio C Hamano \n> ---\n> diff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\n> @@ -186,6 +190,8 @@ existing tag object.\n> + Depending on the given text the shell's word splitting rules might\n> + require additional quoting.\n\ns/text/&,/\n"},{"id":"345969","messageId":"a3253c99-63a4-22bc-53a0-4d94c01a564c@gmail.com","threadId":"48258","inReplyTo":"CAN0heSpdc51+7-uWcLVXU72qLZgZApdME8ywBJaQQy4DiWHAfg@mail.gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T18:28:06Z","receivedAt":"2018-04-27T18:28:13Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 27.04.2018 um 19:18 schrieb Martin Ågren:\n> On 27 April 2018 at 19:04, Andreas Heiduk wrote:\n>> The two '' parameters are not optional but the option\n>> '--no-index' is. Also move the `--options` part to the same\n>> place where the other variants show them.\n> \n> Should this commit message be updated after the changes you did to\n> address Junio's comment? This text suggests you want to place --no-index\n> in [] (and you did in v1) but you do not do that below.\n> \n>> All three items are already correct in the synopsis.\n> \n> Same here, now you actually do change things there.\n> \n>> Signed-off-by: Andreas Heiduk \n>> Reviewed-by: Martin Ågren \n> \n> Strictly speaking, my Reviewed-by was on another patch. I do find this\n\nSorry, I've added that trailer after reading \"The diff LGTM.\", then\napplied Junio's changes and forgot to remove the trailer.\n\n> one better though thanks to Junio's suggestion (except the mismatch with\n> the commit message).\n\nI'll fix that with this:\n\n\tdoc: align 'diff --no-index' in text with synopsis\n\n\tMake the two '' parameters in DESCRIPTION mandatory and\n\tmove the `--options` part to the same place where the other\n\tvariants show them. And finally make `--no-index` in SYNOPSIS\n\tas mandatory as in DESCRIPTION.\n\n"},{"id":"345972","messageId":"bb44e716-a57b-71fb-4446-20809ddb73d1@gmail.com","threadId":"48258","inReplyTo":"CAPig+cTM1J35hvwYwj9BOkFxw0tGQJFZjPL_WnCa=wDxzYzj0g@mail.gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T18:40:39Z","receivedAt":"2018-04-27T18:40:47Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 27.04.2018 um 19:33 schrieb Eric Sunshine:\n> On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n>> The two '' parameters are not optional but the option\n>> '--no-index' is. Also move the `--options` part to the same\n>> place where the other variants show them.\n>>\n>> All three items are already correct in the synopsis.\n>>\n>> Signed-off-by: Andreas Heiduk \n>> ---\n>> diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\n>> @@ -13,7 +13,7 @@ SYNOPSIS\n>> -'git diff' [options] [--no-index] [--] \n>> +'git diff' [options] --no-index [--] \n>> @@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n>> -'git diff' --no-index [--options] [--] [...]::\n>> +'git diff' [--options] --no-index [--] ::\n> \n> Not a problem introduced by this patch, but shouldn't this say\n> \"[options]\" rather than \"[--options]\"? Since the aim of this patch\n> series is to clean up botches and normalize documentation, perhaps it\n> could also fix this oddness(?).\n> \n\nWell, in the SYNOPSIS it is always `[options]` for all variants but in\nthe DESCRIPTION it is always `[--options]` for all variants. Fixing the\nother variants would stretch the \"subject\" line of the patch a little\nbit to far ;-)\n"},{"id":"345973","messageId":"d6325d96-f44d-c4da-c651-cc4b99ab6d0f@gmail.com","threadId":"48258","inReplyTo":"CAPig+cRWn66C6wcmYv5G3qjfZAux6pVCp1H+w9c4+CzevfXMRg@mail.gmail.com","subject":"Re: [PATCH v2 6/6] doc: add note about shell quoting to revision.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T18:42:42Z","receivedAt":"2018-04-27T18:42:48Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 27.04.2018 um 19:36 schrieb Eric Sunshine:\n> On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n>> Signed-off-by: Andreas Heiduk \n>> Reviewed-by: Junio C Hamano \n>> ---\n>> diff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\n>> @@ -186,6 +190,8 @@ existing tag object.\n>> + Depending on the given text the shell's word splitting rules might\n>> + require additional quoting.\n> \n> s/text/&,/\n> \n\nFixed, Thanks\n"},{"id":"345974","messageId":"CAN0heSqSu0g=3V6MueOQaYKv-HWDbXDBwrEQ=o3r4v_xa1xWkw@mail.gmail.com","threadId":"48258","inReplyTo":"bb44e716-a57b-71fb-4446-20809ddb73d1@gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-27T18:43:55Z","receivedAt":"2018-04-27T18:44:04Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 27 April 2018 at 20:40, Andreas Heiduk wrote:\n> Am 27.04.2018 um 19:33 schrieb Eric Sunshine:\n>> On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n>>> The two '' parameters are not optional but the option\n>>> '--no-index' is. Also move the `--options` part to the same\n>>> place where the other variants show them.\n>>>\n>>> All three items are already correct in the synopsis.\n>>>\n>>> Signed-off-by: Andreas Heiduk \n>>> ---\n>>> diff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\n>>> @@ -13,7 +13,7 @@ SYNOPSIS\n>>> -'git diff' [options] [--no-index] [--] \n>>> +'git diff' [options] --no-index [--] \n>>> @@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n>>> -'git diff' --no-index [--options] [--] [...]::\n>>> +'git diff' [--options] --no-index [--] ::\n>>\n>> Not a problem introduced by this patch, but shouldn't this say\n>> \"[options]\" rather than \"[--options]\"? Since the aim of this patch\n>> series is to clean up botches and normalize documentation, perhaps it\n>> could also fix this oddness(?).\n>>\n>\n> Well, in the SYNOPSIS it is always `[options]` for all variants but in\n> the DESCRIPTION it is always `[--options]` for all variants. Fixing the\n> other variants would stretch the \"subject\" line of the patch a little\n> bit to far ;-)\n\nHmm, I do not think it's always though. It's pretty consistent in its\ninconsistency, but \"git diff [options] \" goes the other\nway. Maybe that's patch 1/7...\n"},{"id":"345975","messageId":"CAN0heSquZ9WGQE=SCZ0DkgsbKPc80r5TmYhNXOGKFJ6MaJ_sOQ@mail.gmail.com","threadId":"48258","inReplyTo":"a3253c99-63a4-22bc-53a0-4d94c01a564c@gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-27T18:45:35Z","receivedAt":"2018-04-27T18:45:39Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 27 April 2018 at 20:28, Andreas Heiduk wrote:\n> Am 27.04.2018 um 19:18 schrieb Martin Ågren:\n>> On 27 April 2018 at 19:04, Andreas Heiduk wrote:\n>>> The two '' parameters are not optional but the option\n>>> '--no-index' is. Also move the `--options` part to the same\n>>> place where the other variants show them.\n>>\n>> Should this commit message be updated after the changes you did to\n>> address Junio's comment? This text suggests you want to place --no-index\n>> in [] (and you did in v1) but you do not do that below.\n>>\n>>> All three items are already correct in the synopsis.\n>>\n>> Same here, now you actually do change things there.\n>>\n>>> Signed-off-by: Andreas Heiduk \n>>> Reviewed-by: Martin Ågren \n>>\n>> Strictly speaking, my Reviewed-by was on another patch. I do find this\n>\n> Sorry, I've added that trailer after reading \"The diff LGTM.\", then\n> applied Junio's changes and forgot to remove the trailer.\n>\n>> one better though thanks to Junio's suggestion (except the mismatch with\n>> the commit message).\n>\n> I'll fix that with this:\n>\n> doc: align 'diff --no-index' in text with synopsis\n\ns/with/and/ since they both change? It's not that the first changes to\nmatch the second, but they actually both change to match each other (and\nto be correct, obviously).\n\n> Make the two '' parameters in DESCRIPTION mandatory and\n> move the `--options` part to the same place where the other\n> variants show them. And finally make `--no-index` in SYNOPSIS\n> as mandatory as in DESCRIPTION.\n\nGreat! Junio had some good reasoning about how --no-index is\nsometimes optional, but not always. Not sure if it's worth spelling that\nout. (Although one could argue that it already did trip us up once. :-))\n\nEric's point about \"--options\" vs \"options\" seemed right to me. If you\naddress that, note that this message says \"--options\".\n"},{"id":"345976","messageId":"CAN0heSrd8ej0YMmWrNXw-DRpwEZi=sTL91zNFvHA4tiqz=KAfA@mail.gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-2-asheiduk@gmail.com","subject":"Re: [PATCH v2 1/6] doc: improve formatting in githooks.txt","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-04-27T18:46:40Z","receivedAt":"2018-04-27T18:46:45Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 27 April 2018 at 19:04, Andreas Heiduk wrote:\n> Typeset commands and similar things with as `git foo` instead of\n> 'git foo' or 'git-foo' and add linkgit to the commands which run\n> the hooks.\n>\n> Signed-off-by: Andreas Heiduk \n> Reviewed-by: Martin Ågren \n\nIndeed. The difference between last time (the original patch and the two\nfixups) and this patch is precisely the small tweaks that I suggested.\n\nThanks\nMartin\n"},{"id":"345978","messageId":"61a137f6-f4bd-c15e-b8c9-00c8425c11ac@gmail.com","threadId":"48258","inReplyTo":"CAN0heSquZ9WGQE=SCZ0DkgsbKPc80r5TmYhNXOGKFJ6MaJ_sOQ@mail.gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T19:08:18Z","receivedAt":"2018-04-27T19:08:24Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Am 27.04.2018 um 20:45 schrieb Martin Ågren:\n> On 27 April 2018 at 20:28, Andreas Heiduk wrote:\n>> Am 27.04.2018 um 19:18 schrieb Martin Ågren:\n>>> On 27 April 2018 at 19:04, Andreas Heiduk wrote:\n>>>> The two '' parameters are not optional but the option\n>>>> '--no-index' is. Also move the `--options` part to the same\n>>>> place where the other variants show them.\n>>>\n>>> Should this commit message be updated after the changes you did to\n>>> address Junio's comment? This text suggests you want to place --no-index\n>>> in [] (and you did in v1) but you do not do that below.\n>>>\n>>>> All three items are already correct in the synopsis.\n>>>\n>>> Same here, now you actually do change things there.\n>>>\n>>>> Signed-off-by: Andreas Heiduk \n>>>> Reviewed-by: Martin Ågren \n>>>\n>>> Strictly speaking, my Reviewed-by was on another patch. I do find this\n>>\n>> Sorry, I've added that trailer after reading \"The diff LGTM.\", then\n>> applied Junio's changes and forgot to remove the trailer.\n>>\n>>> one better though thanks to Junio's suggestion (except the mismatch with\n>>> the commit message).\n>>\n>> I'll fix that with this:\n>>\n>> doc: align 'diff --no-index' in text with synopsis\n> \n> s/with/and/ since they both change? It's not that the first changes to\n> match the second, but they actually both change to match each other (and\n> to be correct, obviously).\n\nCorrected\n\n> \n>> Make the two '' parameters in DESCRIPTION mandatory and\n>> move the `--options` part to the same place where the other\n>> variants show them. And finally make `--no-index` in SYNOPSIS\n>> as mandatory as in DESCRIPTION.\n> \n> Great! Junio had some good reasoning about how --no-index is\n> sometimes optional, but not always. Not sure if it's worth spelling that\n> out. (Although one could argue that it already did trip us up once. :-))\n\nThe post-context already explains that.\n\n> Eric's point about \"--options\" vs \"options\" seemed right to me. If you\n> address that, note that this message says \"--options\".\n\n"},{"id":"345980","messageId":"CAPig+cQyVWXfLXGJO2JBxXTB1sT3ba_AukbKtaYwq8SjWcKq8g@mail.gmail.com","threadId":"48258","inReplyTo":"bb44e716-a57b-71fb-4446-20809ddb73d1@gmail.com","subject":"Re: [PATCH v2 2/6] doc: align 'diff --no-index' in text with synopsis","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2018-04-27T19:12:46Z","receivedAt":"2018-04-27T19:12:50Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Fri, Apr 27, 2018 at 2:40 PM, Andreas Heiduk wrote:\n> Am 27.04.2018 um 19:33 schrieb Eric Sunshine:\n>> On Fri, Apr 27, 2018 at 1:04 PM, Andreas Heiduk wrote:\n>>> @@ -13,7 +13,7 @@ SYNOPSIS\n>>> -'git diff' [options] [--no-index] [--] \n>>> +'git diff' [options] --no-index [--] \n>>> @@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n>>> -'git diff' --no-index [--options] [--] [...]::\n>>> +'git diff' [--options] --no-index [--] ::\n>>\n>> Not a problem introduced by this patch, but shouldn't this say\n>> \"[options]\" rather than \"[--options]\"? Since the aim of this patch\n>> series is to clean up botches and normalize documentation, perhaps it\n>> could also fix this oddness(?).\n>\n> Well, in the SYNOPSIS it is always `[options]` for all variants but in\n> the DESCRIPTION it is always `[--options]` for all variants. Fixing the\n> other variants would stretch the \"subject\" line of the patch a little\n> bit to far ;-)\n\nI wasn't suggesting that this patch should fix that issue (it\nshouldn't) but that it could/should be done by a separate new patch\nsince it's a distinct change. (That's why I was careful to say \"aim of\nthis patch _series_\".)\n"},{"id":"345982","messageId":"20180427192350.9910-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180410183224.10780-1-asheiduk@gmail.com","subject":"[PATCH v2 7/6] doc: normalize [--options] to [options] in git-diff","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-04-27T19:23:50Z","receivedAt":"2018-04-27T19:24:17Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"SYNOPSIS and other manuals use [options] but DESCRIPTION\nused [--options].\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-diff.txt | 14 +++++++-------\n 1 file changed, 7 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex 6593b58299..7c2c442700 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -21,7 +21,7 @@ Show changes between the working tree and the index or a tree, changes\n between the index and a tree, changes between two trees, changes between\n two blob objects, or changes between two files on disk.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis form is to view the changes you made relative to\n \tthe index (staging area for the next commit). In other\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' [--options] --no-index [--] ::\n+'git diff' [options] --no-index [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n@@ -38,7 +38,7 @@ two blob objects, or changes between two files on disk.\n \tor when running the command outside a working tree\n \tcontrolled by Git.\n \n-'git diff' [--options] --cached [] [--] [...]::\n+'git diff' [options] --cached [] [--] [...]::\n \n \tThis form is to view the changes you staged for the next\n \tcommit relative to the named . Typically you\n@@ -48,7 +48,7 @@ two blob objects, or changes between two files on disk.\n \t is not given, it shows all staged changes.\n \t--staged is a synonym of --cached.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis form is to view the changes you have in your\n \tworking tree relative to the named . You can\n@@ -56,18 +56,18 @@ two blob objects, or changes between two files on disk.\n \tbranch name to compare with the tip of a different\n \tbranch.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis is to view the changes between two arbitrary\n \t.\n \n-'git diff' [--options] .. [--] [...]::\n+'git diff' [options] .. [--] [...]::\n \n \tThis is synonymous to the previous form. If on\n \tone side is omitted, it will have the same effect as\n \tusing HEAD instead.\n \n-'git diff' [--options] \\... [--] [...]::\n+'git diff' [options] \\... [--] [...]::\n \n \tThis form is to view the changes on the branch containing\n \tand up to the second , starting at a common ancestor\n-- \n2.16.2\n\n"},{"id":"346561","messageId":"20180503184830.11889-1-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180427170440.30418-1-asheiduk@gmail.com","subject":"[PATCH v3 0/7] Some doc-fixes","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:23Z","receivedAt":"2018-05-03T18:48:47Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Changes since the last reroll:\n\n- Better commit comment for \"doc: align 'diff --no-index' in text and synopsis\"\n This includes Martin's `s/with/and/` comment.\n- Eric's typo fix in \"doc: add note about shell quoting to revision.txt\"\n- Added new patch for git-diff.txt with s/--options/options/.\n This addresses Eric's and Martin's comments.\n \n\nAndreas Heiduk (7):\n doc: improve formatting in githooks.txt\n doc: align 'diff --no-index' in text and synopsis\n doc: clarify ignore rules for git ls-files\n doc: add '-d' and '-o' for 'git push'\n git-svn: remove ''--add-author-from' for 'commit-diff'\n doc: add note about shell quoting to revision.txt\n doc: normalize [--options] to [options] in git-diff\n\n Documentation/git-diff.txt | 16 +++---\n Documentation/git-ls-files.txt | 3 +-\n Documentation/git-push.txt | 3 +-\n Documentation/git-svn.txt | 2 +-\n Documentation/githooks.txt | 115 +++++++++++++++++++++--------------------\n Documentation/revisions.txt | 6 +++\n 6 files changed, 77 insertions(+), 68 deletions(-)\n\n-- \n2.16.2\n\n"},{"id":"346562","messageId":"20180503184830.11889-2-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 1/7] doc: improve formatting in githooks.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:24Z","receivedAt":"2018-05-03T18:48:53Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Typeset commands and similar things with as `git foo` instead of\n'git foo' or 'git-foo' and add linkgit to the commands which run\nthe hooks.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/githooks.txt | 115 +++++++++++++++++++++++----------------------\n 1 file changed, 58 insertions(+), 57 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex f877f7b7cd..e3c283a174 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -31,7 +31,7 @@ Hooks can get their arguments via the environment, command-line\n arguments, and stdin. See the documentation for each hook below for\n details.\n \n-'git init' may copy hooks to the new repository, depending on its\n+`git init` may copy hooks to the new repository, depending on its\n configuration. See the \"TEMPLATE DIRECTORY\" section in\n linkgit:git-init[1] for details. When the rest of this document refers\n to \"default hooks\" it's talking about the default template shipped\n@@ -45,9 +45,9 @@ HOOKS\n applypatch-msg\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes a single\n+This hook is invoked by linkgit:git-am[1]. It takes a single\n parameter, the name of the file that holds the proposed commit\n-log message. Exiting with a non-zero status causes 'git am' to abort\n+log message. Exiting with a non-zero status causes `git am` to abort\n before applying the patch.\n \n The hook is allowed to edit the message file in place, and can\n@@ -61,7 +61,7 @@ The default 'applypatch-msg' hook, when enabled, runs the\n pre-applypatch\n ~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter, and is\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter, and is\n invoked after the patch is applied, but before a commit is made.\n \n If it exits with non-zero status, then the working tree will not be\n@@ -76,33 +76,33 @@ The default 'pre-applypatch' hook, when enabled, runs the\n post-applypatch\n ~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git am'. It takes no parameter,\n+This hook is invoked by linkgit:git-am[1]. It takes no parameter,\n and is invoked after the patch is applied and a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git am'.\n+the outcome of `git am`.\n \n pre-commit\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit', and can be bypassed\n+This hook is invoked by linkgit:git-commit[1], and can be bypassed\n with the `--no-verify` option. It takes no parameters, and is\n invoked before obtaining the proposed commit log message and\n making a commit. Exiting with a non-zero status from this script\n-causes the 'git commit' command to abort before creating a commit.\n+causes the `git commit` command to abort before creating a commit.\n \n The default 'pre-commit' hook, when enabled, catches introduction\n of lines with trailing whitespaces and aborts the commit when\n such a line is found.\n \n-All the 'git commit' hooks are invoked with the environment\n+All the `git commit` hooks are invoked with the environment\n variable `GIT_EDITOR=:` if the command will not bring up an editor\n to modify the commit message.\n \n prepare-commit-msg\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git commit' right after preparing the\n+This hook is invoked by linkgit:git-commit[1] right after preparing the\n default log message, and before the editor is started.\n \n It takes one to three parameters. The first is the name of the file\n@@ -114,7 +114,7 @@ commit is a merge or a `.git/MERGE_MSG` file exists); `squash`\n (if a `.git/SQUASH_MSG` file exists); or `commit`, followed by\n a commit SHA-1 (if a `-c`, `-C` or `--amend` option was given).\n \n-If the exit status is non-zero, 'git commit' will abort.\n+If the exit status is non-zero, `git commit` will abort.\n \n The purpose of the hook is to edit the message file in place, and\n it is not suppressed by the `--no-verify` option. A non-zero exit\n@@ -127,7 +127,7 @@ help message found in the commented portion of the commit template.\n commit-msg\n ~~~~~~~~~~\n \n-This hook is invoked by 'git commit' and 'git merge', and can be\n+This hook is invoked by linkgit:git-commit[1] and linkgit:git-merge[1], and can be\n bypassed with the `--no-verify` option. It takes a single parameter,\n the name of the file that holds the proposed commit log message.\n Exiting with a non-zero status causes the command to abort.\n@@ -143,16 +143,16 @@ The default 'commit-msg' hook, when enabled, detects duplicate\n post-commit\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git commit'. It takes no parameters, and is\n+This hook is invoked by linkgit:git-commit[1]. It takes no parameters, and is\n invoked after a commit is made.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git commit'.\n+the outcome of `git commit`.\n \n pre-rebase\n ~~~~~~~~~~\n \n-This hook is called by 'git rebase' and can be used to prevent a\n+This hook is called by linkgit:git-rebase[1] and can be used to prevent a\n branch from getting rebased. The hook may be called with one or\n two parameters. The first parameter is the upstream from which\n the series was forked. The second parameter is the branch being\n@@ -161,17 +161,17 @@ rebased, and is not set when rebasing the current branch.\n post-checkout\n ~~~~~~~~~~~~~\n \n-This hook is invoked when a 'git checkout' is run after having updated the\n+This hook is invoked when a linkgit:git-checkout[1] is run after having updated the\n worktree. The hook is given three parameters: the ref of the previous HEAD,\n the ref of the new HEAD (which may or may not have changed), and a flag\n indicating whether the checkout was a branch checkout (changing branches,\n flag=1) or a file checkout (retrieving a file from the index, flag=0).\n-This hook cannot affect the outcome of 'git checkout'.\n+This hook cannot affect the outcome of `git checkout`.\n \n-It is also run after 'git clone', unless the --no-checkout (-n) option is\n+It is also run after linkgit:git-clone[1], unless the `--no-checkout` (`-n`) option is\n used. The first parameter given to the hook is the null-ref, the second the\n-ref of the new HEAD and the flag is always 1. Likewise for 'git worktree add'\n-unless --no-checkout is used.\n+ref of the new HEAD and the flag is always 1. Likewise for `git worktree add`\n+unless `--no-checkout` is used.\n \n This hook can be used to perform repository validity checks, auto-display\n differences from the previous HEAD if different, or set working dir metadata\n@@ -180,10 +180,10 @@ properties.\n post-merge\n ~~~~~~~~~~\n \n-This hook is invoked by 'git merge', which happens when a 'git pull'\n+This hook is invoked by linkgit:git-merge[1], which happens when a `git pull`\n is done on a local repository. The hook takes a single parameter, a status\n flag specifying whether or not the merge being done was a squash merge.\n-This hook cannot affect the outcome of 'git merge' and is not executed,\n+This hook cannot affect the outcome of `git merge` and is not executed,\n if the merge failed due to conflicts.\n \n This hook can be used in conjunction with a corresponding pre-commit hook to\n@@ -194,10 +194,10 @@ for an example of how to do this.\n pre-push\n ~~~~~~~~\n \n-This hook is called by 'git push' and can be used to prevent a push from taking\n-place. The hook is called with two parameters which provide the name and\n-location of the destination remote, if a named remote is not being used both\n-values will be the same.\n+This hook is called by linkgit:git-push[1] and can be used to prevent\n+a push from taking place. The hook is called with two parameters\n+which provide the name and location of the destination remote, if a\n+named remote is not being used both values will be the same.\n \n Information about what is to be pushed is provided on the hook's standard\n input with lines of the form:\n@@ -216,7 +216,7 @@ SHA-1>` will be 40 `0`. If the local commit was specified by something other\n than a name which could be expanded (such as `HEAD~`, or a SHA-1) it will be\n supplied as it was originally given.\n \n-If this hook exits with a non-zero status, 'git push' will abort without\n+If this hook exits with a non-zero status, `git push` will abort without\n pushing anything. Information about why the push is rejected may be sent\n to the user by writing to standard error.\n \n@@ -224,8 +224,8 @@ to the user by writing to standard error.\n pre-receive\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before starting to update refs on the remote repository, the\n pre-receive hook is invoked. Its exit status determines the success\n or failure of the update.\n@@ -246,7 +246,7 @@ updated. If the hook exits with zero, updating of individual refs can\n still be prevented by the <> hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The number of push options given on the command line of\n@@ -265,8 +265,8 @@ linkgit:git-receive-pack[1] for some caveats.\n update\n ~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n Just before updating the ref on the remote repository, the update hook\n is invoked. Its exit status determines the success or failure of\n the ref update.\n@@ -279,7 +279,7 @@ three parameters:\n - and the new object name to be stored in the ref.\n \n A zero exit from the update hook allows the ref to be updated.\n-Exiting with a non-zero status prevents 'git-receive-pack'\n+Exiting with a non-zero status prevents `git receive-pack`\n from updating that ref.\n \n This hook can be used to prevent 'forced' update on certain refs by\n@@ -299,7 +299,7 @@ membership. See linkgit:git-shell[1] for how you might use the login\n shell to restrict the user's access to only git commands.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'update' hook, when enabled--and with\n@@ -310,8 +310,8 @@ unannotated tags to be pushed.\n post-receive\n ~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -320,7 +320,7 @@ arguments, but gets the same information as the\n <>\n hook does on its standard input.\n \n-This hook does not affect the outcome of 'git-receive-pack', as it\n+This hook does not affect the outcome of `git receive-pack`, as it\n is called after the real work is done.\n \n This supersedes the <> hook in that it gets\n@@ -328,7 +328,7 @@ both old and new values of all the refs in addition to their\n names.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n The default 'post-receive' hook is empty, but there is\n@@ -349,8 +349,8 @@ will be set to zero, `GIT_PUSH_OPTION_COUNT=0`.\n post-update\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository.\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository.\n It executes on the remote repository once after all the refs have\n been updated.\n \n@@ -358,7 +358,7 @@ It takes a variable number of parameters, each of which is the\n name of ref that was actually updated.\n \n This hook is meant primarily for notification, and cannot affect\n-the outcome of 'git-receive-pack'.\n+the outcome of `git receive-pack`.\n \n The 'post-update' hook can tell what are the heads that were pushed,\n but it does not know what their original and updated values are,\n@@ -368,20 +368,20 @@ updated values of the refs. You might consider it instead if you need\n them.\n \n When enabled, the default 'post-update' hook runs\n-'git update-server-info' to keep the information used by dumb\n+`git update-server-info` to keep the information used by dumb\n transports (e.g., HTTP) up to date. If you are publishing\n a Git repository that is accessible via HTTP, you should\n probably enable this hook.\n \n Both standard output and standard error output are forwarded to\n-'git send-pack' on the other end, so you can simply `echo` messages\n+`git send-pack` on the other end, so you can simply `echo` messages\n for the user.\n \n push-to-checkout\n ~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git-receive-pack' when it reacts to\n-'git push' and updates reference(s) in its repository, and when\n+This hook is invoked by linkgit:git-receive-pack[1] when it reacts to\n+`git push` and updates reference(s) in its repository, and when\n the push tries to update the branch that is currently checked out\n and the `receive.denyCurrentBranch` configuration variable is set to\n `updateInstead`. Such a push by default is refused if the working\n@@ -400,8 +400,8 @@ when the tip of the current branch is updated to the new commit, and\n exit with a zero status.\n \n For example, the hook can simply run `git read-tree -u -m HEAD \"$1\"`\n-in order to emulate 'git fetch' that is run in the reverse direction\n-with `git push`, as the two-tree form of `read-tree -u -m` is\n+in order to emulate `git fetch` that is run in the reverse direction\n+with `git push`, as the two-tree form of `git read-tree -u -m` is\n essentially the same as `git checkout` that switches branches while\n keeping the local changes in the working tree that do not interfere\n with the difference between the branches.\n@@ -410,15 +410,16 @@ with the difference between the branches.\n pre-auto-gc\n ~~~~~~~~~~~\n \n-This hook is invoked by 'git gc --auto'. It takes no parameter, and\n-exiting with non-zero status from this script causes the 'git gc --auto'\n-to abort.\n+This hook is invoked by `git gc --auto` (see linkgit:git-gc[1]). It\n+takes no parameter, and exiting with non-zero status from this script\n+causes the `git gc --auto` to abort.\n \n post-rewrite\n ~~~~~~~~~~~~\n \n-This hook is invoked by commands that rewrite commits (`git commit\n---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call\n+This hook is invoked by commands that rewrite commits\n+(linkgit:git-commit[1] when called with `--amend` and\n+linkgit:git-rebase[1]; currently `git filter-branch` does 'not' call\n it!). Its first argument denotes the command it was invoked by:\n currently one of `amend` or `rebase`. Further command-dependent\n arguments may be passed in the future.\n@@ -450,16 +451,16 @@ processed by rebase.\n sendemail-validate\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked by 'git send-email'. It takes a single parameter,\n+This hook is invoked by linkgit:git-send-email[1]. It takes a single parameter,\n the name of the file that holds the e-mail to be sent. Exiting with a\n-non-zero status causes 'git send-email' to abort before sending any\n+non-zero status causes `git send-email` to abort before sending any\n e-mails.\n \n fsmonitor-watchman\n ~~~~~~~~~~~~~~~~~~\n \n-This hook is invoked when the configuration option core.fsmonitor is\n-set to .git/hooks/fsmonitor-watchman. It takes two arguments, a version\n+This hook is invoked when the configuration option `core.fsmonitor` is\n+set to `.git/hooks/fsmonitor-watchman`. It takes two arguments, a version\n (currently 1) and the time in elapsed nanoseconds since midnight,\n January 1, 1970.\n \n@@ -478,7 +479,7 @@ directories are checked for untracked files based on the path names\n given.\n \n An optimized way to tell git \"all files have changed\" is to return\n-the filename '/'.\n+the filename `/`.\n \n The exit status determines whether git will use the data from the\n hook to limit its search. On error, it will fall back to verifying\n-- \n2.16.2\n\n"},{"id":"346563","messageId":"20180503184830.11889-3-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 2/7] doc: align 'diff --no-index' in text and synopsis","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:25Z","receivedAt":"2018-05-03T18:49:02Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Make the two '' parameters in DESCRIPTION mandatory and\nmove the `--options` part to the same place where the other\nvariants show them. And finally make `--no-index` in SYNOPSIS\nas mandatory as in DESCRIPTION.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/git-diff.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex b0c1bb95c8..6593b58299 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -13,7 +13,7 @@ SYNOPSIS\n 'git diff' [options] --cached [] [--] [...]\n 'git diff' [options] [--] [...]\n 'git diff' [options] \n-'git diff' [options] [--no-index] [--] \n+'git diff' [options] --no-index [--] \n \n DESCRIPTION\n -----------\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' --no-index [--options] [--] [...]::\n+'git diff' [--options] --no-index [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n-- \n2.16.2\n\n"},{"id":"346564","messageId":"20180503184830.11889-4-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 3/7] doc: clarify ignore rules for git ls-files","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:26Z","receivedAt":"2018-05-03T18:49:06Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Explain that `git ls-files --ignored` requires at least one\nof the `--exclude*` options to do its job.\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-ls-files.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-ls-files.txt b/Documentation/git-ls-files.txt\nindex 3ac3e3a77d..f3474b2ede 100644\n--- a/Documentation/git-ls-files.txt\n+++ b/Documentation/git-ls-files.txt\n@@ -53,7 +53,8 @@ OPTIONS\n \tShow only ignored files in the output. When showing files in the\n \tindex, print only those matched by an exclude pattern. When\n \tshowing \"other\" files, show only those matched by an exclude\n-\tpattern.\n+\tpattern. Standard ignore rules are not automatically activated,\n+\ttherefore at least one of the `--exclude*` options is required.\n \n -s::\n --stage::\n-- \n2.16.2\n\n"},{"id":"346565","messageId":"20180503184830.11889-5-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 4/7] doc: add '-d' and '-o' for 'git push'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:27Z","receivedAt":"2018-05-03T18:49:09Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Add the missing `-o` shortcut for `--push-option` to the synopsis.\nAdd the missing `-d` shortcut for `--delete` in the main section.\n\nSigned-off-by: Andreas Heiduk \nReviewed-by: Martin Ågren \n---\n Documentation/git-push.txt | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\nindex 5b08302fc2..f2bbda6e32 100644\n--- a/Documentation/git-push.txt\n+++ b/Documentation/git-push.txt\n@@ -11,7 +11,7 @@ SYNOPSIS\n [verse]\n 'git push' [--all | --mirror | --tags] [--follow-tags] [--atomic] [-n | --dry-run] [--receive-pack=]\n \t [--repo=] [-f | --force] [-d | --delete] [--prune] [-v | --verbose]\n-\t [-u | --set-upstream] [--push-option=]\n+\t [-u | --set-upstream] [-o | --push-option=]\n \t [--[no-]signed|--signed=(true|false|if-asked)]\n \t [--force-with-lease[=[:]]]\n \t [--no-verify] [ [...]]\n@@ -123,6 +123,7 @@ already exists on the remote side.\n \twill be tab-separated and sent to stdout instead of stderr. The full\n \tsymbolic names of the refs will be given.\n \n+-d::\n --delete::\n \tAll listed refs are deleted from the remote repository. This is\n \tthe same as prefixing all refs with a colon.\n-- \n2.16.2\n\n"},{"id":"346566","messageId":"20180503184830.11889-6-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 5/7] git-svn: remove ''--add-author-from' for 'commit-diff'","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:28Z","receivedAt":"2018-05-03T18:49:10Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"The subcommand 'commit-diff' does not support the option\n'--add-author-from'.\n\nSigned-off-by: Andreas Heiduk \nSigned-off-by: Eric Wong \n---\n Documentation/git-svn.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt\nindex d59379ee23..e9615951d2 100644\n--- a/Documentation/git-svn.txt\n+++ b/Documentation/git-svn.txt\n@@ -707,7 +707,7 @@ creating the branch or tag.\n config key: svn.useLogAuthor\n \n --add-author-from::\n-\tWhen committing to svn from Git (as part of 'commit-diff', 'set-tree' or 'dcommit'\n+\tWhen committing to svn from Git (as part of 'set-tree' or 'dcommit'\n \toperations), if the existing log message doesn't already have a\n \t`From:` or `Signed-off-by:` line, append a `From:` line based on the\n \tGit commit's author string. If you use this, then `--use-log-author`\n-- \n2.16.2\n\n"},{"id":"346567","messageId":"20180503184830.11889-7-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 6/7] doc: add note about shell quoting to revision.txt","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:29Z","receivedAt":"2018-05-03T18:49:14Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"Signed-off-by: Andreas Heiduk \nReviewed-by: Junio C Hamano \n---\n Documentation/revisions.txt | 6 ++++++\n 1 file changed, 6 insertions(+)\n\ndiff --git a/Documentation/revisions.txt b/Documentation/revisions.txt\nindex dfcc49c72c..e760416d07 100644\n--- a/Documentation/revisions.txt\n+++ b/Documentation/revisions.txt\n@@ -7,6 +7,10 @@ syntax. Here are various ways to spell object names. The\n ones listed near the end of this list name trees and\n blobs contained in a commit.\n \n+NOTE: This document shows the \"raw\" syntax as seen by git. The shell\n+and other UIs might require additional quoting to protect special\n+characters and to avoid word splitting.\n+\n '', e.g. 'dae86e1950b1277e545cee180551750029cfe735', 'dae86e'::\n The full SHA-1 object name (40-byte hexadecimal string), or\n a leading substring that is unique within the repository.\n@@ -186,6 +190,8 @@ existing tag object.\n is matched. ':/!-foo' performs a negative match, while ':/!!foo' matches a\n literal '!' character, followed by 'foo'. Any other sequence beginning with\n ':/!' is reserved for now.\n+ Depending on the given text, the shell's word splitting rules might\n+ require additional quoting.\n \n ':', e.g. 'HEAD:README', ':README', 'master:./README'::\n A suffix ':' followed by a path names the blob or tree\n-- \n2.16.2\n\n"},{"id":"346568","messageId":"20180503184830.11889-8-asheiduk@gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"[PATCH v3 7/7] doc: normalize [--options] to [options] in git-diff","fromName":"Andreas Heiduk","fromEmail":"asheiduk@gmail.com","sentAt":"2018-05-03T18:48:30Z","receivedAt":"2018-05-03T18:49:16Z","isPatch":true,"sender":{"key":"asheiduk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9371344?v=4"},"body":"SYNOPSIS and other manuals use [options] but DESCRIPTION\nused [--options].\n\nSigned-off-by: Andreas Heiduk \n---\n Documentation/git-diff.txt | 14 +++++++-------\n 1 file changed, 7 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/git-diff.txt b/Documentation/git-diff.txt\nindex 6593b58299..7c2c442700 100644\n--- a/Documentation/git-diff.txt\n+++ b/Documentation/git-diff.txt\n@@ -21,7 +21,7 @@ Show changes between the working tree and the index or a tree, changes\n between the index and a tree, changes between two trees, changes between\n two blob objects, or changes between two files on disk.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis form is to view the changes you made relative to\n \tthe index (staging area for the next commit). In other\n@@ -29,7 +29,7 @@ two blob objects, or changes between two files on disk.\n \tfurther add to the index but you still haven't. You can\n \tstage these changes by using linkgit:git-add[1].\n \n-'git diff' [--options] --no-index [--] ::\n+'git diff' [options] --no-index [--] ::\n \n \tThis form is to compare the given two paths on the\n \tfilesystem. You can omit the `--no-index` option when\n@@ -38,7 +38,7 @@ two blob objects, or changes between two files on disk.\n \tor when running the command outside a working tree\n \tcontrolled by Git.\n \n-'git diff' [--options] --cached [] [--] [...]::\n+'git diff' [options] --cached [] [--] [...]::\n \n \tThis form is to view the changes you staged for the next\n \tcommit relative to the named . Typically you\n@@ -48,7 +48,7 @@ two blob objects, or changes between two files on disk.\n \t is not given, it shows all staged changes.\n \t--staged is a synonym of --cached.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis form is to view the changes you have in your\n \tworking tree relative to the named . You can\n@@ -56,18 +56,18 @@ two blob objects, or changes between two files on disk.\n \tbranch name to compare with the tip of a different\n \tbranch.\n \n-'git diff' [--options] [--] [...]::\n+'git diff' [options] [--] [...]::\n \n \tThis is to view the changes between two arbitrary\n \t.\n \n-'git diff' [--options] .. [--] [...]::\n+'git diff' [options] .. [--] [...]::\n \n \tThis is synonymous to the previous form. If on\n \tone side is omitted, it will have the same effect as\n \tusing HEAD instead.\n \n-'git diff' [--options] \\... [--] [...]::\n+'git diff' [options] \\... [--] [...]::\n \n \tThis form is to view the changes on the branch containing\n \tand up to the second , starting at a common ancestor\n-- \n2.16.2\n\n"},{"id":"346688","messageId":"CAN0heSp6_yYfz=tVdZFHMfzWFQ6nhmL8i9BGGF8mi5gctqfFNw@mail.gmail.com","threadId":"48258","inReplyTo":"20180503184830.11889-1-asheiduk@gmail.com","subject":"Re: [PATCH v3 0/7] Some doc-fixes","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-05-04T18:47:29Z","receivedAt":"2018-05-04T18:47:33Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On 3 May 2018 at 20:48, Andreas Heiduk wrote:\n> Changes since the last reroll:\n>\n> - Better commit comment for \"doc: align 'diff --no-index' in text and synopsis\"\n> This includes Martin's `s/with/and/` comment.\n> - Eric's typo fix in \"doc: add note about shell quoting to revision.txt\"\n> - Added new patch for git-diff.txt with s/--options/options/.\n> This addresses Eric's and Martin's comments.\n\nFWIW, this version looks good to me. I was a tiny bit surprised that\npatch 7/7 was not patch 1/7. Could be just a matter of opinion,\nprobably nothing to reroll for. Thanks for getting back to this.\n\nMartin\n"}]}