← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/cmake-reworked into lp:widelands

 

Not sure if this is the right place for this command, regarding the cmake-2.8.12.1 problem:

The problem is caused by target_include_directories with the SYSTEM keyword.
cmake-2.8.11 (saucy) does not have it.

As far as I understand, the SYSTEM keyword is nothing serious for making it work at all. See http://www.cmake.org/cmake/help/git-master/command/target_include_directories.html

So it is pretty straightforward to make a little patch for saucy which removes the SYSTEM keyword and lowers the cmake minimum version to 2.8.11. It may generate more warnings, but it compiles.

This does not help with precise, though. cmake-2.8.7 does not understand target_include_directories. 
-- 
https://code.launchpad.net/~widelands-dev/widelands/cmake-reworked/+merge/222455
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/cmake-reworked.


References