Merge lp:~mterry/ubuntu-archive-tools/or-deps into lp:ubuntu-archive-tools
Status: | Rejected |
---|---|
Rejected by: | Steve Langasek |
Proposed branch: | lp:~mterry/ubuntu-archive-tools/or-deps |
Merge into: | lp:ubuntu-archive-tools |
Diff against target: |
142 lines (+26/-18) 2 files modified
checkrdepends (+14/-11) nbs-report (+12/-7) |
To merge this branch: | bzr merge lp:~mterry/ubuntu-archive-tools/or-deps |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Steve Langasek | Disapprove | ||
Review via email: mp+115445@code.launchpad.net |
Description of the change
I noticed that some of the packages in the NBS report are false-negatives (in the sense that they are reported as not-removable when it would actually be safe to remove them).
In particular, let's say package A depends on "B | C" while B is NBS and C is not. Package A will show up as blocking B from being removed, when B can actually be removed safely.
There are two files I changed:
checkrdepend now reports why a package is include in the rdepend list. The format of the line used to be:
RDEPEND
But now is:
RDEPEND WHY1|WHY2|WHY3
The only consumer that I could see of that output is nbs-report, which was changed to understand the changes. And to examine the "why list" for packages that are not NBS and allow the package to be removed if all of its rdepends are such.
I also added the "why list" to the NBS report HTML, since such imformation will help understand why it is recommending a package for removal when it still has rdepends. It only is shown if the "why list" is not a trivial one-package list.
Obviously, there is a danger here that I will turn false-negatives into false-positives, which are far more disasterous. So please look over with a fine-toothed comb, obviously.
As of this writing, my changes detect several more packages for removal than before, all of which manually are confirmed to be accurate. Specifically: compiz-kde, fb-music-low, libglew1.6-dev, libglewmx1.6, libglewmx1.6-dev, libnode-
Hmm, changing to Work in Progress for a bit. The HTML output is correct, but the remove-package line has some packages it shouldn't. Looking into why.