-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Doxygen: Add 1.9.8, 1.10.0 and 1.11.0; delete all older versions #24074
Conversation
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I am not sure what the failure is about, can't reproduce locally and I also didn't see the actual error in the CI logs yet. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Not sure what to do here. I can perfectly build it locally with Conan 1.x and the profile that fails on CI:
And those errors are not saying anything:
|
This comment has been minimized.
This comment has been minimized.
Pinging @uilianries who has some experience fighting doxygen issues :) |
+ drop their custom FindIconv file
Manpages are now always installed doxygen/doxygen@9ce7f45
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Seems like 1.9.4 is fine, 1.9.5 introduced the problem and the issue still exists until the latest release. |
Conan v1 pipeline ❌Failure in build 11 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Closing for now. I don't know how to debug this remotely in CI nor do I have the time to dig into it. |
Edit: Whoops, I totally missed #22052. Anyway, I am doing things differently here and a few things more. Let's see what CI says about it.
FYI; I have also opened doxygen/doxygen#10887 / doxygen/doxygen#10888 so that we can hopefully get rid of the Iconv patches entirely at some point.