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] Update to 1.16 #39632

Merged
merged 1 commit into from
Jul 2, 2024
Merged

[opentelemetry-cpp] Update to 1.16 #39632

merged 1 commit into from
Jul 2, 2024

Conversation

waywardmonkeys
Copy link
Contributor

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@LilyWangLL LilyWangLL added category:port-update The issue is with a library, which is requesting update new revision requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function labels Jul 2, 2024
@LilyWangLL LilyWangLL changed the title [opentelemtry-cpp] Update to 1.16 [opentelemetry-cpp] Update to 1.16 Jul 2, 2024
@LilyWangLL LilyWangLL removed the requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function label Jul 2, 2024
@LilyWangLL
Copy link
Contributor

Usage test passed on x64-windows.

@LilyWangLL LilyWangLL added the info:reviewed Pull Request changes follow basic guidelines label Jul 2, 2024
@JavierMatosD JavierMatosD merged commit 9afd245 into microsoft:master Jul 2, 2024
17 checks passed
@waywardmonkeys waywardmonkeys deleted the update-opentelemetry-cpp branch July 3, 2024 01:58
@Pravila00 Pravila00 mentioned this pull request Jul 10, 2024
Closed
@Pravila00
Copy link
Contributor

@LilyWangLL have you tested building some of the OTLP features? :)

@BillyONeal BillyONeal mentioned this pull request Jul 10, 2024
Closed
@LilyWangLL
Copy link
Contributor

LilyWangLL commented Jul 11, 2024 •
edited
Loading

@LilyWangLL have you tested building some of the OTLP features? :)

I've tested this port's features before. It was broken in the old version 1.14.2 and hasn't been fixed in the new version.

@Pravila00
Copy link
Contributor

@LilyWangLL this PR contains more detailed information about why this update broke OTLP #39847.
My build broke because of this PR too :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LilyWangLL LilyWangLL LilyWangLL approved these changes

Assignees

@LilyWangLL LilyWangLL

Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@waywardmonkeys @LilyWangLL @Pravila00 @JavierMatosD