-
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
Set boost required version to 1.85.0 to be inline with thrift/0.17.0 … #24458
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! With the new pipeline we'll be able to handle cases like this in a simpler way by being able to modify more than once recipe per PR, sorry that this was an issue meanwhile :)
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Unfortunately, I cannot view the details about the failing required checks c3i/conan-v2/pr-merge and continuous-integration/jenkins/pr-merge. How should I proceed here? |
Hi @maximiliank sorry for the bot noise - our CI got a bit confused, that's also the reason you couldn't see the logs. I've restarted the build now, let's see if it's happier about it this time around :) |
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.
Should be fixed now 🤞 , sorry for the noise @maximiliank - if this gives more problems, I'll have to dig deeper on monday |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Now 60 more fail and I still get a 403 Forbidden on the 3ci/conan-v2/pr-merge and continuous-integration/jenkins/pr-merge checks |
Hello @AbrilRBS, do you have an idea why the checks failed? Thank you |
@AbrilRBS any news on this issue? |
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 10 (
Conan v2 pipeline ✔️
All green in build 10 (
|
This PR has been automatically merged due to |
Adjust boost version dependency for arrow/all
Summary
Changes to recipe: arrow/all
Motivation
In the latest commit to the
thrift
recipe (f8a8a29) theboost
version requirement was set to 1.85.0. This results in a version conflict whenarrow
is requested with any option requiringwith_thrift
(e.q.parquet
requires this)Details
Solution is to increase the
boost
version to 1.85.0. Solves #24455