Bug 5257 - When selecting the 'Edit packages' option in the build fail notification, Hob should not display the "Package list may be incomplete!" dialog
Summary: When selecting the 'Edit packages' option in the build fail notification, Hob...
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: 1.6
Hardware: All Multiple
: Medium normal
Target Milestone: 1.6
Assignee: Cristiana Voicu
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-09-24 09:48 UTC by Belen Barros Pena
Modified: 2013-11-18 12:41 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
edit-packages-dialog.png (110.08 KB, image/png)
2013-09-24 09:48 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 2013-09-24 09:48:25 UTC
Created attachment 1530 [details]
edit-packages-dialog.png

When a Hob build fails during the image building process, we display a notification that provides 3 options:

* Edit packages
* Open log
* File a bug

When you click 'edit packages' you are presented with the dialog shown in the attached screenshot (edit-packages-dialog.png). Hob should not show this dialog: it should simply display the list of packages generated as part of the recipe building process, which has completed.
Comment 1 Cristiana Voicu 2013-10-24 07:20:40 UTC
Hi Belen,

Do you have a test case to make hob fails when bulding packages? I don't have something in mind, in order to reproduce this. Can you please help?
Thanks,
Cristiana
Comment 3 Andreea Brandusa Proca 2013-11-12 13:35:20 UTC
When the build fails and I click "Edit packages", we should be returned to the tab we launched the build from. Hob should remember the tab, not the search state. 
Now we are returned to the search state, if we searched for something before we start the build.
Comment 4 Cristiana Voicu 2013-11-13 12:49:35 UTC
I've sent a new patch to refresh tables of Packages List.
Comment 6 Andreea Brandusa Proca 2013-11-18 12:41:00 UTC
Verified in master:d7ccfc7605f1eb61511b1b2b8db4ac689ce6a468.