Closed Bug 1638947 Opened 11 months ago Closed 5 months ago

Convert 'mach android' to run with Python 3

Categories

(Firefox Build System :: Android Studio and Gradle Integration, task, P3)

task

Tracking

(firefox84 fixed)

RESOLVED FIXED
84 Branch
Tracking Status
firefox84 --- fixed

People

(Reporter: KrisWright, Assigned: gbrown)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

mach android 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/mobile/android/mach_commands.py

Product: GeckoView → Firefox Build System
Component: General → Android Studio and Gradle Integration

Nick -- can you take a look and maybe provide a status update on the triage here?

Flags: needinfo?(nalexander)

(In reply to Ricky Stewart from comment #1)

Nick -- can you take a look and maybe provide a status update on the triage here?

Yes. Most of these commands are internal to the build system and to our CI. They have no third-party dependencies that I am aware of.

Assuming that self.run_process(..., pass_thru=True) works under Python 3 I expect this to either Just Work or be trivially close to Just Working.

I can't really speak to the priority -- I set P3 since it is truly backlog -- but the severity should be S3 or S4.

Severity: -- → S3
Flags: needinfo?(nalexander)
Priority: -- → P3
Blocks: 1674247
Assignee: nobody → whole.grains

Switch 'mach android' to python 3. As far as I can tell, this just works.

Pushed by whole.grains@protonmail.com:
https://hg.mozilla.org/integration/autoland/rev/98b8ae5bce9e
Run 'mach android' with python 3; r=nalexander
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.