summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorpoma <pomidorabelisima@gmail.com>2016-11-25 16:47:34 +0100
committerFrancesco Giudici <fgiudici@redhat.com>2016-11-25 17:07:28 +0100
commit037b61b71ccc8126e2a65504deaf0207fb7e3de4 (patch)
tree3dc8f38b5cc483e8061689ef2e45c778657fe66d
parent0dc0af15ee9ee71f2a711101a139d50c9bd97e32 (diff)
downloadnetwork-manager-applet-037b61b71ccc8126e2a65504deaf0207fb7e3de4.tar.gz
c-e: avoid setting "full" as default duplex value.
NetworkManager 802-3 link properties have been implemented in: https://cgit.freedesktop.org/NetworkManager/NetworkManager/commit/?id=e0c50a97037 'auto-negotiate' default value has been also changed to "false", as together with 'speed' = 0 and duplex unset (NULL) instructs NetworkManager to skip link configuration. Before the implementation, the 'auto-negotiate', 'speed' and 'duplex' 802-3 properties were ignored, without regards of the value they were set. The duplex property in particular was silently set to "full" by c-e for each connection modified. This triggers NetworkManager to force a static link configuration with duplex mode and unspecified speed. [fgiudici@redhat.com: rewritten the commit message]
-rw-r--r--src/connection-editor/page-ethernet.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/connection-editor/page-ethernet.c b/src/connection-editor/page-ethernet.c
index 8593f822..b0e100c1 100644
--- a/src/connection-editor/page-ethernet.c
+++ b/src/connection-editor/page-ethernet.c
@@ -446,7 +446,6 @@ ui_to_setting (CEPageEthernet *self)
NM_SETTING_WIRED_CLONED_MAC_ADDRESS, cloned_mac && *cloned_mac ? cloned_mac : NULL,
NM_SETTING_WIRED_PORT, port,
NM_SETTING_WIRED_SPEED, speed,
- NM_SETTING_WIRED_DUPLEX, gtk_toggle_button_get_active (priv->duplex) ? "full" : "half",
NM_SETTING_WIRED_AUTO_NEGOTIATE, gtk_toggle_button_get_active (priv->autonegotiate),
NM_SETTING_WIRED_WAKE_ON_LAN, wol,
NM_SETTING_WIRED_WAKE_ON_LAN_PASSWORD, wol_passwd && *wol_passwd ? wol_passwd : NULL,