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

Migrate to Qt6 #17167

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Migrate to Qt6 #17167

wants to merge 2 commits into from

Conversation

viachaslavic
Copy link
Contributor

@viachaslavic viachaslavic commented Nov 6, 2024

Description

  • Basic support Qt6 with minimal changes required for compiling
  • Avoid using NULL for Qt UI (keep for further refactoring)
  • Look moc in QT_HOST_LIBEXECS
  • Soft switching to C++17 (required by Qt6). If the compiler does not support it, the build will be performed as before with c++11, but without qt support.
  • cflags for Qt moved to CXXFLAGS to avoid affecting the C code. At least this fixes compiling xaudio driver on Windows.

Tested on ArchLinux/Qt6.8, Debian12/Qt6.4, Win10(MSYS2)/Qt6.8

Related Issues

#16913

Reviewers

[If possible @mention all the people that should review your pull request]

@viachaslavic
Copy link
Contributor Author

Done. Added compatibility with Qt5 for unhurried updating of build environments.

@viachaslavic viachaslavic marked this pull request as ready for review November 8, 2024 14:24
* basic support Qt6 with minimal changes required for compiling
* look `moc` in QT_HOST_LIBEXECS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant