Convert 'mach remote' to run with Python 3
Categories
(Remote Protocol :: Agent, task, P1)
Tracking
(firefox78 fixed)
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: KrisWright, Assigned: whimboo)
References
Details
Attachments
(2 files)
mach remote
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.
Assignee | ||
Comment 1•5 years ago
|
||
Kris, what specifically needs to be done to make it Python 3 compatible? It would be great to get some instructions what needs to be done. If ot here maybe on the meta bug.
Reporter | ||
Comment 2•5 years ago
|
||
(In reply to Henrik Skupin (:whimboo) [⌚️UTC+2] from comment #1)
Kris, what specifically needs to be done to make it Python 3 compatible? It would be great to get some instructions what needs to be done. If ot here maybe on the meta bug.
I'm not super familiar with the process we're using, so I'll divert this to Ricky. Maybe there is a guide or some tips that we can put on the metabug as well?
Comment 3•5 years ago
|
||
Yeah, let me try to throw up some basic guidance on the metabug. Gimme a bit, I'll report back when that's done.
Comment 4•5 years ago
|
||
I updated the user story on the metabug. Please have a look and let me know if there are any improvements I can make or additional questions you have (this is all just off the top of my head).
Assignee | ||
Comment 5•5 years ago
|
||
Thanks Ricky! As it looks like there is nothing to do here for that specific mach command. The sub command vendor-puppeteer
works just fine when it is run with Python 3.
Assignee | ||
Comment 6•5 years ago
|
||
This aligns our code with other mach commands that don't
show the command usage if no subcommand has been specified.
Assignee | ||
Comment 7•5 years ago
|
||
Depends on D75900
Comment 9•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2a54634147e5
https://hg.mozilla.org/mozilla-central/rev/3a73bf75d1ce
Description
•