Review Request 128843: Use standard org.freedesktop.DBus.Properties interfaces for monitoring PropertiesChanged signal for NM 1.4.0+

Jan Grulich jgrulich at redhat.com
Tue Sep 6 11:56:37 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128843/
-----------------------------------------------------------

(Updated Zář. 6, 2016, 11:56 dop.)


Review request for KDE Frameworks, Network Management and Lamarque Souza.


Changes
-------

We need to also call ActiveConnection::propertiesChanged() for VpnConnection which iherits from it, otherwise we won't be notified about changed state and so on.


Repository: networkmanager-qt


Description
-------

Since NM 1.2.x they switched to using o.f.DBus.Properties for sending PropertiesChanged signal. They still support their previous solution which uses their o.f.NetworkManager.Foo interfaces, but with release of NM 1.4.0 this is a bit broken (altough already fixed in upstream). With the fix for that they marked PropertiesChanged signal for their o.f.NM.Foo interfaces as deprecated and we should be slowly moving towards the standardized o.f.DBus.Properties interface instead.

See https://bugzilla.gnome.org/show_bug.cgi?id=770629


Diffs (updated)
-----

  src/accesspoint.cpp d1abe31 
  src/accesspoint_p.h bf111c8 
  src/activeconnection.cpp fc8a1ac 
  src/activeconnection_p.h d021437 
  src/adsldevice.cpp f07b5e6 
  src/bluetoothdevice.cpp 3ccf276 
  src/bonddevice.cpp d9798b0 
  src/bridgedevice.cpp f345ad5 
  src/connection.cpp 5b80a93 
  src/connection_p.h 5f43d93 
  src/device.cpp f7bc7e6 
  src/device_p.h da85faf 
  src/dhcp4config.cpp efcd98d 
  src/dhcp4config_p.h ce5563c 
  src/dhcp6config.cpp 4f0a3b6 
  src/dhcp6config_p.h 093f234 
  src/genericdevice.cpp a6919d5 
  src/gredevice.cpp 1f81ea4 
  src/infinibanddevice.cpp 15b3575 
  src/macvlandevice.cpp db2c594 
  src/manager.cpp 01723d7 
  src/manager_p.h d0a9d8c 
  src/modemdevice.cpp 89394f0 
  src/olpcmeshdevice.cpp 286a080 
  src/settings.cpp f75b86a 
  src/settings_p.h 466d0a7 
  src/teamdevice.cpp 543d26d 
  src/tundevice.cpp 68eb103 
  src/vethdevice.cpp bb7c9fd 
  src/vlandevice.cpp 415dec9 
  src/vpnconnection.cpp d2d17e4 
  src/vpnconnection_p.h 25ba589 
  src/wimaxdevice.cpp daec0a6 
  src/wireddevice.cpp af147b1 
  src/wirelessdevice.cpp dad918d 

Diff: https://git.reviewboard.kde.org/r/128843/diff/


Testing
-------


Thanks,

Jan Grulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160906/8649b6ff/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list