Code

bisect: error out when passing bad path parameters
authorChristian Couder <chriscool@tuxfamily.org>
Sun, 28 Feb 2010 22:19:09 +0000 (23:19 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 1 Mar 2010 09:04:35 +0000 (01:04 -0800)
commit8f69f72fca989b3ba080aab1dc24bfd1709ce83a
treec8527e2c9fb724628b237ad9827bdba3e02a5fa3
parent003c6abdb27c367747847a76b0a7890d67c794be
bisect: error out when passing bad path parameters

As reported by Mark Lodato, "git bisect", when it was started with
path parameters that match no commit was kind of working without
taking account of path parameters and was reporting something like:

Bisecting: -1 revisions left to test after this (roughly 0 steps)

It is more correct and safer to just error out in this case, before
displaying the revisions left, so this patch does just that.

Note that this bug is very old, it exists at least since v1.5.5.
And it is possible to detect that case earlier in the bisect
algorithm, but it is not clear that it would be an improvement to
error out earlier, on the contrary it may change the behavior of
"git rev-list --bisect-all" for example, which is currently correct.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
bisect.c
t/t6030-bisect-porcelain.sh