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
[qtbase] Fix PACKAGE_PREFIX_DIR
substitution, which was broken by CMake 3.29.1
#38088
[qtbase] Fix PACKAGE_PREFIX_DIR
substitution, which was broken by CMake 3.29.1
#38088
Conversation
Since Qt generates the use of |
The PR you linked fixes |
So this complements the other PR. |
close. Upstream CMake reverted the change |
Maybe this patch would be a good idea regardless, to make the substituion robust against other variable names? Especially if Qt moves forward with their patch (regardless of what CMake is doing) |
…se-fix-package-prefix-substitution
Closing since CMake reverted the change. Since CMake is going to pursue a different solution for this problem, we shouldn't do this and we should support the new solution. |
Did Qt also revert the change? I haven't checked, but if not, this is going to bite us on the next port update. |
I think this string replacement no longer applies any way and could be removed in future updates |
This substitution no longer works due to upstream changes in CMake, which changes the
PACKAGE_PREFIX_DIR
variable to a non-fixed name. Strangely I only ran into this issue while building an older vendored version of Qt submodule ports (see e.g. this build), but we should update this code anyway.Checklist
./vcpkg x-add-version --all
and committing the result.