bisect: check for mandatory argument of 'bisect replay'
authorSZEDER Gábor <szeder@ira.uka.de>
Sun, 10 Oct 2010 21:48:58 +0000 (23:48 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 Oct 2010 22:07:50 +0000 (15:07 -0700)
'git bisect replay' has a mandatory logfile argument, but the current
implementation doesn't check whether the user has specified one.  When
the user omits the logfile argument, this leads to the following
unhelpful error message:

  cannot read  for replaying

So, check for the mandatory argument first, and provide a more
meaningful error message when it is omitted.

Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-bisect.sh

index 68fcff6dbf0247e2add6524667aa55f7c6beb669..c21e33c8d133af0e9e0ae3edaffd0a5593ff4ac3 100755 (executable)
@@ -343,6 +343,7 @@ bisect_clean_state() {
 }
 
 bisect_replay () {
+       test "$#" -eq 1 || die "No logfile given"
        test -r "$1" || die "cannot read $1 for replaying"
        bisect_reset
        while read git bisect command rev