Re: [PR] MINIFICPP-2346: Replace CMake FetchContent with Conan Packager for Faster Builds [nifi-minifi-cpp]

2024-05-02 Thread via GitHub
szaszm commented on PR #1775: URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2090806035 1.0.0 will be released some time after NiFi 2.0, but there is no clear target date. We were just discussing the idea (so no commitment), as third party library builds are already t

Re: [PR] MINIFICPP-2346: Replace CMake FetchContent with Conan Packager for Faster Builds [nifi-minifi-cpp]

2024-05-02 Thread via GitHub
james94 commented on PR #1775: URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2090795945 @szaszm thanks for sharing the requirements and what you guys are looking for to make this PR viable. I will shift the updates to meet those requirements. I appreciate it. -- T

Re: [PR] MINIFICPP-2346: Replace CMake FetchContent with Conan Packager for Faster Builds [nifi-minifi-cpp]

2024-05-02 Thread via GitHub
james94 commented on PR #1775: URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2090790581 @lordgamez thanks for the suggestion, I'll switch the default option to USE_FETCH_CONTENT for now. I'm excited to hear it was planned for 1.0.0 release to bring this feature to th

Re: [PR] MINIFICPP-2346: Replace CMake FetchContent with Conan Packager for Faster Builds [nifi-minifi-cpp]

2024-05-02 Thread via GitHub
szaszm commented on PR #1775: URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2090424252 There are a few things we would need for this to be viable: - The python bootstrap should be updated to support selecting each extension with its dependencies installed from Cona

Re: [PR] MINIFICPP-2346: Replace CMake FetchContent with Conan Packager for Faster Builds [nifi-minifi-cpp]

2024-05-02 Thread via GitHub
lordgamez commented on PR #1775: URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2090368183 Hi @james94, thanks for the heads up, this is great stuff you have here! Last week we actually talked about having this feature implemented as part of the 1.0.0 release, so it i