> -----Original Message-----
> From: openembedded-core@lists.openembedded.org <openembedded-
> c...@lists.openembedded.org> On Behalf Of Jose Quaresma
> Sent: den 20 oktober 2021 19:26
> To: openembedded-core@lists.openembedded.org
> Cc: Jose Quaresma <quaresma.j...@gmail.com>
> Subject: [OE-core] [PATCH v2] sstate: fix touching files inside pseudo
> 
> running the 'id' command inside the sstate_create_package
> function shows that this funcion run inside the pseudo:
> 
>  uid=0(root) gid=0(root) groups=0(root)
> 
> The check for touch files [ ! -w ${SSTATE_PKG} ]
> will always return true and the touch can fail
> when the real user don't have permission or
> in readonly filesystem.
> 
> As the documentation refers, the file test operator "-w"
> check if the file has write permission (for the user running the test).
> 
> We can avoid this test running the touch and mask any return errors
> that we have.
> 
> Signed-off-by: Jose Quaresma <quaresma.j...@gmail.com>
> ---
> 
> v2: redirect stderr to /dev/null to avoid messages in the log
> 
>  meta/classes/sstate.bbclass | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> index 3c179259e7..146f56ea2d 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -830,7 +830,7 @@ sstate_task_postfunc[dirs] = "${WORKDIR}"
>  sstate_create_package () {
>       # Exit early if it already exists
>       if [ -e ${SSTATE_PKG} ]; then
> -             [ ! -w ${SSTATE_PKG} ] || touch ${SSTATE_PKG}
> +             touch ${SSTATE_PKG} 2>/dev/null || true
>               return
>       fi
> 
> @@ -865,7 +865,7 @@ sstate_create_package () {
>       else
>               rm $TFILE
>       fi
> -     [ ! -w ${SSTATE_PKG} ] || touch ${SSTATE_PKG}
> +     touch ${SSTATE_PKG} 2>/dev/null || true
>  }
> 
>  python sstate_sign_package () {
> --
> 2.33.1

Can this please be backported to Hardknott and Honister now that 
it has been accepted for master?

//Peter

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157347): 
https://lists.openembedded.org/g/openembedded-core/message/157347
Mute This Topic: https://lists.openembedded.org/mt/86470820/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to