Review Request 123401: Fix usage of the networkmanagerqt_export.h after commit 9966897

[Aleix Pol Gonzalez]

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


Wouldn't it have been easier to add the subdirectory in the interface?

Or you want users of the library to specify the namespace?

Also the former one would have been fine as long as the header and the export file are in the same directory, which seems to be the case...

- Aleix Pol Gonzalez


On April 17, 2015, 5:53 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123401/
> -----------------------------------------------------------
> 
> (Updated April 17, 2015, 5:53 p.m.)
> 
> 
> Review request for KDE Frameworks, David Faure and Jan Grulich.
> 
> 
> Repository: networkmanager-qt
> 
> 
> Description
> -------
> 
> The headers include <networkmanagerqt_export.h>, which is now in lowercase directory, and is not in public INTERFACE includes. We need to change all the includes, and place where the export is generated.
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt 0dc3f2e 
>   src/accesspoint.h 99895ef 
>   src/activeconnection.h 6edd0ae 
>   src/adsldevice.h d7c92ba 
>   src/bluetoothdevice.h 4baf4a3 
>   src/bonddevice.h 82eafa2 
>   src/bridgedevice.h 68b3e00 
>   src/connection.h a20d1f7 
>   src/device.h bca098b 
>   src/dhcp4config.h 35082ba 
>   src/dhcp6config.h 91ca914 
>   src/genericdevice.h d95fab0 
>   src/generictypes.h cd45430 
>   src/gredevice.h eeb682c 
>   src/infinibanddevice.h c9597cb 
>   src/ipaddress.h ebc3e5f 
>   src/ipconfig.h 5eb0227 
>   src/iproute.h 13afe15 
>   src/macvlandevice.h ab02831 
>   src/manager.h e79b933 
>   src/modemdevice.h e40d457 
>   src/olpcmeshdevice.h 9af2342 
>   src/secretagent.h 02e4c9a 
>   src/settings.h f5fe486 
>   src/settings/adslsetting.h 89ab5b5 
>   src/settings/bluetoothsetting.h 9a32851 
>   src/settings/bondsetting.h f3ae1ec 
>   src/settings/bridgeportsetting.h 7a8f74a 
>   src/settings/bridgesetting.h 4628cf5 
>   src/settings/cdmasetting.h ecd30da 
>   src/settings/connectionsettings.h 252f8e9 
>   src/settings/genericsetting.h 610568a 
>   src/settings/gsmsetting.h 1a152b2 
>   src/settings/infinibandsetting.h 713e01e 
>   src/settings/ipv4setting.h bdfcc58 
>   src/settings/ipv6setting.h 7e7ca9e 
>   src/settings/olpcmeshsetting.h 217682d 
>   src/settings/pppoesetting.h a3c5d79 
>   src/settings/pppsetting.h 4755288 
>   src/settings/security8021xsetting.h 486f1bc 
>   src/settings/serialsetting.h fa0bf57 
>   src/settings/setting.h 86c6e83 
>   src/settings/teamsetting.h ba58a5d 
>   src/settings/template.h 4f9f680 
>   src/settings/vlansetting.h 8432eab 
>   src/settings/vpnsetting.h 67aea4e 
>   src/settings/wimaxsetting.h 9fd887a 
>   src/settings/wiredsetting.h fec6860 
>   src/settings/wirelesssecuritysetting.h 5c9a1da 
>   src/settings/wirelesssetting.h 12eed72 
>   src/teamdevice.h e42bc81 
>   src/tundevice.h dd66977 
>   src/utils.h d091c5e 
>   src/vethdevice.h c9322d0 
>   src/vlandevice.h bfe2948 
>   src/vpnconnection.h acd0d2e 
>   src/vpnplugin.h 3a4f181 
>   src/wimaxdevice.h fb202fa 
>   src/wimaxnsp.h c9bd856 
>   src/wireddevice.h 193710c 
>   src/wirelessdevice.h 6137036 
>   src/wirelessnetwork.h 1d5cb7b 
> 
> Diff: https://git.reviewboard.kde.org/r/123401/diff/
> 
> 
> Testing
> -------
> 
> Plasma-nm no longer fails due to original problem, but incorect usage of nm-qt includes:
> 
> vpnuiplugin.h:31:43: fatal error: NetworkManagerQt/generictypes.h: No such file or directory
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150417/eead4001/attachment.html>