Subject: Re: CVS commit: pkgsrc/pkgtools/pkglint
To: Alistair Crooks <agc@pkgsrc.org>
From: Roland Illig <rillig@NetBSD.org>
List: tech-pkg
Date: 11/07/2005 22:18:30
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.

Roland