qa-assistant crashed with ImportError in <module>() due to missing python-rpm dependency

Bug #281066 reported by Steve Beattie
10
Affects Status Importance Assigned to Milestone
qa-assistant (Debian)
Fix Released
Unknown
qa-assistant (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: qa-assistant

Had just started the druid and had poked the "review an SRPM according to fedora rules" for the first time. Examining the traceback, the plugin specific for the fedora srpm review has an unmet dependency on the python-rpm module. Installing the python-rpm package fixes that.

ProblemType: Crash
Architecture: amd64
DistroRelease: Ubuntu 8.10
ExecutablePath: /usr/bin/qa-assistant
InterpreterPath: /usr/bin/python2.5
Package: qa-assistant 0.4.90.5-1
PackageArchitecture: all
ProcAttrCurrent: unconfined
ProcCmdline: /usr/bin/python -tt /usr/bin/qa-assistant
ProcEnviron:
 PATH=/home/username/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=en_US.UTF-8
 SHELL=/bin/bash
PythonArgs: ['/usr/bin/qa-assistant']
SourcePackage: qa-assistant
Title: qa-assistant crashed with ImportError in <module>()
Uname: Linux 2.6.27-6-generic x86_64
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare

Revision history for this message
Steve Beattie (sbeattie) wrote :
Steve Beattie (sbeattie)
description: updated
Changed in qa-assistant:
assignee: nobody → nick.ellery
importance: Undecided → Low
status: New → In Progress
Revision history for this message
Nick Ellery (nick.ellery) wrote :

qa-assistant (0.4.90.5-1ubuntu1) intrepid; urgency=low

  * debian/control:
    - Added dependency on python-rpm (LP: #281066).
    - Bumped standards-version to 3.8.0.
      + Moved homepage to its own field.
    - Updated maintainer field as per DebianMaintainerField policy.

 -- Nick Ellery <email address hidden> Thu, 09 Oct 2008 20:56:27 -0700

Changed in qa-assistant:
status: In Progress → Confirmed
assignee: nick.ellery → nobody
Revision history for this message
James Westby (james-w) wrote :

Hi Nick,

Changing the standards-version has no point for a package pulled from
Debian, so please don't do that. Changing the homepage also
has little point.

Also, convention is now to not document the change in maintainer field.

I assume that Debian has this problem, so please forward the bug there.

Thanks,

James

Revision history for this message
Nick Ellery (nick.ellery) wrote : Re: [Bug 281066] Re: qa-assistant crashed with ImportError in <module>() due to missing python-rpm dependency

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

affects debian

Hi, I reported this to debian, and will attach a new patch with the
suggested changes.

Thanks,
Nick
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkjv4+8ACgkQgovwdl7S/ZGG2QCfWoFgbPYQDjrY9FfQC6tHGzQO
MGYAnjPeV74PBr25Uf4sakFgEkYg0X7x
=aufm
-----END PGP SIGNATURE-----

Revision history for this message
Nick Ellery (nick.ellery) wrote :
Revision history for this message
James Westby (james-w) wrote :

Hi Nick,

Thanks for your work.

After having installed the package I wonder if python-rpm should
be a Recommends rather than a Depends.

However, I think I will sponsor this package tomorrow as is and we
can go with what the Debian maintainer decides when we merge.
I may well tweak the changelog to state this.

Thanks,

James

Revision history for this message
Steve Beattie (sbeattie) wrote : Re: [Bug 281066] Re: qa-assistant crashed with ImportError in <module>() due to missing python-rpm dependency

Hey Nick and James,

Thanks both of you for looking at this.

On Sat, Oct 11, 2008 at 02:25:58AM -0000, James Westby wrote:
> After having installed the package I wonder if python-rpm should
> be a Recommends rather than a Depends.
>
> However, I think I will sponsor this package tomorrow as is and we
> can go with what the Debian maintainer decides when we merge.
> I may well tweak the changelog to state this.

I suspect the ideal solution is to have the package be a recommended
dependency as well but also to have the application trap the module
load error better so that it can report to the user the missing needed
python modules.

--
Steve Beattie
<email address hidden>
http://NxNW.org/~steve/

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qa-assistant - 0.4.90.5-1ubuntu1

---------------
qa-assistant (0.4.90.5-1ubuntu1) intrepid; urgency=low

  * Added dependency on python-rpm (LP: #281066). This is reported in
    Debian as #501853, and we can probably follow whatever the Debian
    maintainer decides here when they fix that. We probably don't want
    to maintain a diff if they go for Reccomends rather than Depends.

 -- Nick Ellery <email address hidden> Fri, 10 Oct 2008 16:43:03 -0700

Changed in qa-assistant:
status: Confirmed → Fix Released
Changed in qa-assistant:
status: Unknown → New
Changed in qa-assistant:
status: New → 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.