Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMake Deprecation Warnings #606

Closed
nabelekt opened this issue Apr 2, 2024 · 4 comments
Closed

CMake Deprecation Warnings #606

nabelekt opened this issue Apr 2, 2024 · 4 comments
Labels
bug This issue is a bug. p3 This is a minor priority issue

Comments

@nabelekt
Copy link

nabelekt commented Apr 2, 2024

Describe the bug

I am using v0.26.4 and getting many warnings like this when building:

CMake Deprecation Warning at libraries/third_party/aws-crt-cpp/crt/aws-c-sdkutils/CMakeLists.txt:4 (cmake_minimum_required):
  Compatibility with CMake < 3.5 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

The fix is given the last two lines there.

Expected Behavior

No warnings when building

Current Behavior

See above

Reproduction Steps

Build with the library

Possible Solution

No response

Additional Information/Context

No response

aws-crt-cpp version used

0.26.4

Compiler and version used

gcc version 13.2.0 (Ubuntu 13.2.0-4ubuntu3)

Operating System and version

Ubuntu 23.10

@nabelekt nabelekt added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Apr 2, 2024
@nabelekt nabelekt changed the title (short issue description) CMake Deprecation Warnings Apr 2, 2024
@nabelekt
Copy link
Author

nabelekt commented Apr 2, 2024

For what it's worth, https://docs.aws.amazon.com/sdk-for-cpp/v1/developer-guide/setup-linux.html specifies the CMake minimum versions as 3.13.

@DmitriyMusatkin
Copy link
Contributor

Note that CRT and SDK are separate projects, with CRT being a dependency of CPP SDK. And while CPP SDK has a min cmake version 3.13, CRT supports much older cmake versions.

We do want to increase our min version in the future, but as of right now we dont have a concrete date on when that might happen

@graebm
Copy link
Contributor

graebm commented Apr 3, 2024

It's possible to shut these warnings up by stating that your project is known to be compatible with later versions, like:
cmake_minimum_required(VERSION 3.1...3.29)

unfortunately, this isn't supported until cmake 3.12, so the code would need to be like:

if (CMAKE_VERSION VERSION_LESS "3.12")
    cmake_minimum_required(VERSION 3.1)
else ()
    cmake_minimum_required(VERSION 3.1...3.29)
endif ()

@jmklix jmklix added p3 This is a minor priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Apr 3, 2024
@jmklix
Copy link
Member

jmklix commented Nov 16, 2024

You should no longer see any deprecation warnings from any of the crt repositories. We have updated our minimum cmake version to 3.9. You might still see some other cmake warnings from other dependencies. One of them is s2n-tls. I opened an issue here asking them to fix it.

@jmklix jmklix closed this as completed Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. p3 This is a minor priority issue
Projects
None yet
Development

No branches or pull requests

4 participants