{"thread":{"id":"61995","subject":"[PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","startedAt":"2024-08-22T17:21:30Z","lastAt":"2024-08-26T17:32:53Z","messageCount":11,"participants":["Celeste Liu","Junio C Hamano","Patrick Steinhardt"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"501550","messageId":"20240822172042.54065-1-CoelacanthusHex@gmail.com","threadId":"61995","inReplyTo":null,"subject":"[PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Celeste Liu","fromEmail":"coelacanthushex@gmail.com","sentAt":"2024-08-22T17:20:41Z","receivedAt":"2024-08-22T17:21:30Z","isPatch":true,"sender":{"key":"coelacanthushex@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43064781?v=4"},"body":"Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\nintroduced this typo.\n\nFixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\nSigned-off-by: Celeste Liu \n---\n Documentation/git-config.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\nindex 65c645d461..79360328aa 100644\n--- a/Documentation/git-config.txt\n+++ b/Documentation/git-config.txt\n@@ -130,7 +130,7 @@ OPTIONS\n --all::\n \tWith `get`, return all values for a multi-valued key.\n \n----regexp::\n+--regexp::\n \tWith `get`, interpret the name as a regular expression. Regular\n \texpression matching is currently case-sensitive and done against a\n \tcanonicalized version of the key in which section and variable names\n-- \n2.46.0\n\n"},{"id":"501553","messageId":"xmqqseuwqvvi.fsf@gitster.g","threadId":"61995","inReplyTo":"20240822172042.54065-1-CoelacanthusHex@gmail.com","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-08-22T18:02:09Z","receivedAt":"2024-08-22T18:02:15Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Celeste Liu writes:\n\n> Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> introduced this typo.\n>\n> Fixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> Signed-off-by: Celeste Liu \n> ---\n> Documentation/git-config.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n> index 65c645d461..79360328aa 100644\n> --- a/Documentation/git-config.txt\n> +++ b/Documentation/git-config.txt\n> @@ -130,7 +130,7 @@ OPTIONS\n> --all::\n> \tWith `get`, return all values for a multi-valued key.\n> \n> ----regexp::\n> +--regexp::\n> \tWith `get`, interpret the name as a regular expression. Regular\n> \texpression matching is currently case-sensitive and done against a\n> \tcanonicalized version of the key in which section and variable names\n\nLooks obviously correct. Thanks.\n"},{"id":"501573","messageId":"Zsgh4LacdOt8N69o@tanuki","threadId":"61995","inReplyTo":"20240822172042.54065-1-CoelacanthusHex@gmail.com","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2024-08-23T05:45:04Z","receivedAt":"2024-08-23T05:45:10Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Fri, Aug 23, 2024 at 01:20:41AM +0800, Celeste Liu wrote:\n> Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> introduced this typo.\n\nWe typically do not word the commit message as a continuation of the\nsubject. The subject often becomes invisible in many contexts, so it is\nhelpful if it can be read standalone while providing all of the context\nnecessary. So mentioning what is wrong would be welcome here.\n\n> Fixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n\nWe do not use Fixes tags in the Git project.\n\n> Signed-off-by: Celeste Liu \n> ---\n> Documentation/git-config.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n> \n> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n> index 65c645d461..79360328aa 100644\n> --- a/Documentation/git-config.txt\n> +++ b/Documentation/git-config.txt\n> @@ -130,7 +130,7 @@ OPTIONS\n> --all::\n> \tWith `get`, return all values for a multi-valued key.\n> \n> ----regexp::\n> +--regexp::\n> \tWith `get`, interpret the name as a regular expression. Regular\n> \texpression matching is currently case-sensitive and done against a\n> \tcanonicalized version of the key in which section and variable names\n\nThis looks obviously correct to me.\n\nThere's one more thing I noticed: the synopsis for `git config get` says\n`--regrexp=`, which is clearly wrong. As you're already at it,\nwould you mind including a second patch that corrects this?\n\nThanks!\n\nPatrick\n"},{"id":"501575","messageId":"35777445-b86a-42c8-93e0-99ac225c11ab@gmail.com","threadId":"61995","inReplyTo":"Zsgh4LacdOt8N69o@tanuki","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Celeste Liu","fromEmail":"coelacanthushex@gmail.com","sentAt":"2024-08-23T07:59:36Z","receivedAt":"2024-08-23T07:59:40Z","isPatch":true,"sender":{"key":"coelacanthushex@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43064781?v=4"},"body":"\nOn 2024-08-23 13:45, Patrick Steinhardt wrote:\n> On Fri, Aug 23, 2024 at 01:20:41AM +0800, Celeste Liu wrote:\n>> Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n>> introduced this typo.\n> \n> We typically do not word the commit message as a continuation of the\n> subject. The subject often becomes invisible in many contexts, so it is\n> helpful if it can be read standalone while providing all of the context\n> necessary. So mentioning what is wrong would be welcome here.\n> \n>> Fixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> \n> We do not use Fixes tags in the Git project.\n> \n>> Signed-off-by: Celeste Liu \n>> ---\n>> Documentation/git-config.txt | 2 +-\n>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n>> index 65c645d461..79360328aa 100644\n>> --- a/Documentation/git-config.txt\n>> +++ b/Documentation/git-config.txt\n>> @@ -130,7 +130,7 @@ OPTIONS\n>> --all::\n>> \tWith `get`, return all values for a multi-valued key.\n>> \n>> ----regexp::\n>> +--regexp::\n>> \tWith `get`, interpret the name as a regular expression. Regular\n>> \texpression matching is currently case-sensitive and done against a\n>> \tcanonicalized version of the key in which section and variable names\n> \n> This looks obviously correct to me.\n> \n> There's one more thing I noticed: the synopsis for `git config get` says\n> `--regrexp=`, which is clearly wrong. As you're already at it,\n> would you mind including a second patch that corrects this?\n\nOk. I will include this in v2 later.\n\n> \n> Thanks!\n> \n> Patrick\n\n"},{"id":"501576","messageId":"6c9f666e-12ad-4b67-a44a-3111686454f3@gmail.com","threadId":"61995","inReplyTo":"Zsgh4LacdOt8N69o@tanuki","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Celeste Liu","fromEmail":"coelacanthushex@gmail.com","sentAt":"2024-08-23T08:21:15Z","receivedAt":"2024-08-23T08:21:18Z","isPatch":true,"sender":{"key":"coelacanthushex@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43064781?v=4"},"body":"\nOn 2024-08-23 13:45, Patrick Steinhardt wrote:\n> On Fri, Aug 23, 2024 at 01:20:41AM +0800, Celeste Liu wrote:\n>> Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n>> introduced this typo.\n> \n> We typically do not word the commit message as a continuation of the\n> subject. The subject often becomes invisible in many contexts, so it is\n> helpful if it can be read standalone while providing all of the context\n> necessary. So mentioning what is wrong would be welcome here.\n> \n>> Fixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> \n> We do not use Fixes tags in the Git project.\n> \n>> Signed-off-by: Celeste Liu \n>> ---\n>> Documentation/git-config.txt | 2 +-\n>> 1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n>> index 65c645d461..79360328aa 100644\n>> --- a/Documentation/git-config.txt\n>> +++ b/Documentation/git-config.txt\n>> @@ -130,7 +130,7 @@ OPTIONS\n>> --all::\n>> \tWith `get`, return all values for a multi-valued key.\n>> \n>> ----regexp::\n>> +--regexp::\n>> \tWith `get`, interpret the name as a regular expression. Regular\n>> \texpression matching is currently case-sensitive and done against a\n>> \tcanonicalized version of the key in which section and variable names\n> \n> This looks obviously correct to me.\n> \n> There's one more thing I noticed: the synopsis for `git config get` says\n> `--regrexp=`, which is clearly wrong. As you're already at it,\n> would you mind including a second patch that corrects this?\n\nI couldn't find this typo. Did I miss something? If something missed, please \ntell me. I will include it in v3.\nv2 has been sent.\n\n> \n> Thanks!\n> \n> Patrick\n"},{"id":"501638","messageId":"ZswfgYsZrzY6kc1E@tanuki","threadId":"61995","inReplyTo":"6c9f666e-12ad-4b67-a44a-3111686454f3@gmail.com","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2024-08-26T06:24:14Z","receivedAt":"2024-08-26T06:24:18Z","isPatch":true,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Fri, Aug 23, 2024 at 04:21:15PM +0800, Celeste Liu wrote:\n> \n> On 2024-08-23 13:45, Patrick Steinhardt wrote:\n> > On Fri, Aug 23, 2024 at 01:20:41AM +0800, Celeste Liu wrote:\n> >> Commit 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> >> introduced this typo.\n> > \n> > We typically do not word the commit message as a continuation of the\n> > subject. The subject often becomes invisible in many contexts, so it is\n> > helpful if it can be read standalone while providing all of the context\n> > necessary. So mentioning what is wrong would be welcome here.\n> > \n> >> Fixes: 4e51389000 (builtin/config: introduce \"get\" subcommand, 2024-05-06)\n> > \n> > We do not use Fixes tags in the Git project.\n> > \n> >> Signed-off-by: Celeste Liu \n> >> ---\n> >> Documentation/git-config.txt | 2 +-\n> >> 1 file changed, 1 insertion(+), 1 deletion(-)\n> >>\n> >> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n> >> index 65c645d461..79360328aa 100644\n> >> --- a/Documentation/git-config.txt\n> >> +++ b/Documentation/git-config.txt\n> >> @@ -130,7 +130,7 @@ OPTIONS\n> >> --all::\n> >> \tWith `get`, return all values for a multi-valued key.\n> >> \n> >> ----regexp::\n> >> +--regexp::\n> >> \tWith `get`, interpret the name as a regular expression. Regular\n> >> \texpression matching is currently case-sensitive and done against a\n> >> \tcanonicalized version of the key in which section and variable names\n> > \n> > This looks obviously correct to me.\n> > \n> > There's one more thing I noticed: the synopsis for `git config get` says\n> > `--regrexp=`, which is clearly wrong. As you're already at it,\n> > would you mind including a second patch that corrects this?\n> \n> I couldn't find this typo. Did I miss something? If something missed, please \n> tell me. I will include it in v3.\n> v2 has been sent.\n\nIt's in line 13 in Documentation/git-config.txt. Seeing that your v2 is\nalready out and queued for `next` by Junio I wouldn't fix that in a v3\nresend though. If you want to fix that as well, I'd just send a separate\npatch.\n\nAlso fine if you just ignore this, I can also fix this myself. Just let\nme know your preference :)\n\nPatrick\n"},{"id":"501671","messageId":"xmqqa5gznwf9.fsf@gitster.g","threadId":"61995","inReplyTo":"ZswfgYsZrzY6kc1E@tanuki","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-08-26T15:19:54Z","receivedAt":"2024-08-26T15:20:02Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Patrick Steinhardt writes:\n\n>> > There's one more thing I noticed: the synopsis for `git config get` says\n>> > `--regrexp=`, which is clearly wrong. As you're already at it,\n>> > would you mind including a second patch that corrects this?\n>> \n>> I couldn't find this typo. Did I miss something? If something missed, please \n>> tell me. I will include it in v3.\n>> v2 has been sent.\n>\n> It's in line 13 in Documentation/git-config.txt. Seeing that your v2 is\n\nAhhhh. You weren't saying \"regexp is misspelt as regrexp\". You're\nright. I too missed it.\n\nThe synopsis should lose \"=\" part, as the option does not\ntake any arguments.\n\nThanks.\n"},{"id":"501673","messageId":"8c1c031b-4f36-4825-93ef-408c50b0ee2f@gmail.com","threadId":"61995","inReplyTo":"xmqqa5gznwf9.fsf@gitster.g","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Celeste Liu","fromEmail":"coelacanthushex@gmail.com","sentAt":"2024-08-26T15:56:44Z","receivedAt":"2024-08-26T15:56:49Z","isPatch":true,"sender":{"key":"coelacanthushex@gmail.com","avatar":"https://avatars.githubusercontent.com/u/43064781?v=4"},"body":"On 2024-08-26 23:19, Junio C Hamano wrote:\n\n> Patrick Steinhardt writes:\n> \n>>>> There's one more thing I noticed: the synopsis for `git config get` says\n>>>> `--regrexp=`, which is clearly wrong. As you're already at it,\n>>>> would you mind including a second patch that corrects this?\n>>>\n>>> I couldn't find this typo. Did I miss something? If something missed, please \n>>> tell me. I will include it in v3.\n>>> v2 has been sent.\n>>\n>> It's in line 13 in Documentation/git-config.txt. Seeing that your v2 is\n> \n> Ahhhh. You weren't saying \"regexp is misspelt as regrexp\". You're\n> right. I too missed it.\n\nOhhhhh. I missed it.\nA comment which include typo and point out a typo led my brain to wrong way.\n\nAnd is there need a new patch to fix this? Or you will fix it directly?\n\n> \n> The synopsis should lose \"=\" part, as the option does not\n> take any arguments.\n> \n> Thanks.\n\n"},{"id":"501679","messageId":"xmqq34mrkxre.fsf@gitster.g","threadId":"61995","inReplyTo":"8c1c031b-4f36-4825-93ef-408c50b0ee2f@gmail.com","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-08-26T17:19:17Z","receivedAt":"2024-08-26T17:19:26Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Celeste Liu writes:\n\n> And is there need a new patch to fix this? Or you will fix it directly?\n>> \n>> The synopsis should lose \"=\" part, as the option does not\n>> take any arguments.\n\nIt is a separate documentation bug. We can fix it separately.\n\n----- >8 --------- >8 --------- >8 -----\nSubject: git-config.1: two random small fixes\n\nThe synopsis says --regexp= but the --regexp option is a\nBoolean that says \"the name given is not literal, but a pattern to\nmatch the name\".\n\nSigned-off-by: Junio C Hamano \n---\n\ndiff --git c/Documentation/git-config.txt w/Documentation/git-config.txt\nindex 65c645d461..1ee5c89ba2 100644\n--- c/Documentation/git-config.txt\n+++ w/Documentation/git-config.txt\n@@ -10,7 +10,7 @@ SYNOPSIS\n --------\n [verse]\n 'git config list' [] [] [--includes]\n-'git config get' [] [] [--includes] [--all] [--regexp=] [--value=] [--fixed-value] [--default=] \n+'git config get' [] [] [--includes] [--all] [--regexp] [--value=] [--fixed-value] [--default=] \n 'git config set' [] [--type=] [--all] [--value=] [--fixed-value] \n 'git config unset' [] [--all] [--value=] [--fixed-value] \n 'git config rename-section' [] \n"},{"id":"501681","messageId":"xmqqplpvjimw.fsf_-_@gitster.g","threadId":"61995","inReplyTo":"xmqq34mrkxre.fsf@gitster.g","subject":"[PATCH] git-config.1: --get-all description update","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-08-26T17:31:19Z","receivedAt":"2024-08-26T17:31:22Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"git config --get-all foo.bar\" shows all values for the foo.bar\nvariable, but does not give the variable name in each output entry.\nHence it is equivalent to \"git config get --all foo.bar\", without\n\"--show-names\", in the more modern syntax.\n\nSigned-off-by: Junio C Hamano \n---\ndiff --git i/Documentation/git-config.txt w/Documentation/git-config.txt\nindex 65c645d461..1ee5c89ba2 100644\n--- i/Documentation/git-config.txt\n+++ w/Documentation/git-config.txt\n@@ -309,7 +309,7 @@ recommended to migrate to the new syntax.\n \tReplaced by `git config get [--value=] `.\n \n --get-all []::\n-\tReplaced by `git config get [--value=] --all --show-names `.\n+\tReplaced by `git config get [--value=] --all `.\n \n --get-regexp ::\n \tReplaced by `git config get --all --show-names --regexp `.\n"},{"id":"501682","messageId":"xmqqle0jjikl.fsf@gitster.g","threadId":"61995","inReplyTo":"xmqq34mrkxre.fsf@gitster.g","subject":"Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-08-26T17:32:42Z","receivedAt":"2024-08-26T17:32:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> Celeste Liu writes:\n>\n>> And is there need a new patch to fix this? Or you will fix it directly?\n>>> \n>>> The synopsis should lose \"=\" part, as the option does not\n>>> take any arguments.\n>\n> It is a separate documentation bug. We can fix it separately.\n>\n> ----- >8 --------- >8 --------- >8 -----\n> Subject: git-config.1: two random small fixes\n\nSorry but, this is a stale subject.\n\n Subject: git-config.1: fix --regexp description in synopsis\n\nor something, probably.\n\n> The synopsis says --regexp= but the --regexp option is a\n> Boolean that says \"the name given is not literal, but a pattern to\n> match the name\".\n>\n> Signed-off-by: Junio C Hamano \n> ---\n>\n> diff --git c/Documentation/git-config.txt w/Documentation/git-config.txt\n> index 65c645d461..1ee5c89ba2 100644\n> --- c/Documentation/git-config.txt\n> +++ w/Documentation/git-config.txt\n> @@ -10,7 +10,7 @@ SYNOPSIS\n> --------\n> [verse]\n> 'git config list' [] [] [--includes]\n> -'git config get' [] [] [--includes] [--all] [--regexp=] [--value=] [--fixed-value] [--default=] \n> +'git config get' [] [] [--includes] [--all] [--regexp] [--value=] [--fixed-value] [--default=] \n> 'git config set' [] [--type=] [--all] [--value=] [--fixed-value] \n> 'git config unset' [] [--all] [--value=] [--fixed-value] \n> 'git config rename-section' [] \n"}]}