Author: adsb
Date: 2009-07-01 17:45:15 +0000 (Wed, 01 Jul 2009)
New Revision: 1919

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
checkbashisms: Detect the use of "source" when the sourced filename
contains a tilde (Closes: #528592) or consists of a single character
(Closes: #535252).  Thanks, Raphael Geissert and Ryan Niebur.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2009-06-25 20:40:39 UTC (rev 1918)
+++ trunk/debian/changelog      2009-07-01 17:45:15 UTC (rev 1919)
@@ -1,3 +1,11 @@
+devscripts (2.10.53) UNRELEASED; urgency=low
+
+  * checkbashisms: Detect the use of "source" when the sourced filename
+    contains a tilde (Closes: #528592) or consists of a single character
+    (Closes: #535252).  Thanks, Raphael Geissert and Ryan Niebur.
+
+ -- Adam D. Barratt <[email protected]>  Wed, 01 Jul 2009 18:43:43 +0100
+
 devscripts (2.10.52) unstable; urgency=low
 
   * debdiff: Make --diffstat work again; accidentally broken in 2.10.48.

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2009-06-25 20:40:39 UTC (rev 1918)
+++ trunk/scripts/checkbashisms.pl      2009-07-01 17:45:15 UTC (rev 1919)
@@ -540,7 +540,7 @@
 
     %singlequote_bashisms = (
        $LEADIN . qr'echo\s+(?:-[^e\s]+\s+)?\'[^\']*(\\[abcEfnrtv0])+.*?[\']' 
=> q<unsafe echo with backslash>,
-       $LEADIN . qr'source\s+[\"\']?(?:\.\/|\/|\$|[\w.-])[^\s]+' =>
+       $LEADIN . qr'source\s+[\"\']?(?:\.\/|\/|\$|[\w~.-])\S*(\s|\z)' =>
                                       q<should be '.', not 'source'>,
     );
 



-- 
To unsubscribe, send mail to [email protected].

Reply via email to