opentelemetry-cpp: add tarball urls (#7981)

* opentelemetry-cpp: add tarball urls

* Add nlohmann_json dependency and update C++ standard

packages/o/opentelemetry-cpp/xmake.lua

@@ -3,10 +3,13 @@ package("opentelemetry-cpp")
     set_description("OpenTelemetry is a collection of tools, APIs, and SDKs. Use it to instrument, generate, collect, and export telemetry data (metrics, logs, and traces) to help you analyze your software’s performance and behavior.")
     set_license("Apache-2.0")
 
-    add_urls("https://github.com/open-telemetry/opentelemetry-cpp.git")
-    add_versions("v1.16.1", "baecbb95bd63df53e0af16e87bc683967962c5f8")
-    add_versions("v1.19.0", "ced79860f8c8a091a2eabfee6d47783f828a9b59")
-    add_versions("v1.22.0", "9ee0c27886f4cc6afc1afbec07a7e8d99b33a9ce")
+    add_urls("https://github.com/open-telemetry/opentelemetry-cpp/archive/refs/tags/$(version).tar.gz",
+             "https://github.com/open-telemetry/opentelemetry-cpp.git", {submodules = false})
+
+    add_versions("v1.22.0", "3428f433f4b435ed1fad64cbdbe75b7288c06f6297786a7036d65d5b9a1d215b")
+    add_versions("v1.21.0", "98e5546f577a11b52a57faed1f4cc60d8c1daa44760eba393f43eab5a8ec46a2")
+    add_versions("v1.19.0", "e0330194b72f2fe4c0ce3ece06b02dc4aa0ab491eb75bf42c6f5e283912e468c")
+    add_versions("v1.16.1", "b8a78bb2a3a78133dbb08bcd04342f4b1e03cb4a19079b8416d408d905fffc37")
 
     add_configs("abseil", {description = "Whether to use Abseil for C++latest features. (not supported for >= v1.21.0)", default = false, type = "boolean"})
     add_configs("api_only", {description = "Only build the API (use as a header-only library). Overrides all options to enable exporters.", default = false, type = "boolean"})
@@ -22,6 +25,11 @@ package("opentelemetry-cpp")
     end
 
     add_deps("cmake")
+    add_deps("nlohmann_json", {configs = {cmake = true}})
+
+    if is_plat("linux", "bsd") then
+        add_syslinks("pthread")
+    end
 
     add_links(
         "opentelemetry_exporter_otlp_http_metric",
@@ -58,6 +66,14 @@ package("opentelemetry-cpp")
     )
 
     on_load(function (package)
+        if package:config("api_only") then
+            package:set("kind", "library", {headeronly = true})
+        end
+
+        if package:version() and package:version():ge("1.21.0") and package:config("abseil") then
+             os.raise("abseil config is not supported for version >= 1.21.0")
+        end
+
         if package:config("otlp_grpc") then
             package:add("deps", "grpc")
         elseif package:config("otlp_http") or package:config("otlp_file") then
@@ -65,22 +81,21 @@ package("opentelemetry-cpp")
         elseif package:config("abseil") then
             package:add("deps", "abseil")
         end
-        if package:config("shared") and package:is_plat("windows") then
-            package:add("defines", "OPENTELEMETRY_BUILD_IMPORT_DLL")
-        end
-    end)
 
-    on_fetch(function (package, opt)
-        if package:version():ge("1.21.0") and package:config("abseil") then
-             os.raise("abseil config is not supported for version >= 1.21.0")
+        if package:config("otlp_http") then
+            package:add("deps", "libcurl")
         end
 
+        if package:config("shared") and package:is_plat("windows") then
+            package:add("defines", "OPENTELEMETRY_BUILD_IMPORT_DLL")
+        end
+    
         local stl = package:config("stl")
         if stl ~= "false" then
             if stl ~= "true" then
                 package:add("defines", "OPENTELEMETRY_STL_VERSION=20" .. stl:sub(4,5))
             else
-                if package:version():ge("1.16.1") and package:version():le("1.22.0") then
+                if package:version() and package:version():ge("1.16.1") and package:version():le("1.22.0") then
                     package:add("defines", "OPENTELEMETRY_STL_VERSION=2023")
                 else
                     os.raise("Update version or define based on https://github.com/open-telemetry/opentelemetry-cpp/blob/main/api/CMakeLists.txt")
@@ -95,16 +110,25 @@ package("opentelemetry-cpp")
             "-DWITH_EXAMPLES=OFF",
             "-DWITH_BENCHMARK=OFF",
             "-DWITH_FUNC_TESTS=OFF",
-            "-DCMAKE_CXX_STANDARD=" .. package:config("cxx_standard")
         }
+
+        if package:dep("protobuf-cpp") then
+            local std = package:dep("abseil"):config("cxx_standard")
+            table.insert(configs, "-DCMAKE_CXX_STANDARD=" .. std)
+            package:data_set("cxx_standard", std)
+        else
+            table.insert(configs, "-DCMAKE_CXX_STANDARD=" .. package:config("cxx_standard"))
+            package:data_set("cxx_standard", package:config("cxx_standard"))
+        end
+
         if package:has_tool("cxx", "clang", "clangxx", "emcc", "emxx") then
             package:add("cxxflags", "-Wno-missing-template-arg-list-after-template-kw")
             table.insert(configs, "-DCMAKE_CXX_FLAGS=-Wno-missing-template-arg-list-after-template-kw")
         end
-        table.insert(configs, "-DCMAKE_BUILD_TYPE=" .. (package:debug() and "Debug" or "Release"))
-        table.insert(configs, "-DCMAKE_POSITION_INDEPENDENT_CODE=" .. (package:config("pic") and "ON" or "OFF"))
+        table.insert(configs, "-DCMAKE_BUILD_TYPE=" .. (package:is_debug() and "Debug" or "Release"))
         if package:config("shared") then
             if package:is_plat("windows") then
+                -- @see https://github.com/open-telemetry/opentelemetry-cpp/issues/2477
                 table.insert(configs, "-DOPENTELEMETRY_BUILD_DLL=ON")
                 table.insert(configs, "-DBUILD_SHARED_LIBS=OFF")
             else
@@ -113,7 +137,7 @@ package("opentelemetry-cpp")
         else
             table.insert(configs, "-DBUILD_SHARED_LIBS=OFF")
         end
-        
+
         table.insert(configs, "-DWITH_OTLP_GRPC=" .. (package:config("otlp_grpc") and "ON" or "OFF"))
         table.insert(configs, "-DWITH_OTLP_HTTP=" .. (package:config("otlp_http") and "ON" or "OFF"))
         table.insert(configs, "-DWITH_OTLP_FILE=" .. (package:config("otlp_file") and "ON" or "OFF"))
@@ -137,5 +161,9 @@ package("opentelemetry-cpp")
     end)
 
     on_test(function (package)
-        assert(package:has_cxxfuncs("opentelemetry::exporter::trace::OStreamSpanExporterFactory::Create()", {configs = {languages = "c++17"}, includes = "opentelemetry/exporters/ostream/span_exporter_factory.h"}))
+        if package:config("api_only") then
+            assert(package:has_cxxincludes("opentelemetry/version.h"))
+        else
+            assert(package:has_cxxfuncs("opentelemetry::exporter::trace::OStreamSpanExporterFactory::Create()", {configs = {languages = "c++" ..  package:data("cxx_standard")}, includes = "opentelemetry/exporters/ostream/span_exporter_factory.h"}))
+        end
     end)