Closed
Bug 985537
Opened 10 years ago
Closed 10 years ago
[B2G] add runEmulatorShell to Marionette
Categories
(Remote Protocol :: Marionette, defect)
Tracking
(blocking-b2g:-, firefox29 wontfix, firefox30 fixed, firefox31 fixed, b2g-v1.4 fixed, b2g-v2.0 fixed)
People
(Reporter: vicamo, Assigned: vicamo)
References
Details
Attachments
(1 file)
11.36 KB,
patch
|
mdas
:
review+
|
Details | Diff | Splinter Review |
We have already runEmulatorCmd, which executes a string command in emulator console through telnet. However, that means you need to commit some changes to "external/qemu/android/console.c" in order to add new commands. It will be very handy if we allow executing arbitrary commands through "adb shell" and then we can examine the response without touching one line in qemu source.
Assignee | ||
Comment 1•10 years ago
|
||
Hi Malini Das, may I have your review?
Assignee: nobody → vyang
Attachment #8393594 -
Flags: review?(mdas)
Assignee | ||
Comment 2•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=c75572eb0d61
Comment 3•10 years ago
|
||
Comment on attachment 8393594 [details] [diff] [review] patch Review of attachment 8393594 [details] [diff] [review]: ----------------------------------------------------------------- This looks really good, thanks!
Attachment #8393594 -
Flags: review?(mdas) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Hi, after some more thoughts, this shouldn't be named as runEmulatorShell because it's common to all adb-enabled devices. runEmulatorCmd is emulator specific because only emulator has that telnet service. I think I should have named it as runAdbShell. How do you think?
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Vicamo Yang [:vicamo][:vyang] from comment #4) > Hi, after some more thoughts, this shouldn't be named as runEmulatorShell > because it's common to all adb-enabled devices. runEmulatorCmd is emulator > specific because only emulator has that telnet service. I think I should > have named it as runAdbShell. How do you think? Sorry, after some more deep diving into Marionette source code, that doesn't work because we don't have anything abstracting ADB in class Marionette when its |self.emulator| is not defined. So let's take that reviewed patch.
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/2a33822ea94b
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2a33822ea94b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Comment 8•10 years ago
|
||
Nominate for 1.4: The test case of bug 990383 needs this marionette command.
blocking-b2g: --- → 1.4?
Comment 9•10 years ago
|
||
Has automatic approval. No need to request approval here
blocking-b2g: 1.4? → -
Comment 10•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/102de3568ed5
status-b2g-v1.4:
--- → fixed
status-b2g-v2.0:
--- → fixed
status-firefox29:
--- → wontfix
status-firefox30:
--- → fixed
status-firefox31:
--- → fixed
Updated•8 months ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•