james94 commented on code in PR #1793:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1793#discussion_r1718960988


##########
cmake/MiNiFiOptions.cmake:
##########
@@ -42,6 +42,7 @@ add_minifi_option(DOCKER_SKIP_TESTS "Skip building tests in 
docker image targets
 add_minifi_option(DOCKER_PUSH "Push created images to the specified tags" OFF)
 
 add_minifi_option(PORTABLE "Instructs the compiler to remove architecture 
specific optimizations" ON)
+add_minifi_option(MINIFI_USE_CONAN_PACKAGER "Instructs cmake to use conan 
packager to install external libs & build MiNiFi C++" OFF)

Review Comment:
   Currently in our conanfile.py, we have the following MINIFI external 
libraries sources set to CONAN:
   
   ~~~python
           tc.variables["MINIFI_LIBCURL_SOURCE"] = "CONAN"
           tc.variables["MINIFI_OPENSSL_SOURCE"] = "CONAN"
           tc.variables["MINIFI_ZLIB_SOURCE"] = "CONAN"
           tc.variables["MINIFI_CIVETWEB_SOURCE"] = "CONAN"
           tc.variables["MINIFI_LIBXML2_SOURCE"] = "CONAN"
           tc.variables["MINIFI_FMT_SOURCE"] = "CONAN"
           tc.variables["MINIFI_SPDLOG_SOURCE"] = "CONAN"
           tc.variables["MINIFI_CATCH2_SOURCE"] = "CONAN"
   ~~~



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to