lintian: r800 - in trunk: checks debian
Author: rra
Date: 2006-12-03 23:16:51 +0100 (Sun, 03 Dec 2006)
New Revision: 800
Modified:
trunk/checks/scripts.desc
trunk/debian/changelog
Log:
* checks/scripts.desc:
+ [RA] Mention SUSv3 in the bashism and non-posix tag descriptions,
note that use of test -a/-o and local aren't bugs but may still be
worth avoiding, and mention short circuiting of && and ||. (Thanks
to Jari Aalto for the latter).
Modified: trunk/checks/scripts.desc
===================================================================
--- trunk/checks/scripts.desc 2006-12-03 19:13:37 UTC (rev 799)
+++ trunk/checks/scripts.desc 2006-12-03 22:16:51 UTC (rev 800)
@@ -245,7 +245,8 @@
Type: warning
Ref: policy 10.4
Info: This script is marked as running under <tt>/bin/sh</tt>, but it seems
- to use a feature found in bash but not in the POSIX shell specification.
+ to use a feature found in bash but not in the SUSv3 or POSIX shell
+ specification.
.
Examples:
'==' in a test, it should use '=' instead
@@ -258,16 +259,14 @@
Tag: possible-non-posix-code-in-maintainer-script
Type: info
-Ref: policy 10.4
Info: This script is marked as running under <tt>/bin/sh</tt>, but it seems
- to use a shell feature that is not defined in the POSIX standard. The
- feature is so common in current shells that this isn't currently
- considered an release critical bug and there are discussion wether
- policy should be amended to specifically allow the use of the feature.
+ to use a shell feature that is not defined in the SUSv3 or POSIX standard.
+ The feature is so common in current shells that this isn't necessarily a
+ bug, but you may wish to avoid this anyway for maximum portability.
.
Examples:
'-a' and '-o' as operators of 'test'. You can use shell conditionals
- (&& and ||) instead.
+ (&& and ||) instead, which also short-circuit correctly.
'local' scoping of variables.
Tag: suidregister-used-in-maintainer-script
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2006-12-03 19:13:37 UTC (rev 799)
+++ trunk/debian/changelog 2006-12-03 22:16:51 UTC (rev 800)
@@ -1,3 +1,13 @@
+lintian (1.23.28) UNRELEASED; urgency=low
+
+ * checks/scripts.desc:
+ + [RA] Mention SUSv3 in the bashism and non-posix tag descriptions,
+ note that use of test -a/-o and local aren't bugs but may still be
+ worth avoiding, and mention short circuiting of && and ||. (Thanks
+ to Jari Aalto for the latter).
+
+ -- Russ Allbery <rra@debian.org> Sun, 3 Dec 2006 14:16:40 -0800
+
lintian (1.23.27) unstable; urgency=low
The "we haven't run this against the whole archive in a while" release
Reply to: