[OE-core] [PATCH] Immediately append TCLIBCAPPEND to TMPDIR in defaultsetup.conf

Chris Larson clarson at kergoth.com
Fri Aug 5 04:59:32 UTC 2011


On Wed, Aug 3, 2011 at 2:54 PM, Daniel Lazzari <dlazzari at leapfrog.com> wrote:
>  Immediately append TCLIBCAPPEND to TMPDIR in defaultsetup.conf
>    instead of lazily appending it with _append. This fixes a bug where
>    the wrong cache was cleared when BB_SRCREV_POLICY = "clear".
>
>
> Signed-off-by: Daniel Lazzari Jr <dlazzari at leapfrog.com>
> ---
>  meta/conf/distro/defaultsetup.conf |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/meta/conf/distro/defaultsetup.conf b/meta/conf/distro/defaultsetup.conf
> index 072f03c..951e5be 100644
> --- a/meta/conf/distro/defaultsetup.conf
> +++ b/meta/conf/distro/defaultsetup.conf
> @@ -14,7 +14,7 @@ require conf/distro/include/tclibc-${TCLIBC}.inc
>
>  # Allow single libc distros to disable this code
>  TCLIBCAPPEND ?= "-${TCLIBC}"
> -TMPDIR_append = "${TCLIBCAPPEND}"
> +TMPDIR := "${TMPDIR}${TCLIBCAPPEND}"
>
>  CACHE = "${TMPDIR}/cache/${TCMODE}-${TCLIBC}${@['', '/' + str(bb.data.getVar('MACHINE', d, 1))][bool(bb.data.getVar('MACHINE', d, 1))]}${@['', '/' + str(bb.data.getVar('SDKMACHINE', d, 1))][bool(bb.data.getVar('SDKMACHINE', d, 1))]}"

If all you want is an immediate concatenation, you can use .=. Using
:= like this forces an early expansion of TMPDIR unnecessarily, as far
as I can tell.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics




More information about the Openembedded-core mailing list