diff options
| author | Manoj Gupta <manojgupta@google.com> | 2020-03-13 11:05:26 -0700 |
|---|---|---|
| committer | Manoj Gupta <manojgupta@chromium.org> | 2020-03-13 21:57:55 +0000 |
| commit | 28979264d4c945fcd9a3f09b385a7b67f051419f (patch) | |
| tree | 3b6b4df4b4058d3286dcc5230bca3f3e5c8b7ea5 /compiler_wrapper/ccache_flag_test.go | |
| parent | 99b3ff928b1c000980d331f30174614484095d37 (diff) | |
| download | toolchain-utils-28979264d4c945fcd9a3f09b385a7b67f051419f.tar.gz | |
compiler_wrapper: Stop setting CCACHE_BASEDIR
CCACHE_BASEDIR used to be set to sysroot so that ccache can reuse
compilation results across different boards. This is not useful
on builders and also less useful for board builds since we set
closer to target CFLAGS now.
It also breaks use of -fmacro-prefix-map option in clang/gcc.
So rip it out and simplify the rest of code.
BUG=chromium:983759
TEST=go test
Change-Id: Ia99ca994c976ee1137c3d904a78f7e25931c70f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2103170
Commit-Queue: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'compiler_wrapper/ccache_flag_test.go')
| -rw-r--r-- | compiler_wrapper/ccache_flag_test.go | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/compiler_wrapper/ccache_flag_test.go b/compiler_wrapper/ccache_flag_test.go index 61abef06..03a74de8 100644 --- a/compiler_wrapper/ccache_flag_test.go +++ b/compiler_wrapper/ccache_flag_test.go @@ -54,17 +54,6 @@ func TestSetCacheDir(t *testing.T) { }) } -func TestSetCacheBaseDirToSysroot(t *testing.T) { - withCCacheEnabledTestContext(t, func(ctx *testContext) { - cmd := ctx.must(callCompiler(ctx, ctx.cfg, - ctx.newCommand(gccX86_64, mainCc))) - if err := verifyEnvUpdate(cmd, - "CCACHE_BASEDIR="+ctx.tempDir+"/usr/x86_64-cros-linux-gnu"); err != nil { - t.Error(err) - } - }) -} - func TestSetCacheUmask(t *testing.T) { withCCacheEnabledTestContext(t, func(ctx *testContext) { cmd := ctx.must(callCompiler(ctx, ctx.cfg, |