From 567c07ece4eace0e335c452a007ca8bda5a38e5f Mon Sep 17 00:00:00 2001 From: rillig Date: Sun, 6 Jan 2008 14:55:00 +0000 Subject: Deactivated the "NOTE: you can write "tool" instead of "${TOOL}", as suggested by darcy@. --- pkgtools/pkglint/files/pkglint.pl | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index abf9f9b92e6..b76f7fe8538 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -1,5 +1,5 @@ #! @PERL@ -# $NetBSD: pkglint.pl,v 1.754 2008/01/06 01:02:14 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.755 2008/01/06 14:55:00 rillig Exp $ # # pkglint - static analyzer and checker for pkgsrc packages @@ -4670,7 +4670,9 @@ sub checkline_mk_shelltext($$) { $line->log_warning("The \"${plain_tool}\" tool is used but not added to USE_TOOLS."); } - if (defined($mkctx_target) && $mkctx_target =~ qr"^(?:pre|do|post)-(?:extract|patch|wrapper|configure|build|install|package|clean)$") { + # Deactivated to allow package developers to write + # consistent code that uses ${TOOL} in all places. + if (false && defined($mkctx_target) && $mkctx_target =~ qr"^(?:pre|do|post)-(?:extract|patch|wrapper|configure|build|install|package|clean)$") { if (!exists(get_required_vartool_varnames()->{$vartool})) { $opt_warn_extra and $line->log_note("You can write \"${plain_tool}\" instead of \"${shellword}\"."); $opt_warn_extra and $line->explain_note( -- cgit v1.2.3