1638966 - Convert 'mach hazards' to run with Python 3

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Kris Wright :KrisWright]

mach hazards still runs with python 2 [1]. We want to convert all mach commands to python 3.

To do this, any dependencies of the command may need to be converted as well as the command itself.

[1] https://searchfox.org/mozilla-central/rev/9193635dca8cfdcb68f114306194ffc860456044/s/src/devtools/rootAnalysis/mach_commands.py

Comment 1

[Steve Fink [:sfink] [:s:]]

The only thing that keeps mach hazards on py2 is that it invokes build commands internally. It should be possible to remove hazards from the list at the same time as build.

Comment 2

[Ricky Stewart]

mach build has been running with Python 3 since bug 1617748, i.e., 5 months ago :)

Comment 3

[Ricky Stewart]

Hi -- is there a status update on this work?

Comment 4

[Steven DeTar [:sdetar]]

Steve, could help provide any new status to Ricky

Comment 5

[Steve Fink [:sfink] [:s:]]

Attachment: Bug 1638966 - Run `mach hazards` under py3

Comment 6

[Steve Fink [:sfink] [:s:]]

Hm, I know at the time I implemented this command, mach build was definitely marked as running under py2. Maybe I was based off an old version. But whatever; I just tried running it under py3 and everything worked fine.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/80a9f72ef6fb
Run `mach hazards` under py3 r=rstewart

Comment 8

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/80a9f72ef6fb