GammaRay
GammaRay copied to clipboard
Upgrade QML Connections {} syntax to functions
Old signal handler declaration style is deprecated and spits warnings at runtime.
This will likely break backward compatibility with older Qt5 versions, wont it? (GammaRay supports Qt5 back to 5.5).
5.5? oh. Is that something that still exists?
Likely will break then. But the current approach is surely broken for Qt 6, isn't it?
QtLocation is gone in Qt6, so this entire thing doesn't work there anyway.
At least Widget Inspector is still legit, I guess
Personally I think it would be fine to upgrade to a newer Qt version nowadays, no? If people want a gammaray that works against Qt 5.5, they can use an older release.
Personally I think it would be fine to upgrade to a newer Qt version nowadays, no?
Too early. The KDE Frameworks are not yet fully ported to Qt6, so application styling UX would be rather suboptimal on Linux platform.
If people want a gammaray that works against Qt 5.5, they can use an older release.
So a GammaRay build against Qt6 could still inspect apps running on Qt5? That would be great. Never tried (I think?)
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
I did, some time ago. And my contributions were already merged in GammaRay in past.
So a GammaRay build against Qt6 could still inspect apps running on Qt5? That would be great. Never tried (I think?)
No, that's not possible because GammaRay injects itself into the program, this is even a problem for same major version but different minor ie Qt6.2 GammaRay vs Qt6.4 App, for best compat you should use that exact same version, so I agree with @milianw that we could drop 5.5, but iirc this warning came around 5.14 or so... 5.14 indeed according to https://doc.qt.io/archives/qt-5.14/qml-qtqml-connections.html
@ratijas Will need a manual rebase now, looks like some stuff was shuffled.
@ratijas now that min required version is 5.15 could you please rebase this?
no problem!
AAAAAaaaa how is this a valid syntax
Widget inspector was removed, so it's not relevant anymore.