A question about "signals" in qt-nm

[Shaheed Haque]

Hi all,

The Python bindings effort recently ran into the need for
https://phabricator.kde.org/D5185 to prevent clashes between Qt's use
of "signals" and libnm via networkmanager-qt's ipconfig.h. However, it
seems to me that the fix is incomplete in current Git master as there
are at least two other files which also include NetworkManager.h:

https://cgit.kde.org/networkmanager-qt.git/tree/src/settings/setting.h
https://cgit.kde.org/networkmanager-qt.git/tree/src/settings/connectionsettings.h

AFAICS, the first, setting.h, definitely needs the fix (it could also
do with a sync up of the version checking around the #include to match
ipconfig.h). The second, connectionsettings.h actually #includes
setting.h, so it might be able to lose the #include of NetworkManager
entirely, else it should likely also get the fixes?

Assuming this makes sense, it would be good if somebody else can
kindly drive the patching as my KDE build/setup/knowledge is very,
VERY rusty.

Thanks, Shaheed

P.S. Please CC me as I'm not subscribed here.