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)

All
Windows 7
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla10

People

(Reporter: wicked, Assigned: wicked)

References

()

Details

Attachments

(1 file)

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?
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+
Keywords: checkin-needed
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
Blocks: 807066
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.