Hi Vincent, Vincent van Ravesteijn wrote: > On the other hand, in the > case without the --octopus parameter, it is necessary to supply a single commit > and a set. Moreover, in the case of --octopus merging, a single commit is also valid, > which is not the case otherwise. So to summarize: unlike plain merge-base, merge-base --octopus only requires at least one commit argument, and this patch updates the synopsis to reflect that. Good catch. > 2) Add a sentence to the discussion that when --octopus is used, we do expect '2' > as the result. Adding an example use of --octopus to the DISCUSSION section makes a lot of sense. At first I thought this was being added to the DESCRIPTION section, which led me to suggest something like the following: DESCRIPTION ----------- 'git merge-base' finds best common ancestors between two commits to use in a three-way merge. [... etc etc ...] can be more than one merge base for a pair of commits. OPERATION MODE -------------- As the most common special case, specifying only two commits on the command line means computing the merge base between the given two commits. More generally, among the two commits to compute the merge base from, one is specified by the first commit argument on the command line; the other commit is a (possibly hypothetical) commit that is a merge across all the remaining commits on the command line. As a consequence, the merge base is not necessarily contained in each of the commit arguments if more than two commits are specified. This is different from git-show-branch(1) when used with the --merge-base option. --octopus:: Compute the best common ancestors of all supplied commits, in preparation for an n-way merge. This mimics the behavior of 'git show-branch --merge-base'. --independent:: Instead of ... OPTIONS ------- -a:: --all:: ... DISCUSSION ---------- What do you think? > --- a/Documentation/git-merge-base.txt > +++ b/Documentation/git-merge-base.txt > @@ -9,7 +9,8 @@ git-merge-base - Find as good common ancestors as possible for a merge [...] > +When the option `--octopus` is given, the result of `git merge-base A B C` > +will be '2', because '2' is the common ancestor of all branches. > + For what it's worth, except for Junio's wording nitpick and the log message, Reviewed-by: Jonathan Nieder <jrnieder@xxxxxxxxx> Thanks. -- 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