ports/144223: ports/textproc/urlview installs non-text browser even when it's not needed

[Masafumi NAKANE]

>Number:         144223
>Category:       ports
>Synopsis:       ports/textproc/urlview installs non-text browser even when it's not needed
>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:   Tue Feb 23 06:30:04 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator:     Masafumi NAKANE
>Release:        FreeBSD 7.0-RELEASE-p5 i386
>Organization:
>Environment:
System: FreeBSD bourbon.accessibility.org 7.0-RELEASE-p5 FreeBSD 7.0-RELEASE-p5 #2: Fri Oct 3 16:27:21 JST 2008 root at bourbon.accessibility.org:/usr/obj/usr/src/sys/GENERIC i386


	
>Description:
	
ports/textproc/urlview depends on one of Firefox, SeaMonkey, or Gecko 
according to WITH_* variable defined at make time.
While this has no problem for desktop system where these browsers are used,
this is not desirable on systems where only text browsers, such as
Lynx, w3m, etc., are used.
The fact that there is no easy way to specify, that you don't need any of
these browsers installed and just urlview to be installed, makes the 
issue somewhat serious.

>How-To-Repeat:
	
>Fix:

	

Patch below adds the config dialog so that users can specify
what they need (or don't need) at build time.

Index: Makefile
===================================================================
RCS file: /home/pcvs/ports/textproc/urlview/Makefile,v
retrieving revision 1.26
diff -u -r1.26 Makefile
--- Makefile	5 Feb 2010 11:44:35 -0000	1.26
+++ Makefile	23 Feb 2010 06:18:06 -0000
@@ -25,15 +25,19 @@
 SUB_FILES=	gecko.sh
 SUB_LIST=	GECKO=${GECKO}
 
-.include <bsd.port.pre.mk>
+OPTIONS=	GECKO	"Use Gecko"	off \
+		SEAMONKEY "Use SeaMonkey"	off \
+		FIREFOX "Use Firefox"	off
 
+.include <bsd.port.pre.mk>
+.include <bsd.port.options.mk>
 .if defined(WITH_GECKO)
 USE_GECKO=	firefox-devel firefox35 seamonkey
 .include "${PORTSDIR}/Mk/bsd.gecko.mk"
 .elif defined(WITH_SEAMONKEY)
 RUN_DEPENDS+=	${LOCALBASE}/lib/seamonkey/seamonkey-bin:${PORTSDIR}/www/seamonkey
 GECKO=		seamonkey
-.else
+.elif defined(WITH_FIREFOX)
 RUN_DEPENDS+=	${LOCALBASE}/lib/firefox3/firefox:${PORTSDIR}/www/firefox35
 GECKO=		firefox3
 .endif

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