Closed
Bug 1256507
Opened 8 years ago
Closed 8 years ago
Remove --enable-xterm-updates
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox48 fixed)
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•8 years ago
|
||
It's an opt-in flag that allows to display where the build is in terminal window titles. The fact that it's opt-in and likely unknown makes it very low-value, and the fact that it was added in an era where builds were not very well parallelized made it have a meaning, but now that builds are parallelized, its meaningfulness is diminished. Let's just remove it. Review commit: https://reviewboard.mozilla.org/r/39909/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/39909/
Attachment #8730472 -
Flags: review?(gps)
Updated•8 years ago
|
Attachment #8730472 -
Flags: review?(gps) → review+
Comment 2•8 years ago
|
||
Comment on attachment 8730472 [details] MozReview Request: Bug 1256507 - Remove --enable-xterm-updates https://reviewboard.mozilla.org/r/39909/#review36473 r+ with pleasure.
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e035b6133eef
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
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
•