Closed Bug 581903 Opened 14 years ago Closed 14 years ago

--enable-xterm-updates disables parallel dirs

Categories

(Firefox Build System :: General, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

References

Details

(Keywords: regression)

Attachments

(1 file)

I didn't really test my patch in bug 463896 thoroughly. So it turns out that the extra code to output xterm updates has the nasty side-effect of disabling building multiple subdirectories in parallel with each other.
Attached patch Possible patchSplinter Review
I remembered to patch js/src this time ;-)
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #460214 - Flags: review?(me)
Comment on attachment 460214 [details] [diff] [review]
Possible patch

These are the only callers of UPDATE_TITLE, no?  Can we remove UPDATE_TITLE entirely?
Attachment #460214 - Flags: review?(me) → review+
Heh, so Bugzilla's diff view tricked me.  The patch is good, but the previous comment is bogus.
Comment on attachment 460214 [details] [diff] [review]
Possible patch

This affects c-c too of course.
Attachment #460214 - Flags: review?(bugspam.Callek)
Attachment #460214 - Flags: review?(bugspam.Callek) → review+
Pushed changeset 34692f1bc2e9 to mozilla-central.

Pushed changeset d8e09f99d49a to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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: