> On Pro. 1, 2014, 10:38 dop., Lamarque Souza wrote: > > src/accesspoint.h, line 172 > > <https://git.reviewboard.kde.org/r/121304/diff/1/?file=331243#file331243line172> > > > > This patch includes binary incompatible changes, you have to bump > > library SO version too. Hmmm, SO version is already 5, what is Frameworks > > policy for SO version? Can we change NMQt's SO version to 6 now? Morever, > > are we allowed to push binary incompatible changes to NMQt now? NMQt is not > > included in KF5 yet, so I guess we can, right?
I guess we can. NMQT is released with Plasma 5 just because it is required for plasma-nm. It is not released as a framework yet. - Jan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121304/#review71149 ----------------------------------------------------------- On Pro. 1, 2014, 9:30 dop., Jan Grulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121304/ > ----------------------------------------------------------- > > (Updated Pro. 1, 2014, 9:30 dop.) > > > Review request for Network Management, Lukáš Tinkl and Lamarque Souza. > > > Repository: libnm-qt > > > Description > ------- > > As David Edmunson suggested, I removed all propertiesChanged() slots from all > classes and moved them to their private classes. This needs to be properly > checked if I didn't break anything. > > I also properly implemented all virtual devices to match the implementation > of other devices. > > > Diffs > ----- > > src/wireddevice_p.h 72411a2 > src/wirelessdevice.h ce68244 > src/wirelessdevice.cpp c80ff22 > src/wirelessdevice_p.h 63c403b > src/wirelessnetwork.h b78a64b > src/vlandevice.cpp 8f5a1fc > src/vlandevice_p.h PRE-CREATION > src/vpnconnection.h eae7a26 > src/vpnconnection.cpp 3015835 > src/vpnconnection_p.h PRE-CREATION > src/wimaxdevice.h c7a616c > src/wimaxdevice.cpp 4d9cb66 > src/wimaxdevice_p.h 1aa767b > src/wimaxnsp.h 92f114c > src/wimaxnsp.cpp 47ca40b > src/wimaxnsp_p.h PRE-CREATION > src/wireddevice.h c3a91da > src/wireddevice.cpp 7835763 > src/infinibanddevice.cpp ea04ebd > src/infinibanddevice_p.h PRE-CREATION > src/macvlandevice.h 19111c1 > src/macvlandevice.cpp ef7f521 > src/macvlandevice_p.h PRE-CREATION > src/modemdevice.h 0205f48 > src/modemdevice.cpp 6d6ba04 > src/modemdevice_p.h beaf2a6 > src/olpcmeshdevice.h b08813b > src/olpcmeshdevice.cpp 814026c > src/olpcmeshdevice_p.h da436d9 > src/teamdevice.h 85ab0f6 > src/teamdevice.cpp 8de1df1 > src/teamdevice_p.h PRE-CREATION > src/tundevice.h fcd1138 > src/tundevice.cpp dcfdbe4 > src/tundevice_p.h PRE-CREATION > src/vethdevice.h bcf17d8 > src/vethdevice.cpp edab51e > src/vethdevice_p.h PRE-CREATION > src/vlandevice.h eca01bf > src/genericdevice.h d954c9e > src/genericdevice.cpp d940f42 > src/genericdevice_p.h PRE-CREATION > src/gredevice.h 6472b0e > src/gredevice.cpp d9468aa > src/gredevice_p.h PRE-CREATION > src/infinibanddevice.h 152c773 > src/dhcp4config.cpp da92b02 > src/dhcp4config_p.h 8c1bb68 > src/dhcp6config.h 7f4d528 > src/dhcp6config.cpp a4a41fc > src/dhcp6config_p.h e287d6a > src/bonddevice.cpp d83b141 > src/bonddevice_p.h PRE-CREATION > src/bridgedevice.h a0fb3ac > src/bridgedevice.cpp e9dae46 > src/bridgedevice_p.h PRE-CREATION > src/connection.h 8b83f29 > src/connection.cpp a2b8afd > src/connection_p.h PRE-CREATION > src/device.h 2013d1f > src/device.cpp b87b0a8 > src/device_p.h 059f6ef > src/dhcp4config.h 9f87b10 > src/bluetoothdevice.cpp e22bdcf > src/bluetoothdevice_p.h 201944a > src/bonddevice.h 5756fc4 > src/accesspoint.h b4b08e7 > src/accesspoint.cpp b058bbd > src/accesspoint_p.h PRE-CREATION > src/activeconnection.h 91c192c > src/activeconnection.cpp 278ab12 > src/activeconnection_p.h fb726af > src/adsldevice.h fb4f44b > src/adsldevice.cpp 102db82 > src/adsldevice_p.h PRE-CREATION > src/bluetoothdevice.h ac7014a > > Diff: https://git.reviewboard.kde.org/r/121304/diff/ > > > Testing > ------- > > I'm using it now with plasma-nm and it seems to work as before. > > > Thanks, > > Jan Grulich > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
