[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

lintian: r1350 - in trunk: checks debian testset



Author: rra
Date: 2008-06-07 02:16:47 +0200 (Sat, 07 Jun 2008)
New Revision: 1350

Modified:
   trunk/checks/scripts
   trunk/debian/changelog
   trunk/testset/tags.maintainer-scripts
Log:
  + [RA] $((var+1)) now works in dash and is in POSIX, so remove it from
    bashisms.  Patch by Adam D. Barratt.  (Closes: #473156)

Modified: trunk/checks/scripts
===================================================================
--- trunk/checks/scripts	2008-06-07 00:14:01 UTC (rev 1349)
+++ trunk/checks/scripts	2008-06-07 00:16:47 UTC (rev 1350)
@@ -571,7 +571,6 @@
 		  '(<\&|>\&)\s*((-|\d+)[^\s;|\)\`&]|[^-\d])', # should be >word 2>&1
 		  '\[\[(?!:)',		       # alternative test command
 		  '(?:^|\s+)select\s+\w+',     # 'select' is not POSIX
-		  '\$\(\([A-Za-z]',	       # cnt=$((cnt + 1)) does not work in dash
 		  '(?:^|\s+)echo\s+-e',        # echo -e
 		  '(?:^|\s+)exec\s+-[acl]',    # exec -c/-l/-a name
 		  '(?:^|\s+)let\s',	       # let ...

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2008-06-07 00:14:01 UTC (rev 1349)
+++ trunk/debian/changelog	2008-06-07 00:16:47 UTC (rev 1350)
@@ -53,6 +53,8 @@
   * checks/scripts:
     + [RA] Catch deprecated chown syntax even when options are given to
       chown.  Patch by Raphael Geissert.  (Closes: #483775)
+    + [RA] $((var+1)) now works in dash and is in POSIX, so remove it from
+      bashisms.  Patch by Adam D. Barratt.  (Closes: #473156)
   * checks/standards-version:
     + [RA] Policy 3.8.0 is now current and anything older than 3.7.2 is
       now ancient.

Modified: trunk/testset/tags.maintainer-scripts
===================================================================
--- trunk/testset/tags.maintainer-scripts	2008-06-07 00:14:01 UTC (rev 1349)
+++ trunk/testset/tags.maintainer-scripts	2008-06-07 00:16:47 UTC (rev 1350)
@@ -53,7 +53,6 @@
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:116 'echo -e'
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:117 '${!foo}'
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:119 'select foo'
-W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:120 '$((c'
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:122 '    exec -l'
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:123 '    exec -c'
 W: maintainer-scripts: possible-bashism-in-maintainer-script postinst:124 '    exec -a'


Reply to: