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

[SCM] Debian package checker branch, master, updated. 2.2.7-5-g83dc423



The following commit has been merged in the master branch:
commit d211885b3c8e8999bbbbe2d6497de3e6ffb3bec0
Author: Russ Allbery <rra@debian.org>
Date:   Mon Mar 9 21:11:31 2009 -0700

    Updates for the forthcoming Policy 3.8.1
    
    * checks/changelog-file.desc:
      + [RA] Using UTF-8 is now a Policy must and hence serious.
    * checks/control-file.desc:
      + [RA] Using UTF-8 is now a Policy must and hence serious.
    * checks/scripts:
      + [RA] Policy allows scripts to assume that /bin/sh supports local
        with simple variable assignments.

diff --git a/checks/changelog-file.desc b/checks/changelog-file.desc
index 82222c8..3aee52a 100644
--- a/checks/changelog-file.desc
+++ b/checks/changelog-file.desc
@@ -158,8 +158,9 @@ Info: The changelog file contains an invalid email address: the domain
  needs at least one dot. This looks like a mistake.
 
 Tag: debian-changelog-file-uses-obsolete-national-encoding
-Severity: important
+Severity: serious
 Certainty: certain
+Ref: policy 4.4
 Info: The Debian changelog file must be valid UTF-8, an encoding of
  the Unicode character set.
  .
diff --git a/checks/control-file.desc b/checks/control-file.desc
index 1cddd68..cd15ba1 100644
--- a/checks/control-file.desc
+++ b/checks/control-file.desc
@@ -7,8 +7,9 @@ Needs-Info: debfiles
 Info: This script checks debian/control files in source packages
 
 Tag: debian-control-file-uses-obsolete-national-encoding
-Severity: important
+Severity: serious
 Certainty: certain
+Ref: policy 5.1
 Info: The Debian control file should be valid UTF-8, an encoding of
  the Unicode character set.
  .
diff --git a/checks/scripts b/checks/scripts
index 8f89194..08d9b3b 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -258,7 +258,6 @@ our @bashism_regexs = (
     $LEADIN . qr'alias\s+-p',      # alias -p
     $LEADIN . qr'unalias\s+-a',    # unalias -a
     $LEADIN . qr'local\s+-[a-zA-Z]+', # local -opt
-    $LEADIN . qr'local\s+\w+=',    # local foo=bar
     qr'(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*\(\)\s*([\{|\(]|\Z)',
 	# function names should only contain [a-z0-9_]
     $LEADIN . qr'(push|pop)d(\s|\Z)',   # (push|pod)d
diff --git a/debian/changelog b/debian/changelog
index 8320d6e..1abf391 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,10 +1,17 @@
 lintian (2.2.8) UNRELEASED; urgency=low
 
+  * checks/changelog-file.desc:
+    + [RA] Using UTF-8 is now a Policy must and hence serious.
+  * checks/control-file.desc:
+    + [RA] Using UTF-8 is now a Policy must and hence serious.
   * checks/menus{,.desc}:
     + [RA] Revert the change to not require update-menus in postinst.
       This is still needed due to #518919.  Don't require update-menus in
       postrm for regular menu files, only for menu-methods, but also don't
       warn if it's present.
+  * checks/scripts:
+    + [RA] Policy allows scripts to assume that /bin/sh supports local
+      with simple variable assignments.
 
  -- Russ Allbery <rra@debian.org>  Mon, 09 Mar 2009 20:31:24 -0700
 
diff --git a/testset/tags.maintainer-scripts b/testset/tags.maintainer-scripts
index 385d19a..bfbcca7 100644
--- a/testset/tags.maintainer-scripts
+++ b/testset/tags.maintainer-scripts
@@ -101,7 +101,6 @@ W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:134 'time '
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:135 'dirs'
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:136 ' <(tac a)'
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:138 'pushd'
-W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:140 'local foo='
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:141 'local -x'
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:143 'popd'
 W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:145 'readonly -f'

-- 
Debian package checker


Reply to: