svn commit: r361201 - in head/games: quake-data quake2-data quake3-data

[Alexey Dokuchaev]

Author: danfe
Date: Tue Jul  8 08:51:31 2014
New Revision: 361201
URL: http://svnweb.freebsd.org/changeset/ports/361201
QAT: https://qat.redports.org/buildarchive/r361201/

Log:
  Reduce the differences between `quake*-data' ports:
  - Consistently use vertical bar (pipe) in :S substitutions
  - Do not abuse .error statement, set IGNORE knob instead
  - Drop some unnecessary quotation marks in comparisons
  - Add a comment on why we cannot depend on particular .pak file
  - Simplify extraction of the `linuxq3apoint-1.32b-3.x86.run'

Modified:
  head/games/quake-data/Makefile.include
  head/games/quake2-data/Makefile.include
  head/games/quake3-data/Makefile
  head/games/quake3-data/Makefile.include

Modified: head/games/quake-data/Makefile.include
==============================================================================
--- head/games/quake-data/Makefile.include	Tue Jul  8 08:47:41 2014	(r361200)
+++ head/games/quake-data/Makefile.include	Tue Jul  8 08:51:31 2014	(r361201)
@@ -5,13 +5,13 @@ Q1PKGNAMEPREFIX?=	quake-
 Q1DIR?=		${LOCALBASE}/share/quake
 
 .if defined(PKGNAMEPREFIX) && ${PKGNAMEPREFIX} == ${Q1PKGNAMEPREFIX} && defined(OPTIONSMKINCLUDED)
-.  error include before <bsd.port.options.mk> if using Q1PKGNAMEPREFIX
+IGNORE=	is using Q1PKGNAMEPREFIX, but <bsd.port.options.mk> included too early
 .endif
 
 MAKE_ENV+=	Q1DIR="${Q1DIR}"
-PLIST_SUB+=	Q1DIR="${Q1DIR:S/${LOCALBASE}\///}"
+PLIST_SUB+=	Q1DIR="${Q1DIR:S|${LOCALBASE}/||}"
 SUB_LIST+=	Q1DIR="${Q1DIR}"
 
-.if ${PORTNAME} != "data"
+.if ${PORTNAME} != data
 RUN_DEPENDS+=	${Q1DIR}/id1/pak0.pak:${PORTSDIR}/games/quake-data
 .endif

Modified: head/games/quake2-data/Makefile.include
==============================================================================
--- head/games/quake2-data/Makefile.include	Tue Jul  8 08:47:41 2014	(r361200)
+++ head/games/quake2-data/Makefile.include	Tue Jul  8 08:51:31 2014	(r361201)
@@ -48,7 +48,7 @@
 Q2PKGNAMEPREFIX?=	quake2-
 Q2DIR?=		${LOCALBASE}/share/quake2
 
-.if defined(PKGNAMEPREFIX) && ${PKGNAMEPREFIX} == "${Q2PKGNAMEPREFIX}" && defined(OPTIONSMKINCLUDED)
+.if defined(PKGNAMEPREFIX) && ${PKGNAMEPREFIX} == ${Q2PKGNAMEPREFIX} && defined(OPTIONSMKINCLUDED)
 IGNORE=	is using Q2PKGNAMEPREFIX, but <bsd.port.options.mk> included too early
 .endif
 
@@ -56,6 +56,7 @@ MAKE_ENV+=	Q2DIR="${Q2DIR}"
 PLIST_SUB+=	Q2DIR="${Q2DIR:S|${LOCALBASE}/||}"
 SUB_LIST+=	Q2DIR="${Q2DIR}"
 
-.if ${PORTNAME} != "data"
+.if ${PORTNAME} != data
+# Not depending on the particular .pak file since they are OPTIONS-dependent
 RUN_DEPENDS+=	${Q2DIR}:${PORTSDIR}/games/quake2-data
 .endif

Modified: head/games/quake3-data/Makefile
==============================================================================
--- head/games/quake3-data/Makefile	Tue Jul  8 08:47:41 2014	(r361200)
+++ head/games/quake3-data/Makefile	Tue Jul  8 08:51:31 2014	(r361201)
@@ -5,13 +5,12 @@ PORTNAME=	data
 DISTVERSION=	1.32b-3
 PORTREVISION=	2
 CATEGORIES=	games
-MASTER_SITES=	${MASTER_SITE_IDSOFTWARE}
-MASTER_SITE_SUBDIR=	quake3/linux
+MASTER_SITES=	IDSOFTWARE/quake3/linux
 PKGNAMEPREFIX=	${Q3PKGNAMEPREFIX}
 DISTNAME=	linuxq3apoint-${DISTVERSION}.x86
 EXTRACT_SUFX=	.run
 
-MAINTAINER=	ports at FreeBSD.org
+MAINTAINER=	games at FreeBSD.org
 COMMENT=	Quake III Arena data files
 
 DATADIR=	${Q3DIR}
@@ -32,8 +31,8 @@ TEAMARENA_DESC=		Install Quake III Team 
 do-extract:
 	@${RM} -rf ${WRKDIR}
 	@${MKDIR} ${WRKDIR}
-	@cd ${WRKDIR} && ${TAIL} +356 \
-		${_DISTDIR}/${DISTNAME}${EXTRACT_SUFX} | ${TAR} xzf -
+	@${TAIL} +356 ${_DISTDIR}/${DISTNAME}${EXTRACT_SUFX} | \
+		${TAR} xzf - -C ${WRKDIR}
 
 do-install:
 	@${MKDIR} ${STAGEDIR}${DATADIR}

Modified: head/games/quake3-data/Makefile.include
==============================================================================
--- head/games/quake3-data/Makefile.include	Tue Jul  8 08:47:41 2014	(r361200)
+++ head/games/quake3-data/Makefile.include	Tue Jul  8 08:51:31 2014	(r361201)
@@ -5,13 +5,13 @@ Q3PKGNAMEPREFIX?=	quake3-
 Q3DIR?=		${LOCALBASE}/share/quake3
 
 .if defined(PKGNAMEPREFIX) && ${PKGNAMEPREFIX} == ${Q3PKGNAMEPREFIX} && defined(OPTIONSMKINCLUDED)
-.  error include before <bsd.port.options.mk> if using Q3PKGNAMEPREFIX
+IGNORE=	is using Q3PKGNAMEPREFIX, but <bsd.port.options.mk> included too early
 .endif
 
 MAKE_ENV+=	Q3DIR="${Q3DIR}"
-PLIST_SUB+=	Q3DIR="${Q3DIR:S/${LOCALBASE}\///}"
+PLIST_SUB+=	Q3DIR="${Q3DIR:S|${LOCALBASE}/||}"
 SUB_LIST+=	Q3DIR="${Q3DIR}"
 
-.if ${PORTNAME} != "data"
+.if ${PORTNAME} != data
 RUN_DEPENDS+=	${Q3DIR}/baseq3/pak1.pk3:${PORTSDIR}/games/quake3-data
 .endif