[Flatpak] Avoid consuming unknown arguments in flatpak wrapper
authorcturner@igalia.com <cturner@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2018 14:40:47 +0000 (14:40 +0000)
committercturner@igalia.com <cturner@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jul 2018 14:40:47 +0000 (14:40 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187951

When running run-webkit-tests --wpe --debug, it was observed that
both self.platform != 'wpe' and self.debug != True. This turned
out to be because of the "args" argument consuming the remainder
of future arguments that came after it. Since the first argument
to load_from_args was the test script to run, it would end up
never processing the other arguments supplied to the wrapper
script.

Tested this by rebuilding GTK in debug mode, running the
minibrowser with various extra options, and running some tests
with various extra options.

Reviewed by Michael Catanzaro.

* flatpak/flatpakutils.py:
(WebkitFlatpak.load_from_args):

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

Tools/ChangeLog
Tools/flatpak/flatpakutils.py

index b92ce28..3c5a75b 100644 (file)
@@ -1,3 +1,25 @@
+2018-07-24  Charlie Turner  <cturner@igalia.com>
+
+        [Flatpak] Avoid consuming unknown arguments in flatpak wrapper
+        https://bugs.webkit.org/show_bug.cgi?id=187951
+
+        When running run-webkit-tests --wpe --debug, it was observed that
+        both self.platform != 'wpe' and self.debug != True. This turned
+        out to be because of the "args" argument consuming the remainder
+        of future arguments that came after it. Since the first argument
+        to load_from_args was the test script to run, it would end up
+        never processing the other arguments supplied to the wrapper
+        script.
+
+        Tested this by rebuilding GTK in debug mode, running the
+        minibrowser with various extra options, and running some tests
+        with various extra options.
+
+        Reviewed by Michael Catanzaro.
+
+        * flatpak/flatpakutils.py:
+        (WebkitFlatpak.load_from_args):
+
 2018-07-24  Thibault Saunier  <tsaunier@igalia.com>
 
         [Flatpak] Remove now unnecessary 'hacks' for running tests
index a2d3f5b..0e90dbf 100644 (file)
@@ -504,9 +504,6 @@ class WebkitFlatpak:
         general.add_argument("-y", "--assumeyes",
                             help="Automatically answer yes for all questions.",
                             action="store_true")
-        general.add_argument("args",
-                            nargs=argparse.REMAINDER,
-                            help="Arguments passed when starting %s" % self.name)
         general.add_argument('--avalaible', action='store_true', dest="check_avalaible", help='Check if required dependencies are avalaible.'),
 
         debugoptions = parser.add_argument_group("Debugging")
@@ -521,7 +518,7 @@ class WebkitFlatpak:
         general.add_argument("--clean", dest="clean", action="store_true",
             help="Clean previous builds and restart from scratch")
 
-        parser.parse_args(args=args, namespace=self)
+        _, self.args = parser.parse_known_args(args=args, namespace=self)
 
         return self