Closed
Bug 689157
Opened 13 years ago
Closed 13 years ago
Add support for building with clcache/ccache on Windows
Categories
(Firefox Build System :: General, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla10
People
(Reporter: wicked, Assigned: wicked)
References
()
Details
Attachments
(1 file)
1.93 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
It seems ccache is not supported on Windows w/MSVC compilers but there's a clcache.py replacement that could be used to speed the build. It's command line is ccache compatible but it's written in Python. However, no matter how I tried to use it the current build system keeps failing at various stages during Configure. I tried to either use clcache.py as a cl replacement via cl.bat or direct python command line or as a preprocessor just like ccache. Eventually I got it working by compiling the Python code to an executable (using py2exe) and by patching few lines in the m-c build system. Patched problem would also prevent use of ccache on Windows too (maybe it's usable with gcc). The attached patch removes the quotation around CC variable, which at this phase already has, if --with-ccache switch was used, the ccache command before cl. The quotes were probably here because a manually give cl command could contain path with spaces but there are other places where these are not used so a path with spaces would fail anyway.
Attachment #562428 -
Flags: review?
Updated•13 years ago
|
Attachment #562428 -
Flags: review? → review?(khuey)
Comment on attachment 562428 [details] [diff] [review] Fix CC variables used in "cl -showIncludes" detection logic, V1 Review of attachment 562428 [details] [diff] [review]: ----------------------------------------------------------------- Yeah, this should be fine. If cl isn't on the PATH things are going to hell anyways.
Attachment #562428 -
Flags: review?(khuey) → review+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 2•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/65e060c81a59
Keywords: checkin-needed
Target Milestone: --- → mozilla10
Comment 3•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/27207582b5df I accidentally pushed this patch to mozilla-central while it was living on inbound. On the next merge, it will be merged. Sorry for the mess!
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 4•13 years ago
|
||
Merged from inbound: https://hg.mozilla.org/mozilla-central/rev/65e060c81a59
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
•