]> code.ossystems Code Review - openembedded-core.git/commitdiff
sstate: fix touching files inside pseudo
authorJose Quaresma <quaresma.jose@gmail.com>
Wed, 20 Oct 2021 17:25:46 +0000 (18:25 +0100)
committerAnuj Mittal <anuj.mittal@intel.com>
Mon, 1 Nov 2021 15:18:14 +0000 (23:18 +0800)
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.jose@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit f6e7445c94443544e92fda97a017ce93393c5f84)
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
meta/classes/sstate.bbclass

index 8b4bfc25b7f878aa607e1ccb349d8be4cd5a21cb..4bc80b607836f537d0e924d1e56156fdd30f3fcc 100644 (file)
@@ -800,7 +800,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
 
@@ -834,7 +834,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 () {