Thread (1 message) 1 message, 1 author, 2016-06-15

Re: Documentation/git-checkout.txt: Inconsistent naming of paths arguments

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:58:13

Duy Nguyen [off-list ref] writes:
On Fri, Jul 19, 2013 at 2:16 PM, Adam Brengesjö [off-list ref] wrote:
quoted
From SYNOPSIS:

git checkout [-p|--patch] [<tree-ish>] [--] [<paths>…]


From DESCRIPTION

git checkout [-p|--patch] [<tree-ish>] [--] <pathspec>…



1. Named <paths> in SYNOPSIS, but <pathspec> in DESCRIPTION. (It's
referred to as <path> in the body text).

2. <paths> is marked as optional in SYNOPSIS, but <pathspec> is not.

I'm not submitting a patch now, as I'm not sure which is correct.
If I'm not mistaken, "git checkout" takes pathspec in all cases.
Correct.

And I think -p form (which I do not use myself so please double
check) can be run with an empty pathspec.

It looks somewhat idiotic that

	git checkout master
        git checout -p next

will finish on the 'master' branch, with random selected bits of
differences for 'master' to go to 'next' applied to the index and
the working tree, but that seems to be how it works.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help