widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #16531
[Merge] lp:~widelands-dev/widelands/fix-wl_include_system_directories into lp:widelands
GunChleoc has proposed merging lp:~widelands-dev/widelands/fix-wl_include_system_directories into lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/fix-wl_include_system_directories/+merge/366266
AppVeyor is in a huff again. Let's see if this will fix it.
--
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/fix-wl_include_system_directories into lp:widelands.
=== modified file 'cmake/WlFunctions.cmake'
--- cmake/WlFunctions.cmake 2017-05-14 21:06:23 +0000
+++ cmake/WlFunctions.cmake 2019-04-18 12:57:44 +0000
@@ -40,7 +40,7 @@
# it works is different. SYSTEM includes silence warnings for included headers etc.
function(wl_include_system_directories TARGET DIR)
_include_directories_internal(${TARGET} ${DIR} TRUE)
-endfunction(wl_include_system_directories TARGET_DIR)
+endfunction(wl_include_system_directories TARGET DIR)
# Add common compile tasks, like includes and libraries to link against for third party
# libraries, and codecheck hook for sources.
Follow ups