add package qpid-proton 0.40.0

[Divix55]

Summary

Changes to recipe: qpid-proton/0.40.0

Motivation

New package qpid-proton

Details

This PR add Apache qpid-proton library to conan (already available on vcpkg). Details:
https://qpid.apache.org/proton/index.html

---

• Read the contributing guidelines
• Checked that this PR is not a duplicate: list of PRs by recipe
• Tested locally with at least one configuration using a recent version of Conan

[CLAassistant]

All committers have signed the CLA.

[ErniGH]

It also requires SWIG
https://github.com/apache/qpid-proton/blob/0.40.0/CMakeLists.txt#L35

[Divix55]

SWIG library is used only for ruby bindings which is not part of this package, its being built without SWIG just fine so I skipped it. Otherwise Conan is complaining that SWIG is not used by any components requires (which is correct, due to disabled ruby bindings).

[uilianries]

Suggested change

self.tool_requires("cpython/[>=3.9]")

Please, avoid using cpython package, usually it result compatibility bugs in the future. Let users consume their system python instead.

[uilianries]

@Divix55 Hello! Thank you for updating your PR so quickly!

I would like to ask if you really need opentemetry-cpp in your scenario or is just an optional dependency?

I did a quick look at other package managers, and nothing is using opentelemetry-cpp as a requirement, for instance:

• Homebrew only uses openssl and libuv: https://formulae.brew.sh/formula/qpid-proton
• Debian only uses openssl: https://packages.debian.org/bookworm/libqpid-proton11
• Conda feedstock has no support for qpid-proton

In case you don't have a real usage for Opentelemetry, I would suggest not including it as optional dependency for now. It only works with Linux and requires extra logic in the recipe.

[uilianries]

@Divix55 About your recent patch, I would suggest something less invasive:

index 920f857d9..a626ca02f 100644
--- a/recipes/qpid-proton/all/conanfile.py
+++ b/recipes/qpid-proton/all/conanfile.py
@@ -94,6 +94,8 @@ class QpidProtonConan(ConanFile):
         deps = CMakeDeps(self)
         deps.set_property("libuv", "cmake_file_name", "Libuv")
         deps.set_property("libuv", "cmake_target_name", "Libuv::Libuv")
+        deps.set_property("jsoncpp", "cmake_additional_variables_prefixes", ["JsonCpp",])
+        deps.set_property("opentelemetry-cpp::opentelemetry_trace", "cmake_target_name", "opentelemetry-cpp::trace")
         deps.generate()
 
     def build(self):

It avoids patching both jsoncpp cmake variable and opentelemetry cmake target.

Still, the target_link_libraries is missing for static library, but is something that's not working for users that are building qpid-proton from source as well. I would suggest opening an issue to the upstream first, if possible, to have a proper fix from there.