Merge lp:~widelands-dev/widelands/cmake_errors_WL_VERSION into lp:widelands

[Toni Förster]

explicitly set CMP0054 to fix warning

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 4053. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/433707130.
Appveyor build 3849. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_cmake_errors_WL_VERSION-3849.

[Tino (tino79)]

I am not a big fan of silencing warnings and do not see this warning on both Appveyor and Travis or my system (cmake 3.11.1).

[GunChleoc (gunchleoc)]

LGTM :)

@bunnbot merge

review: Approve

[Toni Förster (stonerl)]

@Tino I get this warning with Cmake 3.12 on macOS.

We're not silencing the warning. We just tell Cmake explicitly to use the new behavior.

[GunChleoc (gunchleoc)]

@bunnbot merge

[GunChleoc (gunchleoc)]

@bunnybot merge

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2018-09-26 08:47:26 +0000
+++ CMakeLists.txt	2018-09-26 18:57:06 +0000
@@ -1,6 +1,7 @@
 project (widelands)
 
 cmake_minimum_required (VERSION 2.8.7)
+cmake_policy(SET CMP0054 NEW)
 
 include("${CMAKE_SOURCE_DIR}/cmake/WlFunctions.cmake")