From 264fe3720f00c1ad3ae989be51747639a07be465 Mon Sep 17 00:00:00 2001
From: Stephen Birarda <commit@birarda.com>
Date: Thu, 14 Jan 2016 13:18:50 -0800
Subject: [PATCH] use separate targets for npm install/packager

---
 console/CMakeLists.txt | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/console/CMakeLists.txt b/console/CMakeLists.txt
index 576aaf0747..86bb1a38b6 100644
--- a/console/CMakeLists.txt
+++ b/console/CMakeLists.txt
@@ -5,11 +5,18 @@ if (PRODUCTION_BUILD)
 endif()
 
 # add a target that will package the console
-add_custom_target(${TARGET_NAME} ALL
-  npm install
-  COMMAND npm run packager -- --out ${CMAKE_CURRENT_BINARY_DIR} ${PRODUCTION_OPTION}
+add_custom_target(${TARGET_NAME}-npm-install
+  COMMAND npm install
   WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
 )
+add_custom_target(${TARGET_NAME}
+  COMMAND npm run packager -- --out ${CMAKE_CURRENT_BINARY_DIR} ${PRODUCTION_OPTION}
+  WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+  DEPENDS ${TARGET_NAME}-npm-install
+)
+
+set_target_properties(${TARGET_NAME} PROPERTIES FOLDER "Console")
+set_target_properties(${TARGET_NAME}-npm-install PROPERTIES FOLDER "Console")
 
 # add a dependency from the package target to the server components
 add_dependencies(${TARGET_NAME} assignment-client domain-server)