> On Feb. 17, 2011, 3:35 p.m., Sebastian Kügler wrote: > > Patch looks good, apart from some minor coding style issues. Please fix > > those and commit. > > > > (I haven't tested it, but trust those that have =)
done, thanks! http://quickgit.kde.org/?p=networkmanagement.git&a=commit&h=e6a2b8626c998f5ebc2e6bf0bf927462201e039d - Rex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100674/#review1478 ----------------------------------------------------------- On Feb. 17, 2011, 3:24 p.m., Rex Dieter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100674/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2011, 3:24 p.m.) > > > Review request for Network Management. > > > Summary > ------- > > BSSID is provided in a bad format to NM daemon, which causes connection > failures. Also, MAC address is not sent correctly > when you use "Restrict to Interface" combo. For WiFi that causes connection > failure, for wired the setting is just > ignored. > > See also, https://bugs.kde.org/show_bug.cgi?id=238046 > > > Diffs > ----- > > backends/NetworkManager/settingdbus.h e0ef8ba > backends/NetworkManager/settingdbus.cpp 20a4dc9 > backends/NetworkManager/settings/802-11-wirelessdbus.cpp 3ba8964 > backends/NetworkManager/settings/802-3-ethernetdbus.cpp 7d730c1 > libs/ui/802_11_wirelesswidget.cpp e671443 > > Diff: http://git.reviewboard.kde.org/r/100674/diff > > > Testing > ------- > > Patch has been tested in the wild in fedora for awhile, a gentoo user in > aforementioned bug reports success as well. > > > Thanks, > > Rex > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
