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/classes | |
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/classes')
-rw-r--r-- | meta/classes/ccache.bbclass | 25 |
1 files changed, 22 insertions, 3 deletions
diff --git a/meta/classes/ccache.bbclass b/meta/classes/ccache.bbclass index 960902065c..59e1022678 100644 --- a/meta/classes/ccache.bbclass +++ b/meta/classes/ccache.bbclass | |||
@@ -1,4 +1,14 @@ | |||
1 | CCACHE = "${@bb.utils.which(d.getVar('PATH'), 'ccache') and 'ccache '}" | 1 | # |
2 | # Usage: | ||
3 | # - Enable ccache | ||
4 | # Add the following line to a conffile such as conf/local.conf: | ||
5 | # INHERIT += "ccache" | ||
6 | # | ||
7 | # - Disable ccache for a recipe | ||
8 | # Add the following line to the recipe if it can't be built with ccache: | ||
9 | # CCACHE_DISABLE = '1' | ||
10 | # | ||
11 | |||
2 | export CCACHE_DIR ?= "${TMPDIR}/ccache/${MULTIMACH_TARGET_SYS}/${PN}" | 12 | export CCACHE_DIR ?= "${TMPDIR}/ccache/${MULTIMACH_TARGET_SYS}/${PN}" |
3 | 13 | ||
4 | # We need to stop ccache considering the current directory or the | 14 | # We need to stop ccache considering the current directory or the |
@@ -7,5 +17,14 @@ export CCACHE_DIR ?= "${TMPDIR}/ccache/${MULTIMACH_TARGET_SYS}/${PN}" | |||
7 | # ${PV} or ${PR} change. | 17 | # ${PV} or ${PR} change. |
8 | export CCACHE_NOHASHDIR ?= "1" | 18 | export CCACHE_NOHASHDIR ?= "1" |
9 | 19 | ||
10 | DEPENDS_append_class-target = " ccache-native" | 20 | python() { |
11 | DEPENDS[vardepvalueexclude] = " ccache-native" | 21 | """ |
22 | Enable ccache for the recipe | ||
23 | """ | ||
24 | pn = d.getVar('PN') | ||
25 | # quilt-native doesn't need ccache since no c files | ||
26 | if not (pn in ('ccache-native', 'quilt-native') or | ||
27 | bb.utils.to_boolean(d.getVar('CCACHE_DISABLE'))): | ||
28 | d.appendVar('DEPENDS', ' ccache-native') | ||
29 | d.setVar('CCACHE', 'ccache ') | ||
30 | } | ||