Don't pick transitional -dbg over -dbgsym

[Martin Pitt]

Quoting Brian Murray: When trying to add a test like

 $ bzr diff test/test_backend_apt_dpkg.py
=== modified file 'test/test_backend_apt_dpkg.py'
--- test/test_backend_apt_dpkg.py 2015-06-17 15:54:40 +0000
+++ test/test_backend_apt_dpkg.py 2015-06-25 13:53:10 +0000
@@ -848,7 +848,9 @@
                                           ('distro-info-data',
                                            '0.18ubuntu0.2'),
                                           ('qemu-utils',
- '2.0.0+dfsg-2ubuntu1.11')
+ '2.0.0+dfsg-2ubuntu1.11'),
+ ('unity-services',
+ '7.2.5+14.04.20150521.1-0ubuntu1'),
                                          ], False, self.cachedir)

         def sandbox_ver(pkg, debian=True):
@@ -884,6 +886,8 @@
         self.assertIn('distro-info-data 0.18ubuntu0.2', pkglist)
         self.assertIn('qemu-utils-dbgsym 2.0.0+dfsg-2ubuntu1.11',
                       pkglist)
+ self.assertIn('unity-services-dbgsym 7.2.5+14.04.20150521.1-0ubuntu1',
+ pkglist)

         # caches packages, and their versions are as expected
         cache = os.listdir(os.path.join(self.cachedir, 'Foonux 1.2', 'apt',

This fails with

AssertionError: 'unity-services-dbgsym 7.2.5+14.04.20150521.1-0ubuntu1'
not found in ['distro-info-data 0.18ubuntu0.2', 'oxideqt-codecs
1.6.6-0ubuntu0.14.04.1', 'oxideqt-codecs-dbg 1.6.6-0ubuntu0.14.04.1',
'unity-2d-dbg 7.2.5+14.04.20150521.1-0ubuntu1', 'unity-services
7.2.5+14.04.20150521.1-0ubuntu1']

unity-2d-dbg is just a transitional package, and shouldn't trump the -dbgsym package. We need some heuristic there, like "transitional" in the package description, or size being < 10 kB.

[Martin Pitt (pitti)]

Fixed in trunk r2977.

[Launchpad Janitor (janitor)]

This bug was fixed in the package apport - 2.18-0ubuntu2

---------------
apport (2.18-0ubuntu2) wily; urgency=medium

  * Fix PEP-8 error in test/test_backend_apt_dpkg.py.

apport (2.18-0ubuntu1) wily; urgency=medium

  * New upstream release. Changes since our last merge from trunk:
    - apport-gtk: Use GtkWidget::valign property instead of GtkMisc::yalign
      which is deprecated in GTK 3.16. Thanks Iain Lane.
    - sandboxutils, make_sandbox(): Don't exit with 0 (success) if the
      ExecutablePath does not exist. (LP: #1462469)
    - sandboxutils, make_sandbox(): Fix second round of package installs to go
      into permanent root dir as well.
    - apt/dpkg install_packages(): If a requested package version is not
      available from apt in the given archive, try to download it from
      Launchpad. Thanks to Brian Murray!
    - kerneloops: Fix crash when determining the version of a custom kernel.
      Thanks Brian Murray. (LP: #1468154)
    - apt/dpkg install_packages(): Ignore -dbg packages whose descriptions
      contain "transitional". (LP: #1469635)
    - Keep "[origin: ...]" information in Package: and Dependencies: fields
      for native-origins.d/ origins, so that it's possible to retrace them.
      Thanks Brian Murray! (LP: #1470572)
    - Add support for retracing with discovering and enabling foreign
      Launchpad PPA origins, as specified in reports' Package:/Dependencies:
      fields. Thanks Brian Murray!
    - hookutils.attach_wifi(): Shorten value of CRDA if iw isn't available on
      the system. Thanks Brian Murray.
    - Fix wrong assertion in crashdb.test_check_duplicate() which surfaces
      under Python 3.5. (LP: #1474539)
  * test/test_backend_apt_dpkg.py: Disable new test assertion for
    unity-services-dbgsym, which doesn't currently work in the autopkgtest
    (but works fine in trunk and on the retracers). To be investigated.

 -- Martin Pitt <email address hidden> Fri, 17 Jul 2015 11:43:37 +0200