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'
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
