[OE-core] [oe-core][RFCv2] bitbake.conf: store cooker logs in TMPDIR subdirectory

[Chris Larson]

On Thu, Aug 2, 2012 at 9:31 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> * we should probably wait until
>   http://git.openembedded.org/bitbake/commit/?id=1e06d8012868ba5a31503dc99cbf18570be629d9
>   is in next required bitbake version, but error message if the directory doesn't exist is quite clear
> * people with new enough bitbake could just change this in their local.conf
>
> Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
> ---
>  meta/conf/bitbake.conf |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index d5a43e9..8074684 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -753,8 +753,8 @@ DISTRO[unexport] = "1"
>  # Used by canadian-cross to handle string conversions on TARGET_ARCH where needed
>  TRANSLATED_TARGET_ARCH ??= "${@d.getVar('TARGET_ARCH', True).replace("_", "-")}"
>
> -# Complete output from bitbake, if you're changing it, make sure to use existing directory
> -BB_CONSOLELOG ?= "${TMPDIR}/cooker.log.${DATETIME}"
> +# Complete output from bitbake
> +BB_CONSOLELOG ?= "${LOG_DIR}/cooker/${MACHINE}/${DATETIME}.log"

I'd recommend rewording the commit subject. It's not 100% clear that
it refers to 'the tmpdir subdir' or 'a subdir of tmpdir', until you
read the patch :)
-- 
Christopher Larson