[GitHub] [celix] PengZheng commented on a change in pull request #408: Updates conanfile.py and jansson linking for macos setup

[GitBox <gi...@apache.org>]

PengZheng commented on a change in pull request #408:
URL: https://github.com/apache/celix/pull/408#discussion_r841214321



##########
File path: conanfile.py
##########
@@ -193,6 +193,7 @@ def requirements(self):
         self.options['zlib'].shared = True
         self.requires("libuuid/1.0.3")
         self.options['libuuid'].shared = True
+        self.requires("openssl/[>=1.1.1n <2.0.0]")

Review comment:
       Conan upstream requirement can be overridden downstream. I intentional did so for openssl when integrating Celix with legacy application using Conan. May I have a look at the complete console output? I only have Ubuntu PC at hand.
   
   IIRC, openssl does not use semantic versioning, thus version range does not work:
   
   > There are some packages that do not follow semver. A popular one would be the OpenSSL package with versions as 1.0.2n. They cannot be used with version-ranges. 
   
   https://docs.conan.io/en/latest/versioning/version_ranges.html?highlight=version%20range




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org