build: Don't auto opt-in to ccache.

servo / skia

Skia

BSD 3-Clause "New" or "Revised" License

109 stars 61 forks source link

Closed emilio closed 5 years ago

emilio commented 5 years ago

Some distros like fedora auto-opt-in to ccache already if installed by putting symlinks in $PATH (so clang points to /usr/lib64/ccache/clang).

This uses the same approach as mozjs and others to avoid calling ccache recursively.

This change is

emilio commented 5 years ago

r? @jdm

jdm commented 5 years ago

@bors-servo r+

bors-servo commented 5 years ago

:pushpin: Commit 3a05413 has been approved by jdm

bors-servo commented 5 years ago

:hourglass: Testing commit 3a054130e98e9ba6d1b99905125bd70f1559dcd7 with merge b35fbd82a8828fbeb9574fc04f009d249dc02d8b...

bors-servo commented 5 years ago

:sunny: Test successful - checks-travis Approved by: jdm Pushing b35fbd82a8828fbeb9574fc04f009d249dc02d8b to master...