Bug 1772572 - Fix Browsertime installation on Windows. r=perftest-reviewers,kshampur

This patch changes how the browsertime command is called for installation to fix a failure on Windows.

Differential Revision: https://phabricator.services.mozilla.com/D148292
This commit is contained in:
Gregory Mierzwinski 2022-06-06 15:56:53 +00:00
Родитель e09607114a
Коммит d9297919a6
1 изменённых файлов: 4 добавлений и 2 удалений

Просмотреть файл

@ -187,6 +187,7 @@ class RaptorRunner(MozbuildObject):
sys.path.insert(0, os.path.join(self.topsrcdir, "tools", "browsertime"))
try:
import mach_commands as browsertime
import platform
# We don't set `browsertime_{chromedriver,geckodriver} -- those will be found by
# browsertime in its `node_modules` directory, which is appropriate for local builds.
@ -268,13 +269,14 @@ class RaptorRunner(MozbuildObject):
if _should_install():
# TODO: Make this "integration" nicer in the near future
print("Missing browsertime files...attempting to install")
subprocess.check_call(
subprocess.check_output(
[
os.path.join(self.topsrcdir, "mach"),
"browsertime",
"--setup",
"--clobber",
]
],
shell="windows" in platform.system().lower(),
)
if _should_install():
raise Exception(