Date: Sun, 26 Jan 2014 05:10:01 GMT From: Alex Kozlov <spam@rm-rf.kiev.ua> To: freebsd-ports-bugs@FreeBSD.org Subject: Re: ports/186109: [maintainer-patch] converting games/doom-data to OPTIONS_MULTI Message-ID: <201401260510.s0Q5A1ha055307@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR ports/186109; it has been noted by GNATS. From: Alex Kozlov <spam@rm-rf.kiev.ua> To: Jakub Lach <jakub_lach@mailplus.pl> Cc: bug-followup@FreeBSD.org Subject: Re: ports/186109: [maintainer-patch] converting games/doom-data to OPTIONS_MULTI Date: Sun, 26 Jan 2014 07:00:16 +0200 --1yeeQ81UyVL57Vl7 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi, Jakub Can you please review the attached patch? Thanks. -- Alex --1yeeQ81UyVL57Vl7 Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="patch.txt" - Switch to OPTIONS_MULTI - Use options helpers - Support STAGEDIR Index: games/doom-data/Makefile =================================================================== --- games/doom-data/Makefile (revision 341154) +++ games/doom-data/Makefile (working copy) @@ -7,18 +7,20 @@ CATEGORIES= games MASTER_SITES= http://www.ijs.si/~lesi/distfiles/doom/ PKGNAMEPREFIX= ${DMPKGNAMEPREFIX} -EXTRACT_SUFX= .bz2 DIST_SUBDIR= ${PKGNAMEPREFIX}${PORTNAME} MAINTAINER= jakub_lach@mailplus.pl COMMENT= Shareware data files for Doom, Doom II, Hexen, Heretic, and Strife -USE_BZIP2= yes NO_BUILD= yes NO_WRKSUBDIR= yes -OPTIONS_DEFINE= DOOM HERETIC HEXEN STRIFE +OPTIONS_SUB= yes + +OPTIONS_MULTI= WADS +OPTIONS_MULTI_WADS= DOOM HERETIC HEXEN STRIFE OPTIONS_DEFAULT= DOOM + DOOM_DESC= Install Doom shareware data HERETIC_DESC= Install Heretic shareware data HEXEN_DESC= Install Hexen shareware data @@ -29,49 +31,24 @@ LATEST_LINK= ${PKGNAMEPREFIX}${PORTNAME} DATADIR= ${DMDIR} -DOOM_WAD= doom1.wad -HERETIC_WAD= heretic1.wad -HEXEN_WAD= hexen.wad -STRIFE_WAD= strife0.wad +DOOM_DISTFILES= doom1.wad.bz2 +HERETIC_DISTFILES= heretic1.wad.bz2 +HEXEN_DISTFILES= hexen.wad.bz2 +STRIFE_DISTFILES= strife0.wad.bz2 -NO_STAGE= yes .include "Makefile.include" - .include <bsd.port.options.mk> -.for f in ${OPTIONS_DEFINE} -.if ${PORT_OPTIONS:M${f}} -DISTFILES+= ${${f}_WAD}${EXTRACT_SUFX} -PLIST_SUB+= ${f}="" -. else -PLIST_SUB+= ${f}="@comment " -. endif -.endfor - -.if ${DISTFILES} == "" -. undef DIST_SUBDIR -.endif - do-extract: @${MKDIR} ${WRKDIR} -.for f in ${OPTIONS_DEFINE} -. if ${PORT_OPTIONS:M${f}} - @${BZIP2_CMD} -dc ${_DISTDIR}/${${f}_WAD}${EXTRACT_SUFX} > \ - ${WRKDIR}/${${f}_WAD} -. endif +.for f in ${DISTFILES} + @${BZIP2_CMD} -dc ${_DISTDIR}/${f} > ${WRKDIR}/${f:S/.bz2$//} .endfor do-install: - @${MKDIR} ${DATADIR} -.for f in ${OPTIONS_DEFINE} -. if ${PORT_OPTIONS:M${f}} - ${INSTALL_DATA} ${WRKSRC}/${${f}_WAD} ${DATADIR} -. endif + @${MKDIR} ${STAGEDIR}${DATADIR} +.for f in ${DISTFILES} + ${INSTALL_DATA} ${WRKSRC}/${f:S/.bz2$//} ${STAGEDIR}${DATADIR}/ .endfor -post-install: - @${ECHO_CMD} - @${CAT} ${PKGMESSAGE} - @${ECHO_CMD} - .include <bsd.port.mk> --1yeeQ81UyVL57Vl7--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201401260510.s0Q5A1ha055307>