Uplift custom b2g/gaia paths to m-c simulator

RESOLVED FIXED in 2.0 S1 (9may)

Status

defect
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: ochameau, Assigned: ochameau)

Tracking

unspecified
2.0 S1 (9may)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

In bug 929061, we added support for custom gaia/gecko paths in the simulator addon hosted on github. In bug 944451, we are landing the simulator addon in mozilla-central, but that wasn't including bug 929061 changeset.
We need to uplift this from github repo to m-c.
Posted patch patch (obsolete) — Splinter Review
The patch ends up being simplier than the github one,
as there is no need to copy the prosthesis addon!
Attachment #8410308 - Flags: review?(jryans)
Comment on attachment 8410308 [details] [diff] [review]
patch

Review of attachment 8410308 [details] [diff] [review]:
-----------------------------------------------------------------

::: b2g/simulator/lib/simulator-process.js
@@ -172,2 @@
>      args.push("-profile", profile);
> -    Cu.reportError(profile);

You should still log this somehow, maybe |console.log| or |debug|, but it's helpful to have when debugging issues on random machines.
Attachment #8410308 - Flags: review?(jryans) → review+
Posted patch patchSplinter Review
Added a console.log for the profile.
Attachment #8410308 - Attachment is obsolete: true
Attachment #8411184 - Flags: review+
Keywords: checkin-needed
Assignee: nobody → poirot.alex
https://hg.mozilla.org/mozilla-central/rev/1d0a4cd6ec20
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S1 (9may)
Blocks: 1006866
You need to log in before you can comment on or make changes to this bug.