diff options
| author | rillig <rillig> | 2008-01-01 20:44:29 +0000 |
|---|---|---|
| committer | rillig <rillig> | 2008-01-01 20:44:29 +0000 |
| commit | 3944e1323402d5513ef5a2bd3dba60b51d453723 (patch) | |
| tree | 3c65fdd39ee109f998ba709a07c14a4354633f24 | |
| parent | 41614db79579965b31472cf5396ee99d81c3b25a (diff) | |
| download | pkgsrc-3944e1323402d5513ef5a2bd3dba60b51d453723.tar.gz | |
If one of the NO_*_ON_* variables contains something other that
${RESTRICTED}, print the variable name in the warning message.
While here, added an explanation for the warning.
| -rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index 13db7d2b1a9..73188c88a5b 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -1,5 +1,5 @@ #! @PERL@ -# $NetBSD: pkglint.pl,v 1.743 2008/01/01 13:44:11 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.744 2008/01/01 20:44:29 rillig Exp $ # # pkglint - static analyzer and checker for pkgsrc packages @@ -5301,7 +5301,12 @@ sub checkline_mk_vartype_basic($$$$$$$$) { } elsif ($type eq "Restricted") { if ($value ne "\${RESTRICTED}") { - $line->log_warning("The only valid value for this variable is \${RESTRICTED}."); + $line->log_warning("The only valid value for ${varname} is \${RESTRICTED}."); + $line->explain_warning( + +"These variables are used to control which files may be mirrored on FTP", +"servers or CD-ROM collections. They are not intended to mark packages", +"whose only MASTER_SITES are on ftp.NetBSD.org."); } } elsif ($type eq "SVR4PkgName") { |