diff options
author | rillig <rillig> | 2005-05-31 21:32:45 +0000 |
---|---|---|
committer | rillig <rillig> | 2005-05-31 21:32:45 +0000 |
commit | dee4ae90e13a67b187eb019b5f5c57482ec82ebd (patch) | |
tree | 3edf810d51663b782b31a493bc1baa18277afbd3 /pkgtools/pkglint/files | |
parent | 38ecc94c9f3dc953490f68a112d233b2c657e515 (diff) | |
download | pkgsrc-dee4ae90e13a67b187eb019b5f5c57482ec82ebd.tar.gz |
When checking the type of variables, the variable name must begin at the
beginning of a line. This fixes false warnings for variables that are
commented out. Added ${JAM_COMMAND} to the commands that may have
"install" or "test" as arguments.
Diffstat (limited to 'pkgtools/pkglint/files')
-rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index 778dab18f57..aca1b339190 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -11,7 +11,7 @@ # Freely redistributable. Absolutely no warranty. # # From Id: portlint.pl,v 1.64 1998/02/28 02:34:05 itojun Exp -# $NetBSD: pkglint.pl,v 1.190 2005/05/31 21:15:06 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.191 2005/05/31 21:32:45 rillig Exp $ # # This version contains lots of changes necessary for NetBSD packages # done by: @@ -1173,7 +1173,7 @@ sub readmakefile($$) { sub check_Makefile_vartype($) { my ($line) = @_; - if ($line->text =~ qr"([A-Z_a-z0-9.]+)\s*(=|\?=|\+=)\s*(.*)") { + if ($line->text =~ qr"^([A-Z_a-z0-9.]+)\s*(=|\?=|\+=)\s*(.*)") { my ($varname, $op, $value) = ($1, $2, $3); if ($value =~ qr"\$") { # ignore values that contain other variables @@ -1289,7 +1289,7 @@ sub checklines_direct_tools($) { .*_TARGET USE_TOOLS); my @ok_shellcmds = ( - qr"\./Build\s+(?:install|test)", + qr"(?:\./Build|\$\{JAM_COMMAND\})\s+(?:install|test)", qr"\"[^\"]*${regex_tools}[^\"]*\""); my %toolvar = (); |