Closed
Bug 825094
Opened 13 years ago
Closed 12 years ago
improve legibility of manage_masters.py output
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: hwine, Assigned: hwine)
Details
Attachments
(2 files, 1 obsolete file)
|
4.25 KB,
patch
|
rail
:
review+
hwine
:
checked-in+
|
Details | Diff | Splinter Review |
|
3.80 KB,
patch
|
rail
:
review+
hwine
:
checked-in+
|
Details | Diff | Splinter Review |
some of our hostnames have gotten too long for the formatting of show_revisions, leading to a table that is hard to scan for differences.
Also, remote commands (which can't have headers) need to have output explained in help.
Just posting patch for now to see if anyone else has a "oh and this needs it as well" before finalizing
Comment 2•13 years ago
|
||
Hal, could you please hold this patch for a little bit until patches in bug 763929 are landed -- there is a large change for fabric coming.
Comment 4•12 years ago
|
||
Does this still need to wait? (Also, unblocking from the main ship it bug since we seem to be getting along fine in the current state.
No longer depends on: 763929
minor tweaks to output to help infrequent users with short memories (not that we have any here)
Attachment #696166 -
Attachment is obsolete: true
Attachment #713257 -
Flags: review?(bhearsum)
It works - it would be cleaner to create an "actions" object and have these be methods, but that's more hacking on production-code-built-atop-fabric than I'm comfortable with.
More elegant introspection solutions welcome. And, I'm okay with this patch being r-'d
Attachment #713258 -
Flags: review?(bhearsum)
Updated•12 years ago
|
Attachment #713257 -
Flags: review?(bhearsum) → review?(rail)
Updated•12 years ago
|
Attachment #713258 -
Flags: review?(bhearsum) → review?(rail)
Updated•12 years ago
|
Attachment #713257 -
Flags: review?(rail) → review+
Updated•12 years ago
|
Attachment #713258 -
Flags: review?(rail) → review+
Comment on attachment 713257 [details] [diff] [review]
improved help & show_revisions output
http://hg.mozilla.org/build/tools/rev/a3f46cf4d021
Attachment #713257 -
Flags: checked-in+
Comment on attachment 713258 [details] [diff] [review]
add headers to actions run in parallel
http://hg.mozilla.org/build/tools/rev/0fd22dab3cd6
Attachment #713258 -
Flags: checked-in+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•