From: Johannes Sixt <j.sixt@viscovery.net>
Date: Mon, 03 Dec 2007 07:33:27 GMT
Subject: Re: make test failure with latest master
Message-ID: <4753B147.2050404@viscovery.net>
In-Reply-To: <20071203015954.GB8322@coredump.intra.peff.net>

Jeff King schrieb:
> -if ! ( type cvs && type cvsps ) >/dev/null 2>&1
> +if ! type cvs >/dev/null 2>&1
>  then
> -	test_expect_success 'skipping cvsimport tests, cvs/cvsps not found' ''
> +	test_expect_success 'skipping cvsimport tests, cvs not found' ''

FWIW, the idiom to provide informative messages in test cases is

	say 'skipping cvsimport tests, cvs not found'

This gives a distinguishing color, too.

-- Hannes