fix game_packs_all target 77/head
authorThomas Debesse <dev@illwieckz.net>
Sat, 14 Oct 2017 02:06:45 +0000 (04:06 +0200)
committerThomas Debesse <dev@illwieckz.net>
Sat, 14 Oct 2017 21:28:58 +0000 (23:28 +0200)
Previously, game_packs_all target was just calling game_packs target, so it
was just installing free game packs, not all. It's now a proper target
setting proper environment variables before calling install-gamepacks.sh.

CMakeLists.txt

index 00e9814..6fb9b8e 100644 (file)
@@ -249,13 +249,18 @@ endif ()
 #-----------------------------------------------------------------------
 
 option(DOWNLOAD_GAMEPACKS "Download game packs" ON)
-add_custom_target(game_packs
-        COMMAND ${CMAKE_COMMAND} -E make_directory games
-        COMMAND DOWNLOAD_GAMEPACKS=yes SOURCE_DIR="${PROJECT_SOURCE_DIR}" "${PROJECT_SOURCE_DIR}/install-gamepacks.sh" "${PROJECT_BINARY_DIR}"
-        COMMENT "Downloading game packs"
-        )
 if (DOWNLOAD_GAMEPACKS)
-    add_custom_target(game_packs_all ALL DEPENDS game_packs)
+    add_custom_target(game_packs
+            COMMAND ${CMAKE_COMMAND} -E make_directory games
+            COMMAND DOWNLOAD_GAMEPACKS=yes SOURCE_DIR="${PROJECT_SOURCE_DIR}" "${PROJECT_SOURCE_DIR}/install-gamepacks.sh" "${PROJECT_BINARY_DIR}"
+            COMMENT "Downloading free game packs"
+            )
+
+    add_custom_target(game_packs_all
+            COMMAND ${CMAKE_COMMAND} -E make_directory games
+            COMMAND DOWNLOAD_GAMEPACKS=all SOURCE_DIR="${PROJECT_SOURCE_DIR}" "${PROJECT_SOURCE_DIR}/install-gamepacks.sh" "${PROJECT_BINARY_DIR}"
+            COMMENT "Downloading all game packs"
+            )
 endif ()
 
 #-----------------------------------------------------------------------