Move socket host into marionette-js-runner and make it default

RESOLVED FIXED

Status

Testing
JSMarionette
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: lightsofapollo, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Comment 1

3 years ago
Created attachment 8524946 [details] [review]
https://github.com/mozilla-b2g/marionette-js-runner/pull/48

Very minor changes made to the socket host / marionette-js-runner... the patch mostly moves the venv to the top of marionette-js-runner and sets the socket host as the _default_ host.
Attachment #8524946 - Flags: review?(mdas)
Comment on attachment 8524946 [details] [review]
https://github.com/mozilla-b2g/marionette-js-runner/pull/48

yay! Can you also remove these lines from gaia tree: https://github.com/mozilla-b2g/gaia/blob/master/Makefile#L734, we won't need to install it then
Attachment #8524946 - Flags: review?(mdas) → review+
(Reporter)

Comment 3

3 years ago
https://github.com/mozilla-b2g/marionette-js-runner/commit/9035726a2806043ee36b30c563a58df7021d82db
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.