retitle 658997 gwibber-service: Crashes if D-Bus connection to NetworkManager 
fails.
tag 658997 +patch
thanks

* Johannes Brandstätter <jbrandstaet...@gmail.com> [120718 10:05]:
> Package: gwibber-service
> Version: 3.0.0.1-2.1
> Severity: important
> 
> Dear Maintainer,
> 
> I am using Xfce, so no default install of the network-manager package.
> 
> The initial error message i got from starting gwibber-service was a bit
> misleading:

I think rather than unneeded dependency, we should rather fix the
issue that introduced the error.

Patch is attached.

Thanks,
David

diff --git a/debian/patches/debbug_658997.patch b/debian/patches/debbug_658997.patch
new file mode 100644
index 0000000..1393025
--- /dev/null
+++ b/debian/patches/debbug_658997.patch
@@ -0,0 +1,37 @@
+--- a/gwibber/microblog/dispatcher.py
++++ b/gwibber/microblog/dispatcher.py
+@@ -820,20 +820,21 @@
+ 
+     self.NM_STATE_UNKNOWN = 0
+ 
+-    log.logger.debug("NM Version is %s", str(self.nm.Get(NM_DBUS_INTERFACE, "Version")))
++    if self.has_nm:
++      log.logger.debug("NM Version is %s", str(self.nm.Get(NM_DBUS_INTERFACE, "Version")))
+ 
+-    if str(self.nm.Get(NM_DBUS_INTERFACE, "Version")) >= "0.8.998":
+-      log.logger.debug("NM Version is greater than 0.8.997")
+-      self.NM_STATE_ASLEEP = 10
+-      self.NM_STATE_DISCONNECTED = 20
+-      self.NM_STATE_CONNECTING = 40
+-      self.NM_STATE_CONNECTED = 70
+-    else:
+-      log.logger.debug("NM Version is less than 0.8.998")
+-      self.NM_STATE_ASLEEP = 1
+-      self.NM_STATE_CONNECTING = 2
+-      self.NM_STATE_CONNECTED = 3
+-      self.NM_STATE_DISCONNECTED = 4
++      if str(self.nm.Get(NM_DBUS_INTERFACE, "Version")) >= "0.8.998":
++        log.logger.debug("NM Version is greater than 0.8.997")
++        self.NM_STATE_ASLEEP = 10
++        self.NM_STATE_DISCONNECTED = 20
++        self.NM_STATE_CONNECTING = 40
++        self.NM_STATE_CONNECTED = 70
++      else:
++        log.logger.debug("NM Version is less than 0.8.998")
++        self.NM_STATE_ASLEEP = 1
++        self.NM_STATE_CONNECTING = 2
++        self.NM_STATE_CONNECTED = 3
++        self.NM_STATE_DISCONNECTED = 4
+ 
+ 
+ 
diff --git a/debian/patches/series b/debian/patches/series
index e9b0350..96d29df 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -2,3 +2,4 @@ fix_facebook_ext_perms.patch
 lp_735609.patch
 lp_757451.patch
 debbug-625221.patch
+debbug_658997.patch

Reply via email to