requestsync crashed with IndexError in rmadison()

Bug #508794 reported by Jonathan Thomas
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: ubuntu-dev-tools

requesync crashed right after I executed the command. (Apport should attach that)
I guess it didn't like -d being "sid"? Anyways, it shouldn't crash if I give it a bad distro.

ProblemType: Crash
Architecture: i386
Date: Sun Jan 17 10:50:18 2010
DistroRelease: Ubuntu 10.04
ExecutablePath: /usr/bin/requestsync
InstallationMedia: Kubuntu 9.10 "Karmic Koala" - Release i386 (20091028.5)
InterpreterPath: /usr/bin/python2.6
Package: ubuntu-dev-tools 0.91
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/bin/requestsync -d sid skanlite lucid
ProcEnviron:
 SHELL=/bin/bash
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 LANGUAGE=en_US:en
ProcVersionSignature: Ubuntu 2.6.32-10.14-generic-pae
PythonArgs: ['/usr/bin/requestsync', '-d', 'sid', 'skanlite', 'lucid']
SourcePackage: ubuntu-dev-tools
Tags: lucid
Title: requestsync crashed with IndexError in rmadison()
Uname: Linux 2.6.32-10-generic-pae i686
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare

Revision history for this message
Jonathan Thomas (echidnaman) wrote :
visibility: private → public
Revision history for this message
Michael Bienia (geser) wrote :

Yes, you are right. rmadison (the tool) doesn't like sid for suite:
$ rmadison -u debian -a source -s sid skanlite
Traceback (most recent call last):
  File "/usr/local/bin/dak", line 238, in <module>
    main()
  File "/usr/local/bin/dak", line 218, in main
    module.main()
  File "/srv/ftp.debian.org/dak/dak/ls.py", line 106, in main
    utils.parse_args(Options)
  File "/srv/ftp.debian.org/dak/dak/daklib/utils.py", line 979, in parse_args
    if suite.suite_id is None:
AttributeError: 'NoneType' object has no attribute 'suite_id'

The return code is still 0 (else it would trigger an assertation error).
The "fix" is to map "sid" back to "unstable".

Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Low
Revision history for this message
Michael Bienia (geser) wrote :

Fixed in trunk (r566).

Changed in ubuntu-dev-tools (Ubuntu):
status: New → Fix Committed
tags: removed: need-duplicate-check
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.92

---------------
ubuntu-dev-tools (0.92) lucid; urgency=low

  [ Siegfried-Angel Gevatter Pujals ]
  * bash_completion/pbuilder-dist:
     - Enable tab-completion for pbuilder-lucid and cowbuilder-lucid.

  [ Emmet Hikory ]
  * mk-sbuild-lv: support foreign armel schroots
  * mk-sbuild-lv: use --arch=foo rather than --arch foo for debootstrap
  * pbuilder-dist: Allow architecture-switching to armel on i386/amd64
  * pbuilder-dist: use --arch=foo rather than --arch foo for debootstrap
  * pbuilder-dist: change --mirror logic to use -ports when appropriate
  * docs/pbuilder-dist.1: Document architecture-switching for armel
  * debian/control: add qemu-arm-static to Recommends:

  [ Michael Bienia ]
  * ubuntutools/requestsync/mail.py:
    Map "sid" back to "unstable" (and "squeeze" to "testing") else rmadison
    gets a Python traceback from the remote site instead of the expected data
    (lp: #508794).

  [ Kees Cook ]
  * mk-sbuild-lv: adjust schroot.conf template to document the -source
    change further.
 -- Emmet Hikory <email address hidden> Wed, 03 Feb 2010 11:39:12 -0800

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → 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.