assignUser commented on code in PR #13556:
URL: https://github.com/apache/arrow/pull/13556#discussion_r925058675
##########
cpp/cmake_modules/DefineOptions.cmake:
##########
@@ -111,7 +111,10 @@ if(ARROW_DEFINE_OPTIONS)
define_option(ARROW_POSITION_INDEPENDENT_CODE
"Whether to create position-independent target" ON)
-
+
+ define_option(ARROW_USE_SCCACHE "Use sccache when compiling (if available),
takes precdence\
+ over ccache" ON)
Review Comment:
If there are no envvars set sccache will function in local mode similar to
ccache https://github.com/mozilla/sccache#local so I would tend to leave it on
as it allows us to just add the install script to the docker-compose command
and the envvars to the github template to enable this on a lot of jobs, We
could of course also switch it on in the build scripts but we would need to
touch a bunch more files.
Alternatively we could check if a storage back end is configured and only
use sccache over ccache in that case if both are installed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
