diff options
author | Christian Kandeler <christian.kandeler@qt.io> | 2017-08-31 13:12:54 +0200 |
---|---|---|
committer | Christian Kandeler <christian.kandeler@qt.io> | 2017-08-31 13:56:53 +0000 |
commit | 559adb4b176dc19822ba19e17dfb0b6d58caff4a (patch) | |
tree | aebf8f40088e5aaca06c974c9536bcbc055f857e /src/app | |
parent | 315c187f94e343d26eb3bee3d6a06ebdd1b1bfde (diff) |
Do not implicitly resolve a project as part of the "clean" command
It makes no sense to call "clean" without a build graph, and neither do
we need possible project files updates to be reflected in the build
graph before running this operation.
Change-Id: I986793e659e2ca1fdf7e58a52b787096527251ba
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Diffstat (limited to 'src/app')
-rw-r--r-- | src/app/qbs/commandlinefrontend.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/app/qbs/commandlinefrontend.cpp b/src/app/qbs/commandlinefrontend.cpp index f2a0a5f96..261863c38 100644 --- a/src/app/qbs/commandlinefrontend.cpp +++ b/src/app/qbs/commandlinefrontend.cpp @@ -150,8 +150,10 @@ void CommandLineFrontend::start() params.setSettingsDirectory(m_settings->baseDirectory()); params.setOverrideBuildGraphData(m_parser.command() == ResolveCommandType); params.setPropertyCheckingMode(ErrorHandlingMode::Strict); - if (!m_parser.buildBeforeInstalling() || m_parser.command() == DumpNodesTreeCommandType) + if (!m_parser.buildBeforeInstalling() || m_parser.command() == DumpNodesTreeCommandType + || m_parser.command() == CleanCommandType) { params.setRestoreBehavior(SetupProjectParameters::RestoreOnly); + } foreach (const QVariantMap &buildConfig, m_parser.buildConfigurations()) { QVariantMap userConfig = buildConfig; const QString configurationKey = QLatin1String("qbs.configurationName"); |