Closed Bug 1555141 Opened 5 years ago Closed 5 years ago

Simplify process management in android mozrunner

Categories

(Firefox for Android Graveyard :: Testing, defect)

defect
Not set
normal

Tracking

(firefox-esr60 wontfix, firefox-esr68 wontfix, firefox67 wontfix, firefox67.0.1 wontfix, firefox68 wontfix, firefox69 fixed)

RESOLVED FIXED
Firefox 69
Tracking Status
firefox-esr60 --- wontfix
firefox-esr68 --- wontfix
firefox67 --- wontfix
firefox67.0.1 --- wontfix
firefox68 --- wontfix
firefox69 --- fixed

People

(Reporter: gbrown, Assigned: gbrown)

Details

Attachments

(1 file)

No description provided.

There is no real need to use mozprocess from this code, so switched to subprocess.

Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aa6a850c5b95
Simplify process management in android mozrunner; r=jmaher
Regressions: 1555254
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
No longer regressions: 1555254

Geoff, should we uplift this mozrunner fix to ESR 68? Would it make life easier for continued maintenance of Fennec 68 on ESR?

I don't have a strong opinion. I expect no change in behavior with this patch, and it only applies to mach test commands, so having it on esr68 would only be beneficial if we want a future change to mach test commands on esr68.

Flags: needinfo?(gbrown)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: