[79657] trunk/dports/math/eigen3/Portfile

[Ryan Schmidt]

On Jun 22, 2011, at 14:15, eborisch at macports.org wrote:

> Revision: 79657
>          http://trac.macports.org/changeset/79657
> Author:   eborisch at macports.org
> Date:     2011-06-22 12:15:37 -0700 (Wed, 22 Jun 2011)
> Log Message:
> -----------
> eigen3: Use CMake for install by default. Closes #29878. New no_cmake variant drops CMake dependency for those who do not need CMake compatibilty.

Variants named starting with "no_" are deprecated. The variant should instead be called "cmake", do the opposite of what it does now, and be in default_variants. In other words, go back to how it was before, just rename the "configured" variant to "cmake" (and add default_variants).


> Modified Paths:
> --------------
>    trunk/dports/math/eigen3/Portfile
> 
> Modified: trunk/dports/math/eigen3/Portfile
> ===================================================================
> --- trunk/dports/math/eigen3/Portfile	2011-06-22 19:10:53 UTC (rev 79656)
> +++ trunk/dports/math/eigen3/Portfile	2011-06-22 19:15:37 UTC (rev 79657)
> @@ -5,6 +5,7 @@
> 
> name                eigen3
> version             3.0.1
> +revision            1
> license             {{GPL-2+} {LGPL-3+}}
> categories          math science
> maintainers         eborisch \
> @@ -26,20 +27,20 @@
> configure.universal_args  ""
> 
> # Variants
> -variant gcc44 description { Use MacPort's gcc44 compilers } conflicts gcc45 {
> +variant gcc44 description { Use MacPorts' gcc44 compilers } conflicts gcc45 {
>   configure.compiler  macports-gcc-4.4
> }
> 
> -variant gcc45 description { Use MacPort's gcc45 compilers } conflicts gcc44 {
> +variant gcc45 description { Use MacPorts' gcc45 compilers } conflicts gcc44 {
>   configure.compiler  macports-gcc-4.5
> }
> 
> -variant doc description { Place local documentation into <prefix>/share/doc/eigen3/html } requires configured {
> +variant doc description { Place local documentation into <prefix>/share/doc/eigen3/html } conflicts no_cmake {
>   depends_build-append port:doxygen port:texlive-latex
>   build.target-append doc
> }
> 
> -variant blas description { Build eigen-based blas (libeigen_blas*) } requires configured {
> +variant blas description { Build eigen-based blas (libeigen_blas*) } conflicts no_cmake {
>   if {![variant_isset gcc44] && ![variant_isset gcc45]} {
>     ui_error "eigen3 +blas needs either +gcc44 or +gcc45 enabled!"
>     return -code error
> @@ -48,7 +49,7 @@
>   }
> }
> 
> -variant configured description { Enable configure steps. (Enables 'port test' target.) } {
> +variant no_cmake description { Header-only install: Removes CMake dependency, breaks FindEigen3.cmake } {
> }
> 
> # Phase modifications
> @@ -89,8 +90,8 @@
> }
> 
> 
> -# Are we doing a header-only or configured (for 'port test' or +doc/+blas) build?
> -if {![variant_isset configured]} {
> +# Are we doing a header-only or cmake (for 'port test' or +doc/+blas) build?
> +if {[variant_isset no_cmake]} {
>   # We can skip everything and just copy the headers -- fast!
>   use_configure       no
>   build {}
> @@ -104,7 +105,7 @@
>   # Fail with message if user tries to test
>   test.run            yes
>   test {
> -    ui_error "!!! TESTING IS UNSUPPORTED WITHOUT +configured VARIANT !!!"
> +    ui_error "!!! TESTING IS UNSUPPORTED WITH +no_cmake VARIANT !!!"
>   }
> } else {
>   PortGroup           muniversal 1.0
> _______________________________________________
> macports-changes mailing list
> macports-changes at lists.macosforge.org
> http://lists.macosforge.org/mailman/listinfo.cgi/macports-changes