Bug 3329 - Can not close hob from command line if an additional window is opened ("Settings" or "Advanced Configuration")
Summary: Can not close hob from command line if an additional window is opened ("Setti...
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: 1.3
Hardware: x86 Multiple
: Medium normal
Target Milestone: 1.5 M2
Assignee: valentin
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-10-24 10:24 UTC by Andreea Brandusa Proca
Modified: 2013-07-16 07:29 UTC (History)
8 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.