Allow empty submodules property in Depends item.

It just means that no dependency should be pulled in by this item.
Useful for this kind of thing:
Product {
    Depends {
        name: "Qt"
        submodules: theQtSubModulesINeed
    }
    property stringList theQtSubModulesIneed // Can be empty.
}

Change-Id: I8e500c3acfa4374be9005fcf349c0417ad47a76e
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 7 insertions, 1 deletions

diff --git a/src/lib/corelib/language/moduleloader.cpp b/src/lib/corelib/language/moduleloader.cpp
index 3c70fa532..4d22e5efd 100644
--- a/src/lib/corelib/language/moduleloader.cpp
+++ b/src/lib/corelib/language/moduleloader.cpp
@@ -639,7 +639,13 @@ void ModuleLoader::resolveDependsItem(DependsContext *dependsContext, Item *item
     }
 
     QString superModuleName;
-    QStringList submodules = m_evaluator->stringListValue(dependsItem, QLatin1String("submodules"));
+    bool submodulesPropertySet;
+    QStringList submodules = m_evaluator->stringListValue(dependsItem, QLatin1String("submodules"),
+                                                          &submodulesPropertySet);
+    if (submodules.isEmpty() && submodulesPropertySet) {
+        m_logger.qbsTrace() << "Ignoring Depends item with empty submodules list.";
+        return;
+    }
     if (nameParts.count() == 2) {
         if (Q_UNLIKELY(!submodules.isEmpty()))
             throw ErrorInfo(Tr::tr("Depends.submodules cannot be used if name contains a dot."),