Title: [291615] trunk/Tools
Revision
291615
Author
jbed...@apple.com
Date
2022-03-22 07:41:52 -0700 (Tue, 22 Mar 2022)

Log Message

[Python 3] Use sys.executable instead of Python
https://bugs.webkit.org/show_bug.cgi?id=238175
<rdar://problem/90604107>

Reviewed by Ryan Haddad.

* Tools/Scripts/webkitpy/common/system/executive_unittest.py:
(ExecutiveTest.test_run_command_with_bad_command):

Canonical link: https://commits.webkit.org/248708@main

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (291614 => 291615)


--- trunk/Tools/ChangeLog	2022-03-22 14:09:30 UTC (rev 291614)
+++ trunk/Tools/ChangeLog	2022-03-22 14:41:52 UTC (rev 291615)
@@ -1,3 +1,14 @@
+2022-03-21  Jonathan Bedard  <jbed...@apple.com>
+
+        [Python 3] Use sys.executable instead of Python
+        https://bugs.webkit.org/show_bug.cgi?id=238175
+        <rdar://problem/90604107>
+
+        Reviewed by Ryan Haddad.
+
+        * Scripts/webkitpy/common/system/executive_unittest.py:
+        (ExecutiveTest.test_run_command_with_bad_command):
+
 2022-03-22  Carlos Garcia Campos  <cgar...@igalia.com>
 
         [GTK][WPE] Remove the ATK implementation

Modified: trunk/Tools/Scripts/webkitpy/common/system/executive_unittest.py (291614 => 291615)


--- trunk/Tools/Scripts/webkitpy/common/system/executive_unittest.py	2022-03-22 14:09:30 UTC (rev 291614)
+++ trunk/Tools/Scripts/webkitpy/common/system/executive_unittest.py	2022-03-22 14:41:52 UTC (rev 291615)
@@ -107,8 +107,8 @@
     def test_run_command_with_bad_command(self):
         self.assertRaises(OSError, lambda: Executive().run_command(["foo_bar_command_blah"], ignore_errors=True, return_exit_code=True))
         self.assertRaises(OSError, lambda: Executive().run_and_throw_if_fail(["foo_bar_command_blah"], quiet=True))
-        self.assertRaises(ScriptError, lambda: Executive().run_command(['python', '-c', 'import sys; sys.exit(1)']))
-        self.assertRaises(ScriptError, lambda: Executive().run_and_throw_if_fail(['python', '-c', 'import sys; sys.exit(1)'], quiet=True))
+        self.assertRaises(ScriptError, lambda: Executive().run_command([sys.executable, '-c', 'import sys; sys.exit(1)']))
+        self.assertRaises(ScriptError, lambda: Executive().run_and_throw_if_fail([sys.executable, '-c', 'import sys; sys.exit(1)'], quiet=True))
 
     def test_run_command_args_type(self):
         executive = Executive()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to