[Bugs] #616 LOW: battery device: fail gracefully if hald is not running

[SugarLabs Bugs]

#616: battery device: fail gracefully if hald is not running
-----------------------------+----------------------------------------------
    Reporter:  sascha_silbe  |          Owner:  tomeu                      
        Type:  defect        |         Status:  new                        
    Priority:  Low           |      Milestone:  Unspecified by Release Team
   Component:  sugar         |        Version:  Git as of bugdate          
    Severity:  Minor         |       Keywords:                             
Distribution:  Unspecified   |   Status_field:  New                        
-----------------------------+----------------------------------------------
 If hald is not running, the following traceback happens:

 1237823146.126622 ERROR root: Exception while loading extension:
 Traceback (most recent call last):
   File "/home/sugar/sugar-jhbuild/install/lib/python2.5/site-
 packages/jarabe/frame/devicestray.py", line 36, in __init__
     mod.setup(self)
   File "/home/sugar/sugar-
 jhbuild/install/share/sugar/extensions/deviceicon/battery.py", line 247,
 in setup
     '/org/freedesktop/Hal/Manager')
   File "/var/lib/python-support/python2.5/dbus/bus.py", line 244, in
 get_object
     follow_name_owner_changes=follow_name_owner_changes)
   File "/var/lib/python-support/python2.5/dbus/proxies.py", line 241, in
 __init__
     self._named_service = conn.activate_name_owner(bus_name)
   File "/var/lib/python-support/python2.5/dbus/bus.py", line 183, in
 activate_name_owner
     self.start_service_by_name(bus_name)
   File "/var/lib/python-support/python2.5/dbus/bus.py", line 281, in
 start_service_by_name
     'su', (bus_name, flags)))
   File "/var/lib/python-support/python2.5/dbus/connection.py", line 622,
 in call_blocking
     message, timeout)
 DBusException: org.freedesktop.DBus.Error.ServiceUnknown: The name
 org.freedesktop.Hal was not provided by any .service files


 A fallback to acpid would be great (as that one usually needs to run
 anyway for power button etc. to work).
 At the very least, it should catch the exception, though.

-- 
Ticket URL: <http://dev.sugarlabs.org/ticket/616>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system