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

Configure picks Qt6 over Qt5 when both arfe available #592

Closed
1 task done
yurivict opened this issue Feb 18, 2024 · 7 comments · Fixed by #629
Closed
1 task done

Configure picks Qt6 over Qt5 when both arfe available #592

yurivict opened this issue Feb 18, 2024 · 7 comments · Fixed by #629
Milestone

Comments

@yurivict
Copy link

yurivict commented Feb 18, 2024

Setting QT_DEFAULT_MAJOR_VERSION to 6 doesn't seem to do anything.
You need to have a cmake option got QT version, and explicitly separate find_package lines for both.

Version: 24.1.0
FreeBSD 14

TODO

  • Revert WITH_QT6 flag
@ra3xdh
Copy link
Owner

ra3xdh commented Feb 18, 2024

This is done intentionally. You have to set the Qt version explicitly if both are installed. See #217 I am not sure what option may help for FreeBSD.

@ra3xdh ra3xdh closed this as completed Feb 18, 2024
@yurivict
Copy link
Author

@ra3xdh

There used to be the option WITH_QT6, but now there is no such option because it was removed.
QT_DEFAULT_MAJOR_VERSION is set, but it doesn't affect the Qt choice.

Before #217 it was possible to choose the Qt generation, and now it doesn't appear to be possible.

This is ok, I can patch CMakeLists.txt, but this is still a regression IMO.

@ra3xdh
Copy link
Owner

ra3xdh commented Feb 19, 2024

It is not a regression. It is done intentionally. No need to patch CMakeList. Qt version could be selected using environment variables. The reasons to remove WITH_QT6 were described by @dsm in #217

@yurivict
Copy link
Author

FYI: QT_DEFAULT_MAJOR_VERSION doesn't seem to work due to an apparent bug in cmake: https://gitlab.kitware.com/cmake/cmake/-/issues/25691

@yurivict
Copy link
Author

yurivict commented Feb 19, 2024

@ra3xdh

According to the answer to the cmake bug report that I created this line is a mistake, and it has nothing to do with Qt selection in cmake. It only affects some subsequent Qt-related macros. There doesn't appear to be any environment variable for this either.

Could you please consider adding back WITH_QT6, because #217 had no clear explanation why it should have been removed from @dsm .

@ra3xdh
Copy link
Owner

ra3xdh commented Feb 20, 2024

Yes, I will revert the explicit Qt6 selection using WITH_QT6 Also I am planning to switch the build to Qt6 as default and drop Qt5 support till the end of this year. So the issue will be self-fixed in the future.

@ra3xdh ra3xdh reopened this Feb 20, 2024
@ra3xdh ra3xdh added this to the 24.1.1 milestone Feb 20, 2024
@ra3xdh ra3xdh mentioned this issue Mar 6, 2024
@ra3xdh ra3xdh linked a pull request Mar 6, 2024 that will close this issue
@ra3xdh
Copy link
Owner

ra3xdh commented Mar 6, 2024

I have reverted WITH_QT6 flag in #629. @yurivict Please check everything on FreeBSD.

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 a pull request may close this issue.

2 participants