Commit 1e178915 authored by Tobias Hunger's avatar Tobias Hunger

AbstractProcessStep: Free output parser chain in error case

Change-Id: Ie0dc094068bfa56f728d84ee13e70efbc980f8b3
Reviewed-by: default avatarhjk <hjk@qt.io>
parent 6ef5a36b
...@@ -172,6 +172,13 @@ bool CMakeBuildStep::init(QList<const BuildStep *> &earlierSteps) ...@@ -172,6 +172,13 @@ bool CMakeBuildStep::init(QList<const BuildStep *> &earlierSteps)
emit addTask(Task::buildConfigurationMissingTask()); emit addTask(Task::buildConfigurationMissingTask());
canInit = false; canInit = false;
} }
if (!bc->isEnabled()) {
emit addTask(Task(Task::Error,
QCoreApplication::translate("CMakeProjectManager::CMakeBuildStep",
"The build configuration is currently disabled."),
Utils::FileName(), -1, ProjectExplorer::Constants::TASK_CATEGORY_BUILDSYSTEM));
canInit = false;
}
CMakeTool *tool = CMakeKitInformation::cmakeTool(target()->kit()); CMakeTool *tool = CMakeKitInformation::cmakeTool(target()->kit());
if (!tool || !tool->isValid()) { if (!tool || !tool->isValid()) {
......
...@@ -205,6 +205,7 @@ void AbstractProcessStep::run(QFutureInterface<bool> &fi) ...@@ -205,6 +205,7 @@ void AbstractProcessStep::run(QFutureInterface<bool> &fi)
if (!m_process->waitForStarted()) { if (!m_process->waitForStarted()) {
processStartupFailed(); processStartupFailed();
m_process.reset(); m_process.reset();
m_outputParserChain.reset();
reportRunResult(fi, false); reportRunResult(fi, false);
return; return;
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment