[Raptor] Swap out log stream parser

RESOLVED FIXED in 2.2 S14 (12june)

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Eli, Assigned: Eli)

Tracking

unspecified
2.2 S14 (12june)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The adbkit-logcat stream parser does not handle the binary output from a Nexus 5 L device. Swap out the stream parser to something string-non-binary-based to see if we can overcome this issue.
Created attachment 8612876 [details] [review]
[gaia] eliperelman:bug-1169636 > mozilla-b2g:master
(Assignee)

Updated

4 years ago
Attachment #8612876 - Flags: review?(rwood)
Comment on attachment 8612876 [details] [review]
[gaia] eliperelman:bug-1169636 > mozilla-b2g:master

LGTM, with one question in the PR. Applied patch, blew away gaia/node_modules, did a make raptor and ran launch test on flame-kk. R+ assuming that you (or someone) has actually tried this with the Nexus 5L.
Attachment #8612876 - Flags: review?(rwood) → review+
(Assignee)

Updated

4 years ago
Whiteboard: checkin-needed
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
Whiteboard: checkin-needed
Keywords: checkin-needed
http://docs.taskcluster.net/tools/task-graph-inspector/#6EQxayjfRe6-Fju6fj9r_g

The pull request failed to pass integration tests. It could not be landed, please try again.
Target Milestone: --- → 2.2 S14 (12june)
You need to log in before you can comment on or make changes to this bug.