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

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

-- 

Paul Eggleton
Intel Open Source Technology Centre


  reply	other threads:[~2013-10-18 13:38 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 [this message]
2013-10-18 13:57           ` Davide Soldan

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=2090420.FiXqQSde1A@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=davide.soldan@kynetics.it \
    --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.