Subject: Re: CVS commit: pkgsrc/pkgtools/pkglint/files
To: Alistair Crooks <agc@pkgsrc.org>
From: Roland Illig <rillig@NetBSD.org>
List: tech-pkg
Date: 11/10/2005 09:38:14
Alistair Crooks wrote:
> On Thu, Nov 10, 2005 at 07:46:24AM +0000, Roland Illig wrote:
> 
>>Module Name:	pkgsrc
>>Committed By:	rillig
>>Date:		Thu Nov 10 07:46:24 UTC 2005
>>
>>Modified Files:
>>	pkgsrc/pkgtools/pkglint/files: pkglint.pl
>>
>>Log Message:
>>Added (currently disabled) code to scan for variable names in
>>pkgsrc/mk/defaults/mk.conf. All those variable names are then checked as
>>being of type Userdefined. As that file is currently too unstructured
>>and contains too much garbage, this check cannot yet be enabled.
> 
> 
> 
> There is no garbage in pkgsrc/mk/defaults/mk.conf - you must be thinking
> of something else.

Roland Illig wrote:
 > http://mail-index.netbsd.org/tech-pkg/2005/11/10/0001.html
<quote>
Because of the similarity of the filename, I have always assumed that 
all variables in mk/defaults/mk.conf are default values for things that 
can be set by the pkgsrc user in mk.conf. But this is wrong. This file 
has become a placeholder for all kinds of variables, lacking proper 
comments explaining what each variable is meant for.
</quote>

That's what I mean. In mk.conf, there's just a listing of variables. 
Many of them lack the information about whether they are intended to be 
set by the user or a package.

As there had been confusion in the not-so-far past about similar 
variables like PKGREVISION (which should only be set in a package's 
Makefile, not in any other file), PKGBASE and PKGNAME, I want to 
separate the default definitions of user-settable variables from the 
package-settable ones. That's all it is about.

Roland