From 9e3724bca1587ad7063e91683e787bb250cd90a6 Mon Sep 17 00:00:00 2001
From: José Bollo <jose.bollo@iot.bzh>
Date: Fri, 21 Feb 2020 16:13:14 +0100
Subject: Create directories in separate commands
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This change create separate target for creation
separate directories. This fix an obscure error
reported by CMAKE:

  "Deleting primary custom command output ..."

This error appered since recent changes related to
SPEC-2049.

Bug-AGL: SPEC-3207
Bug-AGL: SPEC-2049

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ie531fd203cd054a24e9f5374727bc403341d832d
---
 src/cmake/cmake.d/03-macros.cmake | 87 +++++++++++++++++++++++++--------------
 1 file changed, 56 insertions(+), 31 deletions(-)

diff --git a/src/cmake/cmake.d/03-macros.cmake b/src/cmake/cmake.d/03-macros.cmake
index f36fd7e..389eead 100644
--- a/src/cmake/cmake.d/03-macros.cmake
+++ b/src/cmake/cmake.d/03-macros.cmake
@@ -308,37 +308,60 @@ macro(project_targets_populate)
 	set(PACKAGE_TEST_HTTPDIR ${PROJECT_PKG_TEST_DIR}/${HTTPDIR})
 	set(PACKAGE_TEST_DATADIR ${PROJECT_PKG_TEST_DIR}/${DATADIR})
 
-		add_custom_command(OUTPUT ${PACKAGE_BINDIR}
-					  ${PACKAGE_ETCDIR}
-					  ${PACKAGE_LIBDIR}
-					  ${PACKAGE_HTTPDIR}
-					  ${PACKAGE_DATADIR}
-					  ${PACKAGE_TEST_BINDIR}
-					  ${PACKAGE_TEST_ETCDIR}
-					  ${PACKAGE_TEST_LIBDIR}
-					  ${PACKAGE_TEST_HTTPDIR}
-					  ${PACKAGE_TEST_DATADIR}
-			COMMAND mkdir -p  ${PACKAGE_BINDIR}
-			COMMAND mkdir -p  ${PACKAGE_ETCDIR}
-			COMMAND mkdir -p  ${PACKAGE_LIBDIR}
-			COMMAND mkdir -p  ${PACKAGE_HTTPDIR}
-			COMMAND mkdir -p  ${PACKAGE_DATADIR}
-			COMMAND mkdir -p  ${PACKAGE_TEST_BINDIR}
-			COMMAND mkdir -p  ${PACKAGE_TEST_ETCDIR}
-			COMMAND mkdir -p  ${PACKAGE_TEST_LIBDIR}
-			COMMAND mkdir -p  ${PACKAGE_TEST_HTTPDIR}
-			COMMAND mkdir -p  ${PACKAGE_TEST_DATADIR}
-		)
-		add_custom_target(populate DEPENDS ${PACKAGE_BINDIR}
-						   ${PACKAGE_ETCDIR}
-						   ${PACKAGE_LIBDIR}
-						   ${PACKAGE_HTTPDIR}
-						   ${PACKAGE_DATADIR}
-						   ${PACKAGE_TEST_BINDIR}
-						   ${PACKAGE_TEST_ETCDIR}
-						   ${PACKAGE_TEST_LIBDIR}
-						   ${PACKAGE_TEST_HTTPDIR}
-						   ${PACKAGE_TEST_DATADIR})
+	add_custom_command(OUTPUT ${PROJECT_PKG_BUILD_DIR}
+			   COMMAND mkdir -p ${PROJECT_PKG_BUILD_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_BINDIR}
+			   COMMAND mkdir -p ${PACKAGE_BINDIR}
+			   DEPENDS ${PROJECT_PKG_BUILD_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_ETCDIR}
+			   COMMAND mkdir -p ${PACKAGE_ETCDIR}
+			   DEPENDS ${PROJECT_PKG_BUILD_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_LIBDIR}
+			   COMMAND mkdir -p ${PACKAGE_LIBDIR}
+			   DEPENDS ${PROJECT_PKG_BUILD_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_HTTPDIR}
+			   COMMAND mkdir -p ${PACKAGE_HTTPDIR}
+			   DEPENDS ${PROJECT_PKG_BUILD_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_DATADIR}
+			   COMMAND mkdir -p ${PACKAGE_DATADIR}
+			   DEPENDS ${PROJECT_PKG_BUILD_DIR})
+
+	add_custom_target(prepare_package
+				DEPENDS ${PROJECT_PKG_BUILD_DIR}
+					${PACKAGE_BINDIR}
+					${PACKAGE_ETCDIR}
+					${PACKAGE_LIBDIR}
+					${PACKAGE_HTTPDIR}
+					${PACKAGE_DATADIR})
+
+	add_custom_command(OUTPUT ${PROJECT_PKG_TEST_DIR}
+			   COMMAND mkdir -p ${PROJECT_PKG_TEST_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_TEST_BINDIR}
+			   COMMAND mkdir -p ${PACKAGE_TEST_BINDIR}
+			   DEPENDS ${PROJECT_PKG_TEST_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_TEST_ETCDIR}
+			   COMMAND mkdir -p ${PACKAGE_TEST_ETCDIR}
+			   DEPENDS ${PROJECT_PKG_TEST_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_TEST_LIBDIR}
+			   COMMAND mkdir -p ${PACKAGE_TEST_LIBDIR}
+			   DEPENDS ${PROJECT_PKG_TEST_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_TEST_HTTPDIR}
+			   COMMAND mkdir -p ${PACKAGE_TEST_HTTPDIR}
+			   DEPENDS ${PROJECT_PKG_TEST_TEST_DIR})
+	add_custom_command(OUTPUT ${PACKAGE_TEST_DATADIR}
+			   COMMAND mkdir -p ${PACKAGE_TEST_DATADIR}
+			   DEPENDS ${PROJECT_PKG_TEST_DIR})
+
+	add_custom_target(prepare_package_test
+				DEPENDS ${PROJECT_PKG_TEST_DIR}
+					${PACKAGE_TEST_BINDIR}
+					${PACKAGE_TEST_ETCDIR}
+					${PACKAGE_TEST_LIBDIR}
+					${PACKAGE_TEST_HTTPDIR}
+					${PACKAGE_TEST_DATADIR})
+
+	add_custom_target(populate)
+	add_dependencies(populate prepare_package prepare_package_test)
 
 	# Dirty trick to define a default INSTALL command for app-templates handled
 	# targets
@@ -584,10 +607,12 @@ You need a config.xml template: please specify WIDGET_CONFIG_TEMPLATE correctly.
 	# populate icon
 	add_custom_command(OUTPUT ${PROJECT_PKG_BUILD_DIR}/${PROJECT_ICON}
 		COMMAND cp -d ${ICON_PATH} ${PROJECT_PKG_BUILD_DIR}/${PROJECT_ICON}
+		DEPENDS ${PROJECT_PKG_BUILD_DIR}
 	)
 	list(APPEND widget_files_items ${PROJECT_PKG_BUILD_DIR}/${PROJECT_ICON})
 	add_custom_command(OUTPUT ${PROJECT_PKG_TEST_DIR}/${PROJECT_ICON}
 		COMMAND cp -d ${ICON_PATH} ${PROJECT_PKG_TEST_DIR}/${PROJECT_ICON}
+		DEPENDS ${PROJECT_PKG_TEST_DIR}
 	)
 	list(APPEND test_widget_files_items ${PROJECT_PKG_TEST_DIR}/${PROJECT_ICON})
 
-- 
cgit 1.2.3-korg