james94 commented on code in PR #1793:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1793#discussion_r1741173901
##########
cmake/GetZLIB.cmake:
##########
Review Comment:
@szaszm @lordgamez When building MiNiFi with conan build, **217/219 ctests
PASS**.
34 - ProvenanceTests (Subprocess aborted)
219 - ControllerTests (Failed)
I think **ProvenanceTests** from RocksDB fails because of
double free or corruption (!prev)
Test #34: ProvenanceTests ........ Subprocess aborted***Exception:
0.19 sec
In regards to this **ProvenanceTests**, the main difference between MiNiFi
conan build and standalone CMake build is that in conan build we use **zstd**
and **zlib** conan packages to try to link into CMake's system library
installed **RocksDB** while with standalone CMake approach, we use CMake to
**build and install zlib & zstd external system libs** while installing system
library **RocksDB**
@szaszm any thoughts on what could cause **ProvenanceTest** to have this
**double free or corruption (!prev)** error when building MiNiFi using conan?
This doesn't happen when I build MiNiFi with standalone CMake. Also in my much
larger PR where I added support to install **patched RocksDB conan package**
based on **standalone CMake's patched RocksDB**, I dont think I saw this ctest
error: https://github.com/apache/nifi-minifi-cpp/pull/1775
When building MiNiFi with standalone CMake, **all 275 ctests PASS.**
--
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]
