ModuleLoader: Fix misleading error message

If a product could not get properly loaded and a per-product property
override was given on the command line, we used to erroneously complain
about the property override instead of reporting the real error.

Change-Id: I1cbbc8ff9b6dcd453563c4745778576342862ca3
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>

2 files changed, 4 insertions, 0 deletions

diff --git a/src/lib/corelib/language/moduleloader.cpp b/src/lib/corelib/language/moduleloader.cpp
index 1cc894232..6449655fb 100644
--- a/src/lib/corelib/language/moduleloader.cpp
+++ b/src/lib/corelib/language/moduleloader.cpp
@@ -2087,6 +2087,8 @@ void ModuleLoader::checkProjectNamesInOverrides(const ModuleLoader::TopLevelProj
 void ModuleLoader::checkProductNamesInOverrides()
 {
     for (const QString &productNameInOverride : m_productNamesUsedInOverrides) {
+        if (m_erroneousProducts.contains(productNameInOverride))
+            continue;
         bool found = false;
         for (auto it = m_productsByName.cbegin(); it != m_productsByName.cend(); ++it) {
             // In an override string such as "a.b.c:d, we cannot tell whether we have a product
@@ -3804,6 +3806,7 @@ void ModuleLoader::handleProductError(const ErrorInfo &error,
     productContext->project->result->productInfos.insert(productContext->item,
                                                          productContext->info);
     m_disabledItems << productContext->item;
+    m_erroneousProducts.insert(productContext->name);
 }
 
 static void gatherAssignedProperties(ItemValue *iv, const QualifiedId &prefix,
diff --git a/src/lib/corelib/language/moduleloader.h b/src/lib/corelib/language/moduleloader.h
index eaabdd54f..ce13364d7 100644
--- a/src/lib/corelib/language/moduleloader.h
+++ b/src/lib/corelib/language/moduleloader.h
@@ -444,6 +444,7 @@ private:
     Set<QString> m_projectNamesUsedInOverrides;
     Set<QString> m_productNamesUsedInOverrides;
     Set<QString> m_disabledProjects;
+    Set<QString> m_erroneousProducts;
 
     int m_dependencyResolvingPass = 0;
 };