Author: adam-guest
Date: 2008-05-08 05:59:39 +0000 (Thu, 08 May 2008)
New Revision: 1435
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
Update the check for "echo -e" to only match -n as an extra option
rather than all theoretically possible options, thus reducing false
positives
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-05-06 22:53:24 UTC (rev 1434)
+++ trunk/debian/changelog 2008-05-08 05:59:39 UTC (rev 1435)
@@ -8,6 +8,9 @@
explicitly uses bash
+ Update the check for "function" to more closely match lintian's (and
therefore be less prone to false positives)
+ + Update the check for "echo -e" to only match -n as an extra option
+ rather than all theoretically possible options, thus reducing false
+ positives
* 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-06 22:53:24 UTC (rev 1434)
+++ trunk/scripts/checkbashisms.pl 2008-05-08 05:59:39 UTC (rev 1435)
@@ -382,7 +382,7 @@
q<brace expansion>,
'(?:^|\s+)\w+\[\d+\]=' => q<bash arrays, H[0]>,
$LEADIN . '(read\s*(-[^r])?(?:;|$))' => q<should be read [-r] variable>,
- $LEADIN . 'echo\s+-\w*e' => q<echo -e>,
+ $LEADIN . 'echo\s+-n?en?\s' => q<echo -e>,
$LEADIN . 'exec\s+-[acl]' => q<exec -c/-l/-a name>,
$LEADIN . 'let\s' => q<let ...>,
'(?<![\$\(])\(\(.*\)\)' => q<'((' should be '$(('>,
--
To unsubscribe, send mail to [EMAIL PROTECTED]
