ports/147103: [patch] editors/emacs-nox11: skip OPTIONS and use WITHOUT_DBUS

[Giorgos Keramidas]

The following reply was made to PR ports/147103; it has been noted by GNATS.

From: keramida at freebsd.org (Giorgos Keramidas)
To: wahjava at gmail.com (Ashish SHUKLA)
Cc: Anonymous <swell.k at gmail.com>, phoffman at proper.com,
        bug-followup at freebsd.org
Subject: Re: ports/147103: [patch] editors/emacs-nox11: skip OPTIONS and use WITHOUT_DBUS
Date: Thu, 27 May 2010 14:15:13 +0300

 On Thu, 27 May 2010 14:17:24 +0530, wahjava at gmail.com (Ashish SHUKLA) wrote:
 > Instead of mentioning GTK+ in every OPTION, I've added a 'pre-everything'
 > target.
 >
 > Attached is the diff file of the cleaned up port.
 
 : diff -urN /usr/ports/editors/emacs/Makefile emacs/Makefile
 : --- /usr/ports/editors/emacs/Makefile	2010-05-22 01:21:04.000000000 +0530
 : +++ emacs/Makefile	2010-05-27 14:14:59.000000000 +0530
 : @@ -28,7 +28,8 @@
 :  USE_GMAKE=	yes
 :  USE_BZIP2=	yes
 :
 : -USE_GETTEXT=	yes
 : +LICENSE=	GPLv3
 : +LICENSE_FILE=	${WRKSRC}/COPYING
 :
 :  MAN1=		b2m.1 ctags.1 ebrowse.1 emacs.1 emacsclient.1 etags.1 \
 :  		grep-changelog.1 rcs-checkin.1
 : @@ -43,10 +44,8 @@
 :  		org pcl-cvs pgg rcirc reftex remember sasl sc semantic ses sieve \
 :  		smtpmail speedbar tramp url vip viper widget woman
 :
 : -OPTIONS=	DBUS	"DBus support"			ON \
 : -		GCONF	"GConf support"			ON \
 : +OPTIONS=	GCONF	"GConf support"			ON \
 :  		GIF	"GIF support"			ON \
 : -		GTK2	"GTK+ support"			ON \
 :  		JPEG	"JPEG support"			ON \
 :  		M17N	"M17N support for text-shaping"	ON \
 :  		MOTIF	"Motif support"			OFF \
 : @@ -56,7 +55,6 @@
 :  		SOURCES	"Install source code"		ON \
 :  		SVG	"SVG support"			ON \
 :  		TIFF	"TIFF support"			ON \
 : -		X11	"X11 support"			ON \
 :  		XAW	"XAW support"			OFF \
 :  		XAW3D	"XAW3D support"			OFF \
 :  		SYNC_INPUT	"Synchronously process asynchronous input"	ON \
 : [snip]
 
 It would be nice if we could commit the changes one at a time, e.g. to
 use separate commits/changes for LICENSE_FILE, OPTIONS and the message
 displayed by the pre-everything target.
 
 AFAICT, the patch looks great ;-)