-
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
Force rebuild all packages in CI #8912
Conversation
qt5-tools regression on linux due to automatically picked up external dependency to some XML library in one of the qt5-ports. qt is really annoying with this automatically picked up dependencies. |
It will take some time to fix all of these regressions 😞 |
@vicroms Don't worry, some of them are expected. |
Check port failures on Linux and macOS:
|
Ports failures on windows:
|
I have removed the failing tombstones and will rerun the tests, hopefully this will reduce the number of regressions |
/azp run |
4 ports failed totally on windows In latest CI results: 3 ports failed to download the source file: 1 port libxslt failed with arm64-windows, I can't repro this issue with vs2017 and vs2019 on windows locally, failures: |
I've updated the current commit ID for I'm not sure what to do about I'll take a look at |
|
/azp run |
Current regression ports:
OSX:
Windows(arm64-windows):
|
@cenit thanks for the heads-up, I changed the CONTROL version. |
/azp run |
I can't reproduce the arm64-windows error on my machine, it seems that everything is fine. |
And I can't repro x64-linux error on the machine. @Neumann-A I think qt5 automatically activates some options, what do you think? |
@JackBoosY: Yeah I fear we must be explicit about all the qt options and make them a feature (#8213) or the default. qt-imageformats for example has 4 options and at least 3 can make problems in vcpkg from which two are explicitly set (tiff, webp) and jasper is missing. Those options can be discovered by looking for a config_help.txt or by searching for a number of configure.json in the source tree. But without #8269 solved I fear that static builds might be difficult to link correctly due to all the possible options in building dependent libraries. I already have a small number of qt fixes in different PRs to silence CI: The current errors seen in qt5-tools are probably due to the following options in qt5-base:
To make static builds work you need to probably pass ICU_LIBS and EGL_LIBS with the correct libraries. but there are a few more options which should be checked like:
(basically everything which is auto in the config_help.txt should be checked.) |
/azp run |
/azp run |
In the meantime OTL has been updated again. |
https://sourceforge.net/p/otl/discussion/208859/thread/fdb9d3bacc/ |
@c72578 Thanks for the heads-up!
@cenit Thank you very mcuh for taking the time to report this upstream! I read the thread and I'm glad that the maintainers were so quick to address this. The latest version is still under the dynamic link, so I'm thinking whether to pin this to one version behind the latest for the time being... a possible solution could be to allow |
qt-base requires fontconfig on linux due to automatic detection. Add |
Linux:
OSX:
|
@JackBoosY It is expat not libxml2. libxml2 uses different symbols. (see #8912 (comment)) https://www.xml.com/pub/a/1999/09/expat/reference.html |
/azp run |
1 similar comment
/azp run |
/azp run |
Pull request contains merge conflicts. |
(otl is "broken" on master, you might already know that...) |
@JackBoosY you just broke @vicroms PR in the same way the master is broken.... @vicroms original otl's portfile was ok |
@cenit Ops...I thought this port has been updated. |
@JackBoosY I think the |
/azp run |
I have a clone of that repository, if you want. But not being the original library author, I think it might not be the best idea to point to my repository... |
As CI has been restored, this full rebuild is no longer necessary, I'll separate fixes made in here into their own PRs. Thanks to everyone that helped with solving the flakiness in our ports! |
No description provided.