Closed PengZheng closed 11 months ago
Merging #620 (4e7c7a4) into master (9647e66) will increase coverage by
1.60%
. The diff coverage is50.00%
.
@@ Coverage Diff @@
## master #620 +/- ##
==========================================
+ Coverage 80.00% 81.61% +1.60%
==========================================
Files 260 260
Lines 34674 34674
==========================================
+ Hits 27741 28298 +557
+ Misses 6933 6376 -557
Files Changed | Coverage Δ | |
---|---|---|
libs/dfi/src/json_serializer.c | 82.83% <0.00%> (ø) |
|
...sub_admin_udp_mc/src/pubsub_udpmc_topic_receiver.c | 73.68% <100.00%> (+73.68%) |
:arrow_up: |
... and 10 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Once https://youtrack.jetbrains.com/issue/CPP-34818 is addressed, using Conan with Clion will be enjoyable.
A quick conan install
then everything will be setup automatically in CLion.
This PR updates the whole project to support both Conan 1.x and the current Conan 2:
conanrun.sh
/deactivate_conanrun.sh
configure()
method will be forbidden by Conan 2 but allowed by Conan 1. For an extensive discussion with the Conan upstream, see https://github.com/conan-io/conan/issues/14528configure()
method. Previously this is done inrequirements()
method, which is not supported by Conan 2.self.requires("openssl/1.1.1t", override=True)
to resolve version conflict of openssl caused by civetweb and libcurl. Note that it does not introduce any dependency of openssl into Celix. For an extensive discusson, see https://github.com/conan-io/conan/issues/14535#issuecomment-1687716696conan build . -bf cmake-build-debug --configure
does not work anymore. It will always lead to a full build.conan install
is used instead to configure the build directory.tc.user_presets_path = False
is added to suppress its production. I reported it to JetBrains (https://youtrack.jetbrains.com/issue/CPP-34818). Once this issue solved, we can expect excellent user experience by combining Conan and CLion.