diff options
| author | rillig <rillig> | 2005-05-23 06:47:11 +0000 |
|---|---|---|
| committer | rillig <rillig> | 2005-05-23 06:47:11 +0000 |
| commit | 0a82e6b24e9801bd4280655d7ae0d28f296586d8 (patch) | |
| tree | e6d42e285619540acf32314b73672a51a72a0515 /pkgtools/pkglint/files | |
| parent | 32e60932feb30223f07373c4de3ff9222c0e5fb0 (diff) | |
| download | pkgsrc-0a82e6b24e9801bd4280655d7ae0d28f296586d8.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/pkglint/files')
| -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)."); } } |