Re: [PATCH] optee-client: Switch away from S = WORKDIR

[Jon Mason <jdmason@kudzu.us>]

On Tue, May 28, 2024 at 10:29:16PM +0200, Andrey Zhizhikin wrote:
> Since OE-Core commit 32cba1cc916a ("insane: Error for S == WORKDIR")
> usage of WORKDIR in tasks is fatal, and shall be replaced with UNPACKDIR.
> Similar change is introduced in OE-Core commit d9328e3b0b06 ("recipes:
> Switch away from S = WORKDIR"), follow the same pattern for OP-TEE client.
> 
> Signed-off-by: Andrey Zhizhikin <andrey.z@gmail.com>

Thanks for the patch.  Unfortunately, we need to delay merging it for
a couple of weeks.  This is due to the WORKDIR changes not working on
scarthgap and needing master to be pinned to scarthgap for an upcoming
BSP release.  I've added this patch to my gitlab master-next-master
branch with the other patch(es) being held for the same reason, and
will push it to master once that release occurs.

Thanks,
Jon

> ---
>  meta-arm/recipes-security/optee/optee-client.inc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
> index 77f6a642..ddda2d1a 100644
> --- a/meta-arm/recipes-security/optee/optee-client.inc
> +++ b/meta-arm/recipes-security/optee/optee-client.inc
> @@ -24,8 +24,8 @@ EXTRA_OECMAKE = " \
>  EXTRA_OECMAKE:append:toolchain-clang = " -DCFG_WERROR=0"
>  
>  do_install:append() {
> -    install -D -p -m0644 ${WORKDIR}/tee-supplicant@.service ${D}${systemd_system_unitdir}/tee-supplicant@.service
> -    install -D -p -m0755 ${WORKDIR}/tee-supplicant.sh ${D}${sysconfdir}/init.d/tee-supplicant
> +    install -D -p -m0644 ${UNPACKDIR}/tee-supplicant@.service ${D}${systemd_system_unitdir}/tee-supplicant@.service
> +    install -D -p -m0755 ${UNPACKDIR}/tee-supplicant.sh ${D}${sysconfdir}/init.d/tee-supplicant
>  
>      sed -i -e s:@sysconfdir@:${sysconfdir}:g \
>             -e s:@sbindir@:${sbindir}:g \
> -- 
> 2.34.1
> 
>