Thread (8 messages) flat view 8 messages, 3 authors, 2016-06-15

Re: [PATCH 1/3] test-lib: Add support for multiple test prerequisites

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:49:16

Jonathan Nieder wrote:
quoted hunk ↗ jump to hunk
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -338,7 +338,7 @@ test_have_prereq () {
 		*" $prerequisite "*)
 			: yes, have it ;;
 		*)
-			! : nope ;;
+			return 1
The ";;" is optional for the last arm in a case statement, but
it is probably simpler to include it.  Sorry about that.

(The only intended effect was only to use “return” that terminates and
propagates out of the loop instead of “false” that is overridden by a
later “true”.)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help