gen2brain / volti

Volti is GTK+ application for controlling audio volume from system tray/notification area
GNU General Public License v3.0
28 stars 8 forks source link

Volti freeze with awesome window manager #24

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. use window manager awesome with the notification library naughty
2. launch volti
3. reload awesome (the notification library is reloaded to)
4. change the volume

What is the expected output? What do you see instead?
Volti freeze. I have a DBusException:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/volti/main.py", line 192, in on_volume_changed
    self.update_notify(vol, icon)
  File "/usr/lib/python2.7/site-packages/volti/main.py", line 318, in update_notify
    self.notify.show(icon, self.notify_body, self.notify_timeout, volume)
  File "/usr/lib/python2.7/site-packages/volti/notification.py", line 45, in show
    self.last_id = self.notify.Notify('volume', self.last_id, icon, self.title, body, [], hints, duration * 1000)
  File "/usr/lib/python2.7/site-packages/dbus/proxies.py", line 140, in __call__
    **keywords)
  File "/usr/lib/python2.7/site-packages/dbus/connection.py", line 630, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Error.ServiceUnknown: The 
name :1.129 was not provided by any .service files

What version of the product are you using? On what operating system?
I try with the last stable: 0.2.2 or the svn version.
I am using python 2.7 on Archlinux.

I write a small patch to catch the exception and re-initialize the notification 
object.

Original issue reported on code.google.com by cdemoulins on 13 Dec 2010 at 4:49

Attachments:

GoogleCodeExporter commented 9 years ago
A more clean patch.

Original comment by cdemoulins on 13 Dec 2010 at 4:58

Attachments:

GoogleCodeExporter commented 9 years ago
Hi, I just commited your change. I once also noticed a strange behaviour when I 
changed theme for notification-daemon, I think this should work for other 
cases, not just for awesome. Thanks for patch!

Original comment by gen2br...@gmail.com on 13 Dec 2010 at 6:49

GoogleCodeExporter commented 9 years ago

Original comment by gen2br...@gmail.com on 29 Dec 2010 at 9:46