Closed Bug 1256507 Opened 8 years ago Closed 8 years ago

Remove --enable-xterm-updates

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: glandium, Assigned: glandium)

Details

Attachments

(1 file)

      No description provided.
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)
Attachment #8730472 - Flags: review?(gps) → review+
Comment on attachment 8730472 [details]
MozReview Request: Bug 1256507 - Remove --enable-xterm-updates

https://reviewboard.mozilla.org/r/39909/#review36473

r+ with pleasure.
https://hg.mozilla.org/mozilla-central/rev/e035b6133eef
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: