All of lore.kernel.org
 help / color / mirror / Atom feed
* Where man git-format-patch sends me?
@ 2007-11-01 13:58 Sergei Organov
  2007-11-01 14:38 ` Johannes Schindelin
  0 siblings, 1 reply; 5+ messages in thread
From: Sergei Organov @ 2007-11-01 13:58 UTC (permalink / raw)
  To: git

$ man git-format-patch
[...]
OPTIONS
       -p     Generate patch (see section on generating patches)

       -u     Synonym for "-p".
[...]
$

1. Saying "-p generates patch" suggests that without -p git-format-patch
   will generate something else. It's rather confusing.

2. Where is this "section on generating patches"? I thought it could be
   in man git-diff, but it is not.

-- 
Sergei.

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

* Re: Where man git-format-patch sends me?
  2007-11-01 13:58 Where man git-format-patch sends me? Sergei Organov
@ 2007-11-01 14:38 ` Johannes Schindelin
  2007-11-01 15:47   ` Sergei Organov
  0 siblings, 1 reply; 5+ messages in thread
From: Johannes Schindelin @ 2007-11-01 14:38 UTC (permalink / raw)
  To: Sergei Organov; +Cc: git

Hi,

On Thu, 1 Nov 2007, Sergei Organov wrote:

> $ man git-format-patch
> [...]
> OPTIONS
>        -p     Generate patch (see section on generating patches)
> 
>        -u     Synonym for "-p".

As you can easily see from Documentation/git-format-patch.txt, this part 
is generated from the file Documentation/diff-options.txt.

Hth,
Dscho

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

* Re: Where man git-format-patch sends me?
  2007-11-01 14:38 ` Johannes Schindelin
@ 2007-11-01 15:47   ` Sergei Organov
  2007-11-02 10:03     ` Wincent Colaiuta
  0 siblings, 1 reply; 5+ messages in thread
From: Sergei Organov @ 2007-11-01 15:47 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> Hi,
>
> On Thu, 1 Nov 2007, Sergei Organov wrote:
>
>> $ man git-format-patch
>> [...]
>> OPTIONS
>>        -p     Generate patch (see section on generating patches)
>> 
>>        -u     Synonym for "-p".
>
> As you can easily see from Documentation/git-format-patch.txt, this part 
> is generated from the file Documentation/diff-options.txt.

Sorry, I did saw that, but it doesn't change anything.

1. As a user reading man git-format-patch, where do I find this "section
   on generating patches"? In another man-page? In the html
   documentation? On the moon?

2. I can't find "section on generating patches" in the man git-diff
   either, and I did say this in the part of my original message you've
   snipped.

So the main question remains: could you please point me to the exact
place in 'Documentation/' directory where this "section on generating
patches" resides?

-- 
Sergei.

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

* Re: Where man git-format-patch sends me?
  2007-11-01 15:47   ` Sergei Organov
@ 2007-11-02 10:03     ` Wincent Colaiuta
  2007-11-02 11:25       ` Sergei Organov
  0 siblings, 1 reply; 5+ messages in thread
From: Wincent Colaiuta @ 2007-11-02 10:03 UTC (permalink / raw)
  To: Sergei Organov; +Cc: Johannes Schindelin, git

El 1/11/2007, a las 16:47, Sergei Organov escribió:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>
>> Hi,
>>
>> On Thu, 1 Nov 2007, Sergei Organov wrote:
>>
>>> $ man git-format-patch
>>> [...]
>>> OPTIONS
>>>       -p     Generate patch (see section on generating patches)
>>>
>>>       -u     Synonym for "-p".
>>
>> As you can easily see from Documentation/git-format-patch.txt, this  
>> part
>> is generated from the file Documentation/diff-options.txt.
>
> Sorry, I did saw that, but it doesn't change anything.
>
> 1. As a user reading man git-format-patch, where do I find this  
> "section
>   on generating patches"? In another man-page? In the html
>   documentation? On the moon?
>
> 2. I can't find "section on generating patches" in the man git-diff
>   either, and I did say this in the part of my original message you've
>   snipped.
>
> So the main question remains: could you please point me to the exact
> place in 'Documentation/' directory where this "section on generating
> patches" resides?

I believe the section in question is in Documentation/diff-format.txt

But you've basically found a documentation bug. Someone invoking "man  
git-format-patch" shouldn't be expected to have to grep the source  
tree in order to find out where the referenced documentation is.  
Especially given that he/she may not even have the source.

Cheers,
Wincent

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

* Re: Where man git-format-patch sends me?
  2007-11-02 10:03     ` Wincent Colaiuta
@ 2007-11-02 11:25       ` Sergei Organov
  0 siblings, 0 replies; 5+ messages in thread
From: Sergei Organov @ 2007-11-02 11:25 UTC (permalink / raw)
  To: Wincent Colaiuta; +Cc: Johannes Schindelin, git

Wincent Colaiuta <win@wincent.com> writes:

> El 1/11/2007, a las 16:47, Sergei Organov escribió:
>
>> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>>
>>> Hi,
>>>
>>> On Thu, 1 Nov 2007, Sergei Organov wrote:
>>>
>>>> $ man git-format-patch
>>>> [...]
>>>> OPTIONS
>>>>       -p     Generate patch (see section on generating patches)
>>>>
>>>>       -u     Synonym for "-p".
>>>
>>> As you can easily see from Documentation/git-format-patch.txt, this  
>>> part
>>> is generated from the file Documentation/diff-options.txt.
>>
>> Sorry, I did saw that, but it doesn't change anything.
>>
>> 1. As a user reading man git-format-patch, where do I find this  
>> "section
>>   on generating patches"? In another man-page? In the html
>>   documentation? On the moon?
>>
>> 2. I can't find "section on generating patches" in the man git-diff
>>   either, and I did say this in the part of my original message you've
>>   snipped.
>>
>> So the main question remains: could you please point me to the exact
>> place in 'Documentation/' directory where this "section on generating
>> patches" resides?
>
> I believe the section in question is in Documentation/diff-format.txt

Ah, thank you! Now, diff-format.txt is included by
git-diff-index/git-diff-files/git-diff-tree, but not by git-diff and
git-format-patch. Do you think it is the right thing for the latter two
to include diff-format as well?

-- 
Sergei.

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

end of thread, other threads:[~2007-11-02 11:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-11-01 13:58 Where man git-format-patch sends me? Sergei Organov
2007-11-01 14:38 ` Johannes Schindelin
2007-11-01 15:47   ` Sergei Organov
2007-11-02 10:03     ` Wincent Colaiuta
2007-11-02 11:25       ` Sergei Organov

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.