779247 - add copy/paste output to BaseScript

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Aki Sasaki (not active)]

run_command and get_output_from_command can take a list as a command, but the output could be more friendly to copy/paste.

Comment 1

[Aki Sasaki (not active)]

Attachment: add copy/paste output

Sample output:

12:13:12     INFO - Running command: ['hg', '--config', 'ui.merge=internal:merge', 'update', '-C'] in /src/runcmd/graphs
12:13:12     INFO - Copy/paste: hg --config ui.merge=internal:merge update -C
12:13:13     INFO -  311 files updated, 0 files merged, 0 files removed, 0 files unresolved
12:13:13     INFO - Return code: 0
12:13:13     INFO - Getting output from command: ['hg', '--config', 'ui.merge=internal:merge', 'parent', '--template', '{node|short}'] in /src/runcmd/graphs
12:13:13     INFO - Copy/paste: hg --config ui.merge=internal:merge parent --template {node|short}
12:13:13     INFO - Reading from file tmpfile_stdout
12:13:13     INFO - Output received:
12:13:13     INFO -  da54bac92c1b
12:13:13     INFO - Got revision da54bac92c1b

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 647633 [details] [diff] [review]
add copy/paste output

Review of attachment 647633 [details] [diff] [review]:
-----------------------------------------------------------------

Happy to r+ this aki, not sure if you should accept me instead of rail though

Comment 3

[Aki Sasaki (not active)]

Comment on attachment 647633 [details] [diff] [review]
add copy/paste output

http://hg.mozilla.org/build/mozharness/rev/e93ae357f521