ports/154144: [mainteiner] net/vnc make BROKEN clause more specific

[Mikhail T.]

>Number:         154144
>Category:       ports
>Synopsis:       [mainteiner] net/vnc make BROKEN clause more specific
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Jan 19 15:40:06 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Mikhail T.
>Release:        9.0
>Organization:
FunBox
>Environment:
FreeBSD crush 9.0-CURRENT FreeBSD 9.0-CURRENT #0 r217259: Tue Jan 11 12:35:14 MSK 2011     root at crush:/usr/obj/usr/src/sys/DESK  amd64

>Description:
The port is now marked BROKEN on 9.0, but in fact client part builds and runs fine for me. So I suggest to broke if only WITHOUT_SERVER undefined.
>How-To-Repeat:

>Fix:
diff -uNr vnc.orig/Makefile vnc/Makefile
--- vnc.orig/Makefile	2011-01-19 18:16:37.170655296 +0300
+++ vnc/Makefile	2011-01-19 18:19:06.892654321 +0300
@@ -41,11 +41,12 @@
 
 .include <bsd.port.pre.mk>
 
+.if !defined(WITHOUT_SERVER)
+
 .if ${OSVERSION} >= 900000
-BROKEN=		does not build
+BROKEN=		VNC server part does not build on 9.x
 .endif
 
-.if !defined(WITHOUT_SERVER)
 # This is a placeholder MASTER_SITES entry -- see the pre-fetch target.
 MASTER_SITES+=	${MASTER_SITE_XFREE:S/$/:x/}
 MASTER_SITE_SUBDIR+=	4.3.0/:x

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