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

Bug#917967: marked as done (lintian: drop versioned-dependency-satisfied-by-perl)



Your message dated Mon, 07 Jan 2019 22:51:03 +0000
with message-id <E1ggdjb-000Gtx-FD@fasolo.debian.org>
and subject line Bug#917967: fixed in lintian 2.5.120
has caused the Debian Bug report #917967,
regarding lintian: drop versioned-dependency-satisfied-by-perl
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
917967: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=917967
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: lintian
Version: 2.5.118
Tags: patch
X-Debbugs-Cc: debian-perl@lists.debian.org

Now that perl provides its dual-lived module packages with versions,
this check is unneeded.

In time, a new check could be added recommending that the complex
form previously recommended be dropped, but I suggest that be treated
separately.

Cheers,
Dominic.
>From a60d4fcc63adbfe541c151d9b5079df74dd0fbc9 Mon Sep 17 00:00:00 2001
From: Dominic Hargreaves <dom@earth.li>
Date: Tue, 1 Jan 2019 18:05:46 +0000
Subject: [PATCH] Remove versioned-dependency-satisfied-by-perl

perl now uses versioned provides, so the advice given here is
redundant.
---
 checks/fields.desc                               | 28 ------------------------
 checks/fields.pm                                 | 16 --------------
 t/tests/fields-depends-general/debian/control.in |  4 ++--
 t/tests/fields-depends-general/desc              |  1 -
 t/tests/fields-depends-general/tags              |  2 --
 5 files changed, 2 insertions(+), 49 deletions(-)

diff --git a/checks/fields.desc b/checks/fields.desc
index 4158e6506..9f3e862a0 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -967,34 +967,6 @@ Certainty: certain
 Info: The uploader appears more than once in the <tt>Uploaders</tt>
  field. The duplicate information should be removed.
 
-Tag: versioned-dependency-satisfied-by-perl
-Severity: normal
-Certainty: certain
-Info: This package declares an unnecessary versioned dependency
- on a package that is also provided by one of the Perl core packages
- (perl, perl-base, perl-modules) with at least the required version.
- .
- As versioned dependencies are not satisfied by provided packages,
- this unnecessarily pulls in a separately packaged newer version
- of the module.
- .
- The recommended way to express the dependency without needless
- complications on backporting packages is to use alternative dependencies.
- The perl package should be the preferred alternative and the
- versioned dependency a secondary one.
- .
- Example: perl (&gt;= 5.10.0) | libmodule-build-perl (&gt;= 0.26)
- .
- An exception to this is when the dependency is only satisfied in a
- version of perl in experimental. In this case, the dependency on perl
- should come second.
- .
- Example: libextutils-parsexs-perl (&gt;= 2.210000) | perl (&gt;= 5.14)
- .
- Running <tt>cme fix dpkg -from control -filter Depends</tt> should be able
- to update these dependencies.
-Ref: policy 7.5
-
 Tag: package-superseded-by-perl
 Severity: normal
 Certainty: certain
