[73027] trunk/dports/lang/sbcl/Portfile

[Ryan Schmidt]

On Nov 1, 2010, at 11:59, easieste at macports.org wrote:

> Revision: 73027
>          http://trac.macports.org/changeset/73027
> Author:   easieste at macports.org
> Date:     2010-11-01 09:59:42 -0700 (Mon, 01 Nov 2010)
> Log Message:
> -----------
> Make '+pdf' and '+html' mutually exclusive
> 
> Modified Paths:
> --------------
>    trunk/dports/lang/sbcl/Portfile
> 
> Modified: trunk/dports/lang/sbcl/Portfile
> ===================================================================
> --- trunk/dports/lang/sbcl/Portfile	2010-11-01 16:32:55 UTC (rev 73026)
> +++ trunk/dports/lang/sbcl/Portfile	2010-11-01 16:59:42 UTC (rev 73027)
> @@ -100,13 +100,14 @@
> 
> default_variants      +html
> 
> -variant html description {Installs documentation in HTML and Info formats} {
> +variant html conflicts pdf description {Installs documentation in HTML and Info formats} {
>     patchfiles-append patch-make-doc.diff
>     depends_build-append bin:texi2dvi:texinfo
>     depends_build-append bin:makeinfo:texinfo
> } 
> 
> -variant pdf description {Installs documentation in PDF, HTML and Info formats} {
> +variant pdf conflicts html description {Installs documentation in PDF, HTML and Info formats} {
> +    patchfiles-delete patch-make-doc.diff
>     depends_build-append bin:dvips:texlive-bin
>     depends_build-append bin:texi2dvi:texinfo
>     depends_build-append bin:makeinfo:texinfo

It sounds like the pdf variant does everything the html variant does, and then a little more. So why not make the pdf variant require the html variant, instead of making them conflict? Especially since the html variant is selected by default.