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

[lintian] 02/03: checks/debhelper: Ignore comments in debian/rules



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

jwilk pushed a commit to branch master
in repository lintian.

commit 9fe5079cc36bd34fbe16da6bee98d829172ec9f2
Author: Jakub Wilk <jwilk@debian.org>
Date:   Mon Sep 19 17:15:11 2016 +0200

    checks/debhelper: Ignore comments in debian/rules
---
 checks/debhelper.pm                                  | 1 +
 debian/changelog                                     | 2 ++
 t/tests/debhelper-override-typos/debian/debian/rules | 1 +
 3 files changed, 4 insertions(+)

diff --git a/checks/debhelper.pm b/checks/debhelper.pm
index ad80fc8..cd1dd9f 100644
--- a/checks/debhelper.pm
+++ b/checks/debhelper.pm
@@ -88,6 +88,7 @@ sub run {
             $maybe_skipping--;
         }
 
+        next if /^\s*\#/;
         if (m/^\s+-?(dh_\S+)/) {
             my $dhcommand = $1;
             $build_systems{'debhelper'} = 1
diff --git a/debian/changelog b/debian/changelog
index 4285389..2c806c1 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -8,6 +8,8 @@ lintian (2.5.48) UNRELEASED; urgency=low
       This is because hardening-check appears to be losing its
       "home" with the coming removal of hardening-wrapper and
       hardening-includes.
+  * checks/debhelper.pm:
+    + [JW] Ignore comments in debian/rules.
   * checks/testsuite.pm:
     + [JW] Apply patch from Sean Whitton to recognise autopkgtest-pkg-elpa
       as a valid value for the Testsuite field.  (Closes: #837801)
diff --git a/t/tests/debhelper-override-typos/debian/debian/rules b/t/tests/debhelper-override-typos/debian/debian/rules
index 078ce7d..e57d0fd 100755
--- a/t/tests/debhelper-override-typos/debian/debian/rules
+++ b/t/tests/debhelper-override-typos/debian/debian/rules
@@ -28,3 +28,4 @@ override_dh_python2 override_dh_perl :
 override_dh_systemd_enable override_dh_systemd_disable:
 override_dh_auto_configure-% override_dh_auto_install-%:
 foo override_dh_installdeb bar override_dh_installxmlcatalogs: # combining regular and overrides
+# override_dh_instal: would be a typo, but in a comment it doesn't matter

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


Reply to: