-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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]Fix build failure in linux #7042
Conversation
@TheScarfix can you have a look? |
will try to look into it tomorrow. Still trying to fix the build of 5.12.4. Maybe a major rework of the port for the next update might be useful. |
@TheScarfix BTY, why do we need to execute |
I don't really know, I just made the update to 5.12 and not much related to the scripts here that make qt work. |
A configure step which is used to generate project files.
That's too big a project to take on by one person. Better to just fix what's there for now. The paths in the compiled binaries and the replication of files inside the various plugin directories are the main issues. Try deploying an application using windeployqt which uses qtquickwidgets and some translations for example. |
…macextras/winextras error messages.
@heydojo I only found this document about And I only see the discussions about It seems that we need to pass the parameter When I drilled into the Makefile, I found that if you use |
|
/azp run |
I think this specific workaround might not be needed in 5.12.4. I'll test it with SKIP_MAKEFILES later. Maybe we could work together to bring all the fixes plus the update in one PR? |
@TheScarfix I will help you to fix the update PR. If this PR is resolved earlier, we may first merge this PR. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Open a new PR(#7230) to fix this issue. |
Now we don't need to use
qmake_all
to build qt5 and other modules in linux, it seems to break the disabled parameters during the build. See documentation.Fix the failure of qt5 module build in linux (caused by #5293):
Related: #6962 #5655 #5362 #3859 #3644.