Re: [PATCH 2/2] Document "git log --simplify-by-decoration"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 11, 2008 at 10:47 AM, Nanako Shiraishi <nanako3@xxxxxxxxxxx> wrote:
> Quoting "Santi Béjar" <santi@xxxxxxxxxxx>:
>
>>> +The '\--simplify-by-decoration' option can be used to treat commits that
>>> +are not referenced by tags as TREESAME, and treat commits that are tagged
>>> +as !TREESAME (even when they have exactly the same tree as their parents).
>>> +This can be used when you are only interested in the big picture of the
>>> +topology of the history.
>>> +
>>> +
>>
>> I prefer the other way around, first what it does, and then how it
>> does it (but it is a general comment about the help in "History
>> Simplification", at least when viewing the help for "git log").
>
> Thank you for your comments.
>
> In the earlier part of the description, TREESAME (or !TREESAME) is explained as a prerequisite concept for understanding how the history simplification works.  I think my description first talks about what it does using the concept that was already explained (in other words, "which commits are marked as TREESAME"), and then talks about what the consequence of what it does is (in other words, "only shows the big picture").
>
> I can swap them around.  Let's see if you like this better:
>
>        The '\--simplify-by-decoration' option allows you to view only the
>        big picture of the topology of the history, by omitting commits
>        that are not referenced by tags.  Commits are marked as !TREESAME
>        (in other words, kept after history simplification rules described
>        above) if (1) they are referenced by tags, or (2) they change the
>        contents of the paths given on the command line.  All other
>        commits are marked as TREESAME (subject to be simplified away).
>

Better. I think it also makes sense to present the options at the
beginning of the section, explain what they do, and then explain them
in detail. I'll try to do it.

>> And you should rewrite the line 416:
>>
>> Finally, there is a fourth simplification mode available
>>
>> as it is no longer "Finally".
>
> Yes, I thought about it, but describing the new option at the end does not change the fact that the fourth simplification mode is the final one.

I take "Finally" as the last thing is explained in the section.

>
> The new option is not about a "simplification mode".  Given a set of commits marked as TREESAME and !TREESAME, simplification modes decide how the history is simplified.  The new option does not add any new mode. Instead, it affects which commits are marked as TREESAME/!TREESAME.

Make sense.

Santi
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux