REGRESSION(r263625): run-minibrowser --debug/--release options gone
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Jun 2020 09:32:52 +0000 (09:32 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Jun 2020 09:32:52 +0000 (09:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=213697

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-29
Reviewed by Žan Doberšek.

* Scripts/webkitpy/minibrowser/run_webkit_app.py:
(main): Explicitely skip the --target option. Previous code was also skipping --release and
--debug options because their `dest` is the same as with the --target option.

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

Tools/ChangeLog
Tools/Scripts/webkitpy/minibrowser/run_webkit_app.py

index 5a4818f..0a6101d 100644 (file)
@@ -1,3 +1,14 @@
+2020-06-29  Philippe Normand  <pnormand@igalia.com>
+
+        REGRESSION(r263625): run-minibrowser --debug/--release options gone
+        https://bugs.webkit.org/show_bug.cgi?id=213697
+
+        Reviewed by Žan Doberšek.
+
+        * Scripts/webkitpy/minibrowser/run_webkit_app.py:
+        (main): Explicitely skip the --target option. Previous code was also skipping --release and
+        --debug options because their `dest` is the same as with the --target option.
+
 2020-06-29  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK][WPE] Add webkit_authentication_request_get_security_origin
 2020-06-29  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK][WPE] Add webkit_authentication_request_get_security_origin
index 395c043..1373205 100644 (file)
@@ -37,16 +37,18 @@ def main(argv):
 
         for option in group_options:
             # Skip deprecated option
 
         for option in group_options:
             # Skip deprecated option
-            if option.dest == "configuration":
-                continue
-            default = None
-            if option.default != ("NO", "DEFAULT"):
-                default = option.default
-            option_group.add_argument(option.get_opt_string(), action=option.action, dest=option.dest,
-                                      help=option.help, const=option.const, default=default)
+            if option.get_opt_string() != "--target":
+                default = None
+                if option.default != ("NO", "DEFAULT"):
+                    default = option.default
+                option_group.add_argument(option.get_opt_string(), action=option.action, dest=option.dest,
+                                          help=option.help, const=option.const, default=default)
 
     options, args = option_parser.parse_known_args(argv)
 
 
     options, args = option_parser.parse_known_args(argv)
 
+    if not options.configuration:
+        options.configuration = "Release"
+
     if set(args).issubset(["-h", "--help"]) and not options.platform:
         option_parser.print_help()
         print("\nTo see the available options on a specific platform, supply it on the command-line, for example --gtk --help")
     if set(args).issubset(["-h", "--help"]) and not options.platform:
         option_parser.print_help()
         print("\nTo see the available options on a specific platform, supply it on the command-line, for example --gtk --help")