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

[lintian] 01/01: Also include the offending/unknown shebang in the output of missing (unversioned) interpreters.



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

lamby pushed a commit to branch master
in repository lintian.

commit 8385bfa725a324d8591343179e0207d72aec9511
Author: Chris Lamb <lamby@debian.org>
Date:   Fri Apr 6 11:45:20 2018 +0100

    Also include the offending/unknown shebang in the output of missing (unversioned) interpreters.
---
 checks/scripts.pm           | 2 +-
 debian/changelog            | 3 +++
 t/tests/legacy-scripts/tags | 2 +-
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/checks/scripts.pm b/checks/scripts.pm
index 1b5d96d..0d70e2a 100644
--- a/checks/scripts.pm
+++ b/checks/scripts.pm
@@ -502,7 +502,7 @@ sub run {
             $depends =~ s/\$1/$version/g;
             unless ($all_parsed->implies($depends)) {
                 if ($base =~ /^(python|ruby)/) {
-                    tag "$1-script-but-no-$1-dep", $filename;
+                    tag "$1-script-but-no-$1-dep", $filename,"#!$interpreter";
                 } else {
                     tag 'missing-dep-for-interpreter', "$base => $depends",
                       "($filename)", "#!$interpreter";
diff --git a/debian/changelog b/debian/changelog
index a88cd98..bb2bdda 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -31,6 +31,9 @@ lintian (2.5.81) UNRELEASED; urgency=medium
     + [CL] Stop recommending that packages with repacked tarballs specify a
       get-orig-source target; this was removed in Debian Policy 4.1.4 in
       favour of uscan(1) and debian/watch.
+  * checks/scripts.pm:
+    + [CL] Also include the offending/unknown shebang in the output of
+      missing (unversioned) interpreters.
   * checks/source-copyright.desc:
     + [CL] Change the severity from pedantic ("P:") to info ("I:") for the
       missing-explanation-for-repacked-upstream-tarball tag.
diff --git a/t/tests/legacy-scripts/tags b/t/tests/legacy-scripts/tags
index ff6dbc8..3dbd72e 100644
--- a/t/tests/legacy-scripts/tags
+++ b/t/tests/legacy-scripts/tags
@@ -19,7 +19,7 @@ E: scripts: php-script-but-no-php-cli-dep usr/share/scripts/php7.0foo #!/usr/bin
 E: scripts: php-script-but-no-php-cli-dep usr/share/scripts/phpenvfoo #!php
 E: scripts: php-script-but-no-php-cli-dep usr/share/scripts/phpfoo #!/usr/bin/php
 E: scripts: python-package-missing-depends-on-python
-E: scripts: python-script-but-no-python-dep usr/bin/py2.Xfoo
+E: scripts: python-script-but-no-python-dep usr/bin/py2.Xfoo #!python2.2
 E: scripts: python-script-but-no-python-dep usr/bin/pyfoo #!python
 E: scripts: script-without-interpreter control/prerm
 E: scripts: shell-script-fails-syntax-check usr/bin/sh-broken

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


Reply to: