Hi Junio,

I briefly looked at d84a3da (jc/graph-post-root-gap) in pu, and have
this nit:

> diff --git a/t/t6016-rev-list-graph-simplify-history.sh 
> b/t/t6016-rev-list-graph-simplify-history.sh
> [...]
> +one_independent_branch () {
> +     git checkout --orphan root$1 A1 &&
> +     test_commit root_$1 &&

The naming of root0 etc. makes the test below rather confusing to read,
because test_commit root_0 also creates a tag called root_0.  So you set
up history that has a tag root_0 that points *only* at the root, and a
branch root0 that includes two more commits.

> +test_expect_failure 'multi-root does show necessary post-root gap' '
> +     sed -e "s/ #$/ /" >expect <<-\EOF &&
> +     * further_2
> +     * then_2
> +     * root_2
> +       * further_1
> +       * then_1
> +       * root_1
> +     * further_0
> +     * then_0
> +     * root_0
> +     EOF
> +     git log --graph --format=%s root0 root1 root2 >actual &&
> +     test_cmp expect actual
> +'

-- 
Thomas Rast
t...@thomasrast.ch
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to