Author: adam-guest
Date: 2008-05-11 11:17:41 +0000 (Sun, 11 May 2008)
New Revision: 1438
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
Make the check for "process substitution" ignore quoted strings
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-05-11 11:15:11 UTC (rev 1437)
+++ trunk/debian/changelog 2008-05-11 11:17:41 UTC (rev 1438)
@@ -12,6 +12,7 @@
rather than all theoretically possible options, thus reducing false
positives
+ Further improve the heredoc detection (to allow the space in "<< 'foo'")
+ + Make the check for "process substitution" ignore quoted strings
* cowpoke: New script to upload a Debian source package to a cowbuilder host
and build it, optionally also signing and uploading the result to an
incoming queue (Closes: #479274)
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-05-11 11:15:11 UTC (rev 1437)
+++ trunk/scripts/checkbashisms.pl 2008-05-11 11:17:41 UTC (rev 1438)
@@ -420,6 +420,7 @@
$LEADIN . 'type\s' => q<type>,
$LEADIN . 'time\b' => q<time>,
$LEADIN . 'dirs\b' => q<dirs>,
+ '(?:^|\s+)[<>]\(.*?\)' => q<\<() process substituion>,
);
%string_bashisms = (
@@ -439,7 +440,6 @@
'\$\{?BASH_[A-Z]+\}?\b' => q<$BASH_SOMETHING>,
'\$\{?SHELLOPTS\}?\b' => q<$SHELLOPTS>,
'<<<' => q<\<\<\< here string>,
- '(?:^|\s+)[<>]\(.*?\)' => q<\<() process substituion>,
);
if ($opt_echo) {
--
To unsubscribe, send mail to [EMAIL PROTECTED]
