Re: [PATCH v2 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable
On Sat, Dec 16, 2017 at 2:52 PM,wrote: > [...] > Add a check in check-non-portable-shell.pl to find '"' between > `wc -l` and '=' and hint the user about test_line_count(). > > Reviewed-by: Johannes Schindelin > Signed-off-by: Torsten Bögershausen > --- > diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl > index 03dc9d2852..fadbb1e5a7 100755 > --- a/t/check-non-portable-shell.pl > +++ b/t/check-non-portable-shell.pl > @@ -21,6 +21,7 @@ while (<>) { > /^\s*declare\s+/ and err 'arrays/declare not portable'; > /^\s*[^#]\s*which\s/ and err 'which is not portable (please use > type)'; > /\btest\s+[^=]*==/ and err '"test a == b" is not portable (please use > =)'; > + /\bwc -l.*"\s*=/ and err '`"$(wc -l)"` is not portable, please use > test_line_count'; Nit: Every other "please use" suggestion is parenthesized; for consistency, this probably ought to be, as well. > /\bexport\s+[A-Za-z0-9_]*=/ and err '"export FOO=bar" is not portable > (please use FOO=bar && export FOO)';
[PATCH v2 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable
From: Torsten Bögershausenwc -l was used to count the number if lines in test scripts. $ wc -l Makefile gives a line like this: 105 Makefile while Mac OS has 4 leading spaces: 105 Makefile And this means that shell expressions like test "$(wc -l Signed-off-by: Torsten Bögershausen --- I added Dscho as reviewer, thanks. If there is anything more, please holler. t/check-non-portable-shell.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl index 03dc9d2852..fadbb1e5a7 100755 --- a/t/check-non-portable-shell.pl +++ b/t/check-non-portable-shell.pl @@ -21,6 +21,7 @@ while (<>) { /^\s*declare\s+/ and err 'arrays/declare not portable'; /^\s*[^#]\s*which\s/ and err 'which is not portable (please use type)'; /\btest\s+[^=]*==/ and err '"test a == b" is not portable (please use =)'; + /\bwc -l.*"\s*=/ and err '`"$(wc -l)"` is not portable, please use test_line_count'; /\bexport\s+[A-Za-z0-9_]*=/ and err '"export FOO=bar" is not portable (please use FOO=bar && export FOO)'; # this resets our $. for each file close ARGV if eof; -- 2.15.1.271.g1a4e40aa5d