1370840 - Reftest analyzer doesn't work with reftest log from Android

Status: RESOLVED FIXED

(Testing :: Reftest, enhancement)

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

Attachment: log from android

This is an example log from https://queue.taskcluster.net/v1/task/VAaUj8DlRzCQsvfYe9G5Rg/runs/0/artifacts/public/logs/live_backing.log

Comment 2

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1370840 - Make regexp more general for capturing lines.

Review commit: https://reviewboard.mozilla.org/r/146644/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/146644/

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8875232 [details]
Bug 1370840 - Make regexp more general for capturing lines.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/146644/diff/1-2/

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

Attachment: log from windows

This is an example log from Windows build which doesn't work with current reftest analyzer either. It is https://queue.taskcluster.net/v1/task/eGYj6bZtQXSEh8I0JpAJCw/runs/0/artifacts/public/logs/live_backing.log

Comment 5

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8875232 [details]
Bug 1370840 - Make regexp more general for capturing lines.

https://reviewboard.mozilla.org/r/146644/#review150946

The commit message should mention what sort of output (Android logs) this allows.

r=dbaron with that

Comment 6

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8875232 [details]
Bug 1370840 - Make regexp more general for capturing lines.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/146644/diff/2-3/

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/496a533b6037
Make regexp more general for capturing lines. r=dbaron

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/496a533b6037