Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port Qt backend to Qt 6 #6586

Closed
jgcodes2020 opened this issue Oct 18, 2024 · 3 comments · Fixed by #6624
Closed

Port Qt backend to Qt 6 #6586

jgcodes2020 opened this issue Oct 18, 2024 · 3 comments · Fixed by #6624
Labels
a:backend-qt The Qt backend - including the qt style (mO,bS)
Milestone

Comments

@jgcodes2020
Copy link

The Qt backend should be ported to Qt 6 ASAP, as the current backend runs on Qt 5, set to be discontinued May 2025.

@ogoffart
Copy link
Member

The Qt backend is already ported to Qt6.
It even defaults to Qt6 if both Qt6 and Qt5 are installed.

Although I realized that the linux package are still built with Qt5.
That was because on Linux, most distribution still used KDE that was using Qt5. But I guess now, we should change the package to be built with Qt6.

@ogoffart ogoffart added the a:backend-qt The Qt backend - including the qt style (mO,bS) label Oct 18, 2024
@ogoffart ogoffart added this to the 1.9 milestone Oct 18, 2024
@jgcodes2020
Copy link
Author

I did a bit more digging in the code; seems to be related to woboq/qmetaobject-rs#314. I've submitted a PR that should ensure qttypes defaults to Qt 6 where possible.

@ogoffart
Copy link
Member

Indeed, i was thinking we used Qt6 by default but that's only in CMake with C++.
With Rust, qttypes indeed defaulted to whatever 'qmake' is which is most likely Qt5. New release of qttypes was released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:backend-qt The Qt backend - including the qt style (mO,bS)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants