Review Request 128093: Make network manager version checks in runtime (to avoid compile vs. run-time discrepancies)
Lamarque Souza
lamarque at kde.org
Sat Jul 2 10:06:39 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128093/#review97015
-----------------------------------------------------------
src/manager.cpp (line 407)
<https://git.reviewboard.kde.org/r/128093/#comment65546>
This should be swapped:
return checkVersion(1, 2, 0) ? false : m_isWimaxEnabled;
src/manager.cpp (line 412)
<https://git.reviewboard.kde.org/r/128093/#comment65547>
Here too.
- Lamarque Souza
On June 19, 2016, 8:03 p.m., Palo Kisa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128093/
> -----------------------------------------------------------
>
> (Updated June 19, 2016, 8:03 p.m.)
>
>
> Review request for Network Management, Jan Grulich and Lamarque Souza.
>
>
> Repository: networkmanager-qt
>
>
> Description
> -------
>
> https://bugs.kde.org/show_bug.cgi?id=362736
>
>
> Diffs
> -----
>
> CMakeLists.txt bbd5868
> src/CMakeLists.txt 0470d8f
> src/accesspoint.h 51a1b2f
> src/accesspoint.cpp 62cbc6c
> src/accesspoint_p.h 521f8ce
> src/activeconnection.h 7516ad6
> src/activeconnection.cpp e833f51
> src/activeconnection_p.h 79a0055
> src/connection.h 5dd7a16
> src/connection.cpp 57c5aa0
> src/connection_p.h fb0b90b
> src/device.h f465f78
> src/device.cpp a3f6fad
> src/device_p.h 4d7bcb6
> src/ipconfig.h 24de6d2
> src/ipconfig.cpp d1316fc
> src/manager.h 0f36ba0
> src/manager.cpp bd8752c
> src/manager_p.h 8de789d
> src/settings.h 76cdd8d
> src/settings.cpp a15165f
> src/settings/bondsetting.h 400ed11
> src/settings/bondsetting.cpp 3d52611
> src/settings/bridgesetting.cpp 05ce74c
> src/settings/connectionsettings.h 2dd00af
> src/settings/connectionsettings.cpp b62641e
> src/settings/connectionsettings_p.h d337c99
> src/settings/gsmsetting.h e8dc4ea
> src/settings/gsmsetting.cpp b13274b
> src/settings/infinibandsetting.h c0c854e
> src/settings/infinibandsetting.cpp 6ced9b1
> src/settings/infinibandsetting_p.h 238cd76
> src/settings/setting.cpp ba75623
> src/settings/teamsetting.h c11e1c6
> src/settings/teamsetting.cpp 60ff218
> src/settings/tunsetting.cpp ead153c
> src/settings/vlansetting.h 244cef5
> src/settings/vlansetting.cpp 5c39dc9
> src/settings/wirelesssetting.h 4f84a77
> src/settings/wirelesssetting.cpp aa143c4
> src/settings_p.h aec5423
> src/vlandevice.h cec84ea
> src/vlandevice.cpp 029cab0
> src/vlandevice_p.h 83b6b6f
> src/wimaxdevice.cpp 468f50c
> src/wirelessdevice.h 65313a7
> src/wirelessdevice.cpp 9439606
>
> Diff: https://git.reviewboard.kde.org/r/128093/diff/
>
>
> Testing
> -------
>
> It's a relatively big change... so everything needs to be double checked.
>
>
> File Attachments
> ----------------
>
> 0001-Make-the-networkmanager-version-checks-in-runtime.patch
> https://git.reviewboard.kde.org/media/uploaded/files/2016/06/06/54f5a458-701f-4590-b9b4-21ea0c04f682__0001-Make-the-networkmanager-version-checks-in-runtime.patch
>
>
> Thanks,
>
> Palo Kisa
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-networkmanager/attachments/20160702/112f0dab/attachment.html>
More information about the kde-networkmanager
mailing list