summaryrefslogtreecommitdiffstats
path: root/coin
diff options
context:
space:
mode:
authorSimon Hausmann <simon.hausmann@qt.io>2019-06-17 09:00:38 +0200
committerSimon Hausmann <simon.hausmann@qt.io>2019-06-17 09:10:32 +0000
commit4de70ce95278b29c3e811f1496505459f257adc4 (patch)
tree4f82dd4c4e7e1ab72543719617c0cccc43ffe10f /coin
parentfe3bd212fc85b6c3a64bb8bd5beca4ff4d3493a1 (diff)
Avoid hardcoding the underlying build tool
Since 3.15, not only --build can call the low-level build tool, --install can also be used to run the installation. There's also an environment variable that can be used to control the backend at cmake time, but that may only be useful later. Change-Id: I8c1ee48f946e110af3e824cf8980bbacbb94db99 Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@qt.io> Reviewed-by: Qt CMake Build Bot
Diffstat (limited to 'coin')
-rw-r--r--coin/module_config.yaml8
1 files changed, 7 insertions, 1 deletions
diff --git a/coin/module_config.yaml b/coin/module_config.yaml
index 4de47d1e40..90effa8dbd 100644
--- a/coin/module_config.yaml
+++ b/coin/module_config.yaml
@@ -171,11 +171,17 @@ build_instructions:
maxTimeInSeconds: 60
userMessageOnFailure: Could not print the file cmake config file. This should not be possible.
- type: ExecuteCommand
- command: "{{.Env.ENV_PREFIX}} ninja install"
+ command: "{{.Env.ENV_PREFIX}} cmake --build ."
maxTimeInSeconds: 6000
maxTimeBetweenOutput: 120
userMessageOnFailure: >
Failed to build sources. In the current state bug can be everywhere. Contact Liang first.
+ - type: ExecuteCommand
+ command: "{{.Env.ENV_PREFIX}} cmake --install ."
+ maxTimeInSeconds: 6000
+ maxTimeBetweenOutput: 120
+ userMessageOnFailure: >
+ Failed to install sources. In the current state bug can be everywhere. Contact Liang first.
- type: SignPackage
enable_if:
condition: property