[56794] trunk/dports/print/libspectre/Portfile

[Ryan Schmidt]

On Sep 2, 2009, at 04:45, illogic-al at macports.org wrote:

> Revision: 56794
>          http://trac.macports.org/changeset/56794
> Author:   illogic-al at macports.org
> Date:     2009-09-02 02:45:30 -0700 (Wed, 02 Sep 2009)
> Log Message:
> -----------
> Build libspectre without documentation, to avoid x11 dependencies  
> from doxygen by default. Also adds +docs variant for those wanting  
> documentation and bumps revision.
>
> Modified Paths:
> --------------
>    trunk/dports/print/libspectre/Portfile
>
> Modified: trunk/dports/print/libspectre/Portfile
> ===================================================================
> --- trunk/dports/print/libspectre/Portfile	2009-09-02 09:18:14 UTC  
> (rev 56793)
> +++ trunk/dports/print/libspectre/Portfile	2009-09-02 09:45:30 UTC  
> (rev 56794)
> @@ -5,6 +5,7 @@
>
> name            libspectre
> version         0.2.2
> +revision        1
> description     Libspectre is a small library for rendering  
> PostScript documents.
> long_description \
>                 ${description} \
> @@ -20,12 +21,15 @@
>                 sha1    d5968d34c4bc93cd280ea35d8db53cf41f811edc \
>                 rmd160  aca3ec7b9ffb521c4e38cf32a14cb9af07d2f009
>
> -depends_build   port:pkgconfig \
> -                port:doxygen
> +depends_build   port:pkgconfig
>
> depends_lib     port:ghostscript \
>                 path:lib/pkgconfig/cairo.pc:cairo
>
> +variant docs description "Build documentation" {
> +    depends_build   port:doxygen
> +}

The variant needs to do more than just add a dependency on doxygen. It  
needs to also inform the software it is ok to use doxygen, or  
probably, the port needs to inform the software it is *not* ok to use  
doxygen *unless* the +docs variant is selected.