Skip to content
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.

Sign up for GitHub

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

Jump to bottom

opentelemetry-cpp: bump deps #22237

Merged
merged 11 commits into from
Feb 5, 2024
Merged

opentelemetry-cpp: bump deps #22237

merged 11 commits into from
Feb 5, 2024

Conversation

ericLemanissier
Copy link
Contributor

@ericLemanissier ericLemanissier commented Jan 10, 2024 •
edited
Loading

Specify library name and version: opentelemetry-cpp/*

@valgur
Copy link
Contributor

valgur commented Jan 10, 2024

Thanks!

@prince-chrismc prince-chrismc mentioned this pull request Jan 10, 2024
Open
@conan-center-bot

This comment has been minimized.

Sign in to view
@conan-center-bot

This comment has been minimized.

Sign in to view
@conan-center-bot

This comment has been minimized.

Sign in to view
@valgur valgur mentioned this pull request Jan 20, 2024
Merged
@valgur
Copy link
Contributor

valgur commented Jan 23, 2024

@ericLemanissier Can you maybe bump Boost as well now that Thrift has been updated? It seems to be causing a conflict in #18685 currently.

@conan-center-bot

This comment has been minimized.

Sign in to view
@conan-center-bot

This comment has been minimized.

Sign in to view
@ghost
Copy link

ghost commented Feb 1, 2024

I detected other pull requests that are modifying opentelemetry-cpp/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

This was referenced Feb 1, 2024
Closed
Closed
@uilianries uilianries self-assigned this Feb 1, 2024
@ericLemanissier

This comment was marked as resolved.

Sign in to view
@conan-center-bot conan-center-bot added Failed Missing dependencies Build failed due missing dependencies in Conan Center Version conflict There is a version conflict when solving the dependencies graph labels Feb 1, 2024
@conan-center-bot

This comment has been minimized.

Sign in to view
@conan-center-bot conan-center-bot removed Missing dependencies Build failed due missing dependencies in Conan Center Version conflict There is a version conflict when solving the dependencies graph labels Feb 2, 2024
@conan-center-bot

This comment has been minimized.

Sign in to view
@ericLemanissier
test recipe: bump cppstd
49b86d7 
abseil needs c++14
@ericLemanissier
Copy link
Contributor Author

The build is now finished. You can take a look at the results here

@conan-center-bot

This comment has been minimized.

Sign in to view
@conan-center-bot

This comment has been minimized.

Sign in to view
uilianries
uilianries approved these changes Feb 5, 2024
View reviewed changes
Copy link
Member

@uilianries uilianries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 12 (089a5a8d3dc200a4d357271574695ab6ce8ec283):

  • opentelemetry-cpp/1.12.0:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.8.3:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.9.1:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.7.0:
    All packages built successfully! (All logs)

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 12 (089a5a8d3dc200a4d357271574695ab6ce8ec283):

  • opentelemetry-cpp/1.12.0:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.7.0:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.9.1:
    All packages built successfully! (All logs)

  • opentelemetry-cpp/1.8.3:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit 1a5c959 into conan-io:master Feb 5, 2024
28 checks passed
@ericLemanissier ericLemanissier deleted the opentelemetry-cpp branch February 5, 2024 13:37
Ahajha pushed a commit to Ahajha/conan-center-index that referenced this pull request Feb 10, 2024
@ericLemanissier @uilianries @Ahajha
(conan-io#22237) opentelemetry-cpp: bump deps
fb0bd34 
* opentelemetry-cpp: bump deps

* don't bump boost

* don't bump thrift

* bump thrift

* Update conanfile.py

* bump boost and thrift

* unbump thrift

* Update recipes/opentelemetry-cpp/all/conanfile.py

Co-authored-by: Uilian Ries <[email protected]>

* test recipe: bump cppstd

abseil needs c++14

* fix test_V1_package

---------

Co-authored-by: Uilian Ries <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@uilianries uilianries uilianries approved these changes

@jwillikers jwillikers jwillikers approved these changes

@valgur valgur Awaiting requested review from valgur

Assignees

@uilianries uilianries

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ericLemanissier @valgur @conan-center-bot @uilianries @jwillikers