diff mbox

[1/4] test-suite: remove bashism to avoid test failures

Message ID 54398B60.9080009@ramsay1.demon.co.uk (mailing list archive)
State Mainlined, archived
Headers show

Commit Message

Ramsay Jones Oct. 11, 2014, 7:56 p.m. UTC
The use of the '==' operator in a test/[ conditional is a non-POSIX
bash extension. In order to avoid test failures on systems that do
not have bash as the system shell (/bin/sh), replace the use of the
'==' operator with the POSIX compatible '=' operator.

Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
---

Hi Chris,

Sorry for not spotting this earlier; I just looked at your patch in
my email client and thought 'yep, that looks good', without actually
testing the change. :(

Since Linux Mint is a Ubuntu derivative my /bin/sh is actually dash.
(I hear Fedora are looking to change /bin/sh to dash as well. :-D ).

ATB,
Ramsay Jones

 validation/test-suite | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox

Patch

diff --git a/validation/test-suite b/validation/test-suite
index 7f0f83e..df5a7c6 100755
--- a/validation/test-suite
+++ b/validation/test-suite
@@ -114,7 +114,7 @@  do_test()
 	set -- $cmd
 	base_cmd=`basename $1`
 	for i in $disabled_cmds; do
-		if [ "$i" == "$base_cmd" ] ; then
+		if [ "$i" = "$base_cmd" ] ; then
 			disabled_tests=`expr $disabled_tests + 1`
 			echo "     DISABLE $test_name ($file)"
 			return 3