ports/186334: sysutils/flashrom add STAGE support, disable BUSPIRATE, enable FTDI

[Hakisho Nukama]

On Sat, Feb 1, 2014 at 12:38 PM,  <nemysis at freebsd.org> wrote:
> Synopsis: sysutils/flashrom add STAGE support, disable BUSPIRATE, enable FTDI
>
> Responsible-Changed-From-To: freebsd-ports-bugs->nemysis
> Responsible-Changed-By: nemysis
> Responsible-Changed-When: Sat Feb 1 12:38:51 UTC 2014
> Responsible-Changed-Why:
> I'll take it.
>
> http://www.freebsd.org/cgi/query-pr.cgi?pr=186334

Added svn diff.
-------------- next part --------------
Index: Makefile
===================================================================
--- Makefile	(revision 342366)
+++ Makefile	(working copy)
@@ -1,69 +1,53 @@
 # Created by: Alexander Logvinov <ports at logvinov.com>
 # $FreeBSD$
 
-PORTNAME=	flashrom
-PORTVERSION=	0.9.7
-CATEGORIES=	sysutils
-MASTER_SITES=	http://download.flashrom.org/releases/
+PORTNAME=		flashrom
+PORTVERSION=		0.9.7
+CATEGORIES=		sysutils
+MASTER_SITES=		http://download.flashrom.org/releases/
 
-MAINTAINER=	nukama+maintainer at gmail.com
-COMMENT=	Utility for reading, writing, verifying, and erasing flash ROM chips
+MAINTAINER=		nukama+maintainer at gmail.com
+COMMENT=		Utility for reading, writing, verifying, and erasing flash ROM chips
 
-LICENSE=	GPLv2
+LICENSE=		GPLv2
 
-LIB_DEPENDS=	pci:${PORTSDIR}/devel/libpci
+LIB_DEPENDS=		libpci.so:${PORTSDIR}/devel/libpci
 
-OPTIONS_DEFINE=	DMIDECODE DOCS FTDI
-OPTIONS_DEFAULT=	DMIDECODE DOCS
+OPTIONS_DEFINE=		DMIDECODE FTDI BUSPIRATE
+OPTIONS_DEFAULT=	DMIDECODE FTDI
 
-DMIDECODE_DESC=	Use dmidecode to gather DMI information
-FTDI_DESC=	Enable external FT2232SPI flasher support
+DMIDECODE_DESC=		Use dmidecode to gather DMI information
+FTDI_DESC=		Enable external FT2232SPI flasher support
+BUSPIRATE_DESC=		Enable buspirate support (broken)
 
-USE_BZIP2=	yes
-USE_GMAKE=	yes
-MAKE_JOBS_UNSAFE=	yes
-CPPFLAGS+=	-I${LOCALBASE}/include
-LDFLAGS+=	-L${LOCALBASE}/lib -lc
+USES=			gmake
+USE_BZIP2=		yes
+CPPFLAGS+=		-I${LOCALBASE}/include
+LDFLAGS+=		-L${LOCALBASE}/lib -lc
+MAKE_ENV=		CONFIG_BUSPIRATE_SPI=no
 
-ONLY_FOR_ARCHS=	amd64 i386
+ONLY_FOR_ARCHS=		amd64 i386
 
-PORTDOCS=	ChangeLog COPYING README
-PLIST_FILES=	bin/${PORTNAME}
+PORTDOCS=		ChangeLog COPYING README
+PLIST_FILES=		bin/${PORTNAME} \
+			man/man8/${PORTNAME}.8.gz
 
-.if !defined(NO_INSTALL_MANPAGES)
-MAN8=		${PORTNAME}.8
-.endif
+DMIDECODE_RUN_DEPENDS+=	dmidecode:${PORTSDIR}/sysutils/dmidecode
 
-NO_STAGE=	yes
-.include <bsd.port.options.mk>
+FTDI_LIB_DEPENDS=	libftdi.so:${PORTSDIR}/devel/libftdi
+FTDI_USES=		pkgconfig
+FTDI_LDFLAGS=		-lz
+FTDI_MAKE_ENV_OFF=	2_SPI=no
 
-.if ${PORT_OPTIONS:MDMIDECODE}
-RUN_DEPENDS+=	dmidecode:${PORTSDIR}/sysutils/dmidecode
-.endif
+BUSPIRATE_MAKE_ENV=	CONFIG_BUSPIRATE_SPI=yes
 
-.if ${PORT_OPTIONS:MFTDI}
-USES+=		pkgconfig
-LIB_DEPENDS+=	ftdi:${PORTSDIR}/devel/libftdi
-.endif
-
-post-patch:
-.if ${PORT_OPTIONS:MFTDI}
-	@${REINPLACE_CMD} -e 's|-lusb")|-lusb") -lz|' ${WRKSRC}/Makefile
-.else
-	@${REINPLACE_CMD} -e 's|2_SPI ?= yes|2_SPI ?= no|' ${WRKSRC}/Makefile
-.endif
-
 do-install:
-	${INSTALL_PROGRAM} ${WRKSRC}/${PORTNAME} ${PREFIX}/bin
-.if !defined(NO_INSTALL_MANPAGES)
-	${MKDIR} ${MANPREFIX}/man/man8
-	${INSTALL_MAN} ${WRKSRC}/${MAN8} ${MANPREFIX}/man/man8
-.endif
-.if ${PORT_OPTIONS:MDOCS}
-	${MKDIR} ${DOCSDIR}
+	${INSTALL_PROGRAM} ${WRKSRC}/${PORTNAME} ${STAGEDIR}${PREFIX}/bin
+	${MKDIR} ${STAGEDIR}${MANPREFIX}/man/man8
+	${INSTALL_MAN} ${WRKSRC}/${PORTNAME}.8 ${STAGEDIR}${MANPREFIX}/man/man8
+	${MKDIR} ${STAGEDIR}${DOCSDIR}
 .for i in ${PORTDOCS}
-	${INSTALL_DATA} ${WRKSRC}/${i} ${DOCSDIR}
+	${INSTALL_DATA} ${WRKSRC}/${i} ${STAGEDIR}${DOCSDIR}
 .endfor
-.endif
 
 .include <bsd.port.mk>