Skip to content

Commit

Permalink
opentelemetry-cpp: remove with_logs_preview for >=1.11.0
Browse files Browse the repository at this point in the history
  • Loading branch information
Jenkins authored and Cogitri committed Sep 12, 2023
1 parent c5cd41d commit 7e673e6
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions recipes/opentelemetry-cpp/all/conanfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ def export_sources(self):
def config_options(self):
if self.settings.os == "Windows":
self.options.rm_safe("fPIC")
if Version(self.version) >= "1.11.0":
self.options.rm_safe("with_logs_preview")

def configure(self):
if self.options.shared:
Expand Down Expand Up @@ -205,7 +207,6 @@ def generate(self):
tc.variables["WITH_ZPAGES"] = self.options.with_zpages
tc.variables["WITH_NO_GETENV"] = self.options.with_no_getenv
tc.variables["WITH_ETW"] = self.options.with_etw
tc.variables["WITH_LOGS_PREVIEW"] = self.options.with_logs_preview
tc.variables["WITH_ASYNC_EXPORT_PREVIEW"] = self.options.with_async_export_preview
tc.variables["WITH_METRICS_EXEMPLAR_PREVIEW"] = self.options.with_metrics_exemplar_preview
tc.variables["OTELCPP_PROTO_PATH"] = self._proto_root
Expand All @@ -214,6 +215,9 @@ def generate(self):
tc.variables["WITH_OTLP"] = self.options.with_otlp
tc.variables["WITH_JAEGER"] = self.options.with_jaeger

if Version(self.version) < "1.11":
tc.variables["WITH_LOGS_PREVIEW"] = self.options.with_logs_preview

tc.generate()

tc = CMakeDeps(self)
Expand Down Expand Up @@ -316,7 +320,7 @@ def _otel_libraries(self):
if Version(self.version) >= "1.7.0":
libraries.append("opentelemetry_exporter_otlp_grpc_client")

if self.options.with_logs_preview:
if Version(self.version) >= "1.11" or self.options.with_logs_preview:
libraries.append("opentelemetry_exporter_otlp_grpc_log")

if self.options.get_safe("with_otlp_http"):
Expand All @@ -328,13 +332,13 @@ def _otel_libraries(self):
if Version(self.version) >= "1.5.0":
libraries.append("opentelemetry_exporter_otlp_http_metric")

if self.options.with_logs_preview:
if Version(self.version) >= "1.11" or self.options.with_logs_preview:
libraries.append("opentelemetry_exporter_otlp_http_log")

if self.options.with_prometheus:
libraries.append("opentelemetry_exporter_prometheus")

if self.options.with_elasticsearch and self.options.with_logs_preview:
if self.options.with_elasticsearch and (Version(self.version) >= "1.11" or self.options.with_logs_preview):
libraries.append("opentelemetry_exporter_elasticsearch_logs")

if self.options.with_zipkin:
Expand All @@ -349,7 +353,7 @@ def _otel_libraries(self):
if Version(self.version) >= "1.4.0":
libraries.append("opentelemetry_exporter_ostream_metrics")

if self.options.with_logs_preview:
if Version(self.version) >= "1.11" or self.options.with_logs_preview:
libraries.extend([
"opentelemetry_logs",
"opentelemetry_exporter_ostream_logs",
Expand Down Expand Up @@ -382,7 +386,7 @@ def package_info(self):

self.cpp_info.components["opentelemetry_exporter_in_memory"].libs = []

if self.options.with_logs_preview:
if Version(self.version) >= "1.11" or self.options.with_logs_preview:
self.cpp_info.components["opentelemetry_logs"].requires.extend([
"opentelemetry_resources",
"opentelemetry_common",
Expand Down Expand Up @@ -447,7 +451,7 @@ def package_info(self):
"opentelemetry_exporter_otlp_grpc_client"
])

if self.options.with_logs_preview:
if Version(self.version) >= "1.11.0" or self.options.with_logs_preview:
self.cpp_info.components["opentelemetry_exporter_otlp_grpc_log"].requires.extend([
"opentelemetry_otlp_recordable",
"opentelemetry_exporter_otlp_grpc_client",
Expand Down Expand Up @@ -478,7 +482,7 @@ def package_info(self):
"opentelemetry_exporter_otlp_http_client"
])

if self.options.with_logs_preview:
if Version(self.version) >= "1.11" or self.options.with_logs_preview:
self.cpp_info.components["opentelemetry_exporter_otlp_http_log"].requires.extend([
"opentelemetry_otlp_recordable",
"opentelemetry_exporter_otlp_http_client",
Expand Down

0 comments on commit 7e673e6

Please sign in to comment.