> On June 22, 2012, 2:16 p.m., Lamarque Vieira Souza wrote: > > backends/NetworkManager/connectiondbus.cpp, line 195 > > <http://git.reviewboard.kde.org/r/105324/diff/1/?file=70133#file70133line195> > > > > Some NetworkManager settings when empty strings must not be added to > > the dbus map or NetworkManager invalidates the connection. Is this the case > > for NM_SETTING_CONNECTION_ZONE? If so then you must check if it is empty > > and do not add it here if it is empty. > > > > I cannot answer if that is the case for NM_SETTING_CONNECTION_ZONE > > because that seems to be a implicit rule not described in the > > NetworkManager specification. You must look into NetworkManager source code > > to check if it the case for NM_SETTING_CONNECTION_ZONE. > > Jiri Popelka wrote: > Actually yes, current NetworkManager invalidates the connection when > NM_SETTING_CONNECTION_ZONE is empty. But it's a bug (caused by me) and I > already sent patch to > https://mail.gnome.org/archives/networkmanager-list/2012-June/msg00087.html > > Lamarque Vieira Souza wrote: > Anyway, Plasma NM needs to support older versions of NetworkManager > (distributions are not that eager to use the latest NM version), so I think > it is better not add NM_SETTING_CONNECTION_ZONE to the map if it is empty. > > Jiri Popelka wrote: > We need to be able to add empty NM_SETTING_CONNECTION_ZONE to the map. > That's how we specify 'Default' zone. > > Lamarque Vieira Souza wrote: > Your fix in the link above is going to be added into which NetworkManager > version? Is there any released NetworkManager with that bug? If so then we > need to add a warning somewhere that the user must use NetworkManager x.y.z. > > Jiri Popelka wrote: > NetworkManager-0.9.4 has this bug. I hope it's going to be fixed in next > version, so 0.9.6 I guess. From my point of view it's not a big deal because > to use these 'network zones' in NetworkManager you also need to have > FirewallD running and Fedora is at the moment the only distro, which uses it. > I think it'll take some time before other distributions notice FirewallD. And > regarding Fedora: NetworkManager package in Fedora is done from git snapshots > so it'll probably fixed quite soon there. > > Lamarque Vieira Souza wrote: > Does firewalld support pkgconfig? If so we can add a check in the main > CMakeLists.txt and print a warning if it is found and NetworkManager version > <= 0.9.4. Just for the distribution packagers be aware of the version we > require to run properly.
No, it doesn't AFAIK. - Jiri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105324/#review14997 ----------------------------------------------------------- On June 22, 2012, 9:54 a.m., Lukáš Tinkl wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105324/ > ----------------------------------------------------------- > > (Updated June 22, 2012, 9:54 a.m.) > > > Review request for Network Management and Lamarque Vieira Souza. > > > Description > ------- > > NetworkManager has had support for FirewallD [1] and Network Zones [2] > since [3]. > > This patch adds one combo box that enables to change the network zone. > This box is visible only when FirewallD is running. > > [1] https://fedorahosted.org/firewalld/ > [2] https://fedoraproject.org/wiki/Features/network-zones > [3] > http://cgit.freedesktop.org/NetworkManager/NetworkManager/commit/?id=1c0aeb457546ac5e485ca21dfe23f1a009912701 > > (on behalf of Jiri Popelka <[email protected]>) > > > Diffs > ----- > > backends/NetworkManager/connectiondbus.cpp 4649a19 > libs/internals/connection.h 485b810 > libs/internals/connection.cpp 67f31bc > libs/ui/connection.ui 7b9f873 > libs/ui/connectionwidget.h aac7050 > libs/ui/connectionwidget.cpp f67f157 > > Diff: http://git.reviewboard.kde.org/r/105324/diff/ > > > Testing > ------- > > > Thanks, > > Lukáš Tinkl > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
