[WPE][GTK] flatpakutils.py should respect set-webkit-configuration build type
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Jul 2018 19:02:18 +0000 (19:02 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Jul 2018 19:02:18 +0000 (19:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187218

Reviewed by Philippe Normand.

Using the webkitpy Config module, if the build type is not passed on the command line, we
read it from WebKitBuild/Configuration and respect it.

* flatpak/flatpakutils.py:
(WebkitFlatpak.__init__):
(WebkitFlatpak.clean_args):

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

Tools/ChangeLog
Tools/flatpak/flatpakutils.py

index 4be7133..99206e2 100644 (file)
@@ -1,3 +1,17 @@
+2018-07-02  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [WPE][GTK] flatpakutils.py should respect set-webkit-configuration build type
+        https://bugs.webkit.org/show_bug.cgi?id=187218
+
+        Reviewed by Philippe Normand.
+
+        Using the webkitpy Config module, if the build type is not passed on the command line, we
+        read it from WebKitBuild/Configuration and respect it.
+
+        * flatpak/flatpakutils.py:
+        (WebkitFlatpak.__init__):
+        (WebkitFlatpak.clean_args):
+
 2018-07-02  Zan Dobersek  <zdobersek@igalia.com>
 
         [WPE] Add touch support to WindowViewBackend
index e3bca09..7d5b5ba 100644 (file)
@@ -29,6 +29,9 @@ import sys
 import tempfile
 import re
 
+from webkitpy.common.system.systemhost import SystemHost
+from webkitpy.port.factory import PortFactory
+
 try:
     from urllib.parse import urlparse  # pylint: disable=E0611
 except ImportError:
@@ -515,6 +518,7 @@ class WebkitFlatpak:
         self.finish_args = None
 
         self.no_flatpak_update = False
+        self.release = False
         self.debug = False
         self.clean = False
         self.run_tests = None
@@ -551,8 +555,14 @@ class WebkitFlatpak:
         self.makeargs = ""
 
     def clean_args(self):
-        self.platform = self.platform.upper()
+        if not self.debug and not self.release:
+            factory = PortFactory(SystemHost())
+            port = factory.get(self.platform)
+            self.debug = port.default_configuration() == "Debug"
         self.build_type = "Debug" if self.debug else "Release"
+
+        self.platform = self.platform.upper()
+
         if self.gdb is None and '--gdb' in sys.argv:
             self.gdb = ""