Closed
Bug 1175895
Opened 9 years ago
Closed 9 years ago
automation tier start message buffered by make
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox42 fixed)
RESOLVED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: mshal, Assigned: mshal)
References
Details
Attachments
(2 files)
3.61 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
1.43 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
On windows where we enable output buffering, the TIER_START message is buffered by make along with the rest of the output for each automation step. So for example, buildsymbols will run for several minutes, then make prints the tier start + all buildsymbols output + tier finish all at once. As far as mach knows, the buildsymbols step only ran for the time it took to print the output, which means its stat tracking is inaccurate.
Assignee | ||
Comment 1•9 years ago
|
||
This just aids the greppability of MOZ_AUTOMATION_* - for example, looking up MOZ_AUTOMATION_UPDATE_PACKAGING in dxr would have you believe that it's only used for partial updates, but moz-automation.mk also uses it for complete mar generation.
Attachment #8624201 -
Flags: review?(ted)
Assignee | ||
Comment 2•9 years ago
|
||
Separate the tier start message into a different target. This forces make to flush it to stdout before the automation step starts running, so mach can get accurate stats.
Attachment #8624202 -
Flags: review?(ted)
Comment 3•9 years ago
|
||
Comment on attachment 8624201 [details] [diff] [review] 0001-Bug-1175895-aid-greppability-of-MOZ_AUTOMATION_.patch Review of attachment 8624201 [details] [diff] [review]: ----------------------------------------------------------------- ::: build/moz-automation.mk @@ +55,5 @@ > + MOZ_AUTOMATION_PRETTY_UPDATE_PACKAGING \ > + MOZ_AUTOMATION_L10N_CHECK \ > + MOZ_AUTOMATION_PRETTY_L10N_CHECK \ > + MOZ_AUTOMATION_UPLOAD \ > + MOZ_AUTOMATION_SDK \ You could alternately just wrap this whole block in an `$(addprefix MOZ_AUTOMATION_,`, but gnu make is terrible so whatever.
Attachment #8624201 -
Flags: review?(ted) → review+
Updated•9 years ago
|
Attachment #8624202 -
Flags: review?(ted) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/77d5a7ca90c5 https://hg.mozilla.org/integration/mozilla-inbound/rev/c28f6bfc72c1
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/77d5a7ca90c5 https://hg.mozilla.org/mozilla-central/rev/c28f6bfc72c1
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•