Skip to content

cmake minimal version 2.8.12#3374

Closed
def- wants to merge 2 commits intoddnet:masterfrom
def-:pr-cmake-min
Closed

cmake minimal version 2.8.12#3374
def- wants to merge 2 commits intoddnet:masterfrom
def-:pr-cmake-min

Conversation

@def-
Copy link
Member

@def- def- commented Dec 2, 2020

CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.

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

Checklist

  • Tested the change ingame
  • Provided screenshots if it is a visual change
  • Tested in combination with possibly related configuration options
  • Written a unit test if it works standalone, system.c especially
  • Considered possible null pointers and out of bounds array indexing
  • Changed no physics that affect existing maps
  • Tested the change with ASan+UBSan or valgrind's memcheck (optional)

CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 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.
@heinrich5991
Copy link
Member

bors r+

Not sure if ubuntu-latest is correctly failing…

bors bot added a commit that referenced this pull request Dec 2, 2020
3374: cmake minimal version 2.8.12 r=heinrich5991 a=def-

CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 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.

<!-- What is the motivation for the changes of this pull request -->

## Checklist

- [ ] Tested the change ingame
- [ ] Provided screenshots if it is a visual change
- [ ] Tested in combination with possibly related configuration options
- [ ] Written a unit test if it works standalone, system.c especially
- [ ] Considered possible null pointers and out of bounds array indexing
- [ ] Changed no physics that affect existing maps
- [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--undefinedbehavioursanitizer-or-valgrinds-memcheck) (optional)


Co-authored-by: def <[email protected]>
@bors
Copy link
Contributor

bors bot commented Dec 2, 2020

Build failed:

@heinrich5991
Copy link
Member

Ah, the problem is the automatic GTest download & build.

@def-
Copy link
Member Author

def- commented Dec 2, 2020

Yep, got any good idea how to fix that?

@heinrich5991
Copy link
Member

Looking into it rn.

@heinrich5991
Copy link
Member

set(DDNET_GTEST_VERSION 32f4f52d95dc99c35f51deed552a6ba700567f94) would probably be the easiest way. @def-

@def-
Copy link
Member Author

def- commented Dec 2, 2020

Seems to not exist:

Cloning into 'googletest-src'...
fatal: reference is not a tree: 32f4f52d95dc99c35f51deed552a6ba700567f94
CMake Error at googletest-download/googletest-prefix/tmp/googletest-gitclone.cmake:40 (message):
  Failed to checkout tag: '32f4f52d95dc99c35f51deed552a6ba700567f94'
CMakeFiles/googletest.dir/build.make:110: recipe for target 'googletest-prefix/src/googletest-stamp/googletest-download' failed

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