Sverre Rabbelier [off-list ref] writes:
On Wed, Jun 15, 2011 at 07:59, Junio C Hamano [off-list ref] wrote:
quoted
+perl -MCGI -MCGI::Util -MCGI::Carp -e 0 >/dev/null 2>&1 || {
+ skip_all='skipping gitweb tests, CGI module unusable'
+ test_done
+}
+
Isn't the new style to use test_must_succeed CGI_AVAILABLE '...' '...'
rather than skip_all+test_done?
I do not think that adds any value, at least in this case. Besides, look
at the context line above what you quoted ;-)