Currently portlint stops checking for further issues once the first has been found for the COMMENT field in Makefile. How-To-Repeat: Make COMMENT$of some test port read "a1234567890123456789012345678901234567890123456789012345678901234567890". Run portlint. See how there is no complaint about the length of the COMMENT.
Responsible Changed From-To: freebsd-ports-bugs->marcus Over to maintainer (via the GNATS Auto Assign Tool)
Responsible Changed From-To: marcus->bdrewery I'll take it.
Author: bdrewery Date: Sat Oct 12 10:14:41 2013 New Revision: 330124 URL: http://svnweb.freebsd.org/changeset/ports/330124 Log: - Update to 2.14.5 Changes: * Fix recommending to replace USES=gmake with ${GMAKE} [1] * Ignore USES when recommending to replace direct commands [1] * Fix COMMENT check so it checks for length even if it is not formatted properly [2] PR: ports/181359 [1] Submitted by: Alan Hicks <ahicks@p-o.co.uk> [1] Reported by: many PR: ports/181730 [2] Submitted by: gerald Approved by: maintainer timeout (> 1 month) Modified: head/ports-mgmt/portlint/Makefile head/ports-mgmt/portlint/src/portlint.pl Modified: head/ports-mgmt/portlint/Makefile ============================================================================== --- head/ports-mgmt/portlint/Makefile Sat Oct 12 09:31:18 2013 (r330123) +++ head/ports-mgmt/portlint/Makefile Sat Oct 12 10:14:41 2013 (r330124) @@ -2,7 +2,7 @@ # $FreeBSD$ PORTNAME= portlint -PORTVERSION= 2.14.4 +PORTVERSION= 2.14.5 CATEGORIES= ports-mgmt MASTER_SITES= # none DISTFILES= # none Modified: head/ports-mgmt/portlint/src/portlint.pl ============================================================================== --- head/ports-mgmt/portlint/src/portlint.pl Sat Oct 12 09:31:18 2013 (r330123) +++ head/ports-mgmt/portlint/src/portlint.pl Sat Oct 12 10:14:41 2013 (r330124) @@ -1157,11 +1157,11 @@ sub check_depends_syntax { "USES[+]=gettext."); } - # check USE_GMAKE + # check USES=gmake if ($m{'dep'} =~ /^(gmake|\${GMAKE})$/) { &perror("WARN", $file, -1, "dependency to $1 ". "listed in $j. consider using ". - "USE_GMAKE."); + "USES[+]=gmake."); } # check USE_QT @@ -1867,6 +1867,7 @@ ruby sed sh sort sysctl touch tr which x && $curline !~ /^NO_CDROM(.)?=[^\n]+$i/m && $curline !~ /^MAINTAINER(.)?=[^\n]+$i/m && $curline !~ /^CATEGORIES(.)?=[^\n]+$i/m + && $curline !~ /^USES(.)?=[^\n]+$i/m && $curline !~ /^WX_COMPS(.)?=[^\n]+$i/m && $curline !~ /^\s*#.+$/m && $curline !~ /\-\-$i/m @@ -2839,7 +2840,8 @@ MAINTAINER COMMENT } else { # check for correctness if (($makevar{COMMENT} !~ /^["\[0-9A-Z]/) || ($makevar{COMMENT} =~ m/\.$/)) { #" &perror("WARN", $file, -1, "COMMENT should begin with a capital, and end without a period"); - } elsif (length($makevar{COMMENT}) > 70) { + } + if (length($makevar{COMMENT}) > 70) { &perror("WARN", $file, -1, "COMMENT exceeds 70 characters limit."); } } _______________________________________________ svn-ports-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-ports-all To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
State Changed From-To: open->closed Committed, thanks