Author: adam-guest
Date: 2008-03-28 18:24:49 +0000 (Fri, 28 Mar 2008)
New Revision: 1215

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
  + Remove check for $((var+1)) as dash now supports the syntax

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-03-28 18:13:26 UTC (rev 1214)
+++ trunk/debian/changelog      2008-03-28 18:24:49 UTC (rev 1215)
@@ -13,6 +13,7 @@
     + Temporarily disable the check for "complete" as it's too prone to false
       positives
     + Add checks for pushd and popd
+    + Remove check for $((var+1)) as dash now supports the syntax
   * debcheckout:
     + Tighten the "repository owner" regex for darcs.d.o to correctly check
       http://darcs.d.o/~foo/bar/baz against user foo

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-03-28 18:13:26 UTC (rev 1214)
+++ trunk/scripts/checkbashisms.pl      2008-03-28 18:24:49 UTC (rev 1215)
@@ -152,7 +152,6 @@
                '(?:^|\s+)\w+\+=' =>           q<should be VAR="${VAR}foo">,
                '(?:^|\s+)\w+\[\d+\]=' =>      q<bash arrays, H[0]>,
                '(?:^|\s+)(read\s*(-[^r])?(?:;|$))' => q<should be read [-r] 
variable>,
-               '\$\(\([A-Za-z]' => q<cnt=$((cnt + 1)) does not work in dash>,
                '(?:^|\s+)echo\s+-[e]' =>      q<echo -e>,
                '(?:^|\s+)exec\s+-[acl]' =>    q<exec -c/-l/-a name>,
                '(?:^|\s+)let\s' =>            q<let ...>,



-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to