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

Bug#634112: debhelper deprecations



tags 634112 + patch
thanks

* Joey Hess <joeyh@debian.org>, 2011-07-16, 20:59:
dh_installmanpages is deprecated but not shown by lintian.

Here's a patch.

debian/compress files are deprecated for a while. But only 1 package in the archive has one, and I just filed a bug, so you may not want to bother with that.

I agree that it's not worth effort. (Well, at least my effort. :P)

--
Jakub Wilk
diff --git a/checks/debhelper b/checks/debhelper
--- a/checks/debhelper
+++ b/checks/debhelper
@@ -103,6 +103,9 @@
 	if ($dhcommand eq 'dh_pycentral') {
 	    tag 'dh_pycentral-is-obsolete', "line $.";
 	}
+	if ($dhcommand eq 'dh_installmanpages') {
+	    tag 'dh_installmanpages-is-obsolete', "line $.";
+	}
 
 	# Don't warn about recently deprecated commands in code that may be
 	# optional.  It may be there only for backports.
diff --git a/checks/debhelper.desc b/checks/debhelper.desc
--- a/checks/debhelper.desc
+++ b/checks/debhelper.desc
@@ -109,6 +109,13 @@
  Lenny or older system, please ignore this warning.
 Ref: http://wiki.debian.org/Python/PyCentral2DhPython2
 
+Tag: dh_installmanpages-is-obsolete
+Severity: normal
+Certainty: certain
+Ref: dh_installmanpages(1)
+Info: This package calls dh_installmanpages in its <tt>debian/rules</tt> file.
+ dh_installmanpages is deprecated in favour of dh_installman.
+
 Tag: dh_desktop-is-deprecated
 Severity: normal
 Certainty: certain
diff --git a/debian/changelog b/debian/changelog
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,7 @@
       - boilerplate-copyright-format-uri
       - comma-separated-files-in-dep5-copyright
       - debian-copyright-is-symlink
+      - dh_installmanpages-is-obsolete
       - missing-field-in-dep5-copyright
       - missing-license-paragraph-in-dep5-copyright
       - missing-license-text-in-dep5-copyright
@@ -27,6 +28,8 @@
   * checks/debhelper:
     + [NT] Recognise /usr/share/R/debian/r-cran.mk as a cdbs-based
       build.  This prevents false-positive "unneeded cdbs" warning.
+    + [JW] Check for use of the obsolete dh_installmanpages debhelper
+      program.  Thanks to Joey Hess for the report.  (Closes: #634112).
   * checks/duplicate-files{,.desc}:
     + [NT] Replace the exception for "__init__.py" files by excluding
       all empty files from the check.  The bug (#632789) only requests
diff --git a/t/tests/debhelper-deprecated/debian/debian/rules b/t/tests/debhelper-deprecated/debian/debian/rules
--- a/t/tests/debhelper-deprecated/debian/debian/rules
+++ b/t/tests/debhelper-deprecated/debian/debian/rules
@@ -21,6 +21,7 @@
 	dh_install
 	dh_installchangelogs
 	dh_installdocs
+	dh_installmanpages
 	dh_desktop
 	-dh_dhelp
 	dh_scrollkeeper
diff --git a/t/tests/debhelper-deprecated/desc b/t/tests/debhelper-deprecated/desc
--- a/t/tests/debhelper-deprecated/desc
+++ b/t/tests/debhelper-deprecated/desc
@@ -5,6 +5,7 @@
 Test-For:
  dh_desktop-is-deprecated
  dh_dhelp-is-deprecated
+ dh_installmanpages-is-obsolete
  dh_pycentral-is-obsolete
  dh_python-is-obsolete
  dh_scrollkeeper-is-deprecated
diff --git a/t/tests/debhelper-deprecated/tags b/t/tests/debhelper-deprecated/tags
--- a/t/tests/debhelper-deprecated/tags
+++ b/t/tests/debhelper-deprecated/tags
@@ -1,7 +1,8 @@
-W: debhelper-deprecated source: dh_desktop-is-deprecated line 24
-W: debhelper-deprecated source: dh_dhelp-is-deprecated line 25
-W: debhelper-deprecated source: dh_pycentral-is-obsolete line 30
-W: debhelper-deprecated source: dh_python-is-obsolete line 27
-W: debhelper-deprecated source: dh_scrollkeeper-is-deprecated line 26
-W: debhelper-deprecated source: dh_suidregister-is-obsolete line 40
-W: debhelper-deprecated source: dh_undocumented-is-obsolete line 41
+W: debhelper-deprecated source: dh_desktop-is-deprecated line 25
+W: debhelper-deprecated source: dh_dhelp-is-deprecated line 26
+W: debhelper-deprecated source: dh_installmanpages-is-obsolete line 24
+W: debhelper-deprecated source: dh_pycentral-is-obsolete line 31
+W: debhelper-deprecated source: dh_python-is-obsolete line 28
+W: debhelper-deprecated source: dh_scrollkeeper-is-deprecated line 27
+W: debhelper-deprecated source: dh_suidregister-is-obsolete line 41
+W: debhelper-deprecated source: dh_undocumented-is-obsolete line 42

Reply to: