Message ID | 20240528202916.3082271-1-andrey.z@gmail.com |
---|---|
State | New |
Headers | show |
Series | optee-client: Switch away from S = WORKDIR | expand |
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 > >
On Tue, 28 May 2024 22:29:16 +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. > > Applied, thanks! [1/1] optee-client: Switch away from S = WORKDIR commit: 30872ba12abaab0a53e332a33429e602be30e249 Best regards,
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 \
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> --- meta-arm/recipes-security/optee/optee-client.inc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)