[Toaster] [V2 review-request] show and delete recursive list of reverse dependencies
Lerner, Dave
dave.lerner at windriver.com
Mon Mar 21 12:37:48 PDT 2016
Hi Belen,
Sorry you had problems. Are you certain that you applied all 3 patches? Your results indicate that the customrecipe.js was not replaced with the new v2 patch that was part of the 4-part email (3 patches and the v2 of the review request).
As the attached 3 jpgs indicate, I get the expected results, verified just now by doing the following:
- checked-out my local dlerner/9121-delete-dependent-package-tree branch again,
- wiped out the build and poky/_toaster* poky/_meta* directories,
- rebuilt core-image-sato and customized it
- looked at openssl on a remove package and it looked fine, jpg attched
- customised core-image-minimal
- added atk
- looked at remove libprce and it looks fine, jpg attached.
Here is how I verified that the email and the poky-contrib branch are correct.
- I saved the patches 0001,2,3 from the emails that I sent
- then I diff'd those against a new set of formatted patches and the diffs matched (except for manual insertions of the V2)
- then, just to make sure that the format-patch command wasn't goofed up (in my script) I ran 'patch -R --dry-run -p1 ' on each of the patches and they applied cleanly.
My poky-contrib branch was force-updated and I get this today:
toaster> git status
On branch dlerner/9121-delete-dependent-package-tree
Your branch is up-to-date with 'poky-contrib/dlerner/9121-delete-dependent-package-tree'.
It's been sufficiently long since my original commits that I may need to rebase against toaster-next. I was reluctant to rebase to toaster-next last week after I heard at the Wednesday conference call that toaster-next hung on a build. That seems to have been fixed so if you want a V3 rebased, let me know.
- Dave
________________________________________
From: Barros Pena, Belen [belen.barros.pena at intel.com]
Sent: Monday, March 21, 2016 11:52 AM
To: Lerner, Dave; toaster at yoctoproject.org; SMITH, ELLIOT
Subject: Re: [Toaster] [V2 review-request] show and delete recursive list of reverse dependencies
On 17/03/2016 22:01, "toaster-bounces at yoctoproject.org on behalf of Dave
Lerner" <toaster-bounces at yoctoproject.org on behalf of
dave.lerner at windriver.com> wrote:
>
>Target Branch: toaster-next master
>Contrib Branch: poky-contrib/dlerner/9121-delete-dependant-package-tree
>Defect: 9121
>Summary: Show and delete recurive list of reverse dependencies
In this version the text of the reverse dependencies dialog seems to be
cut. The first sentence before the list of packages says: "in your image
on libcrypto". The last sentence after the list of packages says: "In
order to remove libcrypto, you must remove as well". So there seems to be
some text missing.
>
>Comment: Version 2 patch set per comments from Belen and Elliot.
> Only 1/3 and 3/3 patches are changed, but 2/3 is included.
>
>Commit
>------
>c801079 toaster: get all dependants for pkg for removal
>327ada3 toaster: new customise package-remove modal dlg
>7542b8a toaster: show full list of dependants to remove
>
> .../toaster/toastergui/static/js/customrecipe.js | 72 +++++++++++++++--
> .../toaster/toastergui/templates/customrecipe.html | 5 +-
> bitbake/lib/toaster/toastergui/views.py | 91
>+++++++++++++++++++---
> 3 files changed, 148 insertions(+), 20 deletions(-)
>
>
>Test Cases:
>[V2] denotes additional test for version 2, changes per Belen's comments.
>
>SETUP:
>-----
>Start toaster with empty build directory, sstate-cache
>Create a new project, 'myproject'
>Build core-image-sato
>
>CASE Remove dependant package that was part of original base image
>------------------------------------------------------------------
>Create new custom image on core-image-sato, named mysato
>Remove libcrypto
> Verify that "39 packages will be removed" are reported and that the
> packages are unique and in alpha order
> [V2] Verify that the notification shows
>"You have removed libcrypto and its 39 reverse dependencies:
>gst-player-bin, gstreamer1.0-plugins-bad-dtls,
>gstreamer1.0-plugins-bad-meta, gstreamer1.0-plugins-bad-opengl,
>libegl-mesa and 34 more"
When I remove libcrypto, the notification is missing the dependency
information. It says only: "You have removed 1 packages from sato-belen:
libcrypto". The reverse dependencies seem to have been removed though
(they show the 'add package' button and the new number of packages
installed has been reduced by the number of packages removed). I've
attached a screenshot.
>
>Build mysato
>Click on the All Builds mysato:
> Verify Images : Packages included field shows 40 packages less than if
> you look at All Builds | core-image-sato (eg 567 vs 607)
>
>CASE Add back a root dependency (Issue sent in email Mar9)
>----------------------------------------------------------
>Open custom image mysato
>Add libcrypto back
>Build
>Click All Projects | Custom Images | mysato
> Verify libcrypto 'looks' added: that the entry shows "remove package"
>
>CASE Test no interference removing from a 2nd custom image in project
>----------------------------------------------------------------------
>New custom-image based on core-image-sato 'myminimal'
>Build it
>Add atk which is dependant on glib-2.0, libffi, libpcre (bash appears but
>this
>is busybox system, so don't think that dependency will apply - maybe bug
>unrelated to this work).
>Build myminimal again.
>Verify that atk, libffi, libpcre, glib-2.0 have files in the file system
>For custom image myminimal, enter libpcre, remove package
> Verify modal dlg notes libpcre atk and glib-2.0 will be removed Remove
>Build
> Verify atk, glib2.0 and libpcre are removed.
>
>CASE Test button notifications and banner on removal for package w/
>dependants
>--------------------------------------------------------------------------
>----
>To minimal, add openssl-staticdev which indirectly will bring in libssl
>Build
>Verify that libssl, openssl, openssl-dev are added
>On minimal, enter "openssl" to see a set of matching packages.
>REmove libssl
> Verify modal dialog reports both the direct and indirect dependants
> openssl, openssl-dev, and openssl-staticdev, will be removed.
I tested this with openssl (openssl-staticdev was missing from my list).
Adding openssl brought in libssl. When you click remove package for libssl
you get the dialog saying that openssl will be removed as well, but when
you click 'remove all', openssl is not removed: it still shows the 'remove
package' button. However, when I filter the list to show only the packages
added, openssl is not on the list. This was a bit strange, so I've
recorded it so that you will believe me ;) The video is here
https://drive.google.com/file/d/0B1KD45-8S-9FRF8td29abHNrbHc/view?usp=shari
ng
>Click Remove all
> Verify the buttons for openssl (an others that you see on screen)
>report:
> "4 packages removed"
> Verify that the banner says:
> "You have removed 4 packages from minimal: libssl and it's 3
>dependantso Verify that buttons transition to Add Package.
>
>[V2] CASE Remove package that has only has 1 dependant
>------------------------------------------------------
>To minimal, add back openssl-staticdev
>Build
>Select openssl-dev and remove the package.
> Verify that syntax for 1 rev-dep is appropriate on modal dlg.
> Remove the 2 packages
> Verify that syntax for 1 reverse dependency on main page notifiation.
Same problem with this one: removing openssl-dev doesn't seem to remove
openssl-staticdev. But when I search again for openssl-staticdev, the
button has been updated and changed 'add package'.
Cheers
Belén
>
>--
>_______________________________________________
>toaster mailing list
>toaster at yoctoproject.org
>https://lists.yoctoproject.org/listinfo/toaster
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sato-removing-openssl.png
Type: image/png
Size: 49400 bytes
Desc: sato-removing-openssl.png
URL: <http://lists.yoctoproject.org/pipermail/toaster/attachments/20160321/72149644/attachment-0003.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libpcre.png
Type: image/png
Size: 29185 bytes
Desc: libpcre.png
URL: <http://lists.yoctoproject.org/pipermail/toaster/attachments/20160321/72149644/attachment-0004.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: banner.png
Type: image/png
Size: 18375 bytes
Desc: banner.png
URL: <http://lists.yoctoproject.org/pipermail/toaster/attachments/20160321/72149644/attachment-0005.png>
More information about the toaster
mailing list