From 27586827238ca9079860e77a7b23ae20d163143e Mon Sep 17 00:00:00 2001 From: hjk Date: Fri, 25 Oct 2019 09:55:32 +0200 Subject: ProjectExplorer: Move BuildSystem owership to BuildConfiguration ... or Target. This patch moves build system from conceptually "one per project" to "one per target (i.e. per project-and-kit)" or "per BuildConfigurations" for targets where the builds differ significantly. Building requires usually items from the kit (Qt version, compiler, ...) so a target-agnostic build is practically almost always wrong. Moving the build system to the target also has the potential to solve issues caused by switching targets while parsing, that used Project::activeTarget() regularly, with potentially different results before and after the switch. This patch might create performance/size regressions when several targets are set up per project as the build system implementation's internal data are duplicated in this case. The idea is to fix that by sharing per-project pieces again in the project implementation once these problems occur. Change-Id: I87f640ce418b93175b5029124eaa55f3b8721dca Reviewed-by: Christian Stenger Reviewed-by: Christian Kandeler --- src/plugins/boot2qt/qdbrunconfiguration.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/plugins/boot2qt') diff --git a/src/plugins/boot2qt/qdbrunconfiguration.cpp b/src/plugins/boot2qt/qdbrunconfiguration.cpp index d1abe58229..65df670342 100644 --- a/src/plugins/boot2qt/qdbrunconfiguration.cpp +++ b/src/plugins/boot2qt/qdbrunconfiguration.cpp @@ -91,7 +91,7 @@ QdbRunConfiguration::QdbRunConfiguration(Target *target, Core::Id id) this, &QdbRunConfiguration::updateTargetInformation); connect(target, &Target::kitChanged, this, &QdbRunConfiguration::updateTargetInformation); - connect(target->project(), &Project::parsingFinished, + connect(target, &Target::parsingFinished, this, &QdbRunConfiguration::updateTargetInformation); setDefaultDisplayName(tr("Run on Boot2Qt Device")); -- cgit v1.2.3