Hello community,

here is the log from the commit of package firewalld for openSUSE:Factory 
checked in at 2016-06-19 10:49:54
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/firewalld (Old)
 and      /work/SRC/openSUSE:Factory/.firewalld.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "firewalld"

Changes:
--------
--- /work/SRC/openSUSE:Factory/firewalld/firewalld.changes      2016-06-02 
12:39:19.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.firewalld.new/firewalld.changes 2016-06-19 
10:49:56.000000000 +0200
@@ -1,0 +2,7 @@
+Wed Jun  8 08:10:11 UTC 2016 - mchand...@suse.de
+
+- Add upstream patch to prevent unconditional dependencies to the
+  NetworkManager typelib (gh#t-woerner/firewalld#119)
+  * 0003-firewall.core.fw_nm-Hide-NM-typelib-import-new-nm_ge.patch
+
+-------------------------------------------------------------------

New:
----
  0003-firewall.core.fw_nm-Hide-NM-typelib-import-new-nm_ge.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ firewalld.spec ++++++
--- /var/tmp/diff_new_pack.ytzP1V/_old  2016-06-19 10:49:57.000000000 +0200
+++ /var/tmp/diff_new_pack.ytzP1V/_new  2016-06-19 10:49:57.000000000 +0200
@@ -30,6 +30,9 @@
 # PATCH-FIX-UPSTREAM: 
0002-firewall-core-fw_ifcfg-Quickly-return-if-ifcfg-direc.patch
 # Do not access the ifcfg directory if it does not exist
 Patch1:         0002-firewall-core-fw_ifcfg-Quickly-return-if-ifcfg-direc.patch
+# PATCH-FIX-UPSTREAM: 
0003-firewall.core.fw_nm-Hide-NM-typelib-import-new-nm_ge.patch
+# Hide unneeded NM dependencies
+Patch2:         0003-firewall.core.fw_nm-Hide-NM-typelib-import-new-nm_ge.patch
 BuildRequires:  desktop-file-utils
 BuildRequires:  docbook-xsl-stylesheets
 BuildRequires:  gettext
@@ -92,6 +95,7 @@
 %setup -q
 %patch0 -p1
 %patch1 -p1
+%patch2 -p1
 
 %build
 %configure --enable-sysconfig --enable-rpmmacros

++++++ 0003-firewall.core.fw_nm-Hide-NM-typelib-import-new-nm_ge.patch ++++++
>From dbc30dd2dfee1b4b662aa997cee6a6cb59aa559f Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoer...@redhat.com>
Date: Tue, 7 Jun 2016 16:19:20 +0200
Subject: [PATCH] firewall.core.fw_nm: Hide NM typelib import, new
 nm_get_dbus_interface function

The typelib import of NM should not be added to __all__. The new function
nm_get_dbus_interface has been added to be able to add a signal receiver for
NetworkManager firewall-config and firewall-applet.

Fixes: #119
---
 src/firewall-applet        | 2 +-
 src/firewall-config        | 2 +-
 src/firewall/core/fw_nm.py | 9 +++++++--
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/firewall-applet b/src/firewall-applet
index a078df8..1cf4633 100755
--- a/src/firewall-applet
+++ b/src/firewall-applet
@@ -550,7 +550,7 @@ class TrayApplet(QtGui.QSystemTrayIcon):
         if nm_is_imported():
             self.bus.add_signal_receiver(
                 self.nm_signal_receiver,
-                dbus_interface=NM.DBUS_INTERFACE,
+                dbus_interface=nm_get_dbus_interface(),
                 signal_name='PropertiesChanged',
                 member_keyword='member')
         self.nm_signal_receiver()
diff --git a/src/firewall-config b/src/firewall-config
index b69c69f..e25b058 100755
--- a/src/firewall-config
+++ b/src/firewall-config
@@ -1250,7 +1250,7 @@ class FirewallConfig(object):
         if nm_is_imported():
             self.fw.bus.add_signal_receiver(
                 self.nm_signal_receiver,
-                dbus_interface=NM.DBUS_INTERFACE,
+                dbus_interface=nm_get_dbus_interface(),
                 signal_name='PropertiesChanged',
                 member_keyword='member')
         else:
diff --git a/src/firewall/core/fw_nm.py b/src/firewall/core/fw_nm.py
index 3df94c4..8a8043b 100644
--- a/src/firewall/core/fw_nm.py
+++ b/src/firewall/core/fw_nm.py
@@ -21,10 +21,10 @@
 
 """Functions for NetworkManager interaction"""
 
-__all__ = [ "NM", "check_nm_imported", "nm_is_imported",
+__all__ = [ "check_nm_imported", "nm_is_imported",
             "nm_get_zone_of_connection", "nm_set_zone_of_connection",
             "nm_get_connections", "nm_get_connection_of_interface",
-            "nm_get_bus_name" ]
+            "nm_get_bus_name", "nm_get_dbus_interface" ]
 
 import gi
 try:
@@ -164,3 +164,8 @@ def nm_get_bus_name():
     except Exception as msg:
         log.debug2("Failed to get bus name of NetworkManager")
     return None
+
+def nm_get_dbus_interface():
+    if not _nm_imported:
+        return ""
+    return NM.DBUS_INTERFACE
-- 
2.8.3


Reply via email to