Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in sys-apps/sandbox: ChangeLog sandbox-1.2.18.1-r1.ebuild

[Alec Warner]

On 10/17/07, Doug Goldstein <cardoe@g.o> wrote:
> Donnie Berkholz wrote:
> > On 15:55 Wed 17 Oct     , Daniel Drake (dsd) wrote:
> >> 1.1                  sys-apps/sandbox/sandbox-1.2.18.1-r1.ebuild
> >>
> >> file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/sys-apps/sandbox/sandbox-1.2.18.1-r1.ebuild?rev=1.1&view=markup
> >> plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/sys-apps/sandbox/sandbox-1.2.18.1-r1.ebuild?rev=1.1&content-type=text/plain
> >
> >>      keepdir /var/log/sandbox
> >>      fowners root:portage /var/log/sandbox
> >>      fperms 0770 /var/log/sandbox
> >>
> >>      cd ${S}
> >>      dodoc AUTHORS ChangeLog NEWS README
> >> }
> >>
> >> pkg_preinst() {
> >>      chown root:portage ${D}/var/log/sandbox
> >>      chmod 0770 ${D}/var/log/sandbox
> >> }
> >
> > How come you need to repeat the permissions like this?
> >
> > Also, quoting. =)
> >
> > Thanks,
> > Donnie
>
> Also, pkg_preinst() is not binary package safe. So unless the portage
> group is always the same gid on EVERY Gentoo box. This will break binary
> packages.

Eh?

pkg_* runs for binpkgs...so depending on the target machine it will
chown appropriately in $IMAGE and then merge to the livefs.

If you build the binpkg on machine A and install it on B, pkg_preinst
will run on both (once for the build and one for the install)

-Alec
-- 
gentoo-dev@g.o mailing list