[FFe] piuparts for debian packages doesn't work

Bug #175821 reported by webworm
8
Affects Status Importance Assigned to Milestone
piuparts (Ubuntu)
Fix Released
Medium
Luca Falavigna

Bug Description

Running Gutsy.
running
piuparts -m http://ftp.be.debian.org/debian/ ../cyrscripts_0.1_all.deb or
piuparts -d etch -m http://ftp.be.debian.org/debian/ ../cyrscripts_0.1_all.deb
fails with the following error:

2m2.5s DEBUG: Starting command: ['chroot', '/tmp/tmpY3FJtK', 'dpkg', '--get-selections', '\\*']
2m2.5s DUMP:
  No packages found matching \*.
2m2.5s DEBUG: Command ok: ['chroot', '/tmp/tmpY3FJtK', 'dpkg', '--get-selections', '\\*']
Traceback (most recent call last):
  File "/usr/sbin/piuparts", line 1487, in <module>
    main()
  File "/usr/sbin/piuparts", line 1448, in main
    selections = chroot.get_selections()
  File "/usr/sbin/piuparts", line 579, in get_selections
    for name, status in list:
ValueError: too many values to unpack

Running the same command on Debian Etch doesn't produce the error...

Related branches

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Current package is broken (see attached log, as much as the one in comment #0).
I'm requesting a Feature Freeze exception for Hardy to have a working package for our QA activities.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :
Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Package is native, a diffstat against the whole package is attached.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

As above, debian/changelog is the official upstream changelog.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Hardy build log.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Hardy install/removal/purge log.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Test run with new candidate revision.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Candidate debdiff.

Changed in piuparts:
assignee: nobody → dktrkranz
importance: Undecided → Medium
Revision history for this message
Scott Kitterman (kitterman) wrote : Re: [Bug 175821] Re: [FFe] piuparts for debian packages doesn't work

Ack from me.

Revision history for this message
StefanPotyra (sistpoty) wrote :

Ack #2, confirming.

Changed in piuparts:
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (6.6 KiB)

This bug was fixed in the package piuparts - 0.30ubuntu1

---------------
piuparts (0.30ubuntu1) hardy; urgency=low

  * Merge from Debian unstable, remaining Ubuntu changes:
    - Add /var/log/auth.log to ignored_files.
    - Do not ignore errors during packages install.
  * debian/control
    - Update Maintainer field as per spec.
  * Feature Freeze exception granted (LP: #175821).

piuparts (0.30) unstable; urgency=low

  * piuparts.py:
    - Implement Chroot.create_temp_tgz_file() (since it's used in the VirtServ
      subclass)
    - Fix a typo -- chroot.create_temp_tgz() was being called instead of
      chroot.create_temp_tgz_file() (Closes: #465416)

piuparts (0.29) unstable; urgency=low

  [ Ana Beatriz Guerrero Lopez ]
  * Updated uploaders, remove Amaya and Alastair.

  [ Lars Wirzenius ]
  * piuparts.py: Call "apt-get update" before calling Chroot.minimize, so
    that we can find the debfoster package on Ubuntu.
  * debian/control: Fixed "Uploaders:" to have my preferred e-mail address
    (<email address hidden>) instead of my Debian one.
  * piuparts.py: Added -D option to set Debian flavor, plus two sets of
    default settings depending on the flavor, one for Debian itself, and
    another for Ubuntu. The settings choose default mirror and distribution
    set (sid vs gutsy). This will allow an Ubuntu version of the package
    to set defaults at package building time.
  * piuparts.py: Report target of broken symlink correctly, instead of
    saying "True".
  * piuparts.py: Use lsb-release to guess the Debian flavor.
  * debian/control: Added dependency on lsb-release.
  * piuparts-master.py: Make the master write summary of total pass/fail
    packages, plus status per binary package. This is for having the
    Debian PTS and the Ubuntu developer weather report show summaries
    of piuparts results.
  * integraged changes from Ubuntu by Ian Jackson to support autopkgtest.
  * wrapped long lines in debian/control.

  [ Holger Levsen ]
  * Added myself to uploaders.
  * Added Homepage: header to debian/control and changed the homepage to be
    http://wiki.debian.org/piuparts.
  * Use Vcs-* headers in debian/control instead of XS-Vcs-*.
  * Bumped policy version to 3.7.3.
  * Updated download location in debian/copyright.

  [ John Wright ]
  * Change the --warn-on-others implementation to create a dummy metapackage
    with the needed dependencies and install it in the chroot, rather than
    using "apt-get install package package-". The previous implementation
    had issues when the package was already installed (and especially if the
    package had essential packages in its reverse dependencies). This has
    the pleasant side-effect of making --warn-on-others work both with and
    without --apt. (Closes: #458929)
  * Fix a bug in check_for_broken_symlinks(): the targets for the broken
    symlinks (used for logging) were being read from files on the real root
    filesystem, rather than in the chroot.

piuparts (0.28) unstable; urgency=low

  * Fix an UnboundLocalError introduced in the --warn-on-others fix that would
    occur if run without the --apt option. (Closes: #443641)

piuparts (0.27) unstable; urgency=low

  * Add sup...

Read more...

Changed in piuparts:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.