[ https://issues.apache.org/jira/browse/PROTON-2254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17351821#comment-17351821 ]
ASF GitHub Bot commented on PROTON-2254: ---------------------------------------- codecov-commenter edited a comment on pull request #317: URL: https://github.com/apache/qpid-proton/pull/317#issuecomment-848816517 # [Codecov](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report > Merging [#317](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (d3a2a2b) into [main](https://codecov.io/gh/apache/qpid-proton/commit/ae6c741e3377787ec725fd6e6ebe0aedfcb1e110?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (ae6c741) will **increase** coverage by `19.36%`. > The diff coverage is `n/a`. > :exclamation: Current head d3a2a2b differs from pull request most recent head f4817bc. Consider uploading reports for the commit f4817bc to get more accurate results [![Impacted file tree graph](https://codecov.io/gh/apache/qpid-proton/pull/317/graphs/tree.svg?width=650&height=150&src=pr&token=UKKzV9XnFF&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) ```diff @@ Coverage Diff @@ ## main #317 +/- ## =========================================== + Coverage 68.60% 87.96% +19.36% =========================================== Files 351 47 -304 Lines 70276 2394 -67882 =========================================== - Hits 48212 2106 -46106 + Misses 22064 288 -21776 ``` | [Impacted Files](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | | |---|---|---| | [ruby/lib/util/error\_handler.rb](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cnVieS9saWIvdXRpbC9lcnJvcl9oYW5kbGVyLnJi) | `59.37% <0.00%> (-28.13%)` | :arrow_down: | | [cpp/src/reconnect\_test.cpp](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y3BwL3NyYy9yZWNvbm5lY3RfdGVzdC5jcHA=) | | | | [c/tests/pn\_test.hpp](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Yy90ZXN0cy9wbl90ZXN0LmhwcA==) | | | | [c/tests/pn\_test\_proactor.hpp](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Yy90ZXN0cy9wbl90ZXN0X3Byb2FjdG9yLmhwcA==) | | | | [python/tests/proton-test](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-cHl0aG9uL3Rlc3RzL3Byb3Rvbi10ZXN0) | | | | [c/src/messenger/messenger.c](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Yy9zcmMvbWVzc2VuZ2VyL21lc3Nlbmdlci5j) | | | | [cpp/src/value.cpp](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y3BwL3NyYy92YWx1ZS5jcHA=) | | | | [cpp/include/proton/timestamp.hpp](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Y3BwL2luY2x1ZGUvcHJvdG9uL3RpbWVzdGFtcC5ocHA=) | | | | [c/src/proactor/proactor-internal.h](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Yy9zcmMvcHJvYWN0b3IvcHJvYWN0b3ItaW50ZXJuYWwuaA==) | | | | [c/tools/include/pncompat/misc\_funcs.inc](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-Yy90b29scy9pbmNsdWRlL3BuY29tcGF0L21pc2NfZnVuY3MuaW5j) | | | | ... and [295 more](https://codecov.io/gh/apache/qpid-proton/pull/317/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | | ------ [Continue to review full report at Codecov](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [ae6c741...f4817bc](https://codecov.io/gh/apache/qpid-proton/pull/317?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Relative paths in CMake share > ----------------------------- > > Key: PROTON-2254 > URL: https://issues.apache.org/jira/browse/PROTON-2254 > Project: Qpid Proton > Issue Type: Improvement > Components: proton-c > Affects Versions: proton-c-0.31.0 > Environment: Debian Linux; build Proton then Dispatch into each their > own `DESTDIR`. This is done by our "layered build" system, > https://gitlab.com/arpa2/mkhere/-/blob/master/qpid_proton.sh and > https://gitlab.com/arpa2/mkhere/-/blob/master/qpid_dispatch.sh > Reporter: Rick van Rein > Priority: Minor > Labels: cmake, install, proton > Original Estimate: 0.25h > Remaining Estimate: 0.25h > > This sequency installs well: > {noformat} > cmake > make > make DESTDIR=/some/where install > {noformat} > However, further use of the installation fails, because of strict > dependencies on the *installed* paths in files like `ProtonConfig.cmake`: > {noformat} > set_target_properties(Proton::core > PROPERTIES > IMPORTED_LOCATION "/usr/local/lib/libqpid-proton-core.so" > IMPORTED_LOCATION_DEBUG "/usr/local/lib/libqpid-proton-core.so" > INTERFACE_INCLUDE_DIRECTORIES "${Proton_Core_INCLUDE_DIRS}") > {noformat} > **What would work:** First switch to the `DESTDIR`, then continue building > something like Qpid Dispatch Router. > **Solution:** Use relative directories, like in: > {noformat} > # Compute the installation prefix relative to this file. > get_filename_component(_IMPORT_PREFIX "${CMAKE_CURRENT_LIST_FILE}" PATH) > get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH) > get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH) > get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH) > if(_IMPORT_PREFIX STREQUAL "/") > set(_IMPORT_PREFIX "") > endif() > {noformat} > **Work-around:** An unhappy quickfix is > {noformat} > find /some/where -name *.cmake -exec \ > sed -i "s+/usr/+/some/where/usr/+g" {} \; > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org