Peter Kjellerstedt <peter.kjellerst...@axis.com> escreveu no dia terça, 19/10/2021 à(s) 20:37:
> > -----Original Message----- > > From: openembedded-core@lists.openembedded.org <openembedded- > > c...@lists.openembedded.org> On Behalf Of Jose Quaresma > > Sent: den 18 oktober 2021 20:47 > > To: openembedded-core@lists.openembedded.org > > Cc: Jose Quaresma <quaresma.j...@gmail.com> > > Subject: [OE-core] [PATCH] 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> > > --- > > 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 7f034d746a..d4555150c4 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} || true > > You should pass stderr to /dev/null to avoid messages in the log > if/when touch fails, i.e.: > > touch ${SSTATE_PKG} 2>/dev/null || : > > I only see the logs when the task fails but I will add the redirection and send a V2. Thanks for the review. > > return > > fi > > > > @@ -865,7 +865,7 @@ sstate_create_package () { > > else > > rm $TFILE > > fi > > - [ ! -w ${SSTATE_PKG} ] || touch ${SSTATE_PKG} > > + touch ${SSTATE_PKG} || true > > } > > > > python sstate_sign_package () { > > -- > > 2.33.1 > > //Peter > > -- Best regards, José Quaresma
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#157258): https://lists.openembedded.org/g/openembedded-core/message/157258 Mute This Topic: https://lists.openembedded.org/mt/86422157/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-