All of lore.kernel.org
 help / color / mirror / Atom feed
From: Davide Soldan <davide.soldan@kynetics.it>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: yocto@yoctoproject.org
Subject: Re: preserving file owners in do_package
Date: Fri, 18 Oct 2013 15:57:54 +0200	[thread overview]
Message-ID: <98724567.6MEpULbTr7@fedora-fisso> (raw)
In-Reply-To: <2090420.FiXqQSde1A@helios>

In data venerdì 18 ottobre 2013 14:38:27, Paul Eggleton ha scritto:
> Hi Davide,
> 
> On Friday 18 October 2013 15:31:22 Davide Soldan wrote:
> > In data martedì 15 ottobre 2013 17:18:35, Paul Eggleton ha scritto:
> > > On Tuesday 15 October 2013 18:12:42 Davide Soldan wrote:
> > > > In data martedì 15 ottobre 2013 11:38:21, Paul Eggleton ha scritto:
> > > > > On Tuesday 15 October 2013 12:02:20 Davide Soldan wrote:
> > > > > > in my recipe I want to change owner and permission of a file, so I
> > > > > > wrote:
> > > > > > 
> > > > > > do_install() {
> > > > > > 
> > > > > >         ls -la ${D}/opt/user
> > > > > >         chown -R user ${D}/opt/user
> > > > > >         chgrp -R user ${D}/opt/user
> > > > > >         ls -la ${D}/opt/user
> > > > > >         chmod 754 ${D}/opt/user/user.sh
> > > > > >         ls -la ${D}/opt/user
> > > > > > 
> > > > > > }
> > > > > > 
> > > > > > the output of ls commands is:
> > > > > > 
> > > > > > drwxr-xr-x 8 root root   4096 Oct 15 11:13 .
> > > > > > drwxr-xr-x 3 root root   4096 Oct 15 11:13 ..
> > > > > > -rw-r--r-- 1 *root root*  55753 Aug  2 08:48 user.sh
> > > > > > 
> > > > > > drwxr-xr-x 8 user user   4096 Oct 15 11:13 .
> > > > > > drwxr-xr-x 3 root     root       4096 Oct 15 11:13 ..
> > > > > > -rw-r--r-- 1 *user user*  55753 Aug  2 08:48 user.sh
> > > > > > 
> > > > > > drwxr-xr-x 8 user user   4096 Oct 15 11:13 .
> > > > > > drwxr-xr-x 3 root     root       4096 Oct 15 11:13 ..
> > > > > > *-rwxr-xr--* 1 user user  55753 Aug  2 08:48 user.sh
> > > > > > 
> > > > > > So the change of owner and permission works fine in do_install
> > > > > > phase.
> > > > > > Opening the final rootfs tarball, however, the file user.sh has
> > > > > > still
> > > > > > 754
> > > > > > permission but root as owner...so do_package doesn't preserve owner
> > > > > > change
> > > > > > done in do_install? Or maybe I'm doing something wrong in changing
> > > > > > file
> > > > > > owner? Any ideas? Thanks to all!
> > > > > 
> > > > > Does your recipe create the specified user via useradd.bbclass?
> > > > 
> > > > I solved with a postinstall script (that run at boot):
> > > > 
> > > > pkg_postinst_${PN}() {
> > > > 
> > > >         #!/bin/sh -e
> > > >         if [ x"$D" = "x" ]; then
> > > >         
> > > >           chown -R user /opt/user
> > > >           chgrp -R user /opt/user
> > > >         
> > > >         else
> > > >         
> > > >           exit 1
> > > >         
> > > >         fi
> > > > 
> > > > }
> > > 
> > > This shouldn't be necessary though. I honestly can't tell you why it
> > > doesn't work as specified. I think it might be worth filing a bug for
> > > this.
> > > 
> > > Cheers,
> > > Paul
> > 
> > I'm so sorry Paul, but my mail client missed your last email and so I filed
> > a bug uselessly... I tried, as you suggested, to add the package that
> > create the user to the RDEPENDS_${PN} removing the postinstall scripi. That
> > worked setting the right owner with chown/chgrp in do_install... Thank you
> > so much. I very sorry for the useless bug opened...can I close it by
> > myself?
> 
> I've marked it resolved. No problem about filing it, at least it's now there 
> for reference if anyone hits the same situation :)
> 
> I think it's fair to say we need to look at how to better document all of this 
> stuff in the manual. We could probably afford to dedicate a small section on how 
> to deal with adding/modifying users and file ownership. Scott, could you add 
> this to your todo list?
> 
> Cheers,
> Paul
> 
> 
Very good idea! When I was looking for information about how to add/modify users and file ownership I missed a bit of official documentation!
Cheers
Davide


      reply	other threads:[~2013-10-18 13:58 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-15 10:02 preserving file owners in do_package Davide Soldan
2013-10-15 10:38 ` Paul Eggleton
2013-10-15 10:50   ` Davide Soldan
2013-10-15 16:12   ` Davide Soldan
2013-10-15 16:18     ` Paul Eggleton
2013-10-15 21:54       ` Paul Eggleton
2013-10-18 13:31       ` Davide Soldan
2013-10-18 13:38         ` Paul Eggleton
2013-10-18 13:57           ` Davide Soldan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=98724567.6MEpULbTr7@fedora-fisso \
    --to=davide.soldan@kynetics.it \
    --cc=paul.eggleton@linux.intel.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.