[Flatpak] Manifest expanding fails for qt WPE extension
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Mar 2019 17:40:14 +0000 (17:40 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Mar 2019 17:40:14 +0000 (17:40 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196148

Patch by Philippe Normand <pnormand@igalia.com> on 2019-03-22
Reviewed by Alex Christensen.

* flatpak/flatpakutils.py:
(expand_submodules_recurse): The last recurse manifest expand
iteration might lead to a single module so take it into account
and don't try to recurse further.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243387 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/flatpak/flatpakutils.py

index 383a631..85a6bcd 100644 (file)
@@ -1,3 +1,15 @@
+2019-03-22  Philippe Normand  <pnormand@igalia.com>
+
+        [Flatpak] Manifest expanding fails for qt WPE extension
+        https://bugs.webkit.org/show_bug.cgi?id=196148
+
+        Reviewed by Alex Christensen.
+
+        * flatpak/flatpakutils.py:
+        (expand_submodules_recurse): The last recurse manifest expand
+        iteration might lead to a single module so take it into account
+        and don't try to recurse further.
+
 2019-03-22  Chris Dumez  <cdumez@apple.com>
 
         [ iOS Simulator] REGRESSION (r241821) Layout Test platform/ipad/fast/viewport/viewport-unchanged-by-minimum-effective-width-if-not-ignore-meta-viewport.html is a flaky failure on bots
index 0e3d132..04d7c8d 100644 (file)
@@ -172,6 +172,11 @@ def expand_submodules_recurse(modules, manifest_path, port_name, command):
         all_modules.extend(expand_submodules_recurse(submanifest, submanifest_path, port_name, command))
         return all_modules
 
+    # The last recurse manifest expand iteration might lead to a single module.
+    if not isinstance(modules, list):
+        all_modules.append(modules)
+        return all_modules
+
     for module in modules:
         if type(module) is str:
             submanifest_path = os.path.join(os.path.dirname(manifest_path), module)