From 7ebd7a2213a254dc227b3f3a1885ccf1c506978d Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Tue, 18 Aug 2020 14:06:19 +0200
Subject: [PATCH] Nim: Fix use of non-default arguments for Nimble build step

Amends 200d81d3, which mis-placed the call for argument setup,
effectively resetting it to default on each run.

Change-Id: I4b4f3ab90570d14e7ffc41346eb4f06241eb4f4c
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
---
 src/plugins/nim/project/nimblebuildstep.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/plugins/nim/project/nimblebuildstep.cpp b/src/plugins/nim/project/nimblebuildstep.cpp
index 973b70942da..3bac97c4e20 100644
--- a/src/plugins/nim/project/nimblebuildstep.cpp
+++ b/src/plugins/nim/project/nimblebuildstep.cpp
@@ -103,6 +103,7 @@ NimbleBuildStep::NimbleBuildStep(BuildStepList *parentList, Id id)
     m_arguments = addAspect<ArgumentsAspect>();
     m_arguments->setSettingsKey(Constants::C_NIMBLEBUILDSTEP_ARGUMENTS);
     m_arguments->setResetter([this] { return defaultArguments(); });
+    m_arguments->setArguments(defaultArguments());
 
     setCommandLineProvider([this] {
         return CommandLine(QStandardPaths::findExecutable("nimble"),
@@ -119,7 +120,6 @@ NimbleBuildStep::NimbleBuildStep(BuildStepList *parentList, Id id)
 
 bool NimbleBuildStep::init()
 {
-    m_arguments->setArguments(defaultArguments());
     ProcessParameters *params = processParameters();
     setupProcessParameters(params);