guake crashed with ShowableError in reload_globals()

Bug #1018643 reported by Alan Neilan
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
guake (Ubuntu)
Expired
Undecided
Unassigned

Bug Description

i was using it and there was an issue with the keybinding

ProblemType: Crash
DistroRelease: Ubuntu 12.10
Package: guake 0.4.3-2
ProcVersionSignature: Ubuntu 3.5.0-2.2-generic 3.5.0-rc4
Uname: Linux 3.5.0-2-generic x86_64
ApportVersion: 2.2.5-0ubuntu1
Architecture: amd64
Date: Wed Jun 27 18:51:31 2012
ExecutablePath: /usr/bin/guake
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha amd64 (20120627)
InterpreterPath: /usr/bin/python2.7
ProcCmdline: python /usr/bin/guake
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
PythonArgs: ['/usr/bin/guake']
SourcePackage: guake
Title: guake crashed with ShowableError in reload_globals()
Traceback:
 Traceback (most recent call last):
   File "/usr/bin/guake", line 377, in reload_globals
     -1)
 ShowableError
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo

Revision history for this message
Alan Neilan (alanwneilan) wrote :
tags: removed: need-duplicate-check
Revision history for this message
Sasa Paporovic (melchiaros) wrote :

Please give a step by step description like pushing this button, choosing that, waiting for, and so on). Give a good algorithmic description, that is complete and can guide a beginner in guake to the point where the problem happends.

(step1,
 step2,
.
.
.
finished)

After that set the status of this report back to new.

Changed in guake (Ubuntu):
status: New → Incomplete
Revision history for this message
Alan Neilan (alanwneilan) wrote : Re: [Bug 1018643] Re: guake crashed with ShowableError in reload_globals()

it seems to have only happened once..

On Sat, Jun 30, 2012 at 10:17 AM, melchiaros <email address hidden>wrote:

> Please give a step by step description like pushing this button,
> choosing that, waiting for, and so on). Give a good algorithmic
> description, that is complete and can guide a beginner in guake to the
> point where the problem happends.
>
> (step1,
> step2,
> .
> .
> .
> finished)
>
> After that set the status of this report back to new.
>
> ** Changed in: guake (Ubuntu)
> Status: New => Incomplete
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1018643
>
> Title:
> guake crashed with ShowableError in reload_globals()
>
> Status in “guake” package in Ubuntu:
> Incomplete
>
> Bug description:
> i was using it and there was an issue with the keybinding
>
> ProblemType: Crash
> DistroRelease: Ubuntu 12.10
> Package: guake 0.4.3-2
> ProcVersionSignature: Ubuntu 3.5.0-2.2-generic 3.5.0-rc4
> Uname: Linux 3.5.0-2-generic x86_64
> ApportVersion: 2.2.5-0ubuntu1
> Architecture: amd64
> Date: Wed Jun 27 18:51:31 2012
> ExecutablePath: /usr/bin/guake
> InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha amd64
> (20120627)
> InterpreterPath: /usr/bin/python2.7
> ProcCmdline: python /usr/bin/guake
> ProcEnviron:
> PATH=(custom, no user)
> LANG=en_US.UTF-8
> SHELL=/bin/bash
> PythonArgs: ['/usr/bin/guake']
> SourcePackage: guake
> Title: guake crashed with ShowableError in reload_globals()
> Traceback:
> Traceback (most recent call last):
> File "/usr/bin/guake", line 377, in reload_globals
> -1)
> ShowableError
> UpgradeStatus: No upgrade log present (probably fresh install)
> UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu/+source/guake/+bug/1018643/+subscriptions
>

Revision history for this message
consolation (midden-heap) wrote :

go to change key binding to "super+`"
hit couple of times to try out
crash...

also, binding "ctrl+`" is equivalent to "`"

Revision history for this message
consolation (midden-heap) wrote :

on further testing, crash occurs when guake key bindings overlap with system key bindings...

Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for guake (Ubuntu) because there has been no activity for 60 days.]

Changed in guake (Ubuntu):
status: Incomplete → Expired
Changed in guake (Ubuntu):
status: Expired → New
Revision history for this message
anatoly techtonik (techtonik) wrote :

Trying to set Win-` as a hot key crashed it for me on Ubuntu 13.04.

Step by step:
1. F12 - bring Guake to front
2. Right click, Preferences
3. Keyboard shortcuts
4. Toggle Guake visibility
5. Press Win-`
6. Watch
     1. label changes to Super+`
     2. error dialog box "key binding error" appears
7. Close, Close
8. Press Win-`, F12, whatever. Shoud crash.

-- console output --
** (guake:18429): WARNING **: Binding '<Super>grave' failed!

/usr/lib/python2.7/dist-packages/guake/common.py:46: GtkWarning: Failed to set text from markup due to error parsing markup: Error on line 1 char 49: Element 'b' was closed, but the currently open element is 'Super'
  d.format_secondary_markup(msg)
/usr/lib/python2.7/dist-packages/guake/common.py:47: GtkWarning: Failed to set text from markup due to error parsing markup: Error on line 1 char 49: Element 'b' was closed, but the currently open element is 'Super'
  d.run()
Traceback (most recent call last):
  File "/usr/bin/guake", line 381, in reload_globals
    -1)
guake.common.ShowableError

Revision history for this message
Jadrian Miles (jadrian.miles) wrote :

I can confirm techtonik's bug, described above in their post from 2013-06-21.

Confirmed under Ubuntu 12.04 with Unity, both with the current version of Guake from the repository (0.4.2) and current master copy of Guake from github (cloned on 2014-06-26; last update 2014-05-02).

Specifically, my way to reproduce it is to try to set Win-space as the "Toggle Guake visibility" shortcut.

I googled around and also took a look at the Guake source code. The problem is twofold:

1. Unity, by default, reserves the Super (Win) key as the global shortcut for showing the Unity launcher. Therefore any global shortcuts that use an unmodified Super will conflict. There's a simple workaround (described here: http://askubuntu.com/questions/34145/can-i-define-keyboard-shortcuts-using-the-super-key ) --- namely, use the CompizConfig Settings Manager to change the launcher shortcut (I chose Alt-space). If you do this, then Guake will accept Win-space as a Show/Hide shortcut without complaint.

2. Let's say you don't perform this workaround, though. When the conflict occurs, Guake tries to process this fact and create a warning dialog box. This happens within the reload_globals() function in guake (around line 381 in the version from the Ubuntu repository, as indicated in the bug report above). This function raises a ShowableError, which later gets processed to produce a dialog box. The second argument to the ShowableError constructor can be a simple HTML string (to provide bold and italic formatting), but the string used to represent "Super" in the key combination is "<Super>". Hence the HTML-parsing warnings from Gtk, and the weird, empty dialog box that results.

The solution is simply to XML-encode the name of the key combination, so it turns into "&lt;Super&gt;" (for example) instead.

I've submitted a pull request to the Guake developers to fix this issue.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in guake (Ubuntu):
status: New → Confirmed
Revision history for this message
Robert Matusewicz (matekm) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. We are sorry that we do not always have the capacity to look at all reported bugs in a timely manner. There have been many changes in Ubuntu since that time you reported the bug and your problem may have been fixed with some of the updates. It would help us a lot if you could test it on a currently supported Ubuntu version. When you test it and it is still an issue, kindly upload the updated logs by running only once:
apport-collect 1018643

Changed in guake (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for guake (Ubuntu) because there has been no activity for 60 days.]

Changed in guake (Ubuntu):
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.