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

[Doug Goldstein]

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.

-- 
Doug Goldstein <cardoe@g.o>
http://dev.gentoo.org/~cardoe/
-- 
gentoo-dev@g.o mailing list