diff --git a/checks/fields.pm b/checks/fields.pm
index 37d885b96..274b685b1 100644
--- a/checks/fields.pm
+++ b/checks/fields.pm
@@ -906,14 +906,6 @@ sub run {
                       if ("$d_pkg-doc" eq $pkg
                         && $field =~ /^(?:pre-)?depends$/);
 
-                    # only trigger this for the preferred alternative
-                    tag 'versioned-dependency-satisfied-by-perl',
-                      "$field: $part_d_orig"
-                      if $alternatives[0][-1] eq $part_d_orig
-                      && &$is_dep_field($field)
-                      && perl_core_has_version($d_pkg, $d_version->[0],
-                        $d_version->[1]);
-
                     tag 'package-relation-with-perl-modules', "$field: $d_pkg"
                       # matches "perl-modules" (<= 5.20) as well as
                       # perl-modules-5.xx (>> 5.20)
@@ -1182,14 +1174,6 @@ sub run {
                           # perl-modules-5.xx (>> 5.20)
                           if $d_pkg =~ /^perl-modules/
                           && $proc->pkg_src ne 'perl';
-
-                        # only trigger this for the preferred alternative
-                        tag 'versioned-dependency-satisfied-by-perl',
-                          "$field: $part_d_orig"
-                          if $alternatives[0][-1] eq $part_d_orig
-                          && &$is_dep_field($field)
-                          && perl_core_has_version($d_pkg, $d_version->[0],
-                            $d_version->[1]);
                     }
 
                     my $all_obsolete = 0;
diff --git a/t/tests/fields-depends-general/debian/control.in b/t/tests/fields-depends-general/debian/control.in
index 4b6672bf2..abedf22b5 100644
--- a/t/tests/fields-depends-general/debian/control.in
+++ b/t/tests/fields-depends-general/debian/control.in
@@ -3,14 +3,14 @@ Priority: optional
 Section: {$section}
 Maintainer: {$author}
 Standards-Version: {$standards_version}
-Build-Depends: {$build_depends}, libtest-simple-perl (>= 0.98), perl-modules
+Build-Depends: {$build_depends}, perl-modules
 Rules-Requires-Root: no
 
 Package: {$source}
 Architecture: all
 Depends: $\{shlibs:Depends\}, $\{misc:Depends\}, xorg, bash,
  conflict-dep, gawk | awk, new-package | xbase-clients (>= 0.1), {$source},
- gaim (>= 0.1), emacs21, emacs22, emacs23, makedev, libtest-simple-perl (>= 0.98),
+ gaim (>= 0.1), emacs21, emacs22, emacs23, makedev,
  packaging-dev, xfont-a, emacs22-gtk, emacs23-el, emacs23-nox, emacs23-lucid,
  emacs222, perl-modules, debhelper
 Breaks: package-without-version, {$source} (<< 0.1),
diff --git a/t/tests/fields-depends-general/desc b/t/tests/fields-depends-general/desc
index 47223aded..640dd76a8 100644
--- a/t/tests/fields-depends-general/desc
+++ b/t/tests/fields-depends-general/desc
@@ -22,4 +22,3 @@ Test-For:
  package-depends-on-itself
  package-relation-with-perl-modules
  package-relation-with-self
- versioned-dependency-satisfied-by-perl
diff --git a/t/tests/fields-depends-general/tags b/t/tests/fields-depends-general/tags
index a505d1bf7..3b1349079 100644
--- a/t/tests/fields-depends-general/tags
+++ b/t/tests/fields-depends-general/tags
@@ -12,7 +12,6 @@ E: py-fields-depends-general: depends-on-python-minimal depends
 I: fields-depends-general: conflicts-with-version package-with-version (<< 3.0)
 I: fields-depends-general: ored-depends-on-obsolete-package depends: xbase-clients (>= 0.1)
 W: fields-depends-general source: package-depends-on-itself fields-depends-general depends
-W: fields-depends-general source: versioned-dependency-satisfied-by-perl build-depends: libtest-simple-perl (>= 0.98)
 W: fields-depends-general-doc: doc-package-depends-on-main-package depends
 W: fields-depends-general: binary-package-depends-on-toolchain-package depends: debhelper
 W: fields-depends-general: breaks-without-version package-without-version
@@ -26,7 +25,6 @@ W: fields-depends-general: depends-on-old-emacs depends: emacs23-lucid
 W: fields-depends-general: depends-on-old-emacs depends: emacs23-nox
 W: fields-depends-general: depends-on-packaging-dev depends
 W: fields-depends-general: package-relation-with-self breaks: fields-depends-general (<< 0.1)
-W: fields-depends-general: versioned-dependency-satisfied-by-perl depends: libtest-simple-perl (>= 0.98)
 W: libfields-depends-general: binary-package-depends-on-toolchain-package recommends: cdbs
 W: libfields-depends-general: empty-binary-package
 W: libfields-depends-general: lib-recommends-documentation recommends: debconf-doc
-- 
2.11.0


--- End Message ---
--- Begin Message ---
Source: lintian
Source-Version: 2.5.120

We believe that the bug you reported is fixed in the latest version of
lintian, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 917967@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Chris Lamb <lamby@debian.org> (supplier of updated lintian package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Mon, 07 Jan 2019 22:19:07 +0000
Source: lintian
Binary: lintian
Architecture: source all
Version: 2.5.120
Distribution: unstable
Urgency: medium
Maintainer: Debian Lintian Maintainers <lintian-maint@debian.org>
Changed-By: Chris Lamb <lamby@debian.org>
Description:
 lintian    - Debian package checker
Closes: 917964 917967 918444 918459 918621
Changes:
 lintian (2.5.120) unstable; urgency=medium
 .
   * Summary of tag changes:
     + Removed:
       - versioned-dependency-satisfied-by-perl
 .
   * checks/binaries.pm:
     + [CL] Guile object files do not objdump/strip correctly so exclude
       them from a number of tests.  (Closes: #918444)
   * checks/fields.{desc,pm}:
     + [CL] Drop versioned-dependency-satisfied-by-perl; Perl now provides
       its dual-lived module packages with versions.  (Closes: #917967)
   * checks/{menus,patch-systems}.desc:
     + [CL] Clarify that spelling-* tags also correct various grammatical
       errors too.
   * checks/rules.desc:
     + [AB] override_dh_auto_test-does-not-check-DEB_BUILD_OPTIONS: Replace
       misleading "profile" with "option" in long description. See #889746
       and https://lists.debian.org/debian-devel/2019/01/msg00047.html
   * checks/systemd.pm:
     + [CL] Apply a patch from Robin Jarry to handle service
       templates for missing init scripts.  (MR: !115)
   * checks/testsuite.pm:
     + [CL] Apply a patch from Ondřej Nový fix false-positives in the
       unnecessary-testsuite-autopkgtest-field tag with non-"autopkgtest"
       values in the Testsuite control field.  (Closes: #918621)
 .
   * data/systemd/init-whitelist:
     + [CL] Add "brightness" to the list of initscripts that do not need a
       corresponding service file.  (Closes: #918459)
 .
   * debian/control:
     + [AB] Build-conflict with libperl-critic-pulp-perl (except with
       nocheck build profile) as it adds additional perlcritic rules which
       cause several checks under t/scripts/01-critic/ to fail.
 .
   * lib/Lintian/Util.pm:
     + [CL] Add optional support for stripping comments at EOL in
       read_dpkg_control via DCTRL_COMMENTS_AT_EOL.  (Closes: #917964)
Checksums-Sha1:
 43c6449adc4cb6372a57d6a675efd9ebe8968c1e 3746 lintian_2.5.120.dsc
 dae8b17fb39f846c23b07c5082c11b2d90778300 1626980 lintian_2.5.120.tar.xz
 46f5b24c3c39d1c5a8aa4e6ea297e16af55b4fc6 1174552 lintian_2.5.120_all.deb
 2535f8ab895b11e7ca8244b39a7eb00c90c77e3a 17582 lintian_2.5.120_amd64.buildinfo
Checksums-Sha256:
 f8017e7645079c3a2c8a9bf19d206f66b9c152b5b3e998406954f52cc9a1bc87 3746 lintian_2.5.120.dsc
 c8bacad6d4ac54d14db579a8738d480133de4c8fabd330c85217703d47b6f64e 1626980 lintian_2.5.120.tar.xz
 fcf9a875555068529a0b3f46a8ec367e1ba940bb54585344e0f240b8be1a2286 1174552 lintian_2.5.120_all.deb
 a7fd43692f30b89b9431ff6ad7fcdce89af53cfb4dee2d08f3944a2d50930b56 17582 lintian_2.5.120_amd64.buildinfo
Files:
 d3302a3df8674ed636b1b3338e2d93de 3746 devel optional lintian_2.5.120.dsc
 ed8114001fec9cd6886a03edcde087f9 1626980 devel optional lintian_2.5.120.tar.xz
 1f72bcedd72acbac759d68d5f35feab1 1174552 devel optional lintian_2.5.120_all.deb
 6710c53b413eac43a9e4f68ba2a9756b 17582 devel optional lintian_2.5.120_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEwv5L0nHBObhsUz5GHpU+J9QxHlgFAlwz0+UACgkQHpU+J9Qx
HlhK1g//dwnfpHBFxwq0Kxi7Rv9ZkUc7OVunT5HdBWzuzHu0VLwFn+0gNeQAL7hz
2Unk8fjMLG9mWh9bCQvE/vddEah7C35MD0dIxrPqGtQPUOl3CjKiuJvjwGwgw3y1
7fLz7hZA1uHQUWDIASfONI2/Cu2/UpTV6nOWYbKwM6hBnlKXTUG9vJOC+ldieFb+
86dsCMhVnDFoKSBumXt/HX3eVKwjJOyuATE9UDo9ydGoVPuMV8hoI+1tWTBNUsQ/
Hmt6n7LIETT2O4tFeFJntKWAIecj/YgXfvY1DZ5h94GzAg4PZmMeXMcSDt8iFZ0L
7E7ynubw+INy2NhtCdaquEKXNeLe86/MvFY+nKUM7B4KjiDXK/j07+4zY9mDJCWy
pey5DYzznlmIxsQXjDvlYcogz70ACCC6Yui9pQBPZJowXmNK//KoV8J0ZdMaL9hp
OQUWE7JS+lj/E2SpjJhG/DMzRqQ56oeTRsEKGASCbdGGUpMOPj+BDn19ZzqYaBqm
ck+2K5blRFjmGMPukAV45fDI/xJrQCtkZqKceTOj7OvZFpv5dRiab5wGI3tUyXvH
h4qvzvAP4pFBvW+DCDCH5+FufAt289QCAUR3SjGPlGmpAsSIBHA5F4tovpHCtVUT
RiikdI9kuy3X7RCdC1/oiUc+Ol7ilISun89lS6Xl0Yrrt/71fOs=
=XXDQ
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: