[142729] trunk/dports/science/alps/Portfile

[dstrubbe at macports.org]

Revision: 142729
          https://trac.macports.org/changeset/142729
Author:   dstrubbe at macports.org
Date:     2015-11-20 20:02:18 -0800 (Fri, 20 Nov 2015)
Log Message:
-----------
alps: The openmpi variant did nothing: a dependency was added and MPI was requested, but the appropriate configuration variables for the compilers were not set, so MPI was not found and used. Now MPI variants are supported through the mpi portgroup, and consistency for HDF5 is checked. clang will not be used since it does not work for MPI or +applications in this code. gcc is set as default compiler. openmaintainer is added, as per the guide: "Port maintainers who are not committers are encouraged to add <openmaintainer> to their ports."

Maintainer time-out. Ticket #49720.

Modified Paths:
--------------
    trunk/dports/science/alps/Portfile

Modified: trunk/dports/science/alps/Portfile
===================================================================
--- trunk/dports/science/alps/Portfile	2015-11-21 02:14:05 UTC (rev 142728)
+++ trunk/dports/science/alps/Portfile	2015-11-21 04:02:18 UTC (rev 142729)
@@ -4,6 +4,7 @@
 PortSystem              1.0
 PortGroup               cmake 1.0
 PortGroup               conflicts_build 1.0
+PortGroup               mpi 1.0
 
 name                    alps
 version                 2.1.1
@@ -12,7 +13,7 @@
 platforms               darwin
 license                 Restrictive
 # http://alps.comp-phys.org/static/software/ALPS/LICENSE.txt/
-maintainers             gmail.com:gamperl
+maintainers             gmail.com:gamperl openmaintainer
 
 description             Algorithms and Libraries for Physics Simulations
 
@@ -31,9 +32,19 @@
 
 depends_lib             port:hdf5
 
+mpi.enforce_variant     hdf5
+
 # alps has its own internal boost it wants to use, and fails if MacPorts' newer boost is active.
 conflicts_build         boost
 
+compilers.choose        cc cxx
+# clang doesn't work for +applications or with MPI
+mpi.setup               -clang
+# replace ancient llvm we would otherwise be using by default
+if {![c_variant_isset]} {
+    default_variants-append +${compilers.gcc_default}
+}
+
 patchfiles              patch-CMakeLists.txt.diff \
                         patch-config-FindLapack.cmake.diff
 
@@ -61,18 +72,14 @@
     # alps/applications/dmrg/dmrg/dmrg.h:610:49: error: expected expression
     #     std::string name = simplify_name(it->get<1>());
     #                                                 ^
-    compiler.blacklist  *clang*
-    
     configure.args-delete -DALPS_BUILD_APPLICATIONS=OFF
 }
 
-# why not mpich also as a variant?
-variant openmpi         description {Build with MPI support} {
-    depends_lib-append  port:openmpi
-    
-    configure.args-delete -DALPS_ENABLE_MPI=OFF    
-#-- Could NOT find MPI_C (missing:  MPI_C_LIBRARIES MPI_C_INCLUDE_PATH)
-#-- Could NOT find MPI_CXX (missing:  MPI_CXX_LIBRARIES MPI_CXX_INCLUDE_PATH)
+pre-configure {
+    if {[mpi_variant_isset]} {
+        configure.args-delete -DALPS_ENABLE_MPI=OFF    
+        configure.args-append -DMPI_C=${mpi.cc} -DMPI_CXX=${mpi.cxx}
+    }
 }
 
 set pythons_suffixes {26 27}
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20151212/31ba0659/attachment.html>