Date: Fri, 16 Mar 2012 11:05:18 +0800 (CST) From: Sunpoet Po-Chuan Hsieh <sunpoet@FreeBSD.org> To: FreeBSD-gnats-submit@FreeBSD.org Cc: marcus@FreeBSD.org Subject: ports/166173: [PATCH] ports-mgmt/portlint: add TEST_DEPENDS support Message-ID: <3V8BPB2bd5z37Y@sunpoet.net> Resent-Message-ID: <201203160310.q2G3A4aR040685@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 166173 >Category: ports >Synopsis: [PATCH] ports-mgmt/portlint: add TEST_DEPENDS support >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: change-request >Submitter-Id: current-users >Arrival-Date: Fri Mar 16 03:10:04 UTC 2012 >Closed-Date: >Last-Modified: >Originator: Sunpoet Po-Chuan Hsieh >Release: FreeBSD 9.0-STABLE amd64 >Organization: The FreeBSD Project >Environment: System: FreeBSD bonjour.sunpoet.net 9.0-STABLE FreeBSD 9.0-STABLE #0: Sun Jan 29 23:36:49 CST 2012 >Description: - Add TEST_DEPENDS support Port maintainer (marcus@FreeBSD.org) is cc'd. Generated with FreeBSD Port Tools 0.99_4 (mode: change, diff: CVS) >How-To-Repeat: >Fix: --- portlint-2.13.9.patch begins here --- Index: src/portlint.pl =================================================================== RCS file: /home/ncvs/ports/ports-mgmt/portlint/src/portlint.pl,v retrieving revision 1.126 diff -u -u -r1.126 portlint.pl --- src/portlint.pl 4 Mar 2012 18:46:31 -0000 1.126 +++ src/portlint.pl 16 Mar 2012 02:15:53 -0000 @@ -1195,8 +1195,8 @@ $ENV{'PORTSDIR'} //= $portsdir; - foreach my $i (grep(/^(PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|FETCH_)*DEPENDS[?+]?=/, split(/\n/, $tmp))) { - $i =~ s/^((PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|FETCH_)*DEPENDS)[?+]?=[ \t]*//; + foreach my $i (grep(/^(PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|TEST_|FETCH_)*DEPENDS[?+]?=/, split(/\n/, $tmp))) { + $i =~ s/^((PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|TEST_|FETCH_)*DEPENDS)[?+]?=[ \t]*//; $j = $1; $seen_depends{$j}++; if ($j ne 'DEPENDS' && @@ -1222,7 +1222,7 @@ print "OK: checking dependency value for $j.\n" if ($verbose); - if ($k =~ /\${((PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|FETCH_)*DEPENDS)}/) { + if ($k =~ /\${((PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|TEST_|FETCH_)*DEPENDS)}/) { &perror("WARN", $file, -1, "do not set $j to $k. ". "Instead, explicity list out required $j dependencies."); } @@ -2885,10 +2885,10 @@ # NOTE: EXEC_DEPENDS is obsolete, so it should not be listed. @linestocheck = qw( EXTRACT_DEPENDS LIB_DEPENDS PATCH_DEPENDS BUILD_DEPENDS RUN_DEPENDS -FETCH_DEPENDS DEPENDS_TARGET +TEST_DEPENDS FETCH_DEPENDS DEPENDS_TARGET ); - if ($tmp =~ /^(PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|FETCH_)DEPENDS/m) { + if ($tmp =~ /^(PATCH_|EXTRACT_|LIB_|BUILD_|RUN_|TEST_|FETCH_)DEPENDS/m) { &checkearlier($file, $tmp, @varnames); check_depends_syntax($tmp, $file); --- portlint-2.13.9.patch ends here --- >Release-Note: >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3V8BPB2bd5z37Y>