All of lore.kernel.org
 help / color / mirror / Atom feed
* Documentation/git-checkout.txt: Inconsistent naming of paths arguments
@ 2013-07-19  7:16 Adam Brengesjö
  2013-07-19  7:40 ` Duy Nguyen
  0 siblings, 1 reply; 3+ messages in thread
From: Adam Brengesjö @ 2013-07-19  7:16 UTC (permalink / raw)
  To: git

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.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Documentation/git-checkout.txt: Inconsistent naming of paths arguments
  2013-07-19  7:16 Documentation/git-checkout.txt: Inconsistent naming of paths arguments Adam Brengesjö
@ 2013-07-19  7:40 ` Duy Nguyen
  2013-07-19 14:36   ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Duy Nguyen @ 2013-07-19  7:40 UTC (permalink / raw)
  To: Adam Brengesjö; +Cc: Git Mailing List

On Fri, Jul 19, 2013 at 2:16 PM, Adam Brengesjö <ca.brengesjo@gmail.com> wrote:
> 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.
--
Duy

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Documentation/git-checkout.txt: Inconsistent naming of paths arguments
  2013-07-19  7:40 ` Duy Nguyen
@ 2013-07-19 14:36   ` Junio C Hamano
  0 siblings, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2013-07-19 14:36 UTC (permalink / raw)
  To: Duy Nguyen; +Cc: Adam Brengesjö, Git Mailing List

Duy Nguyen <pclouds@gmail.com> writes:

> On Fri, Jul 19, 2013 at 2:16 PM, Adam Brengesjö <ca.brengesjo@gmail.com> wrote:
>> 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.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-07-19 14:36 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-19  7:16 Documentation/git-checkout.txt: Inconsistent naming of paths arguments Adam Brengesjö
2013-07-19  7:40 ` Duy Nguyen
2013-07-19 14:36   ` Junio C Hamano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.