On Sunday 18 November 2007 19:47:07 Reuben Thomas wrote:
> On Sun, 18 Nov 2007, Felipe Sateler wrote:
> > On Tuesday 06 November 2007 17:50:24 Reuben Thomas wrote:
> >> Package: checkinstall
> >> Version: 1.6.1-4
> >> Followup-For: Bug #447779
> >>
> >> Instead of changing the permissions to 644, the patch should set the
> >> "other" perms to the same as those for "group", i.e. "o=g".
> >
> > I'm not sure if this is a good thing. Usually the software developer
> > knows better than checkinstall which files should have which permissions.
>
> I am using packages that fail to set the permissions. Specifically, they
> fail to override the permissions set by my umask, which is 0027, so files
> end up being unreadable once installed (except by the root user).
>
> The whole point of this patch is to help fix badly set permissions, just
> like the patch already fixes badly-set uids, which the software developer
> should also get right, but doesn't always.

Hmm, good point. Another question is to wether add a new option or reuse the 
reset-uids one. I think it makes sense to use htat one, but it breaks the 
principle of least surprise. I'll ask upstream for input.


-- 

        Felipe Sateler

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to