Subject: Re: CVS commit: pkgsrc/pkgtools/pkglint
To: Roland Illig <rillig@NetBSD.org>
From: Alistair Crooks <agc@pkgsrc.org>
List: tech-pkg
Date: 11/07/2005 22:15:41
On Mon, Nov 07, 2005 at 10:18:30PM +0100, Roland Illig wrote:
> Alistair Crooks wrote:
> >On Sun, Nov 06, 2005 at 10:38:36PM +0000, Roland Illig wrote:
> >
> >>Module Name:	pkgsrc
> >>Committed By:	rillig
> >>Date:		Sun Nov  6 22:38:36 UTC 2005
> >>
> >>Modified Files:
> >>	pkgsrc/pkgtools/pkglint: TODO
> >>
> >>Log Message:
> >>+ check for magic numbers like 0755, 755, 644
> >
> >
> >If this is an attempt to eradicate octal numbers from chmod(1)
> >statements, please do not do this.  There are numerous occasions when
> >absolute modes are needed and essential, and a pkglint warning for
> >those statements would quickly just become noise.
> 
> This is just the TODO list. When I implement an additional check I 
> always run my pkglint-diff program and examine the output carefully. 
> When I feel that most of the uses should be fixed, I commit that change. 
> Otherwise I change the heuristics.
> 
> I think that in most cases these magic numbers can be replaced by 
> ${BINMODE}, ${LIBMODE} or ${DATAMODE}, although I still have to verify 
> this assumption. I'll tell you about it when I have found out.

Off the top of my head, there are such things as setuid programs,
specific directory modes, and others which I can think of.

Please don't add a noise generator - I have enough trouble with
all this fatal white space, and wondering what's so problematic
with it.

Thanks,
Alistair