Merge branch 'jc/test-say-color-avoid-echo-escape'

peff · peff · commit 4cd31a6320d2 · 2012-10-25T06:42:49.000-04:00
Recent nd/wildmatch series was the first to reveal this ancient bug
in the test scaffolding.

* jc/test-say-color-avoid-echo-escape:
  test-lib: Fix say_color () not to interpret \a\b\c in the message
diff --git a/t/test-lib.sh b/t/test-lib.sh
@@ -230,7 +230,7 @@ else
 	say_color() {
 		test -z "$1" && test -n "$quiet" && return
 		shift
-		echo "$*"
+		printf "%s\n" "$*"
 	}
 fi
 

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ else`
`230`	`230`	`say_color() {`
`231`	`231`	`test -z "$1" && test -n "$quiet" && return`
`232`	`232`	`shift`
`233`		`- echo "$*"`
	`233`	`+ printf "%s\n" "$*"`
`234`	`234`	`}`
`235`	`235`	`fi`
`236`	`236`