git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] test-lint-shell-syntax: Allow bash syntax in bash tests

From
Torsten Bögershausen <tboegi@web.de>
Date
Apr 25, 2013, 18:48 UTC
Message-ID
<201304252048.33803.tboegi@web.de>

t/check-non-portable-sh.pl checks for bash only syntax like test $a == $b This is wrong when bash is used. Switch to bash mode and skip this test

Signed-off-by: Torsten Bögershausen <tboegi@web.de>
---
 t/check-non-portable-shell.pl | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl
index 58f8e2c..1ca8c8b 100755
--- a/t/check-non-portable-shell.pl
+++ b/t/check-non-portable-shell.pl
@@ -7,23 +7,26 @@ use strict;
 use warnings;
 
 my $exit_code=0;
+my $bashmode=0;
 
 sub err {
 	my $msg = shift;
-	print "$ARGV:$.: error: $msg: $_\n";
+	print "$ARGV:$.: error: (bashmode=$bashmode) $msg: $_\n";
 	$exit_code = 1;
 }
 
 while (<>) {
 	chomp;
+	/^\. \.\/lib\-bash.sh/ and $bashmode=1;
 	/ e?grep\s+\"[^"]*\\([st])/ and err "grep \\$1 is not portable)";
 	/ [^e]grep.*\\\+/ and err 'grep with \+ is not portable (please use egrep)';
 	/^\s*sed\s+-i/ and err 'sed -i is not portable';
 	/^\s*echo\s+-(\S)\s+/ and err "echo -$1 is not portable (please use printf)";
 	/^\s*declare\s+/ and err 'arrays/declare not portable';
 	/^\s*[^#]\s*which\s/ and err 'which is not portable (please use type)';
-	/test\s+[^=]*==/ and err '"test a == b" is not portable (please use =)';
+	$bashmode==0 and /test\s+[^=]*==/ and err '"test a == b" is not portable (please use =)';
 	# this resets our $. for each file
 	close ARGV if eof;
+	$bashmode=0 if eof;
 }
 exit $exit_code;
-- 
1.8.2.1.614.g66d7af5
Message 1 of 1 in “test-lint-shell-syntax: Allow bash syntax in bash tests”
  1. test-lint-shell-syntax: Allow bash syntax in bash testsTorsten Bögershausen, Apr 25, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.