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