Bug 9121 - Remove reverse dependencies when removing packages from custom images
Summary: Remove reverse dependencies when removing packages from custom images
Status: VERIFIED FIXED
Alias: None
Product: Toaster
Classification: Build System, Metadata & Runtime
Component: toaster (show other bugs)
Version: 2.0
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 2.1 M4
Assignee: Dave Lerner
QA Contact: Alexandru Roman
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-02-16 09:36 UTC by Belen Barros Pena
Modified: 2016-04-21 11:13 UTC (History)
6 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments
reverse-deps-dialog.png (130.55 KB, image/png)
2016-02-16 09:36 UTC, Belen Barros Pena
no flags Details
new-reverse-deps-dialog.png (84.69 KB, image/png)
2016-02-16 09:37 UTC, Belen Barros Pena
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Belen Barros Pena 2016-02-16 09:36:16 UTC
Currently, when you remove a package from a custom image, we tell you about the reverse dependencies of the package you are about to remove, but we don't remove the reverse dependencies (see attached reverse-deps-dialog.png). 

This creates a very cumbersome removal workflow, since you effectively must remember the list of reverse dependencies and remove them one by one. 

The dialog should instead remove the selected package and its reverse dependencies when you click the 'remove packages' button. It should look like the attached new-reverse-deps-dialog.png

Note that: 

* The dialog shows the number of reverse dependencies. This is not a random thing. This number is a very good high level indicator of the impact that removing the package will have, particularly since the number of reverse dependencies can be quite high and you might not be able to see the entire package list without scrolling.

* The list of reverse dependencies should include package groups, since the package group break down mechanism we have in place for removals means that Toaster is likely to do the right thing (remove the package group itself, and the package you selected for removal).
Comment 1 Belen Barros Pena 2016-02-16 09:36:49 UTC
Created attachment 2989 [details]
reverse-deps-dialog.png
Comment 2 Belen Barros Pena 2016-02-16 09:37:19 UTC
Created attachment 2990 [details]
new-reverse-deps-dialog.png
Comment 3 Dave Lerner 2016-03-18 18:54:36 UTC
Version 2 patch set submitted for review
Comment 4 Alexandru Roman 2016-04-21 11:13:10 UTC
Verified on master.