ports/179344: [patch] x11-wm/fluxbox: rename option PDF -> PDFDOCS and some more

[Boris Samorodov]

>Number:         179344
>Category:       ports
>Synopsis:       [patch] x11-wm/fluxbox: rename option PDF -> PDFDOCS and some more
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Jun 05 19:20:00 UTC 2013
>Closed-Date:
>Last-Modified:
>Originator:     Boris Samorodov
>Release:        FreeBSD 9-i386
>Organization:
BSDprint
>Environment:
FreeBSD srv.bb.tel.ru 9.1-STABLE FreeBSD 9.1-STABLE #9 r250778: Sat May 18 23:01:30 SAMT 2013     bsam at srv.bb.tel.ru:/usr/obj/usr/src/sys/BB  i386
>Description:
Yesterday I installed a brand new PC. When installing x11-wm/fluxbox I was curious what is the option "PDF document support". At a nearer glance it happened to be only an option to install PDF documentation. It seems to me that it should be an other option, say PDFDOCS.
>How-To-Repeat:
Do "make configure" for x11-wm/fluxbox.
>Fix:
. introduce a new option PDFDOCS (rather then PDF);
. sort options *_DESC;
. remove the indefinite article from COMMENT.


Patch attached with submission follows:

Index: Makefile
===================================================================
--- Makefile	(revision 320012)
+++ Makefile	(working copy)
@@ -9,7 +9,7 @@
 EXTRACT_ONLY=	${PORTNAME}-${PORTVERSION}${EXTRACT_SUFX}
 
 MAINTAINER=	beat at FreeBSD.org
-COMMENT=	A small and fast window manager based on BlackBox
+COMMENT=	Small and fast window manager based on BlackBox
 
 RUN_DEPENDS=	xmessage:${PORTSDIR}/x11/xmessage
 
@@ -25,14 +25,15 @@
 		fluxbox-style.5
 DOCSVERSION=	20060629
 
-OPTIONS_DEFINE=	DOCS DOCHTML PDF DEBUG GNOME IMLIB2 XINERAMA \
+OPTIONS_DEFINE=	DOCS DOCHTML PDFDOCS DEBUG GNOME IMLIB2 XINERAMA \
 		XRENDER NLS SLIT REMEMBER TOOLBAR
 OPTIONS_DEFAULT=	XRENDER SLIT REMEMBER TOOLBAR
 DOCHTML_DESC=	Install html documentation
-XRENDER_DESC=	Enable xrender support
+PDFDOCS_DESC=	Build and/or install PDF documentation
+REMEMBER_DESC=	Enable remember feature
 SLIT_DESC=	Enable slit feature
-REMEMBER_DESC=	Enable remember feature
 TOOLBAR_DESC=	Enable toolbar feature
+XRENDER_DESC=	Enable xrender support
 
 .include <bsd.port.options.mk>
 
@@ -51,7 +52,7 @@
 PLIST_SUB+=		DOCHTML:="@comment "
 .endif
 
-.if ${PORT_OPTIONS:MPDF}
+.if ${PORT_OPTIONS:MPDFDOCS}
 # http://fluxbox.org/docbook/en/pdf/
 MASTER_SITES+=		${MASTER_SITE_LOCAL:S,%SUBDIR%,mezz,}:pdf \
 			http://people.freebsd.org/~mezz/distfiles/:pdf
@@ -136,7 +137,7 @@
 	@${INSTALL_DATA} ${WRKSRC}/${i} ${DOCSDIR}
 .endfor
 .endif
-.if ${PORT_OPTIONS:MPDF}
+.if ${PORT_OPTIONS:MPDFDOCS}
 	@${MKDIR} ${DOCSDIR}/pdf
 	@${INSTALL_DATA} ${DISTDIR}/fluxbook-${DOCSVERSION}.pdf \
 		${DOCSDIR}/pdf/


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