Re: [PATCH v2] Teach git-rev-list --simplify-forks
- From
- Antonio Russo <antonio.e.russo@gmail.com>
- Date
- Apr 29, 2020, 13:04 UTC
- Message-ID
- <ceeb706d-83ad-c994-121d-d66724d82a49@gmail.com>
- In-Reply-To
- <eda71f1e-b883-82c3-72f2-dc5221804c4b@iee.email>
On 4/29/20 4:08 AM, Philip Oakley wrote:
Show 17 quoted lines
>> +--simplify-forks:: >> + Convert the commit graph into a spanning subgraph produced by a >> + depth-first-search of the history graph, searching the leftmost >> + parent first, and discarding edges to commits already visited. >> + Useful with `--graph` to visualize repositories with many merges >> + when you are interested in was added to master, and not when the >> + branch was last rebased. > > Does this also need to actually mention that it effectively discard > edges to fork points, as per the option name?. No rebasing required for > it to be useful. > s/when/where/ > "...and not where the branch was last rebased or forked from." - not > great but actually mentions the option. > > -- > Philip
Good point. More generally, my description of the option is not very useful unless you already basically understand what it's doing.
How about:
Show 9 quoted lines
> --simplify-forks:: > Omits the parent relationship of the first commit of merged branches. > Effectively discards all information about the fork point of merged > branches. It does so by converting the commit graph into a > spanning subgraph produced by a depth-first-search of the history > graph, searching the leftmost parent first, and discarding edges > to commits already visited. Useful with `--graph` to visualize > repositories with many merges when you are most interested in > when branches were merged.
Antonio