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

[lintian] 01/01: c/scripts.desc: Demote certainity of shell syntax check



This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit 297fd8aa2eb7436a9a51f28bf43b90d8caa02415
Author: Niels Thykier <niels@thykier.net>
Date:   Sat Sep 21 21:04:07 2013 +0200

    c/scripts.desc: Demote certainity of shell syntax check
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 checks/scripts.desc |   10 ++++++++--
 debian/changelog    |    4 ++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/checks/scripts.desc b/checks/scripts.desc
index 66b2b66..f8f30f6 100644
--- a/checks/scripts.desc
+++ b/checks/scripts.desc
@@ -298,23 +298,29 @@ Info: Packages that include tclsh scripts must depend on the virtual
 
 Tag: shell-script-fails-syntax-check
 Severity: important
-Certainty: certain
+Certainty: possible
 Info: Running this shell script with the shell's -n option set fails,
  which means that the script has syntax errors.  The most common cause of
  this problem is a script expecting <tt>/bin/sh</tt> to be bash checked on
  a system using dash as <tt>/bin/sh</tt>.
  .
  Run e.g. <tt>sh -n yourscript</tt> to see the errors yourself.
+ .
+ Note this can have false-positives, for an example with bash scripts
+ using "extglob".
 
 Tag: example-shell-script-fails-syntax-check
 Severity: pedantic
-Certainty: certain
+Certainty: possible
 Info: Running this shell script with the shell's -n option set fails,
  which means that the script has syntax errors.  The most common cause of
  this problem is a script expecting <tt>/bin/sh</tt> to be bash checked on
  a system using dash as <tt>/bin/sh</tt>.
  .
  Run e.g. <tt>sh -n yourscript</tt> to see the errors yourself.
+ .
+ Note this can have false-positives, for an example with bash scripts
+ using "extglob".
 
 Tag: maintainer-shell-script-fails-syntax-check
 Severity: serious
diff --git a/debian/changelog b/debian/changelog
index 569985e..0d567c1 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -47,6 +47,10 @@ lintian (2.5.18) UNRELEASED; urgency=low
     + [BR] Add check for missing for recommended "get-orig-source"
       target in d/rules for packages that look like they have been
       repacked.  (Closes: #460174)
+  * checks/scripts.desc:
+    + [NT] Demote the certainty of shell-script-fails-syntax-check
+      due to "extglob" in bash.  Thanks To Trent W. Buck for the
+      report.  (Closes: #699410)
   * checks/testsuite.pm:
     + [NT] Add "allow-stderr" as known autopkgtest "restriction".
       Thanks to Jérémy Bobbio for the report and the initial

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: