Skip to content

[opentelemetry-cpp] Add opentracing shim feature#44012

Merged
JavierMatosD merged 2 commits into
microsoft:masterfrom
JonLiu1993:dev/Jon/opentelemetry-cpp
Feb 26, 2025
Merged

[opentelemetry-cpp] Add opentracing shim feature#44012
JavierMatosD merged 2 commits into
microsoft:masterfrom
JonLiu1993:dev/Jon/opentelemetry-cpp

Conversation

@JonLiu1993
Copy link
Copy Markdown
Contributor

Fixes #43974

Add opentracing shim feature

  • 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.

@JonLiu1993 JonLiu1993 added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist category:port-bug The issue is with a library, which is something the port should already support info:internal labels Feb 25, 2025
HEAD_REF main
PATCHES
cmake-quirks.diff
fix-target_link.patch
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fix error:

CMake Error at opentracing-shim/CMakeLists.txt:26 (target_link_libraries):
  Target "opentelemetry_opentracing_shim" links to:

    OpenTracing::opentracing

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

Tested usage successfully in the following triplet:

  • x64-windows-static

@JonLiu1993 JonLiu1993 marked this pull request as ready for review February 25, 2025 10:20
@jimwang118 jimwang118 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Feb 26, 2025
@JavierMatosD JavierMatosD merged commit 4f79328 into microsoft:master Feb 26, 2025
@dg0yt
Copy link
Copy Markdown
Contributor

dg0yt commented Feb 26, 2025

Another 👎 PR. 😢

dg0yt added a commit to dg0yt/vcpkg that referenced this pull request Feb 26, 2025
@JonLiu1993 JonLiu1993 deleted the dev/Jon/opentelemetry-cpp branch February 27, 2025 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[opentelemetry-cpp] Add opentracing shim feature

4 participants