Re: [Suggestion] Alternative way for displaying help menu of patch flow

2017-04-28 Thread Jeff King
On Fri, Apr 28, 2017 at 10:44:22AM +0530, Kaartic Sivaraam wrote: > I guess it would be better to display the help menu in a separate flow, like > when the user click the `?` option in the `patch` flow then the whole screen > is cleared and the help menu is displayed in some appropri

[Suggestion] Alternative way for displaying help menu of patch flow

2017-04-27 Thread Kaartic Sivaraam
commits. So, you would like to split the hunk into smaller parts but you forgot the correct option for it and you use `?` option. In most cases you wouldn't see the help menu completely. Moreover, if the hunk's size is reasonable enough to take up the whole screen of the terminal you would

[PATCH] worktree: change 'worktree' to 'working tree' in help text

2017-04-12 Thread Nguyễn Thái Ngọc Duy
We have been trying to keep the terminology consistent on the user-facing front. Let's keep sticking to "working tree". While at there, change 'other' to 'another'. The latter is probably more correct. Signed-off-by: Nguyễn Thái Ngọc Duy --- builtin/worktree.c | 2 +- 1 file

Dear, I need your help to invest in your country,

2017-03-30 Thread aishagaddaf...@ono.com
I came across your contact during my private search Mrs Aisha Al-Qaddafi is my name, the only daughter of late Libyan president, I have funds the sum of $27.5 million USD for investment, I am interested in you for investment project assistance in your country, i shall compensate you 30% of

I need your urgent help...

2017-03-30 Thread Sgt. Britta Lopez
Hello, I have a personal Project in which i need your assistance I would like to be sure of your willingness and commitment to execute this transaction with me. I seek your partnership in receiving this fund worth (Twenty five million United States Dollars). If interested, reply immediately for

[PATCH v4 08/16] for-each-ref: partly change to in help

2017-03-24 Thread Ævar Arnfjörð Bjarmason
Change mentions of to in the help output of for-each-ref as appropriate. Both --[no-]merged and --contains only take commits, but --points-at can take any object, such as a tag pointing to a tree or blob. Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com> --- builtin/for-each-ref

[PATCH v3 08/16] for-each-ref: partly change to in help

2017-03-23 Thread Ævar Arnfjörð Bjarmason
Change mentions of to in the help output of for-each-ref as appropriate. Both --[no-]merged and --contains only take commits, but --points-at can take any object, such as a tag pointing to a tree or blob. Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com> --- builtin/for-each-ref

Re: [PATCH v2 08/16] for-each-ref: partly change to in help

2017-03-21 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason writes: > On Tue, Mar 21, 2017 at 7:32 PM, Junio C Hamano wrote: > >> Do these strictly require commit, or does any commit-ish also do? > > commit-ish, but that's a good point, and could be the subject of a > future follow-up patch.

Re: [PATCH v2 08/16] for-each-ref: partly change to in help

2017-03-21 Thread Ævar Arnfjörð Bjarmason
On Tue, Mar 21, 2017 at 7:32 PM, Junio C Hamano <gits...@pobox.com> wrote: > Ævar Arnfjörð Bjarmason <ava...@gmail.com> writes: > >> Change mentions of to in the help output of >> for-each-ref as appropriate. >> >> Both --[no-]merged and --contains only

Re: [PATCH v2 08/16] for-each-ref: partly change to in help

2017-03-21 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason <ava...@gmail.com> writes: > Change mentions of to in the help output of > for-each-ref as appropriate. > > Both --[no-]merged and --contains only take commits, but --points-at > can take any object, such as a tag pointing to a tree or blob. &g

[PATCH v2 08/16] for-each-ref: partly change to in help

2017-03-21 Thread Ævar Arnfjörð Bjarmason
Change mentions of to in the help output of for-each-ref as appropriate. Both --[no-]merged and --contains only take commits, but --points-at can take any object, such as a tag pointing to a tree or blob. Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com> --- builtin/for-each-ref

[PATCH v3 1/5] grep: fix help text typo

