From f92ccd380b278e81cae41365395889b7c3829dc5 Mon Sep 17 00:00:00 2001 From: Wez Furlong Date: Thu, 6 Jun 2019 10:30:20 -0700 Subject: [PATCH] getdeps: disable ccache on sandcastle Summary: We sometimes see some flakeyness in our internal CI, so we can live without it there. Reviewed By: pkaush Differential Revision: D15695124 fbshipit-source-id: 1d76ae89e245d9c95937e8818826c544c0ae8fc3 --- build/fbcode_builder/getdeps/builder.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/build/fbcode_builder/getdeps/builder.py b/build/fbcode_builder/getdeps/builder.py index 0b5e7d2ce..733426605 100644 --- a/build/fbcode_builder/getdeps/builder.py +++ b/build/fbcode_builder/getdeps/builder.py @@ -271,9 +271,13 @@ class CMakeBuilder(BuilderBase): "CMAKE_BUILD_TYPE": "RelWithDebInfo", } env = self._compute_env(install_dirs) - ccache = path_search(env, "ccache") - if ccache: - defines["CMAKE_CXX_COMPILER_LAUNCHER"] = ccache + if "SANDCASTLE" not in os.environ: + # We sometimes see intermittent ccache related breakages on some + # of the FB internal CI hosts, so we prefer to disable ccache + # when running in that environment. + ccache = path_search(env, "ccache") + if ccache: + defines["CMAKE_CXX_COMPILER_LAUNCHER"] = ccache if self.build_opts.is_darwin(): # Try to persuade cmake to set the rpath to match the lib # dirs of the dependencies. This isn't automatic, and to