Launchpad itself

"X-Launchpad-Bug: component" wrong

Bug #297709 reported by Daniel Holbach on 2008-11-13

Affects		Status	Importance	Assigned to	Milestone
	Launchpad itself	Fix Released	Low	Cody A.W. Somerville

Bug Description

I just ran into a few cases where the 'component' piece is of the X-Launchpad-Bug is wrong.

Subject: [Bug 285390] Re: [Intrepid] X freezes
X-Launchpad-Bug: distribution=ubuntu; sourcepackage=xserver-xorg-video-intel;
component=universe; status=New; importance=Undecided;
assignee=None;

Subject: [Bug 288037] Re: compiz.real crashed with SIGSEGV
X-Launchpad-Bug: distribution=ubuntu; sourcepackage=compiz; component=universe;
status=Incomplete; importance=Medium; assignee=None;

This gets them mistakingly routed to <email address hidden>

Tags:

Related branches

lp:~cody-somerville/launchpad/fix-bug-297709

Merged into lp:launchpad

Brad Crittenden (community): Approve (code) on 2009-11-17

Julian Edwards: Pending requested 2009-11-16

Revision history for this message

Graham Binns (gmb) wrote on 2008-11-13: Re: [Bug 297709] [NEW] "X-Launchpad-Bug: component" wrong

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

What should the component be in these cases?

status incomplete
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: http://getfiregpg.org

iEYEARECAAYFAkkcVDcACgkQwutMq02tGPr0SwCfSlN5KovTmK7kiBM+Ng1apeE9
msoAn2gRWnYXdeeI6qCllJJInldk02pD
=hbo1
-----END PGP SIGNATURE-----

Changed in malone:
status:	New → Incomplete

Revision history for this message

Eleanor Berger (intellectronica) wrote on 2008-11-13:

> What should the component be in these cases?

main

Changed in malone:
status:	Incomplete → New

Revision history for this message

Brian Murray (brian-murray) wrote on 2009-06-04:

This is because these packages were originally in universe and have subsequently moved to main, which happens quite often. I think it'd make sense to use the "current" version of the package as seen at the +source page for the package.

tags:

added: ubuntu-qa

Revision history for this message

Daniel Holbach (dholbach) wrote on 2009-06-04:

Almost all package start out in universe. :-)

Revision history for this message

Cody A.W. Somerville (cody-somerville) wrote on 2009-11-12:

fix-incorrect-component-in-bugmail-header.patch Edit (620 bytes, text/plain)

Attached is a patch to fix this issue by fixing the order by clause (the issue was that it was sorting the publishing status in the wrong direction - obsolete (5) should not become before published (2)) used when trying to fetch the latest overall publication of a distribution source package.

Revision history for this message

Graham Binns (gmb) wrote on 2009-11-12:

Thanks for the patch, Cody. It's great that you're helping out with bugs in Launchpad.

However:

- There are no tests to go with this patch, and I've no idea what tests it's going to break.
- It would be much better if this were a branch rather than a patch. That would allow us to use the code review system and would give us a better chance getting this landed sooner.

I'm happy to help you out with writing the tests; just ping me if you've any questions. I'll also review the final branch for you and make sure it lands on devel, too.

Cody A.W. Somerville (cody-somerville) on 2009-11-12