-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[qt5] update qt to 5.15 and make ports depend on qt5-base[core]. #11596
Conversation
Why the qt5-base[core] edit? Without default features, it does not look necessary at all |
@cenit. Correctness. It should mostly always be |
Hmm...I can't get the log if there is no failure in the CI test. |
Vcpkg has a bug when using |
Co-authored-by: Jack·Boos·Yu <[email protected]>
@JackBoosY: That is not a bug but a feature of vcpkg. The user has to be explicit about what he wants if he don't want to have the default set of features (as written in the issue itself) |
@JackBoosY: I don't think the Linux CI should be green |
Please get logs here.
I don't think that has error there, maybe just a misjudgment in |
|
The egl checks fails and that is why Edit: Solved |
@Neumann-A VMs have been updated with your changes, dropping the depends:vm-update tag. If there are still CI regressions you'll want to look at those. |
# Conflicts: # ports/paraview/portfile.cmake # ports/qt5-base/CONTROL # ports/qt5-base/portfile.cmake # scripts/ci.baseline.txt
…_5_15_0 # Conflicts: # ports/paraview/portfile.cmake # ports/qt5-base/portfile.cmake
quickfix seems to be a bit flakky? (Why is it even rebuild here it only depends on openssl?) Since linux and windows are already green I assume the same for osx. I highly recommend to make paraview tools a default feature if the embree issue in netCDF is resolved to make paraview a test of Qt. |
Quickfix is broken in master right now, your change is an innocent bystander. @JackBoosY you were previously 'Request Changes' on this PR, does it look good to you now? @Neumann-A Thanks for your contribution! |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@BillyONeal: Please rerun x64-windows. error is: |
Due to the long duration of the PR and the long test time, the format problem can be waited for the next update and fix. |
It is merged! Thank you @Neumann-A once again for the marathon of effort required to pilot the Qt ship :) |
THANK YOU!!!!!!! |
Thanks for your hard work @Neumann-A! |
closes #11594
and so it starts again ....
lets see if ci is able to build it. @heydojo: Mind testing?)Cannot create parent directory
) (seems to be a bit flaky had the error two times on my pc and i just retried. Maybe disable parallel build if too flaky for ci?)Note for reviewers: The patches are the same as for Qt 5.14. I just removed the old 5.12.8 patches and renamed the patches from Qt 5.14