Closed
Bug 1262569
Opened 8 years ago
Closed 5 years ago
Turn off printing of the files that we're building as we build by default
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox48 fixed)
RESOLVED
INACTIVE
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: jrmuizel, Assigned: gps)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
This list of files doesn't really provide any value. Because we parallelize better, it's basically not possible to use the list of files as a proxy for progress.
Comment 1•8 years ago
|
||
If we do this, can we make it controllable by a setting in mozconfig? I definitely like seeing the output as a sign of progress. A percentage complete output would cover this, though.
Assignee | ||
Comment 3•8 years ago
|
||
Multiple people have complained that the build output of printing the source files being built adds little value. I agree. The extra output doesn't give really helpful progress info because sources can be built in non-deterministic order. Furthermore, the extra output hides useful output like compiler warnings. This patch makes the default build system output even less verbose. We no longer print the individual source targets when they are built. We do still print the targets for binaries, so some sense of progress can be inferred. If people like verbosity, they can export the undocumented BUILD_VERBOSE_LOG environment variable can be set to restore the old behavior. Review commit: https://reviewboard.mozilla.org/r/44687/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/44687/
Attachment #8738732 -
Flags: review?(ted)
Comment 4•8 years ago
|
||
Comment on attachment 8738732 [details] MozReview Request: Bug 1262569 - Do not print individual source targets being built; r?ted https://reviewboard.mozilla.org/r/44687/#review41425
Attachment #8738732 -
Flags: review?(ted) → review+
Comment 6•8 years ago
|
||
https://reviewboard.mozilla.org/r/44687/#review41431 ::: config/rules.mk:870 (Diff revision 1) > > $(OBJS) $(HOST_OBJS) $(PROGOBJS) $(HOST_PROGOBJS): $(GLOBAL_DEPS) > > # Rules for building native targets must come first because of the host_ prefix > $(HOST_COBJS): > - $(REPORT_BUILD) > + $(REPORT_BUILD_VERBOSE) REPORT_BUILD is also emitted by the backend and used in other Makefiles. You'd have better results changing the definition of REPORT_BUILD than by adding REPORT_BUILD_VERBOSE.
Updated•8 years ago
|
Keywords: leave-open
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a4cc25ae5b7d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 8•8 years ago
|
||
I tried something like this a couple of years ago and too many people complained so it got reverted. Worth trying again, though :)
Comment 9•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/45187/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/45187/
Attachment #8739310 -
Flags: review?(gps)
Assignee | ||
Comment 10•8 years ago
|
||
Comment on attachment 8739310 [details] MozReview Request: Bug 1262569 - Actually disable all REPORT_BUILDs. r?gps https://reviewboard.mozilla.org/r/45187/#review41765 In the original patch, I specifically limited the printing of source related targets and retaining printing of binaries. The reason was I wanted to retain some indication of progress during the build. With this patch, we have no progress indicators during compilation except for compiler warnings. With this patch, we can go multiple minutes without printing anything. This can lead to the impression the build has stalled. I favor eventually not printing anything except warnings and other important messages (basically what this patch does). However, I insist we have some kind of progress indicator before we do that. I was toying with the idea of restoring the X/Y progress indicator during the compile tier. I know you had good reasons to remove it. Perhaps we can come up with something better than directories for the thing being measured.
Attachment #8739310 -
Flags: review?(gps)
Comment 11•8 years ago
|
||
> I favor eventually not printing anything except warnings and other important > messages (basically what this patch does). However, I insist we have some > kind of progress indicator before we do that. Would the progress indicator work if you redirect the output of `mach build` to file or through a pipe? I ask because I pipe the output through my own script that pulls out error messages. (See https://lists.mozilla.org/pipermail/dev-platform/2015-January/008270.html and replies for old discussion on dev-platform about this.)
Updated•6 years ago
|
Product: Core → Firefox Build System
Comment 12•5 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months. :gps, maybe it's time to close this bug?
Flags: needinfo?(gps)
Updated•5 years ago
|
Status: REOPENED → RESOLVED
Closed: 8 years ago → 5 years ago
Flags: needinfo?(gps)
Resolution: --- → INACTIVE
Updated•5 years ago
|
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•