diff options
author | rillig <rillig> | 2005-05-23 06:47:11 +0000 |
---|---|---|
committer | rillig <rillig> | 2005-05-23 06:47:11 +0000 |
commit | fa0303566cb7bb743eb34df867d575ca87b0b723 (patch) | |
tree | e6d42e285619540acf32314b73672a51a72a0515 /pkgtools | |
parent | 5322316cde96f09be0b672641aff8dd3addab593 (diff) | |
download | pkgsrc-fa0303566cb7bb743eb34df867d575ca87b0b723.tar.gz |
Only warn about use of $(VARIABLE) if it is not preceded by another $.
This avoids false positives in sed(1) expressions.
Diffstat (limited to 'pkgtools')
-rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index f6369cefa37..a84db776a04 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.169 2005/05/23 00:10:42 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.170 2005/05/23 06:47:11 rillig Exp $ # # This version contains lots of changes necessary for NetBSD packages # done by: @@ -1264,7 +1264,7 @@ sub checkfile_Makefile($) { # if ($opt_warn_paren) { log_info(NO_FILE, NO_LINE_NUMBER, "Checking for \$(VARIABLE)."); - if ($whole =~ /\$\([\w\d]+\)/) { + if ($whole =~ /[^\$]\$\([\w\d]+\)/) { log_warning(NO_FILE, NO_LINE_NUMBER, "Use \${VARIABLE} instead of \$(VARIABLE)."); } } |