diff options
author | Robert Yang <liezhi.yang@windriver.com> | 2018-12-06 18:55:29 -0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-01-14 11:35:55 +0000 |
commit | 0c98ff2c31d462ea139ebf1a3e0dbd97088d5466 (patch) | |
tree | 34f52ca5fdc645585ee8f4a5ded9dc0273a00271 /meta/lib/oe/utils.py | |
parent | 87413eb34a810f3c267f1185a4f2af211838530f (diff) | |
download | poky-0c98ff2c31d462ea139ebf1a3e0dbd97088d5466.tar.gz |
ccache.bbclass: Refactor it to make it more reliable
The previous ccache.bbclass has the following problems:
- It uses host's ccache for native recipes, but this may not work on some
hosts, for example, it nerver works on my Ubuntu 14.04.4, there are always
build failures (m4-native failed at do_configure, and others will also be
failed if I disable CCACHE for m4-native)
- native/nativesdk/cross/crosssdk recipes use host's ccache, but target uses
ccache-native, this may confuse user.
- The target recipes may use both host's ccache and ccache-native, this may
cause unexpected problems and be hard to debug. This is because ccache-native is
in SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS, so ccache-native may not be present when
rebuild target recipes, and then it would use hosttools/ccache, but the
previous ccache files were generated by ccache-native.
- Target recipes can't use ccache when no ccache is installed on the host:
CCACHE = "${@bb.utils.which(d.getVar('PATH'), 'ccache') and 'ccache '}"
After refactored:
All types recipes (native, target and others) will use ccache-native except
ccache-native itself, host's cache won't be used any more. It is more
reliable now, which will work everywhere when ccache-native can be built.
And now we need use "CCACHE_DISABLE = '1'" to disable ccache for the recipe
rather than "CCACHE = ''" since we set CCACHE in anonymous function, and
d.getVar('CCACHE') works after "CCACHE ??=" which is set in bitbake.conf, so we
can't check whether CCACHE is set or not in anonymous function since it is
always set. Use CCACHE_DISABLE to disable it would be more clear.
(From OE-Core rev: b25271b65262f70d849a4861da216c9be6c54d53)
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oe/utils.py')
-rw-r--r-- | meta/lib/oe/utils.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/meta/lib/oe/utils.py b/meta/lib/oe/utils.py index 29b41151d4..ee6f0e6647 100644 --- a/meta/lib/oe/utils.py +++ b/meta/lib/oe/utils.py | |||
@@ -363,6 +363,9 @@ def host_gcc_version(d, taskcontextonly=False): | |||
363 | return | 363 | return |
364 | 364 | ||
365 | compiler = d.getVar("BUILD_CC") | 365 | compiler = d.getVar("BUILD_CC") |
366 | # Get rid of ccache since it is not present when parsing. | ||
367 | if compiler.startswith('ccache '): | ||
368 | compiler = compiler[7:] | ||
366 | try: | 369 | try: |
367 | env = os.environ.copy() | 370 | env = os.environ.copy() |
368 | env["PATH"] = d.getVar("PATH") | 371 | env["PATH"] = d.getVar("PATH") |