Bug 1223

Summary: [HOB] Stop vs. Force Stop needs more visual guidance
Product: [Build System, Metadata & Runtime] Hob Reporter: Jiajun Xu <jiajun.xu>
Component: hobAssignee: Joshua Lock - Disabled <josh>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: jessica.zhang, lianhao.lu, liping.ke, poky.bs.watcher, poky.watcher
Version: unspecified   
Target Milestone: 1.1 M3   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---

Description Jiajun Xu 2011-07-07 01:39:42 UTC 
Tree/Branch: Poky/Master
Commit: d4132fa12885fc050313a5c9aa6903e4fa92c94f

If I choose some packages or images in hob and start building, then click "Stop
Build", there will be 3 options for me - "Cancel", "Stop" and "Force Stop". If
I choose "Force Stop" and select back to build something other again. It will show nothing and no any build happen.
Comment 1 Joshua Lock - Disabled 2011-07-07 10:22:08 UTC 
I am familiar with this issue however I'm not certain what to do about it. The force stop option could be removed or the dialog message could be enhanced to be more informative as to what the two options will do.

Any thoughts?
Comment 2 Joshua Lock - Disabled 2011-07-07 10:24:27 UTC 
For more information - force stop is the equivalent of hitting Ctrl-C at the console (knotty) UI until the build stops. Which can also result in an unusable build directory.

Unless anyone objects I will remove the Force Stop option. It seems right that the UI will protect the user from being unable to use it next run.
Comment 3 Jessica 2011-07-07 10:34:22 UTC 
I think that's a much clearer approach by removing the "force stop"
Comment 4 Joshua Lock - Disabled 2011-07-07 10:49:20 UTC 
Be aware that this means the build won't actually be stopped until all of the running tasks have ended. If that's a long task like packaging the kernel you may be waiting some time. This is what I added the Force Stop option in the first place.

We could possibly add a two phased Stop -> (optional) Force Stop for the impatient but I suppose we'd want to add large warnings if we did that.
Comment 5 Joshua Lock - Disabled 2011-07-19 16:11:07 UTC 
I've developed a patch which leaves both Stop and Force Stop but explains them both and their ramifications.
Comment 7 Jiajun Xu 2011-08-01 23:11:37 UTC 
verify the bug with commit 46cf540e63a848512617b20fd8492f81bfb2f704, it's fixed.