[PATCH] Fix install rule/hook for setuid/setgid

Doug Goldstein cardoe at gentoo.org
Thu Sep 6 20:55:35 PDT 2007


Richard Hughes wrote:
> On 06/09/07, Doug Goldstein <cardoe at gentoo.org> wrote:
>> Daniel Stone wrote:
>>> On Thu, Sep 06, 2007 at 05:16:01PM -0400, ext Doug Goldstein wrote:
>>>
>>>>      -chown :$(POLKIT_GROUP) $(DESTDIR)$(libexecdir)/polkit-grant-helper
>>>>      -chown :$(POLKIT_GROUP) $(DESTDIR)$(libexecdir)/polkit-grant-helper-pam
>>>>
>>> Maybe I'm missing something, but is this not what chgrp is for?
>>>
>>> Cheers,
>>> Daniel
>>>
>> That's what I was thinking but I decided not to touch it... so I left it
>> as was. I figured David had some reason for doing it this way.
> 
> To be honest, I wrote that because it worked. My UNIX skills are not
> l33t, so there is probably a better way to do it. There's no
> cleverness involved.
> 
> Richard.

Well, here's the follow up patch to make the change to chgrp like Daniel 
requested.


-- 
Doug Goldstein <cardoe at gentoo.org>
http://dev.gentoo.org/~cardoe/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0003-use-chgrp-instead-of-chown-with-group-only.patch
Url: http://lists.freedesktop.org/archives/hal/attachments/20070906/6e80d2a8/attachment.ksh 


More information about the hal mailing list