better review of package changes

Bug #576748 reported by Robert Collins on 2010-05-07
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Medium
Unassigned
Launchpad itself
Low
Unassigned
Ubuntu Distributed Development
High
Unassigned

Bug Description

When packages are changed and proposed as a merge in launchpad, it can be hard to tell whats going on.

This is a complex topic, and will be getting discussed at UDS I hear.

This bug may need to be split into smaller ones, but acking the issue is a good start.

Both launchpad-code and bzr need to make it easier for distro developer accepting a change to assess whether the change is right. There are many sorts of changes people submit, and some are pretty easy to work with:
 - cherrypicked fixes from upstream
 - small changes to packaging or upstream

Others are harder:
 - new upstream releases (we don't usually want to audit those changes)
 - debian accepting ubuntu changes, with the changelog merging badly [this looks particularly ugly]
 - patches to patches (e.g. a patch to fix a bug wasn't quite right)

Looms may help with this, but unless we had everything in looms it wouldn't be better; there will be a branch->loom importer in bzr-builddeb at some point.

I'm filing a separate bug for using the bzr-builddeb changelog merge on launchpad, which should help today.

tags: added: udd
Tim Penhey (thumper) on 2010-05-07
tags: added: code-review
Changed in launchpad-code:
status: New → Triaged
importance: Undecided → Medium
Changed in bzr:
status: New → Confirmed
importance: Undecided → Medium
James Westby (james-w) on 2010-10-01
tags: added: launchpad usability
Changed in udd:
status: New → Triaged
importance: Undecided → High
Changed in launchpad:
importance: Medium → High
Robert Collins (lifeless) wrote :

The Launchpad team won't be working on this anytime soon, but we'd be delighted to mentor someone that wants to put it together.

Changed in launchpad:
importance: High → Low
Jelmer Vernooij (jelmer) on 2017-11-09
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers