Bug 182919

Summary: [PATCH] [ports-mgmt/portlint] port standard updates and maintainership to portmgr
Product: Ports & Packages Reporter: Bryan Drewery <bdrewery>
Component: Individual Port(s)Assignee: Joe Marcus Clarke <marcus>
Status: Closed FIXED    
Severity: Affects Only Me CC: portmgr
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
portlint-updates-portmgr.txt none

Description Bryan Drewery freebsd_committer freebsd_triage 2013-10-12 11:40:00 UTC
	portlint is a critical tool for port development. We have been making a lot of changes lately that
	portlint fails to properly find or recommend. The recent ports/181359 one that recommended
	USES=gmake to use USES=${GMAKE} and USE_GMAKE was especially wrong. We need more people to be able
	to approve or work on getting updates into portlint faster.

	- Support staging
	- Remove indefinite article from COMMENT
	- Pass maintainership to portmgr
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-10-12 14:26:49 UTC
Responsible Changed
From-To: freebsd-ports-bugs->marcus

Over to maintainer (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=182919 

Date: Sat, 12 Oct 2013 12:17:35 -0400
Comment 2 Bryan Drewery freebsd_committer freebsd_triage 2013-10-12 17:25:41 UTC
> I have been maintaining portlint for a while now, and I do my best to
> batch changes to avoid a lot of constant churn.  I would appreciate not
> being summarily replaced as maintainer.


I don't think you understand. In just 2 months now, it has become
unusable, wrong, and more and more irrelevant. We need to be able to
update it more frequently. It is not OK to sit on valid patches for
months at a time just to avoid updating a port that has 0 build time.
Porters need this tool updated ASAP when we make framework changes.


-- 
Regards,
Bryan Drewery
Comment 3 Joe Marcus Clarke freebsd_committer freebsd_triage 2013-10-12 17:34:06 UTC
State Changed
From-To: open->closed

Committed, retaining maintainership.  Thanks.
Comment 4 dfilter service freebsd_committer freebsd_triage 2013-10-12 17:34:07 UTC
Author: marcus
Date: Sat Oct 12 16:33:59 2013
New Revision: 330137
URL: http://svnweb.freebsd.org/changeset/ports/330137

Log:
  * Support staging
  * Adjust COMMENT
  
  PR:		182919
  Submitted by:	bdrewery

Modified:
  head/ports-mgmt/portlint/Makefile

Modified: head/ports-mgmt/portlint/Makefile
==============================================================================
--- head/ports-mgmt/portlint/Makefile	Sat Oct 12 16:15:07 2013	(r330136)
+++ head/ports-mgmt/portlint/Makefile	Sat Oct 12 16:33:59 2013	(r330137)
@@ -8,7 +8,7 @@ MASTER_SITES=	# none
 DISTFILES=	# none
 
 MAINTAINER=	marcus@FreeBSD.org
-COMMENT=	A verifier for FreeBSD port directory
+COMMENT=	Verifier for FreeBSD port directory
 
 NO_BUILD=	yes
 WRKSRC=		${WRKDIR}/src
@@ -17,15 +17,16 @@ USE_PERL5=	run
 
 SRC=		${.CURDIR}/src
 
-MAN1=		portlint.1
-PLIST_FILES=	bin/portlint
+PLIST_FILES=	bin/portlint \
+		man/man1/portlint.1.gz
 
-.if !defined(NOPORTEXAMPLES)
+.include <bsd.port.options.mk>
+
+.if ${PORT_OPTIONS:MEXAMPLES}
 PLIST_FILES+=	%%EXAMPLESDIR%%/portlintgrep
 PLIST_DIRS=	%%EXAMPLESDIR%%
 .endif
 
-NO_STAGE=	yes
 do-fetch:
 	@${DO_NADA}
 
@@ -38,11 +39,15 @@ post-patch:
 		${WRKSRC}/portlintgrep.pl
 
 do-install:
-	${INSTALL_SCRIPT} ${WRKSRC}/portlint.pl ${PREFIX}/bin/portlint
-	${INSTALL_MAN} ${WRKSRC}/portlint.1 ${MAN1PREFIX}/man/man1
-.if !defined(NOPORTEXAMPLES)
-	${INSTALL} -o ${SHAREOWN} -g ${SHAREGRP} -m ${BINMODE} -d ${EXAMPLESDIR}
-	${INSTALL_SCRIPT} ${WRKSRC}/portlintgrep.pl ${EXAMPLESDIR}/portlintgrep
+	${INSTALL_SCRIPT} ${WRKSRC}/portlint.pl \
+	    ${STAGEDIR}${PREFIX}/bin/portlint
+	${INSTALL_MAN} ${WRKSRC}/portlint.1 \
+	    ${STAGEDIR}${MAN1PREFIX}/man/man1
+.if ${PORT_OPTIONS:MEXAMPLES}
+	${INSTALL} -o ${SHAREOWN} -g ${SHAREGRP} -m ${BINMODE} -d \
+	    ${STAGEDIR}${EXAMPLESDIR}
+	${INSTALL_SCRIPT} ${WRKSRC}/portlintgrep.pl \
+	    ${STAGEDIR}${EXAMPLESDIR}/portlintgrep
 .endif
 
 .include <bsd.port.mk>
_______________________________________________
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"