-
-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ccache to the build if available #553
Conversation
Also enables ccache in TravisCI
Seems that linux + clang build is failing, but might be a ccache + clang bug (http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html) |
Oh, nice find. I’ve just learnt to live with those warnings in Travis on my
own projects. I’ll submit a fix once I get a couple of spare minutes 😊
…On Sun, 22 Apr 2018 at 11:24, Cong ***@***.***> wrote:
Seems that linux + clang build is failing, but might be a ccache + clang
bug (http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html)
This could be fixed by having clang ignore unused arguments
-Qunused-arguments, setting envvar CCACHE_CPP2=yes (
http://petereisentraut.blogspot.com/2011/09/ccache-and-clang-part-2.html),
or using an updated ccache which fixes this bug (
https://bugzilla.samba.org/show_bug.cgi?id=8118 version 3.2). I can look
into this when I next get time and access to a linux environment, or do you
have time?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#553 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAKAx178xEqQLw_sPj-hiX_Ko_mAhr5Eks5trExGgaJpZM4Tarw2>
.
|
7e16299
to
25e266f
Compare
Ok, didn't realize that there were multiple CMakeLists.txt files. I'll need to fiddle more than I thought. I'll try again later 😃 |
CMakeLists.txt
Outdated
link_directories(${SDL2_LIBRARY_DIRS} ${SDL2_IMAGE_LIBRARY_DIRS} ${SDL2_MIXER_LIBRARY_DIRS}) | ||
|
||
if(CCACHE_FOUND) | ||
set(ENV{CCACHE_CPP2} "yes") | ||
if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use CLANG
; see line 33
Added ccache to the build to benefit those who have it installed. But it comes bundled in most Linux package managers and really speeds up successive compiles. There should be no impact if you don't have ccache installed.
Also enabled ccache in TravisCI.