Author: adam-guest
Date: 2008-05-14 18:06:12 +0000 (Wed, 14 May 2008)
New Revision: 1446
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
Fix false positives in the test for "test with unary -a"
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-05-12 22:38:03 UTC (rev 1445)
+++ trunk/debian/changelog 2008-05-14 18:06:12 UTC (rev 1446)
@@ -15,7 +15,7 @@
positives
+ Further improve the heredoc detection (to allow the space in "<< 'foo'")
+ Make the check for "process substitution" ignore quoted strings
- + Don't flag "time)" as matching "time"
+ + Fix false positives in the tests for "time" and "test with unary -a"
+ Apply various improvements to "quoted block" detection
+ Correctly match 'echo "$(< $0)"'
+ Add a check for "$[" (a deprecated form of arithmetic expansion)
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-05-12 22:38:03 UTC (rev 1445)
+++ trunk/scripts/checkbashisms.pl 2008-05-14 18:06:12 UTC (rev 1446)
@@ -387,7 +387,7 @@
$LEADIN . 'let\s' => q<let ...>,
'(?<![\$\(])\(\(.*\)\)' => q<'((' should be '$(('>,
'\$\[' => q<'$[' should be '$(('>,
- '(\[|test)\s+-a' => q<test with unary -a (should be -e)>,
+ '(?:^|\s+)(\[|test)\s+-a' => q<test with unary -a (should be
-e)>,
'\&>' => q<should be \>word 2\>&1>,
'(<\&|>\&)\s*((-|\d+)[^\s;|)`&\\\\]|[^-\d\s])' =>
q<should be \>word 2\>&1>,
--
To unsubscribe, send mail to [EMAIL PROTECTED]