2017-03-17 Thread Brandon Williams
Signed-off-by: Brandon Williams --- builtin/grep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/grep.c b/builtin/grep.c index 9304c33e7..4694e68f3 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -979,7 +979,7 @@ int cmd_grep(int argc, const

i need your help for this Investment

2017-03-15 Thread mrs.aich...@ono.com
Hello Dear, My names are Aisha Gaddafi, 39, I need a very honest and reliable person that can assist me for investment project for a profitable business/ company to invest into in your country than if you are interested let me know. I will details you more when I hear from you. Aisha

Re: [PATCH v2 1/4] grep: fix help text typo

2017-03-14 Thread Brandon Williams
On 03/14, Stefan Beller wrote: > On Tue, Mar 14, 2017 at 3:10 PM, Brandon Williams wrote: > > Missing SoB here, too. I guess I'm having an off day...Will fix. > > > --- > > builtin/grep.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git

Re: [PATCH v2 1/4] grep: fix help text typo

2017-03-14 Thread Stefan Beller
On Tue, Mar 14, 2017 at 3:10 PM, Brandon Williams wrote: Missing SoB here, too. > --- > builtin/grep.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/builtin/grep.c b/builtin/grep.c > index 9304c33e7..4694e68f3 100644 > --- a/builtin/grep.c > +++

[PATCH v2 1/4] grep: fix help text typo

2017-03-14 Thread Brandon Williams
--- builtin/grep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/grep.c b/builtin/grep.c index 9304c33e7..4694e68f3 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -979,7 +979,7 @@ int cmd_grep(int argc, const char **argv, const char *prefix)

Re: [RFC PATCH] help: add optional instructions for reporting bugs

2017-03-13 Thread Stefan Beller
> I think it's better to do things like the following: >> >> - add `git version -h` >> - add proper documentation for `git version` so that `git help version` works >> - in `git help version` talk about the REPORTING BUGS section in `git help >> git` >> - add `git versi

Re: [RFC PATCH] help: add optional instructions for reporting bugs

2017-03-12 Thread Jeff King
proper documentation for `git version` so that `git help version` works > - in `git help version` talk about the REPORTING BUGS section in `git help > git` > - add `git version --full` or other such options to also print other > stuff like the current OS, processor arc

Re: [RFC PATCH] help: add optional instructions for reporting bugs

2017-03-10 Thread Christian Couder
t; > It's tempting to put the custom information in --build-options --- e.g. > > $ git version > git version 2.12.0.190.g6e60aba09d.dirty > hint: use "git version --build-options" for more detail > hint: and bug reporting instructions > $ > $ git version --build-options &g

Re: [RFC PATCH] help: add optional instructions for reporting bugs

2017-03-10 Thread Jonathan Nieder
nt: use "git version --build-options" for more detail hint: and bug reporting instructions $ $ git version --build-options git version 2.12.0.190.g6e60aba09d.dirty sizeof-long: 8 reporting-bugs: see REPORTING BUGS section in "git help git" Using 'hint:' would put this in the a

[RFC PATCH] help: add optional instructions for reporting bugs

2017-03-09 Thread Stefan Beller
Git is distributed in various ways by various organizations. The Git community prefers to have bugs reported on the mailing list, whereas other organizations may rather want to have filed a bug in a bug tracker or such. The point of contact is different by organization as well. When reporting

Re: [PATCH] git add -i: replace \t with blanks in the help message

2017-02-22 Thread Junio C Hamano
Ralf Thielow <ralf.thie...@gmail.com> writes: > Within the help message of 'git add -i', the 'diff' command uses one > tab character and blanks to create the space between the name and the > description while the others use blanks only. So if the tab size is > n

[PATCH] git add -i: replace \t with blanks in the help message

2017-02-22 Thread Ralf Thielow
Within the help message of 'git add -i', the 'diff' command uses one tab character and blanks to create the space between the name and the description while the others use blanks only. So if the tab size is not at 4 characters, this description will not be in range. Replace the tab character

Re: [PATCH] help: show help for aliases

2017-02-09 Thread Junio C Hamano
Tom Kunze <m...@tom-kunze.de> writes: > If an alias is a single git command show the man page of the > aliased git command with --help. > > Signed-off-by: Tom Kunze <m...@tom-kunze.de> > ... > diff --git a/builtin/help.c b/builtin/help.c > index 49f7a07..655e

[PATCH] help: show help for aliases

2017-02-09 Thread Tom Kunze
If an alias is a single git command show the man page of the aliased git command with --help. Signed-off-by: Tom Kunze <m...@tom-kunze.de> --- Hi, I noticed that when I pass --help to an alias which is only a git command it tells me a information about the alias. But it would b

Re: A little help understanding output from git blame --reverse

2017-02-06 Thread Edmundo Carmona Antoranz
On Mon, Feb 6, 2017 at 6:38 AM, Edmundo Carmona Antoranz wrote: > I'm "difflaming" HEAD~100 (02db2d0421b97fcb6211) and HEAD > (066fb0494e6398eb). Specifically file abspath.c. I just noticed that I'm standing on a private branch. Replace HEAD for "4e59582ff" when doing your

A little help understanding output from git blame --reverse

2017-02-06 Thread Edmundo Carmona Antoranz
Hi! While doing some research developing difflame I found some output from git blame --reverse that I can't quite understand. Perhaps another set of eyeballs could help me. I'm "difflaming" HEAD~100 (02db2d0421b97fcb6211) and HEAD (066fb0494e6398eb). Specifically file abspath.c.

Re: [PATCH v2] help: improve is_executable() on Windows

2017-01-30 Thread Junio C Hamano
ng line by opening the file in question, and executing > it via the specified script interpreter. > > To figure out whether files in the `PATH` are executable, `git help` has > code that imitates this behavior. With one exception: it *always* opens > the files and looks for a she-ban

Re: [PATCH] help: correct behavior for is_executable on Windows

2017-01-30 Thread Johannes Schindelin
Hi Junio, On Fri, 27 Jan 2017, Junio C Hamano wrote: > Johannes Schindelin writes: > > > From: Heiko Voigt > > > > The previous implementation said that the filesystem information on > > Windows is not reliable to determine whether a file is

[PATCH v2] help: improve is_executable() on Windows

2017-01-30 Thread Johannes Schindelin
preter. To figure out whether files in the `PATH` are executable, `git help` has code that imitates this behavior. With one exception: it *always* opens the files and looks for a she-bang line *or* an `MZ` tell-tale (nevermind that files with the magic `MZ` but without file extension `.exe` would

Re: [PATCH] help: correct behavior for is_executable on Windows

2017-01-27 Thread Junio C Hamano
Johannes Schindelin writes: > From: Heiko Voigt > > The previous implementation said that the filesystem information on > Windows is not reliable to determine whether a file is executable. To > gather this information it was peeking into the first

[PATCH] help: correct behavior for is_executable on Windows

2017-01-27 Thread Johannes Schindelin
r.exe): 0.000180 before open (git-blame.exe): 0.000718 after open (git-blame.exe): 0.000724 ... With this patch I get: $ time git help git Launching default browser to display HTML ... real0m8.723s user0m0.000s sys 0m0.000s and without $ time git help git Launching default browser

HELP ME PLEASE

2017-01-16 Thread Abdoul Issouf
HELP ME PLEASE Dear Friend I am Mr. Abdoul Issouf ,I work for BOA bank Ouagadougou Burkina Faso. I have a business proposal which concerns the transfer of $13.5 Million USD, into a foreign account. Everything about this transaction shall be legally done without any problem. If you

[PATCH] submodule absorbgitdirs: mention in docstring help

2017-01-11 Thread Stefan Beller
This part was missing in f6f85861 (submodule: add absorb-git-dir function, 2016-12-12). Noticed-by: Jonathan Nieder Signed-off-by: Stefan Beller --- git-submodule.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/git-submodule.sh

[PATCHv2] Tweak help auto-correct phrasing.

2016-12-20 Thread Marc Branchaud
When auto-correct is enabled, an invalid git command prints a warning and a continuation message, which differs depending on whether or not help.autoCorrect is positive or negative. With help.autoCorrect = 15: WARNING: You called a Git command named 'lgo', which does not exist. Continuing

Re: [PATCH] Tweak help auto-correct phrasing.

2016-12-19 Thread Junio C Hamano
Marc Branchaud writes: > Signed-off-by: Marc Branchaud > --- > > On 2016-12-18 07:48 PM, Chris Packham wrote: >> >> This feature already exists (although it's not interactive). See >> help.autoCorrect in the git-config man page. "git config >>

[PATCH] Tweak help auto-correct phrasing.

2016-12-19 Thread Marc Branchaud
Signed-off-by: Marc Branchaud --- On 2016-12-18 07:48 PM, Chris Packham wrote: > > This feature already exists (although it's not interactive). See > help.autoCorrect in the git-config man page. "git config > help.autoCorrect -1" should to the trick. Awesome, I was unaware

I want to present you as the owner of funds In my Bank.which will be Invested in your Country with your help, Kindly Get back to me with your Phone number

2016-12-05 Thread Minoru
information

RE: merge --no-ff is NOT mentioned in help

2016-11-18 Thread Vanderhoof, Tzadik
HIDDEN > "--ff". I thought we had done that in other cases, but I can't seem to find > any. But it would make "--no-ff" the primary form, which makes sense, as > "--ff" is already the default. > > Another option would be to teach parse-options to somehow treat

Re: merge --no-ff is NOT mentioned in help

2016-11-17 Thread Jeff King
But it would make "--no-ff" the primary form, which makes sense, as "--ff" is already the default. Another option would be to teach parse-options to somehow treat the negated form as primary in the help text. That's a bit more code, but might be usable in other places. -Peff

Re: merge --no-ff is NOT mentioned in help

2016-11-17 Thread Torsten Bögershausen
e sense to word the output to include both. I think that was a deliberate design decision to avoid cluttering the short help text with mention of both --option and --no-option. People interested may want to try the attached single-liner patch to see how the output from _ALL_ commands that

Re: merge --no-ff is NOT mentioned in help

2016-11-17 Thread Junio C Hamano
automatic counter > argument with "no-" in there ('--no-ff') to disable the option. Maybe > it would make sense to word the output to include both. I think that was a deliberate design decision to avoid cluttering the short help text with mention of both --option and --no-optio

RE: merge --no-ff is NOT mentioned in help

2016-11-17 Thread Vanderhoof, Tzadik
-Original Message- From: Mike Rappazzo [mailto:rappa...@gmail.com] Sent: Wednesday, November 16, 2016 7:58 AM To: Vanderhoof, Tzadik Cc: git@vger.kernel.org Subject: Re: merge --no-ff is NOT mentioned in help >(Please reply inline) > >On Wed, Nov 16, 2016 at 10:48 AM, Vanderhoo

Re: merge --no-ff is NOT mentioned in help

2016-11-16 Thread Mike Rappazzo
(Please reply inline) On Wed, Nov 16, 2016 at 10:48 AM, Vanderhoof, Tzadik wrote: > I am running:git version 2.10.1.windows.1 > > I typed: git merge -h > > and got: > > usage: git merge [] [...] >or: git merge [] HEAD >or: git merge --abort > >

RE: merge --no-ff is NOT mentioned in help

2016-11-16 Thread Vanderhoof, Tzadik
I am running:git version 2.10.1.windows.1 I typed: git merge -h and got: usage: git merge [] [...] or: git merge [] HEAD or: git merge --abort -ndo not show a diffstat at the end of the merge --statshow a diffstat at the end of the merge

Re: merge --no-ff is NOT mentioned in help

2016-11-16 Thread Mike Rappazzo
On Wed, Nov 16, 2016 at 10:16 AM, Vanderhoof, Tzadik <tzadik.vanderh...@optum360.com> wrote: > When I do: "git merge -h" to get help, the option "--no-ff" is left out of > the list of options. I am running git version 2.10.0, and running git merge --help co

merge --no-ff is NOT mentioned in help

2016-11-16 Thread Vanderhoof, Tzadik
When I do: "git merge -h" to get help, the option "--no-ff" is left out of the list of options. This e-mail, including attachments, may include confidential and/or proprietary information, and may be used only by the person or entity to which it is addressed. If the r

Re: New to git, need help!

2016-11-15 Thread Pranit Bauva
Hey Mayank, On Tue, Nov 15, 2016 at 6:00 PM, Mayank Gupta wrote: > Hi All, > > I'm new to open source and have recently joined this mailing list. > Since I'm new at this, I think I can initially contribute to the > community by fixing some small bugs or errors but as

New to git, need help!

2016-11-15 Thread Mayank Gupta
Hi All, I'm new to open source and have recently joined this mailing list. Since I'm new at this, I think I can initially contribute to the community by fixing some small bugs or errors but as the documentation is too large, I don't know where to start. So if anybody could guide me on how to go

Re: Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Johannes Schindelin
Hi Peff, On Thu, 3 Nov 2016, Jeff King wrote: > On Thu, Nov 03, 2016 at 11:34:53AM -0400, Jeff King wrote: > > > This is missing a Content-Transfer-Encoding. I think the default is the > > traditional 7-bit ascii encoding, but your body has characters with the > > high-bit set (your UTF-8

Re: Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Jeff King
On Thu, Nov 03, 2016 at 11:38:45AM -0400, Jeff King wrote: > On Thu, Nov 03, 2016 at 11:34:53AM -0400, Jeff King wrote: > > > This is missing a Content-Transfer-Encoding. I think the default is the > > traditional 7-bit ascii encoding, but your body has characters with the > > high-bit set (your

Re: Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Jeff King
On Thu, Nov 03, 2016 at 11:34:53AM -0400, Jeff King wrote: > This is missing a Content-Transfer-Encoding. I think the default is the > traditional 7-bit ascii encoding, but your body has characters with the > high-bit set (your UTF-8 bullet). > > Try adding: > > Content-Transfer-Encoding:

Re: Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Jeff King
On Thu, Nov 03, 2016 at 04:15:05PM +0100, Johannes Schindelin wrote: > When it finally sent out the mail, and I thought everything was alright, > thinking that I could turn out for the night with a well-deserved drink, I > got this from vger.kernel.org: > > -- snip -- > SMTP error from remote

Re: Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Junio C Hamano
Johannes Schindelin writes: > thinking that I could turn out for the night with a well-deserved drink, I > got this from vger.kernel.org: > > -- snip -- > SMTP error from remote server for TEXT command, host: vger.kernel.org > (209.132.180.67) reason: 550 5.7.1

Help with vger's rules, was Re: [ANNOUNCE] Git for Windows 2.10.2

2016-11-03 Thread Johannes Schindelin
o keep automating things to make the release engineering of Git for Windows more painless and boring, but this thing, this unhelpful vger error message is blocking me from doing so right now. Help, anyone? Thanks, Dscho Footnote *1*: The release notes are actually written using Markdown

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Ralf Thielow
2016-08-26 22:20 GMT+02:00 Junio C Hamano : > > Because the whole thing is inside a double-quote pair, $() and $name > are all interpolated even before test_expect_success is called. > So the above becomes equivalent to > >>> test_expect_success "two commits do not have the same

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Junio C Hamano
Ralf Thielow writes: > 2016-08-26 21:42 GMT+02:00 Junio C Hamano : >> Junio C Hamano writes: >> >> >> Taking all of these together, I'll queue this as a proposed fix-up >> directly on top of yours. >> > > Thanks! Thank you for

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Junio C Hamano
>> >> I think this is OK; with s|As we pass a URL|As we pass a string with >> :// in it|, the first sentence can be a in-code comment in the test >> that does this and will help readers of the code in the future. > > Hmm. The "://" is really a URL thing. Perha

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Ralf Thielow
2016-08-26 21:42 GMT+02:00 Junio C Hamano : > Junio C Hamano writes: > > > Taking all of these together, I'll queue this as a proposed fix-up > directly on top of yours. > Thanks! -- To unsubscribe from this list: send the line "unsubscribe git" in the body

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Ralf Thielow
2016-08-26 21:06 GMT+02:00 Junio C Hamano <gits...@pobox.com>: > Ralf Thielow <ralf.thie...@gmail.com> writes: > >> Introduce option --exclude-guides to the help command. With this option >> being passed, "git help" will open man pages only for actual c

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Junio C Hamano
Junio C Hamano <gits...@pobox.com> writes: > Let's hide this option from command help of "git help" itself, drop > the short-and-sweet "-e", not command-line complete it, and leave it > not-mentioned here. > ... > Unless there is a good reason you

Re: [PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Junio C Hamano
Ralf Thielow <ralf.thie...@gmail.com> writes: > Introduce option --exclude-guides to the help command. With this option > being passed, "git help" will open man pages only for actual commands. Let's hide this option from command help of "git help" itse

[PATCH v2 0/3] help: make option --help open man pages only for Git commands

2016-08-26 Thread Ralf Thielow
Changes in v2 are: - add a patch from Dscho to make config variable 'help.browser' work on Windows again - rename option "--command-only" to "--exclude-guides" which is less ambiguous in 'help' context - improve test script - refactor usage of argv_array in handle_builtin Jo

[PATCH v2 3/3] help: make option --help open man pages only for Git commands

2016-08-26 Thread Ralf Thielow
If option --help is passed to a Git command, we try to open the man page of that command. However, we do it for both commands and concepts. Make sure it is an actual command. This makes "git --help" not working anymore, while "git help " still works. Signed-off-by: Ral

[PATCH v2 2/3] help: introduce option --exclude-guides

2016-08-26 Thread Ralf Thielow
Introduce option --exclude-guides to the help command. With this option being passed, "git help" will open man pages only for actual commands. Since we know it is a command, we can use function help_unknown_command to give the user advice on typos. Helped-by: Johannes

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-24 Thread Johannes Schindelin
or Windows [*1*] (and that did not make it upstream yet) which > > reverts that change whereby web--browse was sidestepped. That sidestepping > > was well-intentioned but turned out to cause more harm than good. > > > > So I'll pickup the patch you sent [1] to my series and prepa

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-23 Thread Ralf Thielow
> was well-intentioned but turned out to cause more harm than good. > So I'll pickup the patch you sent [1] to my series and prepare the test cases the way you described to verify that the 'help' command works. Thanks! [1] http://public-inbox.org/git/03ae6a9d47cb95a54960bfdc90c5392f890ff1e

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-23 Thread Ralf Thielow
2016-08-19 10:39 GMT+02:00 Remi Galan Alfonso <remi.galan-alfo...@ensimag.grenoble-inp.fr>: > Hi Ralf, > > Ralf Thielow <ralf.thie...@gmail.com> writes: >> [...] >> +test_expect_success "works for commands and guides by default" " >> +

Re: [PATCH 2/2] help: make option --help open man pages only for Git commands

2016-08-23 Thread Ralf Thielow
ed differently by different people. For example, > "git co --help" and "git help co" would work, with or without 1/2 in > place when you have "[alias] co = checkout", so we are calling "Git > subcommands that we ship, custom commands 'git-$foo' the us

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-19 Thread Junio C Hamano
Johannes Schindelin writes: > - configure help.format = html (for "man", the current code would always > add $(prefix)/share/man to the MANPATH when testing, not what we want, > and hacking this code *just* for testing is both ugly and unnecessary). A very

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-19 Thread Johannes Schindelin
Hi Ralf, On Thu, 18 Aug 2016, Ralf Thielow wrote: > diff --git a/t/t0012-help.sh b/t/t0012-help.sh > new file mode 100755 > index 000..e20f907 > --- /dev/null > +++ b/t/t0012-help.sh > @@ -0,0 +1,21 @@ > +#!/bin/sh > + > +test_description='h

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-19 Thread Remi Galan Alfonso
Hi Ralf, Ralf Thielow <ralf.thie...@gmail.com> writes: > [...] > +test_expect_success "works for commands and guides by default" " > +git help status && > +git help revisions > +" > + > +test_expect_success "--command-on

[PATCH 2/2] help: make option --help open man pages only for Git commands

2016-08-19 Thread Ralf Thielow
If option --help is passed to a Git command, we try to open the man page of that command. However, we do it even for commands we don't know. Make sure it is a Git command. This breaks "git --help" while "git help " still works. Signed-off-by: Ralf Thielow <ralf.thie..

Re: [PATCH 1/2] help: introduce option --command-only

2016-08-18 Thread Philip Oakley
From: "Ralf Thielow" <ralf.thie...@gmail.com> Introduce option --command-only to the help command. With this option being passed, "git help" will open man pages only for commands. Since we know it is a command, we can use function help_unknown_command to give the use

[PATCH 0/2] help: make option --help open man pages only for Git commands

2016-08-18 Thread Ralf Thielow
In this version, one patch has been turned into two. The first introduces the option "command-only" to make 'help' only working for commands and additionally give some nice help on typos. The second makes option --help only work for actual Git commands. Ralf Thielow (2): help:

[PATCH 1/2] help: introduce option --command-only

2016-08-18 Thread Ralf Thielow
Introduce option --command-only to the help command. With this option being passed, "git help" will open man pages only for commands. Since we know it is a command, we can use function help_unknown_command to give the user advice on typos. Signed-off-by: Ralf Thielow <ralf.thie

Re: [PATCH 2/2] help: make option --help open man pages only for Git commands

2016-08-18 Thread Junio C Hamano
Ralf Thielow <ralf.thie...@gmail.com> writes: > If option --help is passed to a Git command, we try to open > the man page of that command. However, we do it even for commands > we don't know. Make sure it is a Git command. What the patch does is correct, I think, but the

Re: [PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread Junio C Hamano
Ralf Thielow writes: > 2016-08-16 19:27 GMT+02:00 Junio C Hamano : >> Ralf Thielow writes: >>> + >>> + if (swapped) >>> + return help_unknown_cmd(cmd); >> >> I am guilty of suggesting "swapped"; even if we are

Re: [PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread Ralf Thielow
2016-08-16 19:27 GMT+02:00 Junio C Hamano : > Ralf Thielow writes: >> + >> + if (swapped) >> + return help_unknown_cmd(cmd); > > I am guilty of suggesting "swapped"; even if we are going to mark > this as OPT_HIDDEN, I think we should be

Re: [PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread Junio C Hamano
at = HELP_FORMAT_NONE; > +static int swapped = 0; This is not the first offender (show_guides above does so, too), but please do not initialize static explicitly to 0 or NULL. > static struct option builtin_help_options[] = { > + OPT_BOOL('s', "swapped", , "mark as being

Re: [PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread Ralf Thielow
2016-08-16 18:33 GMT+02:00 John Keeping <j...@keeping.me.uk>: > On Tue, Aug 16, 2016 at 06:20:30PM +0200, Ralf Thielow wrote: >> static struct option builtin_help_options[] = { >> + OPT_BOOL('s', "swapped", , "mark as being called by >> --

Re: [PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread John Keeping
On Tue, Aug 16, 2016 at 06:20:30PM +0200, Ralf Thielow wrote: > If option --help is passed to a Git command, we try to open > the man page of that command. However, we do it even for commands > we don't know. Make sure it is a Git command by using "help_unknown_cmd" &

[PATCH v3] help: make option --help open man pages only for Git commands

2016-08-16 Thread Ralf Thielow
If option --help is passed to a Git command, we try to open the man page of that command. However, we do it even for commands we don't know. Make sure it is a Git command by using "help_unknown_cmd" which is even able to assume a command if the user made a typo. This breaks "git

Re: [PATCH v2] help: make option --help open man pages only for Git commands

2016-08-16 Thread John Keeping
>> introduced the --guides option (65f9835 (builtin/help.c: add --guide > >> option, 2013-04-02)) was that we had no easy way of determining what > >> guides were available, especially given the *nix/Windows split where > >> the help defaults are different (--

Re: [PATCH v2] help: make option --help open man pages only for Git commands

2016-08-15 Thread Junio C Hamano
rward. To implement typo correction for "git help ", having guide-list would be very useful. A related tangent is that I think "git --help" shouldn't fall back to "git help ", regardless of typo correction. It happens to "work" only because we blindly turned &qu

Re: [PATCH v2] help: make option --help open man pages only for Git commands

2016-08-15 Thread Philip Oakley
no easy way of determining what guides were available, especially given the *nix/Windows split where the help defaults are different (--man/--html). At the time[1] we (I) punted on trying to determine which guides were actually installed, and just created a short list of the important guide

Re: [PATCH v2] help: make option --help open man pages only for Git commands

2016-08-15 Thread Junio C Hamano
s were available, especially given the *nix/Windows split where > the help defaults are different (--man/--html). > > At the time[1] we (I) punted on trying to determine which guides were > actually installed, and just created a short list of the important > guides, which I believe yo

Re: [PATCH v2] help: make option --help open man pages only for Git commands

2016-08-15 Thread Philip Oakley
From: "Ralf Thielow" <ralf.thie...@gmail.com> If option --help is passed to a Git command, we try to open the man page of that command. However, we do it even for commands we don't know. Make sure the command is known to Git before try to open the man page. If we don't know t

[PATCH v2] help: make option --help open man pages only for Git commands

2016-08-14 Thread Ralf Thielow
If option --help is passed to a Git command, we try to open the man page of that command. However, we do it even for commands we don't know. Make sure the command is known to Git before try to open the man page. If we don't know the command, give the usual advice. Signed-off-by: Ralf Thielow

Re: [PATCH] help: make option --help open man pages only for Git commands

2016-08-13 Thread Junio C Hamano
"Philip Oakley" <philipoak...@iee.org> writes: > But does it cope with the Guides? Should it cope if spelt that way? > > git help revisions > git revisions --help Hmph. Ralf's patch is not just "I wonder if we could do a bit better" but is also a r

Re: [PATCH] help: make option --help open man pages only for Git commands

2016-08-12 Thread Philip Oakley
From: "Junio C Hamano" <gits...@pobox.com> To: "Ralf Thielow" <ralf.thie...@gmail.com> Cc: <git@vger.kernel.org>; <larsxschnei...@gmail.com>; <m...@jnm2.com> Sent: Friday, August 12, 2016 11:53 PM Subject: Re: [PATCH] help: make option --help

Re: [PATCH] help: make option --help open man pages only for Git commands

2016-08-12 Thread Junio C Hamano
id it [*1*] ;-) Having said that, I wonder if we could do a bit better. $ git -c help.autocorrect=1 whatchange --help WARNING: You called a Git command named 'whatchange', which does not exist. Continuing under the assumption that you meant 'whatchanged' in 0.1 seconds automati

Re: [PATCH] help: make option --help open man pages only for Git commands

2016-08-12 Thread Junio C Hamano
Ralf Thielow <ralf.thie...@gmail.com> writes: > If option --help is passed to a Git command, we try to open > the man page of that command. However, we do it even for commands > we don't know. Make sure the command is known to Git before try > to open the man page. If we don'

[PATCH] help: make option --help open man pages only for Git commands

2016-08-12 Thread Ralf Thielow
If option --help is passed to a Git command, we try to open the man page of that command. However, we do it even for commands we don't know. Make sure the command is known to Git before try to open the man page. If we don't know the command, give the usual advice. Signed-off-by: Ralf Thielow

Re: `git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-12 Thread Jacob Keller
On Fri, Aug 12, 2016 at 9:25 AM, Junio C Hamano <gits...@pobox.com> wrote: > On Fri, Aug 12, 2016 at 9:15 AM, Joseph Musser <m...@jnm2.com> wrote: >> Oh, I'm embarrassed. The typo was mine, I must have typed `git stack >> --help`. I would have expected a syntax error

Re: `git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-12 Thread Junio C Hamano
On Fri, Aug 12, 2016 at 9:15 AM, Joseph Musser <m...@jnm2.com> wrote: > Oh, I'm embarrassed. The typo was mine, I must have typed `git stack > --help`. I would have expected a syntax error message or "did you > mean" suggestions; it didn't even enter my mind that it w

Re: `git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-12 Thread Joseph Musser
Oh, I'm embarrassed. The typo was mine, I must have typed `git stack --help`. I would have expected a syntax error message or "did you mean" suggestions; it didn't even enter my mind that it would look up whatever I typed before --help and assume it existed on disk. I'm sorry! On F

Re: `git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-12 Thread Lars Schneider
> On 12 Aug 2016, at 17:48, Junio C Hamano wrote: > > Joseph Musser writes: > >> Looks like a simple typo. > > Unfortunately this does not reproduce to me (built from source on > Ubuntu Linux). I tried it with the latest released version on Windows and OSX

Re: `git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-12 Thread Junio C Hamano
Joseph Musser writes: > Looks like a simple typo. Unfortunately this does not reproduce to me (built from source on Ubuntu Linux). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at

`git stash --help` tries to pull up nonexistent file gitstack.html

2016-08-11 Thread Joseph Musser
Looks like a simple typo. Joseph Musser -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

<    1   2   3   4   5   6   7   8   9   10   >