Re: [PATCH 5/5] uboot-sign: do_concat_dtb(): cd to $B

From: Denys Dmytriyenko <denis@denix.org>
To: George McCollister <george.mccollister@gmail.com>
Cc: marex@denx.de, trini@konsulko.com, denys@ti.com,
	andrew.bradford@kodakalaris.com,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 5/5] uboot-sign: do_concat_dtb(): cd to $B
Date: Mon, 08 Aug 2016 17:32:43 -0400	[thread overview]
Message-ID: <20160808213243.GS8521@denix.org> (raw)
In-Reply-To: <d37bec4f77f0670b35347c367371d2cd4887b29a.1470672689.git.george.mccollister@gmail.com>

On Mon, Aug 08, 2016 at 11:37:57AM -0500, George McCollister wrote:
> Prior to running oe_runmake make sure $B is the cwd. This is required
> due to bitbake commit 67a7b8b021badc17d8fdf447c250e79d291e75f7
> "build: don't use $B as the default cwd for functions".
> 
> Without this change, do_concat_dtb fails with:
> | ERROR: oe_runmake failed
> | make: *** No targets specified and no makefile found.  Stop.
> 
> Signed-off-by: George McCollister <george.mccollister@gmail.com>

Tested-by: Denys Dmytriyenko <denys@ti.com>
Acked-by: Denys Dmytriyenko <denys@ti.com>


> ---
>  meta/classes/uboot-sign.bbclass | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/classes/uboot-sign.bbclass b/meta/classes/uboot-sign.bbclass
> index d56ad8e..3c56db8 100644
> --- a/meta/classes/uboot-sign.bbclass
> +++ b/meta/classes/uboot-sign.bbclass
> @@ -66,6 +66,7 @@ do_concat_dtb () {
>  	if [ "x${UBOOT_SIGN_ENABLE}" = "x1" ]; then
>  		if [ "x${UBOOT_SUFFIX}" = "ximg" -o "x${UBOOT_SUFFIX}" = "xrom" ] && \
>  			[ -e "${DEPLOYDIR}/${UBOOT_DTB_IMAGE}" ]; then
> +			cd ${B}
>  			oe_runmake EXT_DTB=${DEPLOYDIR}/${UBOOT_DTB_IMAGE}
>  			install ${S}/${UBOOT_BINARY} ${DEPLOYDIR}/${UBOOT_IMAGE}
>  			install ${S}/${UBOOT_BINARY} ${DEPLOY_DIR_IMAGE}/${UBOOT_IMAGE}
> -- 
> 2.8.0
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core