{"thread":{"id":"57306","subject":"[PATCH 0/2] docs: apply minor fixes to Documentation/config/pgp.txt","startedAt":"2022-01-25T12:40:38Z","lastAt":"2022-01-27T17:50:39Z","messageCount":11,"participants":["Greg Hurrell","Ævar Arnfjörð Bjarmason","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"446841","messageId":"20220125123716.66991-1-greg@hurrell.net","threadId":"57306","inReplyTo":null,"subject":"[PATCH 0/2] docs: apply minor fixes to Documentation/config/pgp.txt","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-25T12:37:14Z","receivedAt":"2022-01-25T12:40:38Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Two little fixes to the docs, adding a missing apostrophe and fixing a\nmid-sentence tab to a space, respectively.\n\n Documentation/config/gpg.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\n\n"},{"id":"446842","messageId":"20220125123716.66991-2-greg@hurrell.net","threadId":"57306","inReplyTo":"20220125123716.66991-1-greg@hurrell.net","subject":"[PATCH 1/2] docs: Fix bad whitespace in Documentation/config/pgp.txt","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-25T12:37:15Z","receivedAt":"2022-01-25T12:40:46Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Signed-off-by: Greg Hurrell \n---\n Documentation/config/gpg.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\nindex 0cb189a077..abfabd6d62 100644\n--- a/Documentation/config/gpg.txt\n+++ b/Documentation/config/gpg.txt\n@@ -37,7 +37,7 @@ gpg.minTrustLevel::\n gpg.ssh.defaultKeyCommand::\n \tThis command that will be run when user.signingkey is not set and a ssh\n \tsignature is requested. On successful exit a valid ssh public key is\n-\texpected in the\tfirst line of its output. To automatically use the first\n+\texpected in the first line of its output. To automatically use the first\n \tavailable key from your ssh-agent set this to \"ssh-add -L\".\n \n gpg.ssh.allowedSignersFile::\n-- \n2.35.0\n\n"},{"id":"446843","messageId":"20220125123716.66991-3-greg@hurrell.net","threadId":"57306","inReplyTo":"20220125123716.66991-1-greg@hurrell.net","subject":"[PATCH 2/2] docs: add missing apostrophe to Documentation/config/gpg.txt","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-25T12:37:16Z","receivedAt":"2022-01-25T12:40:47Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Add an apostrophe to \"signatures\" to indicate the possessive\nrelationship in \"the signature's creation\".\n\nSigned-off-by: Greg Hurrell \n---\n Documentation/config/gpg.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\nindex abfabd6d62..86892ada77 100644\n--- a/Documentation/config/gpg.txt\n+++ b/Documentation/config/gpg.txt\n@@ -66,7 +66,7 @@ This way only committers with an already valid key can add or change keys in the\n +\n Since OpensSSH 8.8 this file allows specifying a key lifetime using valid-after &\n valid-before options. Git will mark signatures as valid if the signing key was\n-valid at the time of the signatures creation. This allows users to change a\n+valid at the time of the signature's creation. This allows users to change a\n signing key without invalidating all previously made signatures.\n +\n Using a SSH CA key with the cert-authority option\n-- \n2.35.0\n\n"},{"id":"446855","messageId":"220125.861r0vhh8r.gmgdl@evledraar.gmail.com","threadId":"57306","inReplyTo":"20220125123716.66991-2-greg@hurrell.net","subject":"Re: [PATCH 1/2] docs: Fix bad whitespace in Documentation/config/pgp.txt","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-01-25T17:09:57Z","receivedAt":"2022-01-25T17:16:38Z","isPatch":true,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Tue, Jan 25 2022, Greg Hurrell wrote:\n\nnit @ subject: I'd suggest:\n\n\tDocumentation/config/pgp.txt: replace stray character with \n\nOr something, i.e. the \"docs\" can just be replaced by the filename, so\nwe don't need to put it at the end.\n\nAs general style, we tend to not have the part of \":\n\" start with a capital letter, i.e. \"fix\" not \"Fix\".\n\n> Signed-off-by: Greg Hurrell \n> ---\n> Documentation/config/gpg.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\n> index 0cb189a077..abfabd6d62 100644\n> --- a/Documentation/config/gpg.txt\n> +++ b/Documentation/config/gpg.txt\n> @@ -37,7 +37,7 @@ gpg.minTrustLevel::\n> gpg.ssh.defaultKeyCommand::\n> \tThis command that will be run when user.signingkey is not set and a ssh\n> \tsignature is requested. On successful exit a valid ssh public key is\n> -\texpected in the\tfirst line of its output. To automatically use the first\n> +\texpected in the first line of its output. To automatically use the first\n\nSince I had to look it's a s/\\t/ / of a tab between \"the\" and \"first\" in\nthe pre-image>\n"},{"id":"446938","messageId":"20220126121426.53799-1-greg@hurrell.net","threadId":"57306","inReplyTo":"220125.861r0vhh8r.gmgdl@evledraar.gmail.com","subject":"[PATCH v2 0/2] Apply minor fixes to Documentation/config/pgp.txt","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-26T12:14:24Z","receivedAt":"2022-01-26T12:14:33Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Just applying changes to commit messages as suggested by Ævar Arnfjörð\nBjarmason:\n\n> nit @ subject: I'd suggest:\n>\n>\tDocumentation/config/pgp.txt: replace stray character with \n>\n> Or something, i.e. the \"docs\" can just be replaced by the filename, so\n> we don't need to put it at the end.\n>\n> As general style, we tend to not have the part of \":\n> \" start with a capital letter, i.e. \"fix\" not \"Fix\".\n\n\n"},{"id":"446939","messageId":"20220126121426.53799-2-greg@hurrell.net","threadId":"57306","inReplyTo":"20220126121426.53799-1-greg@hurrell.net","subject":"[PATCH v2 1/2] Documentation/config/pgp.txt: replace stray character with ","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-26T12:14:25Z","receivedAt":"2022-01-26T12:14:42Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Specifically, replace the tab between \"the\" and \"first\" with a space.\n\nSigned-off-by: Greg Hurrell \n---\n Documentation/config/gpg.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\nindex 0cb189a077..abfabd6d62 100644\n--- a/Documentation/config/gpg.txt\n+++ b/Documentation/config/gpg.txt\n@@ -37,7 +37,7 @@ gpg.minTrustLevel::\n gpg.ssh.defaultKeyCommand::\n \tThis command that will be run when user.signingkey is not set and a ssh\n \tsignature is requested. On successful exit a valid ssh public key is\n-\texpected in the\tfirst line of its output. To automatically use the first\n+\texpected in the first line of its output. To automatically use the first\n \tavailable key from your ssh-agent set this to \"ssh-add -L\".\n \n gpg.ssh.allowedSignersFile::\n-- \n2.35.0\n\n"},{"id":"446940","messageId":"20220126121426.53799-3-greg@hurrell.net","threadId":"57306","inReplyTo":"20220126121426.53799-1-greg@hurrell.net","subject":"[PATCH 2/2] Documentation/config/pgp.txt: add missing apostrophe","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-26T12:14:26Z","receivedAt":"2022-01-26T12:14:47Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"Add an apostrophe to \"signatures\" to indicate the possessive\nrelationship in \"the signature's creation\".\n\nSigned-off-by: Greg Hurrell \n---\n Documentation/config/gpg.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\nindex abfabd6d62..86892ada77 100644\n--- a/Documentation/config/gpg.txt\n+++ b/Documentation/config/gpg.txt\n@@ -66,7 +66,7 @@ This way only committers with an already valid key can add or change keys in the\n +\n Since OpensSSH 8.8 this file allows specifying a key lifetime using valid-after &\n valid-before options. Git will mark signatures as valid if the signing key was\n-valid at the time of the signatures creation. This allows users to change a\n+valid at the time of the signature's creation. This allows users to change a\n signing key without invalidating all previously made signatures.\n +\n Using a SSH CA key with the cert-authority option\n-- \n2.35.0\n\n"},{"id":"447029","messageId":"xmqq5yq56hbu.fsf@gitster.g","threadId":"57306","inReplyTo":"20220126121426.53799-2-greg@hurrell.net","subject":"Re: [PATCH v2 1/2] Documentation/config/pgp.txt: replace stray character with ","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-27T02:30:29Z","receivedAt":"2022-01-27T02:30:33Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg Hurrell writes:\n\n> Specifically, replace the tab between \"the\" and \"first\" with a space.\n\nGood eyes. Does it have ANY practical downside to leave it unfixed?\n\nIn other words, how did you spot it (e.g. \"asciidoctor passes tab as-is\nand screwed up an alignment of the text, which was how I noticed it\")?\n\nThanks.\n\n>\n> Signed-off-by: Greg Hurrell \n> ---\n> Documentation/config/gpg.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\n> index 0cb189a077..abfabd6d62 100644\n> --- a/Documentation/config/gpg.txt\n> +++ b/Documentation/config/gpg.txt\n> @@ -37,7 +37,7 @@ gpg.minTrustLevel::\n> gpg.ssh.defaultKeyCommand::\n> \tThis command that will be run when user.signingkey is not set and a ssh\n> \tsignature is requested. On successful exit a valid ssh public key is\n> -\texpected in the\tfirst line of its output. To automatically use the first\n> +\texpected in the first line of its output. To automatically use the first\n> \tavailable key from your ssh-agent set this to \"ssh-add -L\".\n> \n> gpg.ssh.allowedSignersFile::\n"},{"id":"447030","messageId":"xmqq1r0t6hb3.fsf@gitster.g","threadId":"57306","inReplyTo":"20220126121426.53799-3-greg@hurrell.net","subject":"Re: [PATCH 2/2] Documentation/config/pgp.txt: add missing apostrophe","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-27T02:30:56Z","receivedAt":"2022-01-27T02:31:01Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg Hurrell writes:\n\n> Add an apostrophe to \"signatures\" to indicate the possessive\n> relationship in \"the signature's creation\".\n>\n> Signed-off-by: Greg Hurrell \n> ---\n> Documentation/config/gpg.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n\nThanks. Will queue.\n\n> diff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt\n> index abfabd6d62..86892ada77 100644\n> --- a/Documentation/config/gpg.txt\n> +++ b/Documentation/config/gpg.txt\n> @@ -66,7 +66,7 @@ This way only committers with an already valid key can add or change keys in the\n> +\n> Since OpensSSH 8.8 this file allows specifying a key lifetime using valid-after &\n> valid-before options. Git will mark signatures as valid if the signing key was\n> -valid at the time of the signatures creation. This allows users to change a\n> +valid at the time of the signature's creation. This allows users to change a\n> signing key without invalidating all previously made signatures.\n> +\n> Using a SSH CA key with the cert-authority option\n"},{"id":"447055","messageId":"4216fdca-cf73-4bf6-b911-90c6c7ba0e2d@beta.fastmail.com","threadId":"57306","inReplyTo":"xmqq5yq56hbu.fsf@gitster.g","subject":"Re: [PATCH v2 1/2] Documentation/config/pgp.txt: replace stray character with ","fromName":"Greg Hurrell","fromEmail":"greg@hurrell.net","sentAt":"2022-01-27T08:20:40Z","receivedAt":"2022-01-27T08:21:17Z","isPatch":true,"sender":{"key":"greg@hurrell.net","avatar":"https://avatars.githubusercontent.com/u/7074?v=4"},"body":"On Thu, Jan 27, 2022, at 3:30 AM, Junio C Hamano wrote:\n> Greg Hurrell writes:\n> \n> > Specifically, replace the tab between \"the\" and \"first\" with a space.\n> \n> Good eyes. Does it have ANY practical downside to leave it unfixed?\n> \n> In other words, how did you spot it (e.g. \"asciidoctor passes tab as-is\n> and screwed up an alignment of the text, which was how I noticed it\")?\n\nThe nearby fix (the apostrophe) in the other commit was what had me\nlooking at this file in the first place. But my editor (Neovim)\nmade the mid-line tab visually obvious, so I figured may as well clean\nit up along the way; on my screen it looked something like this:\n\n ▷┅┅┅signature is requested. On successful exit a valid ssh public key is\n ▷┅┅┅expected in the▷┅┅┅first line of its output. To automatically use the first\n ▷┅┅┅available key from your ssh-agent set this to \"ssh-add -L\".\n\nVim's \"listchars\" setting is responsible for making tabs visible like that:\n\n :set listchars=extends:»,nbsp:⦸,precedes:«,tab:▷⋯,trail:•\n\nGreg\n"},{"id":"447103","messageId":"xmqqee4t3w5x.fsf@gitster.g","threadId":"57306","inReplyTo":"4216fdca-cf73-4bf6-b911-90c6c7ba0e2d@beta.fastmail.com","subject":"Re: [PATCH v2 1/2] Documentation/config/pgp.txt: replace stray character with ","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-27T17:50:34Z","receivedAt":"2022-01-27T17:50:39Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Greg Hurrell\" writes:\n\n> On Thu, Jan 27, 2022, at 3:30 AM, Junio C Hamano wrote:\n>> Greg Hurrell writes:\n>> \n>> > Specifically, replace the tab between \"the\" and \"first\" with a space.\n>> \n>> Good eyes. Does it have ANY practical downside to leave it unfixed?\n>> \n>> In other words, how did you spot it (e.g. \"asciidoctor passes tab as-is\n>> and screwed up an alignment of the text, which was how I noticed it\")?\n>\n> The nearby fix (the apostrophe) in the other commit was what had me\n> looking at this file in the first place. But my editor (Neovim)\n> made the mid-line tab visually obvious, so I figured may as well clean\n\nAh, yes, highlighting editors and pagers. I forgot about them.\n\nIn any case, thanks for the fixes. Queued.\n"}]}