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.

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

[package] opentelemetry-cpp/1.9.1: fails to build due to broken opentelemetry-proto dependency #18967

Closed
dfszb opened this issue Jul 30, 2023 · 2 comments · Fixed by #21332
Closed
Labels
bug Something isn't working

Comments

@dfszb
Copy link

dfszb commented Jul 30, 2023

Description

run conan install . --build=missing with a conanfile:

[requires]
opentelemetry-cpp/1.9.1

[generators]
cmake

The package installation fails with message:

'ERROR: opentelemetry-cpp/1.9.1: required component package 'opentelemetry-proto::' not in dependencies'
The output is observed after 
'opentelemetry-cpp/1.9.1: Package '8865730c41263c8e8733581b5e0ae10b5378ffa9' created' 

Package and Environment Details

  • Package Name/Version: opentelemetry-cpp/1.9.1
  • Operating System+version: Linux Ubuntu 22.04

Conan profile

[settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.cppstd=gnu17
compiler.libcxx=libstdc++11
compiler.version=13
os=Linux

Steps to reproduce

create conanfile as:

[requires]
opentelemetry-cpp/1.9.1

[generators]
cmake
run conan install . --build=missing

Logs

Click to expand log
ERROR: opentelemetry-cpp/1.9.1: required component package 'opentelemetry-proto::' not in dependencies
Note: the cmake build passes successfully, and the package seems to be installed, just that this dependency check makes it unusable.
@dfszb dfszb added the bug Something isn't working label Jul 30, 2023
@sebsnyk
Copy link

sebsnyk commented Sep 22, 2023

@dfszb are you aware of a workaround?

@nic-godz
Copy link
Contributor

nic-godz commented Oct 9, 2023

I have the same issue. Same goes for version 1.8.1 as well.

As far as I can see the recipe is broken as target "opentelemetry-proto::opentelemetry-proto" does not exist. I'm playing around with some ugly patching until this is solved.

@sebsnyk Maybe you are doing the same? But currently I'm starting with downloading opentelemetry-cpp/1.9.1, then removing the "opentelemetry-proto::opentelemetry-proto" line and then the "conan install" as usual. I can of course share the ugly bash code if you like 😃

valgur added a commit to valgur/conan-center-index that referenced this issue Nov 23, 2023
conan-center-bot pushed a commit that referenced this issue Jan 10, 2024
…d fix options, fix build requirements

* opentelemetry-cpp: drop old versions, bump deps

* opentelemetry-cpp: fix with_otlp=False handling

* opentelemetry-cpp: fix opentelemetry-proto usage

Fixes #18967.

* opentelemetry-cpp: simplify patching

* opentelemetry-cpp: add v1.12.0, update options

* opentelemetry-cpp: add transitive_headers=True

* opentelemetry-cpp: rmdir lib/pkgconfig

* opentelemetry-cpp: disable failing configuration

* Apply suggestions from code review

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* opentelemetry-cpp: fix formatting

---------

Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>
Co-authored-by: Daniel <danimanzaneque@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants