From df8112eca589bc449c43bb2fded69cd20e6e5b71 Mon Sep 17 00:00:00 2001 From: marcus Date: Sat, 21 Jan 2006 21:54:05 +0000 Subject: Update to 2.8.2. * Update quoting requirements for new bsd.port.mk * Warn on use of USE_REINPLACE [1] * Recommend @dirrmtry instead of @unexec rmdir [1] Requested by: pav [1] --- devel/portlint/Makefile | 2 +- devel/portlint/src/portlint.pl | 46 ++++++++++++++++++++---------------------- 2 files changed, 23 insertions(+), 25 deletions(-) (limited to 'devel') diff --git a/devel/portlint/Makefile b/devel/portlint/Makefile index f87d57a4ea34..afcbbf64a3e3 100644 --- a/devel/portlint/Makefile +++ b/devel/portlint/Makefile @@ -8,7 +8,7 @@ # PORTNAME= portlint -PORTVERSION= 2.8.1 +PORTVERSION= 2.8.2 CATEGORIES= devel MASTER_SITES= # none DISTFILES= # none diff --git a/devel/portlint/src/portlint.pl b/devel/portlint/src/portlint.pl index 278881eda5a6..3badcf8a5c3d 100644 --- a/devel/portlint/src/portlint.pl +++ b/devel/portlint/src/portlint.pl @@ -17,7 +17,7 @@ # OpenBSD and NetBSD will be accepted. # # $FreeBSD$ -# $MCom: portlint/portlint.pl,v 1.94 2005/12/17 16:35:05 marcus Exp $ +# $MCom: portlint/portlint.pl,v 1.99 2006/01/21 21:51:52 marcus Exp $ # use vars qw/ $opt_a $opt_A $opt_b $opt_C $opt_c $opt_h $opt_t $opt_v $opt_M $opt_N $opt_B $opt_V /; @@ -40,7 +40,7 @@ $portdir = '.'; # version variables my $major = 2; my $minor = 8; -my $micro = 1; +my $micro = 2; sub l { '[{(]'; } sub r { '[)}]'; } @@ -610,10 +610,8 @@ sub checkplist { &perror("WARN: $file [$.]: use \"%D\" or \"%B\" to ". "specify prefix."); } - if ($_ !~ /true$/) { - &perror("WARN: $file [$.]: use \"\@dirrm\" ". + &perror("WARN: $file [$.]: use \"\@dirrmtry\" ". "instead of \"\@unexec rmdir\"."); - } } elsif (!$autoinfo && $_ =~ /^\@exec[ \t]+install-info\s+(.+)\s+(.+)$/) { $infoinstallseen = $.; push(@exec_info, $1); @@ -1189,19 +1187,6 @@ sub checkmakefile { "overridden."); } - # - # whole file: BROKEN et al. - # - my($var); - foreach $var (qw(BROKEN FORBIDDEN MANUAL_PACKAGE_BUILD NO_CDROM NO_PACKAGE RESTRICTED)) { - print "OK: checking ${var}.\n" if ($verbose); - if ($whole =~ /\n${var}[+?]?=[ \t]?[^"]+\w+/) { - my $lineno = &linenumber($`); - &perror("WARN: $file [$lineno]: ${var} messages should be ". - "quoted."); - } - } - # # whole file: DEPRECATED # @@ -1229,13 +1214,16 @@ sub checkmakefile { } # - # whole file: IGNORE + # whole file: BROKEN et al. # - print "OK: checking IGNORE.\n" if ($verbose); - if ($whole =~ /\nIGNORE[+?]?=[ \t]+"/) { - my $lineno = &linenumber($`); - &perror("WARN: $file [$lineno]: IGNORE messages should not ". - "be quoted."); + my ($var); + foreach $var (qw(IGNORE BROKEN FORBIDDEN MANUAL_PACKAGE_BUILD NO_CDROM NO_PACKAGE RESTRICTED)) { + print "OK: checking ${var}.\n" if ($verbose); + if ($whole =~ /\n${var}[+?]?=[ \t]+"/) { + my $lineno = &linenumber($`); + &perror("WARN: $file [$lineno]: ${var} messages should not ". + "be quoted."); + } } if ($whole =~ /\nIGNORE[+?]?=[ \t]+[^a-z \t]/ || @@ -1255,6 +1243,16 @@ sub checkmakefile { "PORTVERSION, PKGNAMEPREFIX and PKGNAMESUFFIX."); } + # + # whole file: USE_REINPLACE + # + print "OK: checking for USE_REINPLACE.\n" if ($verbose); + if ($whole =~ /\nUSE_REINPLACE.?=/) { + my $lineno = &linenumber($`); + &perror("WARN: $file [$lineno]: USE_REINPLACE is now obsolete. ". + "You can safely use REINPLACE_CMD without it."); + } + # # whole file: IS_INTERACTIVE/NOPORTDOCS # -- cgit