# [PATCH] Fix a typo in line 117 of git-gui/lib/sshkeys.tcl. 8 messages from 2011-10-30 to 2017-10-11. Participants: Dejan Ribič, Erik Faye-Lund, Pat Thoyts, Takahito Ogawa, Thomas Gummerer. Thread: https://gitlist.dev/t/28801 ## Dejan Ribič, 2011-10-30 17:18 Subject: [PATCH] Fix a typo in line 117 of git-gui/lib/sshkeys.tcl. Message-ID: <1319995124-7509-1-git-send-email-dejan.ribic@gmail.com> URL: https://gitlist.dev/e/1319995124-7509-1-git-send-email-dejan.ribic%40gmail.com In-Reply-To: ``` "succeded" changed to "succeeded". Reference Launchpad bug #879427. modified: git-gui/lib/sshkey.tcl Signed-off-by: Dejan Ribič --- git-gui/lib/sshkey.tcl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/git-gui/lib/sshkey.tcl b/git-gui/lib/sshkey.tcl index 5f75bc9..aa6457b 100644 --- a/git-gui/lib/sshkey.tcl +++ b/git-gui/lib/sshkey.tcl @@ -117,7 +117,7 @@ proc read_sshkey_output {fd w} { } else { set finfo [find_ssh_key] if {$finfo eq {}} { - set sshkey_title [mc "Generation succeded, but no keys found."] + set sshkey_title [mc "Generation succeeded, but no keys found."] $w.contents insert end $sshkey_output } else { set sshkey_title [mc "Your key is in: %s" [lindex $finfo 0]] -- 1.7.5.4 ``` ## Erik Faye-Lund, 2011-10-30 23:57 Subject: Re: [PATCH] Fix a typo in line 117 of git-gui/lib/sshkeys.tcl. Message-ID: URL: https://gitlist.dev/e/CABPQNSaLsS9P8hWMEBYVe4FEo5x11%2BJ%2BpMw29NN%2BEF2siCO35w%40mail.gmail.com In-Reply-To: <1319995124-7509-1-git-send-email-dejan.ribic@gmail.com> ``` Thanks for re-posting inline and with a sign-off. But there's still a few minor nits: 2011/10/30 Dejan Ribič : > "succeded" changed to "succeeded". We write commit messages in imperative mood, so this should be something like 'change "succeded" to "succeeded"' instead. This is documented in Documentation/SubmittingPatches. > modified:   git-gui/lib/sshkey.tcl We don't normally include a list of changed files in the commit message; the diffstat already provides that information. ``` ## Pat Thoyts, 2011-10-31 08:56 Subject: Re: [PATCH] Fix a typo in line 117 of git-gui/lib/sshkeys.tcl. Message-ID: <87aa8h7d4w.fsf@fox.patthoyts.tk> URL: https://gitlist.dev/e/87aa8h7d4w.fsf%40fox.patthoyts.tk In-Reply-To: ``` Erik Faye-Lund writes: >Thanks for re-posting inline and with a sign-off. But there's still a >few minor nits: > >2011/10/30 Dejan Ribič : >> "succeded" changed to "succeeded". > >We write commit messages in imperative mood, so this should be >something like 'change "succeded" to "succeeded"' instead. This is >documented in Documentation/SubmittingPatches. > >> modified:   git-gui/lib/sshkey.tcl > >We don't normally include a list of changed files in the commit >message; the diffstat already provides that information. > The change is fine so I've applied this to the git-gui repository with a fixed commit message. Thank you. git-gui is hosted separately and merged into git-core at intervals so the development version of git-gui is in fact hosted at git://repo.or.cz/git-gui.git although patches against the git-core version seldom pose a problem to apply and are gratefully received. (git am -p2 usually fixes them). -- Pat Thoyts http://www.patthoyts.tk/ PGP fingerprint 2C 6E 98 07 2C 59 C8 97 10 CE 11 E6 04 E0 B9 DD ``` ## Takahito Ogawa, 2017-10-11 18:06 Subject: [PATCH 1/1] git-stash.txt: correct "git stash" behavior with no arguments Message-ID: <20171011180627.153-1-takahito.ogawa@datagrid.co.jp> URL: https://gitlist.dev/e/20171011180627.153-1-takahito.ogawa%40datagrid.co.jp In-Reply-To: ``` "git stash" behavior without any arguments was changed in 1ada5020b ("stash: use stash_push for no verb form", 2017-02-28). This is equivalent to "git stash push" but document says "git stash save". Correct it. Signed-off-by: Takahito Ogawa --- Documentation/git-stash.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt index 00f95fee1..63642c145 100644 --- a/Documentation/git-stash.txt +++ b/Documentation/git-stash.txt @@ -33,7 +33,7 @@ and reverts the working directory to match the `HEAD` commit. The modifications stashed away by this command can be listed with `git stash list`, inspected with `git stash show`, and restored (potentially on top of a different commit) with `git stash apply`. -Calling `git stash` without any arguments is equivalent to `git stash save`. +Calling `git stash` without any arguments is equivalent to `git stash push`. A stash is by default listed as "WIP on 'branchname' ...", but you can give a more descriptive message on the command line when you create one. -- 2.13.1 ``` ## Takahito Ogawa, 2017-10-11 18:37 Subject: [PATCH 1/1] git-stash.txt: correct "git stash" behavior with no arguments Message-ID: <20171011183718.2404-1-aiueogawa217@gmail.com> URL: https://gitlist.dev/e/20171011183718.2404-1-aiueogawa217%40gmail.com In-Reply-To: ``` "git stash" behavior without any arguments was changed in 1ada5020b ("stash: use stash_push for no verb form", 2017-02-28). This is equivalent to "git stash push" but documents says "git stash save". Correct it. Signed-off-by: Takahito Ogawa --- Documentation/git-stash.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt index 00f95fee1..63642c145 100644 --- a/Documentation/git-stash.txt +++ b/Documentation/git-stash.txt @@ -33,7 +33,7 @@ and reverts the working directory to match the `HEAD` commit. The modifications stashed away by this command can be listed with `git stash list`, inspected with `git stash show`, and restored (potentially on top of a different commit) with `git stash apply`. -Calling `git stash` without any arguments is equivalent to `git stash save`. +Calling `git stash` without any arguments is equivalent to `git stash push`. A stash is by default listed as "WIP on 'branchname' ...", but you can give a more descriptive message on the command line when you create one. -- 2.13.1 ``` ## Thomas Gummerer, 2017-10-11 19:09 Subject: Re: [PATCH 1/1] git-stash.txt: correct "git stash" behavior with no arguments Message-ID: <20171011190935.GA15399@hank> URL: https://gitlist.dev/e/20171011190935.GA15399%40hank In-Reply-To: <20171011183718.2404-1-aiueogawa217@gmail.com> ``` On 10/12, Takahito Ogawa wrote: > "git stash" behavior without any arguments was changed in > 1ada5020b ("stash: use stash_push for no verb form", 2017-02-28). > This is equivalent to "git stash push" but documents says > "git stash save". > > Correct it. Thanks for fixing this! I recently sent a patch that would advertise git stash push more in general, which would also fix this occurrence [1], but it didn't seem like it got much interest. However this is obviously correct, and should definitely be fixed, while the other places can still mention 'git stash save'. For what it's worth this is Reviewed-by: Thomas Gummerer > Signed-off-by: Takahito Ogawa > --- > Documentation/git-stash.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt > index 00f95fee1..63642c145 100644 > --- a/Documentation/git-stash.txt > +++ b/Documentation/git-stash.txt > @@ -33,7 +33,7 @@ and reverts the working directory to match the `HEAD` commit. > The modifications stashed away by this command can be listed with > `git stash list`, inspected with `git stash show`, and restored > (potentially on top of a different commit) with `git stash apply`. > -Calling `git stash` without any arguments is equivalent to `git stash save`. > +Calling `git stash` without any arguments is equivalent to `git stash push`. > A stash is by default listed as "WIP on 'branchname' ...", but > you can give a more descriptive message on the command line when > you create one. > -- > 2.13.1 > ``` ## Thomas Gummerer, 2017-10-11 19:12 Subject: Re: [PATCH 1/1] git-stash.txt: correct "git stash" behavior with no arguments Message-ID: <20171011191214.GB15399@hank> URL: https://gitlist.dev/e/20171011191214.GB15399%40hank In-Reply-To: <20171011190935.GA15399@hank> ``` On 10/11, Thomas Gummerer wrote: > On 10/12, Takahito Ogawa wrote: > > "git stash" behavior without any arguments was changed in > > 1ada5020b ("stash: use stash_push for no verb form", 2017-02-28). > > This is equivalent to "git stash push" but documents says > > "git stash save". > > > > Correct it. > > Thanks for fixing this! I recently sent a patch that would advertise > git stash push more in general, which would also fix this occurrence [1], > but it didn't seem like it got much interest. However this is > obviously correct, and should definitely be fixed, while the other > places can still mention 'git stash save'. > > For what it's worth this is > > Reviewed-by: Thomas Gummerer And I forgot to include the link, sorry. Here it is: [1]: https://public-inbox.org/git/20171005200049.GF30301@hank/ > > > Signed-off-by: Takahito Ogawa > > --- > > Documentation/git-stash.txt | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt > > index 00f95fee1..63642c145 100644 > > --- a/Documentation/git-stash.txt > > +++ b/Documentation/git-stash.txt > > @@ -33,7 +33,7 @@ and reverts the working directory to match the `HEAD` commit. > > The modifications stashed away by this command can be listed with > > `git stash list`, inspected with `git stash show`, and restored > > (potentially on top of a different commit) with `git stash apply`. > > -Calling `git stash` without any arguments is equivalent to `git stash save`. > > +Calling `git stash` without any arguments is equivalent to `git stash push`. > > A stash is by default listed as "WIP on 'branchname' ...", but > > you can give a more descriptive message on the command line when > > you create one. > > -- > > 2.13.1 > > ``` ## Takahito Ogawa, 2017-10-11 20:01 Subject: [PATCH 1/1] git-stash.txt: correct "git stash" behavior with no arguments Message-ID: <20171011200147.4377-1-aiueogawa217@gmail.com> URL: https://gitlist.dev/e/20171011200147.4377-1-aiueogawa217%40gmail.com In-Reply-To: ``` "git stash" behavior without any arguments was changed in 1ada5020b ("stash: use stash_push for no verb form", 2017-02-28). This is equivalent to "git stash push" but documents says "git stash save". Correct it. Reviewed-by: Thomas Gummerer Signed-off-by: Takahito Ogawa --- Documentation/git-stash.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt index 00f95fee1..63642c145 100644 --- a/Documentation/git-stash.txt +++ b/Documentation/git-stash.txt @@ -33,7 +33,7 @@ and reverts the working directory to match the `HEAD` commit. The modifications stashed away by this command can be listed with `git stash list`, inspected with `git stash show`, and restored (potentially on top of a different commit) with `git stash apply`. -Calling `git stash` without any arguments is equivalent to `git stash save`. +Calling `git stash` without any arguments is equivalent to `git stash push`. A stash is by default listed as "WIP on 'branchname' ...", but you can give a more descriptive message on the command line when you create one. -- 2.13.1 ```