Allow retrieving addon/test output when run on android device

RESOLVED FIXED

Status

P2
normal
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: ochameau, Assigned: ochameau)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Bug 688651 introduce preliminary Android devices support in cfx,
but I delayed implementation of addon run or tests execution console messages.
Being able to run an addon on the device is already cool, but it would be even easier if we can see messages directly through cfx command line, instead of going in device JS console.
(Assignee)

Updated

7 years ago
Assignee: nobody → poirot.alex

Updated

7 years ago
Priority: -- → P2
Summary: Allow retrieving addon/test output when runned on android device → Allow retrieving addon/test output when run on android device
(Assignee)

Comment 1

7 years ago
Created attachment 565221 [details]
Pull request 244
(Assignee)

Comment 2

7 years ago
Comment on attachment 565221 [details]
Pull request 244

This patch is finally ready for review.
It was based on top of the very first pull request about mobile support,
and should have landed way sooner but I almost forgot about this work.
The patch has been merge with last master and adapted to last mobile work done by Matteo.

If you want to play with it, connect an Android thing (with a fennec nightly instellad) and create a naive addon that just do some console.log(), then run:
  
  $ cfx run --force-mobile -a fennec-on-device -b /path/to/adb/executable

You may have to add "-m firefox_nightly" if you have multiple version of fennec installed.
Attachment #565221 - Flags: review?(warner-bugzilla)
Comment on attachment 565221 [details]
Pull request 244

looked good to me.. r+ with a few minor comments on the pull-request
Attachment #565221 - Flags: review?(warner-bugzilla) → review+

Comment 4

7 years ago
Commit pushed to https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/b34e5f742e7c12c3d2f870fc94f53052a0e75345
Merge pull request #244 from ochameau/device-log

Bug 689777: Allow retrieving addon/test output when run on android device r=@warner
(Assignee)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 5

7 years ago
fkjfkljfdjlkfdjklfdjkfdjklfdjklgjlkfdrpoerekjklfdjkljgnvcxmndfjklgneiojnersjg;prjokgf;okjfdl;gjlkfdjgklfdjgrpoirepjklfjfkldjgrepoklfjgklfdjklfdjgf
You need to log in before you can comment on or make changes to this bug.