ports/156833: games/fairymax: O-Flags option and preparing change into masterport

[Kalten]

>Number:         156833
>Category:       ports
>Synopsis:       games/fairymax: O-Flags option and preparing change into masterport
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 05 17:50:07 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Kalten
>Release:        8.2-RELEASE
>Organization:
>Environment:
FreeBSD freeHugin.Walhalla.Leben 8.2-RELEASE FreeBSD 8.2-RELEASE #0: Thu Feb 17 02:41:51 UTC 2011     root at mason.cse.buffalo.edu:/usr/obj/usr/src/sys/GENERIC  amd64
>Description:
I had removed the -O2 option as mentioned in Porter's Handbook[1] earlier. Now there is an option to re enable it (on by default).

And it is being prepared to be changed into a master port, as my next PR with the title ``New port: games/fairymax-devel latest version of games/fairymax'' (hen egg problem: can not tell you its number yet) will introduce a slave port.

(See attachment.)

[1] http://www.freebsd.org/doc/en/books/porters-handbook/dads-cflags.html

>How-To-Repeat:

>Fix:


Patch attached with submission follows:

--- Makefile.orig	2011-05-05 00:47:54.000000000 +0200
+++ Makefile	2011-05-05 01:55:54.000000000 +0200
@@ -1,4 +1,4 @@
-# New ports collection makefile for:	fairy-max
+# New ports collection makefile for:	fairymax
 # Date created:				22 April 2011
 # Whom:					Kalten <kalten at gmx.at>
 #
@@ -6,15 +6,20 @@
 #
 
 PORTNAME=	fairymax
-PORTVERSION=	4.8Q
+PORTVERSION?=	4.8Q
+PORTREVISION?=	1
 CATEGORIES=	games
 MASTER_SITES=	http://freeheimdall.homeunix.org/files/
-DISTNAME=	${PORTNAME}-c7a129e
+DISTNAME?=	${PORTNAME}-c7a129e
 # http://hgm.nubati.net/cgi-bin/gitweb.cgi?p=fairymax.git;a=snapshot;h=c7a129ead81286990efb8409f238c48c7109c83b;sf=tgz
 
 MAINTAINER=	kalten at gmx.at
 COMMENT=	Chess engine (e.g. xboard) for shatranj, courier chess, ...
 
+CONFLICTS?=	fairymax-4.8Q.[0-9]* fairymax-4.8Q.[0-9]*_[0-9]*
+
+OPTIONS+=	OCFLAGS "Use optimized compiler flags" on
+
 DIST_SUBDIR=	${PORTNAME}
 
 PLIST_FILES=	bin/fairymax \
@@ -33,6 +38,14 @@
 MAKE_ARGS+=	INI_FILE_fmax=${DATADIR}/fmax.ini \
 		INI_FILE_qmax=${DATADIR}/qmax.ini
 
+.include <bsd.port.options.mk>
+
+.if defined(WITHOUT_OCFLAGS)
+CFLAGS=
+.else
+CFLAGS=		-O2
+.endif
+
 post-patch:
 	@${CP} ${WRKSRC}/fairymax.pod ${WRKSRC}/fairymax.pod.ori
 	@${SED} -e 's|%%PREFIX%%|${PREFIX}|g;s|%%DATADIR%%|${DATADIR}|g' \
--- distinfo.orig	2011-05-05 01:51:04.000000000 +0200
+++ distinfo	2011-05-05 01:52:03.000000000 +0200
@@ -1,2 +1,4 @@
 SHA256 (fairymax/fairymax-c7a129e.tar.gz) = ec8e75c92e345e0f0db56095ac576156f54fb9320af05e86f8e589a8300c5a9a
 SIZE (fairymax/fairymax-c7a129e.tar.gz) = 34229
+SHA256 (fairymax/fairymax-507c6ba.tar.gz) = 18727b04bf4a3a9d1e3bac55a2729727735a9103f3beb03afb8bbbfd21a6112a
+SIZE (fairymax/fairymax-507c6ba.tar.gz) = 34289


>Release-Note:
>Audit-Trail:
>Unformatted: