Re: [PATCH] rev-list: Demonstrate breakage with --ancestry-path --all

Junio C Hamano [off-list ref] writes:

Brad King [off-list ref] writes:

quoted

The option added by commit ebdc94f3 (revision: --ancestry-path,
2010-04-20) does not work properly in combination with --all, at least
in the case of a criss-cross merge:

    b---bc
   / \ /
  a   X
   \ / \
    c---cb

Hmm, what should --ancestry-path do given more than one positive commit to
begin with, let alone --all?

I actually think that this does not have much to do with "criss-cross"-ness. 
Instead of computing those that can be reached from cb, we are computing
those that can be reached from either b, c or cb.

This needs fixing, but it takes a bit more than a quick hack. Stay tuned
;-)

Thanks.