Skip to content

Commit 0d3dca7

Browse files
amachado-pieMrSparc
authored andcommitted
opentelemetry-cpp: add version 1.8.2
1 parent 479d2eb commit 0d3dca7

File tree

5 files changed

+53
-10
lines changed

5 files changed

+53
-10
lines changed

recipes/opentelemetry-cpp/all/conandata.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
sources:
2+
"1.8.2":
3+
url: "https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.8.2.tar.gz"
4+
sha256: "20fa97e507d067e9e2ab0c1accfc334f5a4b10d01312e55455dc3733748585f4"
25
"1.8.1":
36
url: "https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.8.1.tar.gz"
47
sha256: "3d640201594b07f08dade9cd1017bd0b59674daca26223b560b9bb6bf56264c2"
@@ -25,6 +28,10 @@ sources:
2528
sha256: "32f12ff15ec257e3462883f84bc291c2d5dc30055604c12ec4b46a36dfa3f189"
2629

2730
patches:
31+
"1.8.2":
32+
- patch_file: "patches/1.8.2-0001-fix-cmake.patch"
33+
patch_description: "fix lack of linking libraries due to conan not generating the variables that are expected"
34+
patch_type: "conan"
2835
"1.8.1":
2936
- patch_file: "patches/1.8.1-0001-fix-cmake.patch"
3037
patch_description: "fix lack of linking libraries due to conan not generating the variables that are expected"

recipes/opentelemetry-cpp/all/conanfile.py

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def layout(self):
5555
def requirements(self):
5656
self.requires("abseil/20220623.0")
5757
self.requires("grpc/1.50.1")
58-
self.requires("libcurl/7.86.0")
58+
self.requires("libcurl/7.87.0")
5959
self.requires("nlohmann_json/3.11.2")
6060
self.requires("openssl/1.1.1s")
6161
if Version(self.version) <= "1.4.1":
@@ -65,7 +65,7 @@ def requirements(self):
6565
self.requires("protobuf/3.21.4")
6666
self.requires("thrift/0.17.0")
6767
if Version(self.version) >= "1.3.0":
68-
self.requires("boost/1.80.0")
68+
self.requires("boost/1.81.0")
6969

7070
def validate(self):
7171
if self.info.settings.compiler.cppstd:
@@ -103,8 +103,11 @@ def generate(self):
103103
tc.variables["WITH_ABSEIL"] = True
104104
tc.variables["WITH_ETW"] = True
105105
tc.variables["WITH_EXAMPLES"] = False
106-
tc.variables["WITH_JAEGER"] = True
106+
if Version(self.version) < "1.8.2":
107+
tc.variables["WITH_JAEGER"] = True
107108
tc.variables["WITH_OTLP"] = True
109+
if Version(self.version) >= "1.8.2":
110+
tc.variables["WITH_OTLP_HTTP"] = True
108111
tc.variables["WITH_ZIPKIN"] = True
109112
if self.options.with_logs_preview:
110113
tc.variables["WITH_LOGS_PREVIEW"] = True
@@ -170,7 +173,6 @@ def _otel_libraries(self):
170173
self._http_client_name,
171174
"opentelemetry_common",
172175
"opentelemetry_exporter_in_memory",
173-
"opentelemetry_exporter_jaeger_trace",
174176
"opentelemetry_exporter_ostream_span",
175177
"opentelemetry_exporter_otlp_grpc",
176178
"opentelemetry_exporter_otlp_http",
@@ -198,6 +200,9 @@ def _otel_libraries(self):
198200
if Version(self.version) >= "1.7.0":
199201
libraries.append("opentelemetry_exporter_otlp_grpc_client")
200202

203+
if Version(self.version) < "1.8.2":
204+
libraries.append("opentelemetry_exporter_jaeger_trace")
205+
201206
if self.options.with_logs_preview:
202207
libraries.extend([
203208
"opentelemetry_logs",
@@ -306,6 +311,11 @@ def package_info(self):
306311
"opentelemetry_trace",
307312
])
308313

314+
if Version(self.version) >= "1.8.2" and self.options.with_logs_preview:
315+
self.cpp_info.components["opentelemetry_otlp_recordable"].requires.extend([
316+
"opentelemetry_logs",
317+
])
318+
309319
self.cpp_info.components["opentelemetry_proto"].requires.extend([
310320
"opentelemetry-proto::opentelemetry-proto",
311321
"protobuf::protobuf",
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
diff --git a/CMakeLists.txt b/CMakeLists.txt
2+
index 9df8f5c..43b9d82 100644
3+
--- a/CMakeLists.txt
4+
+++ b/CMakeLists.txt
5+
@@ -251,7 +251,6 @@ if(WITH_JAEGER)
6+
find_package(Thrift QUIET)
7+
if(Thrift_FOUND)
8+
find_package(Boost REQUIRED)
9+
- include_directories(${Boost_INCLUDE_DIR})
10+
else()
11+
# Install Thrift and propagate via vcpkg toolchain file
12+
if(WIN32 AND (NOT DEFINED CMAKE_TOOLCHAIN_FILE))
13+
diff --git a/cmake/opentelemetry-proto.cmake b/cmake/opentelemetry-proto.cmake
14+
index a21c0f1..a467ffd 100644
15+
--- a/cmake/opentelemetry-proto.cmake
16+
+++ b/cmake/opentelemetry-proto.cmake
17+
@@ -280,6 +280,10 @@ else() # cmake 3.8 or lower
18+
target_link_libraries(opentelemetry_proto INTERFACE ${Protobuf_LIBRARIES})
19+
endif()
20+
21+
+if(TARGET gRPC::grpc++)
22+
+ target_link_libraries(opentelemetry_proto PUBLIC gRPC::grpc++)
23+
+endif()
24+
+
25+
if(BUILD_SHARED_LIBS)
26+
set_property(TARGET opentelemetry_proto PROPERTY POSITION_INDEPENDENT_CODE ON)
27+
endif()
Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
11
cmake_minimum_required(VERSION 3.8)
22

3-
project(test_package CXX)
3+
project(test_package)
44

55
include(${CMAKE_BINARY_DIR}/conanbuildinfo.cmake)
66
conan_basic_setup(TARGETS)
77

8-
find_package(opentelemetry-cpp REQUIRED CONFIG)
9-
10-
add_executable(${PROJECT_NAME} ../test_package/test_package.cpp)
11-
target_link_libraries(${PROJECT_NAME} PRIVATE opentelemetry-cpp::opentelemetry-cpp)
12-
target_compile_features(${PROJECT_NAME} PRIVATE cxx_std_11)
8+
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/../test_package/
9+
${CMAKE_CURRENT_BINARY_DIR}/test_package/)

recipes/opentelemetry-cpp/config.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
versions:
2+
"1.8.2":
3+
folder: all
24
"1.8.1":
35
folder: all
46
"1.7.0":

0 commit comments

Comments
 (0)