[network-manager-applet] Revert "bridge: add support for group-forward-mask property"



commit 3721371ee8b9762725ddd163204fdaa086812b33
Author: Thomas Haller <thaller redhat com>
Date:   Mon Aug 21 11:30:20 2017 +0200

    Revert "bridge: add support for group-forward-mask property"
    
    The patch makes use of libnm-1-10 API. At this moment, master of
    nm-applet is still supposed to build against 1.8 (current stable).
    
    This should be either resolved by making the use of 1.10 API condtional
    (either at compile time or at runtime), or by bumping the required libnm
    version. For the moment, revert the patch that breaks build.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=786537
    
    This reverts commit 4a780cc25d69d2dce983a218f5eb435b696a86df.

 src/connection-editor/ce-page-bridge.ui |   35 -------------------------------
 src/connection-editor/page-bridge.c     |   15 +-----------
 2 files changed, 2 insertions(+), 48 deletions(-)
---
diff --git a/src/connection-editor/ce-page-bridge.ui b/src/connection-editor/ce-page-bridge.ui
index ddb19a9..639b133 100644
--- a/src/connection-editor/ce-page-bridge.ui
+++ b/src/connection-editor/ce-page-bridge.ui
@@ -32,11 +32,6 @@
     <property name="step_increment">1</property>
     <property name="page_increment">10</property>
   </object>
-  <object class="GtkAdjustment" id="bridge_group_fwd_mask_adjustment">
-    <property name="upper">65535</property>
-    <property name="step_increment">1</property>
-    <property name="page_increment">10</property>
-  </object>
   <object class="GtkListStore" id="master_connections_model">
     <columns>
       <!-- column-name connection -->
@@ -468,35 +463,5 @@
         <property name="top_attach">3</property>
       </packing>
     </child>
-    <child>
-      <object class="GtkLabel" id="bridge_group_fwd_mask_label">
-        <property name="visible">True</property>
-        <property name="can_focus">False</property>
-        <property name="label" translatable="yes">Group _forward mask:</property>
-        <property name="use_underline">True</property>
-        <property name="mnemonic_widget">bridge_group_fwd_mask</property>
-        <property name="xalign">0</property>
-      </object>
-      <packing>
-        <property name="left_attach">0</property>
-        <property name="top_attach">10</property>
-      </packing>
-    </child>
-    <child>
-      <object class="GtkSpinButton" id="bridge_group_fwd_mask">
-        <property name="visible">True</property>
-        <property name="can_focus">True</property>
-        <property name="hexpand">True</property>
-        <property name="invisible_char">●</property>
-        <property name="text">0</property>
-        <property name="adjustment">bridge_group_fwd_mask_adjustment</property>
-        <property name="climb_rate">1</property>
-        <property name="numeric">True</property>
-      </object>
-      <packing>
-        <property name="left_attach">1</property>
-        <property name="top_attach">10</property>
-      </packing>
-    </child>
   </object>
 </interface>
diff --git a/src/connection-editor/page-bridge.c b/src/connection-editor/page-bridge.c
index 0a8870a..62881bf 100644
--- a/src/connection-editor/page-bridge.c
+++ b/src/connection-editor/page-bridge.c
@@ -42,7 +42,6 @@ typedef struct {
        GtkSpinButton *forward_delay;
        GtkSpinButton *hello_time;
        GtkSpinButton *max_age;
-       GtkSpinButton *group_fwd_mask;
 
 } CEPageBridgePrivate;
 
@@ -61,7 +60,6 @@ bridge_private_init (CEPageBridge *self)
        priv->forward_delay = GTK_SPIN_BUTTON (gtk_builder_get_object (builder, "bridge_forward_delay"));
        priv->hello_time = GTK_SPIN_BUTTON (gtk_builder_get_object (builder, "bridge_hello_time"));
        priv->max_age = GTK_SPIN_BUTTON (gtk_builder_get_object (builder, "bridge_max_age"));
-       priv->group_fwd_mask = GTK_SPIN_BUTTON (gtk_builder_get_object (builder, "bridge_group_fwd_mask"));
 
        priv->toplevel = GTK_WINDOW (gtk_widget_get_ancestor (GTK_WIDGET (priv->stp),
                                                              GTK_TYPE_WINDOW));
@@ -100,7 +98,7 @@ populate_ui (CEPageBridge *self)
        NMSettingBridge *s_bridge = priv->setting;
        gboolean stp, mcast_snoop;
        int priority, forward_delay, hello_time, max_age;
-       int ageing_time, group_fwd_mask;
+       int ageing_time;
 
        /* Ageing time */
        ageing_time = nm_setting_bridge_get_ageing_time (s_bridge);
@@ -149,13 +147,6 @@ populate_ui (CEPageBridge *self)
        g_signal_connect (priv->max_age, "value-changed",
                          G_CALLBACK (stuff_changed),
                          self);
-
-       /* Group forward mask */
-       group_fwd_mask = nm_setting_bridge_get_group_forward_mask (s_bridge);
-       gtk_spin_button_set_value (priv->group_fwd_mask, (gdouble) group_fwd_mask);
-       g_signal_connect (priv->group_fwd_mask, "value-changed",
-                         G_CALLBACK (stuff_changed),
-                         self);
 }
 
 static void
@@ -244,16 +235,14 @@ ui_to_setting (CEPageBridge *self)
 {
        CEPageBridgePrivate *priv = CE_PAGE_BRIDGE_GET_PRIVATE (self);
        int ageing_time, priority, forward_delay, hello_time, max_age;
-       gboolean stp, mcast_snoop, group_fwd_mask;
+       gboolean stp, mcast_snoop;
 
        ageing_time = gtk_spin_button_get_value_as_int (priv->ageing_time);
-       group_fwd_mask = gtk_spin_button_get_value_as_int (priv->group_fwd_mask);
        mcast_snoop = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->mcast_snoop));
        stp = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (priv->stp));
        g_object_set (G_OBJECT (priv->setting),
                      NM_SETTING_BRIDGE_AGEING_TIME, ageing_time,
                      NM_SETTING_BRIDGE_MULTICAST_SNOOPING, mcast_snoop,
-                     NM_SETTING_BRIDGE_GROUP_FORWARD_MASK, group_fwd_mask,
                      NM_SETTING_BRIDGE_STP, stp,
                      NULL);
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]