Closed Bug 759076 Opened 12 years ago Closed 12 years ago

Enable ccache for Linux 64 debug builds

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Attached patch The fixSplinter Review
It looks like Bug 658644 initially meant to enable ccache for Linux 64 bit debug builds, but didn't quite make it into the final patch.

We've been using it on the Thunderbird builds for a while, so I can't see any reason not to have it on the debug builds.

I've just pushed the attached patch to try, so I'll request review if it succeeds.
Comment on attachment 627661 [details] [diff] [review]
The fix

Worked on try. Not sure who reviews mozconfig changes...
Attachment #627661 - Flags: review?(khuey)
Attachment #627661 - Flags: review?(bhearsum)
Assignee: nobody → mbanner
Attachment #627661 - Flags: review?(khuey) → review?(ted.mielczarek)
Attachment #627661 - Flags: review?(bhearsum) → review+
Comment on attachment 627661 [details] [diff] [review]
The fix

Review of attachment 627661 [details] [diff] [review]:
-----------------------------------------------------------------

Generally one of Build Config or RelEng review is sufficient. We're sort of co-owning them.
Attachment #627661 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/9df54b979922
Flags: in-testsuite-
Whiteboard: [inbound]
Target Milestone: --- → mozilla15
This landed earlier:

https://hg.mozilla.org/mozilla-central/rev/9df54b979922
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
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: