<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100947/">http://git.reviewboard.kde.org/r/100947/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 111a9f8711f97b8b66b978fda84ccfe0ad25cfcb by Lamarque V. Souza.</pre>
<br />
<p>- Commit</p>
<br />
<p>On March 30th, 2011, 6:04 p.m., Andrey Borzenkov wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Network Management.</div>
<div>By Andrey Borzenkov.</div>
<p style="color: grey;"><i>Updated March 30, 2011, 6:04 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The patch is based on work of Gökçen Eraslan and extends it by retrofitting user connections support. Design summary:
kded gets system wide connections from NM org.freedesktop.NetworkManagerSystemSettings and reads user connections directly from disk and exports them via org.freedesktop.NetworkManagerUserSettings
Connections, received from org.freedesktop.NetworkManagerSystemSettings, are marked as having System scope; connections read from disk are marked as having User scope.
KCM module (manage connections widget) does the same. Additionally, it checks whether user marked connection as System connection and forwards it to NM or saves directly on disk.
Plasma applet is not touched and continues to receive activatables list from kded, which is now merged list of system + user connections.
The patch set of Gökçen changed parts of KNM core to act directly on Connection object to handle connections received via D-Bus. My patches additionally move connection storing up from connectioneditor into manageconnectionwidet, so lower code does not need to know whether connection is user or systemwide.
Additionally handling of VPN plugins for systemwide connections (or more general - for connections from D-Bus) is fixed.
As discussed on ml, patch is against current master and includes both Gökçen and my changes. Breakdown is available in my clone repository in branch systemwide.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I am using it myself and am able to see and use system connections. User connections work as well (tested with VPN). Creation of user connections work; system connections not tested so far. According to information I got system connections were successfully used by at least one distribution.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=204340">204340</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>backends/NetworkManager/CMakeLists.txt <span style="color: grey">(f6ff372)</span></li>
<li>backends/NetworkManager/connectiondbus.h <span style="color: grey">(428429b)</span></li>
<li>backends/NetworkManager/connectiondbus.cpp <span style="color: grey">(4a142e9)</span></li>
<li>backends/NetworkManager/generateDbusAdapters.sh <span style="color: grey">(58f2e00)</span></li>
<li>backends/NetworkManager/nm-exported-connection-secrets-interface.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>backends/NetworkManager/nm-exported-connection-secrets-interface.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>backends/NetworkManager/nmdbussettingsconnectionprovider.h <span style="color: grey">(3245f84)</span></li>
<li>backends/NetworkManager/nmdbussettingsconnectionprovider.cpp <span style="color: grey">(b799694)</span></li>
<li>backends/NetworkManager/settings/802-11-wireless-securitydbus.cpp <span style="color: grey">(4667103)</span></li>
<li>backends/NetworkManager/settings/802-11-wirelessdbus.cpp <span style="color: grey">(05bc049)</span></li>
<li>backends/NetworkManager/settings/802-1xdbus.cpp <span style="color: grey">(0707ee2)</span></li>
<li>backends/NetworkManager/settings/gsmdbus.cpp <span style="color: grey">(9912df6)</span></li>
<li>backends/NetworkManager/settings/ipv4dbus.cpp <span style="color: grey">(f77d421)</span></li>
<li>backends/NetworkManager/settings/pppoedbus.cpp <span style="color: grey">(e889177)</span></li>
<li>libs/client/remoteactivatablelist.cpp <span style="color: grey">(035531f)</span></li>
<li>libs/internals/connection.h <span style="color: grey">(8c69782)</span></li>
<li>libs/internals/connection.cpp <span style="color: grey">(fdf3431)</span></li>
<li>libs/internals/connectionpersistence.cpp <span style="color: grey">(7c52bb1)</span></li>
<li>libs/internals/settings/802-11-wireless-security.h <span style="color: grey">(a9328bb)</span></li>
<li>libs/internals/settings/802-11-wireless-security.cpp <span style="color: grey">(a803c61)</span></li>
<li>libs/internals/settings/802-11-wireless-securitypersistence.cpp <span style="color: grey">(f223d92)</span></li>
<li>libs/internals/settings/802-11-wireless.h <span style="color: grey">(1283980)</span></li>
<li>libs/internals/settings/802-1x.cpp <span style="color: grey">(f3bdb37)</span></li>
<li>libs/internals/settings/802-3-ethernet.h <span style="color: grey">(24d64a2)</span></li>
<li>libs/internals/settings/gsm.h <span style="color: grey">(b380af0)</span></li>
<li>libs/internals/settings/gsmpersistence.cpp <span style="color: grey">(ce87fbe)</span></li>
<li>libs/service/connectionlistpersistence.cpp <span style="color: grey">(b90e8cf)</span></li>
<li>libs/ui/802_11_wirelesswidget.cpp <span style="color: grey">(b654bc4)</span></li>
<li>libs/ui/cdmaconnectioneditor.h <span style="color: grey">(b3e53d3)</span></li>
<li>libs/ui/cdmaconnectioneditor.cpp <span style="color: grey">(f12638e)</span></li>
<li>libs/ui/connection.ui <span style="color: grey">(b6e9bb7)</span></li>
<li>libs/ui/connectioneditor.h <span style="color: grey">(423de5a)</span></li>
<li>libs/ui/connectioneditor.cpp <span style="color: grey">(e8831d5)</span></li>
<li>libs/ui/connectionprefs.h <span style="color: grey">(4a36dfa)</span></li>
<li>libs/ui/connectionprefs.cpp <span style="color: grey">(f532dbb)</span></li>
<li>libs/ui/connectionsecretsjob.cpp <span style="color: grey">(fce537a)</span></li>
<li>libs/ui/connectionwidget.cpp <span style="color: grey">(47daa17)</span></li>
<li>libs/ui/gsm.ui <span style="color: grey">(eeaaa41)</span></li>
<li>libs/ui/gsmconnectioneditor.h <span style="color: grey">(2a3ccb1)</span></li>
<li>libs/ui/gsmconnectioneditor.cpp <span style="color: grey">(5a26c7c)</span></li>
<li>libs/ui/gsmwidget.cpp <span style="color: grey">(82e015a)</span></li>
<li>libs/ui/networkmanagement_vpnuiplugin.desktop <span style="color: grey">(695cb6d)</span></li>
<li>libs/ui/pppoepreferences.h <span style="color: grey">(6f39150)</span></li>
<li>libs/ui/pppoepreferences.cpp <span style="color: grey">(0cabcd0)</span></li>
<li>libs/ui/security/eapmethodstack.h <span style="color: grey">(552f5c9)</span></li>
<li>libs/ui/security/eapmethodstack.cpp <span style="color: grey">(e89e887)</span></li>
<li>libs/ui/security/nullsecuritywidget.h <span style="color: grey">(a0faa46)</span></li>
<li>libs/ui/security/nullsecuritywidget.cpp <span style="color: grey">(5edc5cd)</span></li>
<li>libs/ui/security/securityeap.h <span style="color: grey">(26ed203)</span></li>
<li>libs/ui/security/securityeap.cpp <span style="color: grey">(db35037)</span></li>
<li>libs/ui/security/securityleap.h <span style="color: grey">(5d9b034)</span></li>
<li>libs/ui/security/securityleap.cpp <span style="color: grey">(95ba530)</span></li>
<li>libs/ui/security/securitywidget.h <span style="color: grey">(71ebf4a)</span></li>
<li>libs/ui/security/securitywidget.cpp <span style="color: grey">(53cdc5e)</span></li>
<li>libs/ui/security/securitywidget_p.h <span style="color: grey">(6ec8d9c)</span></li>
<li>libs/ui/security/wepwidget.h <span style="color: grey">(e78338c)</span></li>
<li>libs/ui/security/wepwidget.cpp <span style="color: grey">(88b422f)</span></li>
<li>libs/ui/security/wirelesssecuritysettingwidget.cpp <span style="color: grey">(6fff4a6)</span></li>
<li>libs/ui/security/wpapskwidget.h <span style="color: grey">(7077836)</span></li>
<li>libs/ui/security/wpapskwidget.cpp <span style="color: grey">(d2d221d)</span></li>
<li>libs/ui/vpnpreferences.h <span style="color: grey">(39dd52b)</span></li>
<li>libs/ui/vpnpreferences.cpp <span style="color: grey">(5274d14)</span></li>
<li>libs/ui/wiredpreferences.h <span style="color: grey">(59b4f16)</span></li>
<li>libs/ui/wiredpreferences.cpp <span style="color: grey">(60b7932)</span></li>
<li>libs/ui/wiredwidget.cpp <span style="color: grey">(2229f74)</span></li>
<li>libs/ui/wirelesspreferences.h <span style="color: grey">(5d6a30c)</span></li>
<li>libs/ui/wirelesspreferences.cpp <span style="color: grey">(7f901c2)</span></li>
<li>monolithic/tooltipbuilder.cpp <span style="color: grey">(a09734a)</span></li>
<li>settings/config/CMakeLists.txt <span style="color: grey">(0d4cd00)</span></li>
<li>settings/config/manageconnectionwidget.h <span style="color: grey">(c0fccd9)</span></li>
<li>settings/config/manageconnectionwidget.cpp <span style="color: grey">(15a6251)</span></li>
<li>settings/config/managetraywidget.cpp <span style="color: grey">(38680ce)</span></li>
<li>settings/config/mobileproviders.cpp <span style="color: grey">(1ef26fc)</span></li>
<li>settings/configshell/CMakeLists.txt <span style="color: grey">(d900ae7)</span></li>
<li>settings/configshell/main.cpp <span style="color: grey">(43248e2)</span></li>
<li>vpnplugins/novellvpn/networkmanagement_novellvpnui.desktop <span style="color: grey">(6bc2b56)</span></li>
<li>vpnplugins/openvpn/networkmanagement_openvpnui.desktop <span style="color: grey">(33b2b89)</span></li>
<li>vpnplugins/pptp/networkmanagement_pptpui.desktop <span style="color: grey">(95a6bf7)</span></li>
<li>vpnplugins/strongswan/networkmanagement_strongswanui.desktop <span style="color: grey">(a5b016e)</span></li>
<li>vpnplugins/vpnc/networkmanagement_vpncui.desktop <span style="color: grey">(b408112)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100947/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>