Merge lp:~roadmr/checkbox/phoronix-verbose-output into lp:checkbox
Proposed by
Daniel Manrique
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Zygmunt Krynicki | ||||
Approved revision: | 1907 | ||||
Merged at revision: | 1908 | ||||
Proposed branch: | lp:~roadmr/checkbox/phoronix-verbose-output | ||||
Merge into: | lp:checkbox | ||||
Diff against target: |
39 lines (+18/-2) 2 files modified
debian/changelog (+2/-0) scripts/pts_run (+16/-2) |
||||
To merge this branch: | bzr merge lp:~roadmr/checkbox/phoronix-verbose-output | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Zygmunt Krynicki (community) | Approve | ||
Review via email: mp+145966@code.launchpad.net |
Description of the change
changes to log the full phoronix console output as the comment, this will ease debugging and will be clearer than the current output, which is just a number representing FPS. Without context this number is a bit hard to interpret.
This doesn't entirely fix the linked bug, but that bug showed that without proper information it's very painful and slow to debug problems like the one reported.
To post a comment you must log in.
Looks good, crappy phoronix