Bug 3329

Summary: Can not close hob from command line if an additional window is opened ("Settings" or "Advanced Configuration")
Product: [Build System, Metadata & Runtime] Hob Reporter: Andreea Brandusa Proca <andreea.b.proca>
Component: hobAssignee: valentin <valentin.popa>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: andreea.b.proca, belen.barros.pena, cristian.iorga, ioanax.grigoropol, jessica.zhang, poky.bs.watcher, poky.watcher, valentin.popa
Version: 1.3   
Target Milestone: 1.5 M2   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---

Description Andreea Brandusa Proca 2012-10-24 10:24:14 UTC

    
Comment 1 Andreea Brandusa Proca 2012-10-24 10:30:58 UTC
Steps to reproduce:

1. launch hob
2. click icon "Settings"
3. from command line (from where you started hob), press CTR+C.

Hob does not stop until you close the window for "Settings".
Comment 2 Belen Barros Pena 2013-05-14 08:41:52 UTC
Apparently related to GTK model. The workaround would be killing the process, which is not advisable. So this is hard to fix.
Comment 3 valentin 2013-06-25 10:50:58 UTC
I'm back...

Hi Belen,
When a user is pressing CTRL+C he/she wants to kill the entire app, so maybe it is not bad idea to kill it after all. What do you think?
Comment 4 Belen Barros Pena 2013-07-01 09:11:00 UTC
(In reply to comment #3)
> I'm back...

Welcome back! :) 

> 
> Hi Belen,
> When a user is pressing CTRL+C he/she wants to kill the entire app, so maybe
> it is not bad idea to kill it after all. What do you think?

I agree. I think the key is responding to the user's input (Ctrl+C). The problem we currently have is that Hob doesn't respond in any way: users need to figure out by themselves they need to close the dialog to exit the application.

So yes: let's kill Hob in this case.

Thanks!
Comment 5 valentin 2013-07-03 11:41:37 UTC
patch sent on mailing list.
Comment 6 Andreea Brandusa Proca 2013-07-16 07:29:27 UTC
Verified in master:74e403705e25afec656b5e40921771a9c29bdc40.