-
Notifications
You must be signed in to change notification settings - Fork 2.2k
qt 6.10: use newer pipenv for build-time QtWebEngine dependency #29181
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
base: master
Are you sure you want to change the base?
Conversation
… dependency for QtWebEngine
|
So pipenv can be used for libs too ? Very interesting! |
yes - it creates a |
|
Ok, great news. I guess the documentation could be amended then. Thanks for the details. |
|
I just tried to build your branch, but unfortnualtey it seems to fail on windows : https://github.com/eirikb/proof-of-conan/actions/runs/20363954687/job/58514707969#step:11:811, macos https://github.com/eirikb/proof-of-conan/actions/runs/20363954687/job/58514707976#step:11:27627 and linux https://github.com/eirikb/proof-of-conan/actions/runs/20364672430/job/58516935991#step:11:5745 in spite of earlier installation: log |
|
I suspect we need to pass based on the logs, I suspect this is because cmake tends to prefer |
|
Except if I messed up something, I don't think it fixes the problem : https://github.com/eirikb/proof-of-conan/actions/runs/20373449826/job/58545639892#step:11:5771 Linux log: windows logs, showing no difference regarding the python exe used
|
Summary
html5lib, which is required by the build scripts of QtWebEngineMotivation
The Qt build configurator will silently disable the build of QtWebEngine if
html5libis not visible by the interpreterClose #27285
Details
Add a 👍 reaction to pull requests you find important to help the team prioritize, thanks!