Skip to content

Updates for CMake 3, CMAKE_MODULE_PATH as subproject - #201

Merged
PJK merged 1 commit into
PJK:masterfrom
thewtex:cmake-improvements
Mar 19, 2022
Merged

PJK merged 1 commit into
PJK:masterfrom
thewtex:cmake-improvements

Conversation

@thewtex

@thewtex thewtex commented Feb 22, 2022 •

Copy link
Copy Markdown
Contributor

Update minimum required version to 3.16 (supported by most current Linux
distributions). CMake 2 is no longer supported.

CMAKE_MODULE_PATH set with CMAKE_CURRENT_SOURCE_DIR instead of
CMAKE_SOURCE_DIR -- this enables libcbor use in an add_subdirectory
context, e.g. with CMake FetchContent.

subdirs is deprecated in favor of add_subdirectory.

PR: Short description of change

Description

A clear and concise description of what the PR does and why.

Checklist

  • I have read followed CONTRIBUTING.md
    • I have added tests
    • I have updated the documentation
    • I have updated the CHANGELOG
  • Are there any breaking changes? If so, are they documented?
  • Does this PR introduce any platform specific code? If so, is this captured in the description?
  • Security: Does this PR potentially affect security? If so, is this captured in the description?
  • Performance: Does this PR potentially affect performance? If so, is this captured in the description?

Update minimum required version to 3.16 (supported by most current Linux
distributions). CMake 2 is no longer supported.

CMAKE_MODULE_PATH set with CMAKE_CURRENT_SOURCE_DIR instead of
CMAKE_SOURCE_DIR -- this enables libcbor use in an add_subdirectory
context, e.g. with CMake FetchContent.

subdirs is deprecated in favor of add_subdirectory.
@PJK

PJK commented Feb 24, 2022

Copy link
Copy Markdown
Owner

Thanks for the PR. The change looks good, but we cannot test it on CI due to #202, so it will have to wait until I fix that.

@PJK

PJK commented Mar 15, 2022

Copy link
Copy Markdown
Owner

Hi @thewtex, apologies for the delay, could you please merge master?

@PJK
PJK merged commit de97626 into PJK:master Mar 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

⌂ 换网址