Closed Bug 1540589 Opened 6 months ago Closed 4 months ago

Allow ./mach run to run emulator on Windows

Categories

(Firefox Build System :: Android Studio and Gradle Integration, enhancement, P5)

Unspecified
Android
enhancement

Tracking

(firefox67 wontfix, firefox68 fixed, firefox69 fixed)

RESOLVED FIXED
mozilla69
Tracking Status
firefox67 --- wontfix
firefox68 --- fixed
firefox69 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.

See also Bug 1521996, which doesn't have a blocking relationship with this ticket, but needs to take this ticket into account.

(In reply to Nick Alexander :nalexander [he/him] from comment #1)

See also Bug 1521996, which doesn't have a blocking relationship with this ticket, but needs to take this ticket into account.

This issue is that it is unnecessary -enable-kvm option on windows emulator. and _get_host_platform() in testing/mozbase/mozrunner/mozrunner/devices/android_device.py returns null on Windows.

Type: defect → enhancement
Priority: -- → P5

./mach run doesn't work since _get_host_platform returns None. So we should
return win32 on Windows.

Attachment #9063965 - Attachment description: Bug 1540589 - _get_host_platform shouldn't return None. → Bug 1540589 - _get_host_platform shouldn't return None. r=nalexander
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/44c42283d83c
_get_host_platform shouldn't return None. r=nalexander
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69

Makoto, should we uplift your mozrunner Windows fix to 68 Beta?

Fennec will move to the ESR 68 channel so we should uplift fixes from 69 to 68 if they will help Fennec developers maintain the ESR 68 branch over the next year.

Flags: needinfo?(m_kato)

(In reply to Chris Peterson [:cpeterson] from comment #6)

Makoto, should we uplift your mozrunner Windows fix to 68 Beta?

Fennec will move to the ESR 68 channel so we should uplift fixes from 69 to 68 if they will help Fennec developers maintain the ESR 68 branch over the next year.

OK, I will request uplift for Windows workstation.

Flags: needinfo?(m_kato)

Comment on attachment 9063965 [details]
Bug 1540589 - _get_host_platform shouldn't return None. r=nalexander

Beta/Release Uplift Approval Request

  • User impact if declined: This is required to support building Fennec on Windows. Fennec is moving to ESR, so our contributor doesn't use Windows workstation to build Fennec.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is ./mach command issue when using Windows to build Fennec. If not applying, detecting windows becomes script error.
  • String changes made/needed: no
Attachment #9063965 - Flags: approval-mozilla-beta?

This sounds like it can land a=npotb.

Whiteboard: [checkin-needed-beta]
Attachment #9063965 - Flags: approval-mozilla-beta?
Whiteboard: [checkin-needed-beta]
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 69 → mozilla69
You need to log in before you can comment on or make changes to this bug.