Merge lp://staging/~juliank/update-manager/remove-breaks into lp://staging/update-manager

Proposed by Julian Andres Klode
Status: Needs review
Proposed branch: lp://staging/~juliank/update-manager/remove-breaks
Merge into: lp://staging/update-manager
Diff against target: 34 lines (+11/-3)
2 files modified
UpdateManager/Core/MyCache.py (+3/-3)
debian/changelog (+8/-0)
To merge this branch: bzr merge lp://staging/~juliank/update-manager/remove-breaks
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+420537@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Julian Andres Klode (juliank) wrote :

I switched the order of checks because I figured Replaces will be shorter than Breaks+Conflicts.

Revision history for this message
Julian Andres Klode (juliank) wrote :

Arguably this maybe should only happen for unversioned Breaks, but also if you have a versioned Breaks: other (<< 2) because you took over a file from a package, and other=1 is installed, but other=2 was removed from the new release, then other=1 should be removed.

Revision history for this message
Steve Langasek (vorlon) wrote :

Approved, begrudgingly

review: Approve

Unmerged revisions

2948. By Julian Andres Klode

Allow removing packages on Replaces+Breaks in addition to
Replaces+Conflicts, as there are people using the former despite the
latter being documented in policy 7.6.2.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: