From da7f40f7c2eb1a5928876de7b89abb55a2f8fdab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C4=99drzej=20Nowacki?= <jedrzej.nowacki@qt.io>
Date: Fri, 21 Jun 2019 14:25:44 +0200
Subject: [PATCH] Update the instructions after CI changed way of splitting
 arguments
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Now execute command can directly specified how the arguments should
be divided. The default option doesn't work well for CONFIGURE_ARGS,
because we want to pass it's result as a separate subprocess
arguments.

Change-Id: I6b7adb04164f5568d9ad2f45e3334d7a7a49bc79
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@qt.io>
---
 coin/module_config.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/coin/module_config.yaml b/coin/module_config.yaml
index 868a106d7e..840310ed66 100644
--- a/coin/module_config.yaml
+++ b/coin/module_config.yaml
@@ -173,6 +173,7 @@ build_instructions:
       equals_value: Windows
   - type: ExecuteCommand
     command: "{{.Env.ENV_PREFIX}} cmake {{.Env.CONFIGURE_ARGS}} -DCMAKE_INSTALL_PREFIX:PATH={{.InstallDir}} -DBUILD_TESTING=OFF {{.SourceDir}}"
+    executeCommandArgumentSplitingBehavior: SplitAfterVariableSubstitution
     maxTimeInSeconds: 6000
     maxTimeBetweenOutput: 120
     userMessageOnFailure: >