diff options
author | hjk <hjk@qt.io> | 2019-12-05 16:19:42 +0100 |
---|---|---|
committer | hjk <hjk@qt.io> | 2019-12-06 14:49:59 +0000 |
commit | 3c41b91d44838fcadfa50c4044e338d43a27cc10 (patch) | |
tree | ade3a866f972b8b4b367a21ccb88d7fa2417caff /src/plugins/nim | |
parent | f0316dbae9405166c4a6981db55295440ded7962 (diff) | |
download | qt-creator-3c41b91d44838fcadfa50c4044e338d43a27cc10.tar.gz |
ProjectExplorer: Settle on fixed set of build step lists
Even if the implementation allowed different lists than 'build'
and 'clean', that's the only set that has been used, ever.
If really needed, this could be re-instated, but for now having
them fixed removes part of the neeed for a two-phase construction
and helps to simplify user code.
Change-Id: I3df09a1829a7d020ef8963d358ea80f8d199ba13
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/plugins/nim')
-rw-r--r-- | src/plugins/nim/project/nimblebuildconfiguration.cpp | 5 | ||||
-rw-r--r-- | src/plugins/nim/project/nimbuildconfiguration.cpp | 16 |
2 files changed, 5 insertions, 16 deletions
diff --git a/src/plugins/nim/project/nimblebuildconfiguration.cpp b/src/plugins/nim/project/nimblebuildconfiguration.cpp index a49485aed5..873b88de66 100644 --- a/src/plugins/nim/project/nimblebuildconfiguration.cpp +++ b/src/plugins/nim/project/nimblebuildconfiguration.cpp @@ -65,8 +65,6 @@ BuildConfiguration::BuildType NimbleBuildConfiguration::buildType() const void NimbleBuildConfiguration::initialize() { - BuildConfiguration::initialize(); - m_buildType = initialBuildType(); setBuildDirectory(project()->projectDirectory()); @@ -78,8 +76,7 @@ void NimbleBuildConfiguration::initialize() // // Don't add a nimble build step when the package has no binaries (i.e a library package) // if (!m_nimbleBuildSystem->metadata().bin.empty()) // { - BuildStepList *buildSteps = stepList(ProjectExplorer::Constants::BUILDSTEPS_BUILD); - buildSteps->appendStep(new NimbleBuildStep(buildSteps)); + buildSteps()->appendStep(new NimbleBuildStep(buildSteps())); // } } diff --git a/src/plugins/nim/project/nimbuildconfiguration.cpp b/src/plugins/nim/project/nimbuildconfiguration.cpp index c04685f238..42f0177ceb 100644 --- a/src/plugins/nim/project/nimbuildconfiguration.cpp +++ b/src/plugins/nim/project/nimbuildconfiguration.cpp @@ -75,8 +75,6 @@ NimBuildConfiguration::NimBuildConfiguration(Target *target, Core::Id id) void NimBuildConfiguration::initialize() { - BuildConfiguration::initialize(); - // Create the build configuration and initialize it from build info setBuildDirectory(defaultBuildDirectory(target()->kit(), project()->projectFilePath(), @@ -85,8 +83,7 @@ void NimBuildConfiguration::initialize() // Add nim compiler build step { - BuildStepList *buildSteps = stepList(ProjectExplorer::Constants::BUILDSTEPS_BUILD); - auto nimCompilerBuildStep = new NimCompilerBuildStep(buildSteps); + auto nimCompilerBuildStep = new NimCompilerBuildStep(buildSteps()); NimCompilerBuildStep::DefaultBuildOptions defaultOption; switch (initialBuildType()) { case BuildConfiguration::Release: @@ -107,14 +104,11 @@ void NimBuildConfiguration::initialize() if (!nimFiles.isEmpty()) nimCompilerBuildStep->setTargetNimFile(nimFiles.first()); - buildSteps->appendStep(nimCompilerBuildStep); + buildSteps()->appendStep(nimCompilerBuildStep); } // Add clean step - { - BuildStepList *cleanSteps = stepList(ProjectExplorer::Constants::BUILDSTEPS_CLEAN); - cleanSteps->appendStep(Constants::C_NIMCOMPILERCLEANSTEP_ID); - } + cleanSteps()->appendStep(Constants::C_NIMCOMPILERCLEANSTEP_ID); } FilePath NimBuildConfiguration::cacheDirectory() const @@ -131,9 +125,7 @@ FilePath NimBuildConfiguration::outFilePath() const const NimCompilerBuildStep *NimBuildConfiguration::nimCompilerBuildStep() const { - BuildStepList *steps = stepList(ProjectExplorer::Constants::BUILDSTEPS_BUILD); - QTC_ASSERT(steps, return nullptr); - foreach (BuildStep *step, steps->steps()) + foreach (BuildStep *step, buildSteps()->steps()) if (step->id() == Constants::C_NIMCOMPILERBUILDSTEP_ID) return qobject_cast<NimCompilerBuildStep *>(step); return nullptr; |