requestsync crashed with IndexError in rmadison()

[Jonathan Thomas]

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

[Michael Bienia (geser)]

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".

[Michael Bienia (geser)]

Fixed in trunk (r566).

[Launchpad Janitor (janitor)]

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