[patch|test] CVSup/amd64

[Roman Neuhauser]

# bzeeb-lists at lists.zabbadoz.net / 2005-04-05 18:13:14 +0000:
> attached patch also available at
> http://sources.zabbadoz.net/freebsd/ports/experimental/cvsup-amd64.diff
> updates binary only port for amd64.

> @@ -73,6 +71,10 @@ M3FLAGS+=	-DSTATIC
>  pre-fetch:
>  .ifndef WITHOUT_X11
>  	@${ECHO_MSG} "*****************************************************"
> +.if ${ARCH} == "amd64"
> +	@${ECHO_MSG} "* CVSup with X11 not yet supported on amd64.        *"
> +	@${ECHO_MSG} "* Will install non-X11 version. You might want      *"
> +.endif
>  	@${ECHO_MSG} "* To build CVSup without X11 (and without the GUI), *"
>  	@${ECHO_MSG} "* use the \"net/cvsup-without-gui\" port.             *"
>  	@${ECHO_MSG} "*****************************************************"

Hmm, this doesn't look very natural:

    CVSup with X11 not yet supported on amd64.
    Will install non-X11 version. You might want
    To build CVSup without X11 (and without the GUI),
    use the "net/cvsup-without-gui" port.

And it's misleading, too: it won't actually build anything. This is better:

@@ -73,10 +71,15 @@
 pre-fetch:
 .ifndef WITHOUT_X11
        @${ECHO_MSG} "*****************************************************"
+.if ${ARCH} == "amd64"
+       @${ECHO_MSG} "* CVSup with X11 not yet supported on amd64.        *"
+       @${ECHO_MSG} "* This port will install the non-X11 version.       *"
+.else
        @${ECHO_MSG} "* To build CVSup without X11 (and without the GUI), *"
        @${ECHO_MSG} "* use the \"net/cvsup-without-gui\" port.             *"
+.endif
        @${ECHO_MSG} "*****************************************************"
 .endif
        @${TRUE}
 
 post-install:


-- 
How many Vietnam vets does it take to screw in a light bulb?
You don't know, man.  You don't KNOW.
Cause you weren't THERE.             http://bash.org/?255991