<pre style='margin:0'>
David Strubbe (dstrubbe) pushed a commit to branch master
in repository macports-ports.

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/37160e466d7a72e51d01d13125ca22923b5e6f4c">https://github.com/macports/macports-ports/commit/37160e466d7a72e51d01d13125ca22923b5e6f4c</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 37160e4  octopus: Update to version 8.1.
</span>37160e4 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 37160e466d7a72e51d01d13125ca22923b5e6f4c
</span>Author: David Strubbe <dstrubbe@macports.org>
AuthorDate: Fri Jul 27 14:47:14 2018 -0700

<span style='display:block; white-space:pre;color:#404040;'>    octopus: Update to version 8.1.
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Since clang is not working, remove +gfortran so that a gcc C/C++ always is used. Remove unnecessary checks for MPI variant with parallel dependencies -- mpi.enforce_variant handles this anyway. Fix residual logic for clang options (for when that can be re-enabled).
</span>---
 science/octopus/Portfile | 44 +++++++++-----------------------------------
 1 file changed, 9 insertions(+), 35 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/science/octopus/Portfile b/science/octopus/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index c163e43..4e4beb4 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/science/octopus/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/science/octopus/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -5,7 +5,7 @@ PortGroup           mpi 1.0
</span> PortGroup           linear_algebra 1.0
 
 name                octopus
<span style='display:block; white-space:pre;background:#ffe0e0;'>-version             8.0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+version             8.1
</span> categories          science
 platforms           darwin
 license             GPL-2+
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -21,12 +21,12 @@ long_description    Octopus is a scientific program aimed at ab initio virtual e
</span> homepage            http://www.tddft.org/programs/octopus
 master_sites        ${homepage}/download/${version}
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-checksums           rmd160  967ab8a9d1a7c1e45703eaa7c012d9519def0d13 \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    sha256  0aca270e22895fa2d1c5b8034389e21ff22697aed28e4a6be36f11e943bda0df \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    size    49312804
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+checksums           rmd160  89b1d3cf161d433c38539c1ee7a566324ac7be26 \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    sha256  20d84142e8d4356c9113b15b0d0cf561f34c381a9962c8a1b5c40c212edbbc67 \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                    size    49316675
</span> 
 compilers.choose    fc cc cxx
<span style='display:block; white-space:pre;background:#ffe0e0;'>-mpi.setup           require_fortran -g95 -clang
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+mpi.setup           require_fortran -g95 -clang -gfortran
</span> #-clang33 -clang34 -clang38
 # not clear if clang33 and clang34 work; clang38 gives cxx11 error as does default clang.
 # clang fails in tests, relating to pseudopotentials and XML
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -61,11 +61,12 @@ if {[mpi_variant_isset]} {
</span> pre-configure {
     configure.args-append --with-blas="-L${prefix}/lib ${linalglib}"
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    # not necessary unless mixing clang++ and gfortran;
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # found automatically when using MPI wrappers
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[clang_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if {![gcc_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # needed for clang; clang_variant_isset would miss case of default clang compiler.
</span>         configure.cxxflags-append "-std=c++11"
     }
<span style='display:block; white-space:pre;background:#e0ffe0;'>+    # not necessary unless mixing clang++ and gfortran;
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    # found automatically when using MPI wrappers
</span>     if {![gcc_variant_isset] && ![mpi_variant_isset]} {
         configure.ldflags-append "-lc++"
     }
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -121,18 +122,6 @@ variant scalapack description {Build with ScaLAPACK for parallel linear algebra}
</span>     configure.args-delete   --with-scalapack=no
     configure.args-append   --with-blacs=${prefix}/lib/libscalapack.dylib
     depends_lib-append      port:scalapack
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {![mpi_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        ui_error "+scalapack requires an MPI variant. Choose +mpich, +mpich_devel, +openmpi, or +openmpi_devel."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        return -code error "+scalapack requires an MPI variant."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    mpi.setup               require
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # FIXME: this unnecessarily enforces scalapack had same C compiler as we are using.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # also, if +scalapack but no MPI is used, an internal error from active_variants
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # is produced rather than saying you need MPI or just adding it to variants as intended.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # Error: active_variants: Error: invalid port depspec ''
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # Error:   expecting either: port or (bin|lib|path):foo:port
</span>     mpi.enforce_variant     scalapack
 }
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -141,11 +130,6 @@ variant scalapack description {Build with ScaLAPACK for parallel linear algebra}
</span> #    depends_lib-append      port:pfft
 #    configure.args-delete   --with-pfft-prefix=no
 #    configure.args-append   --with-pfft-prefix=${prefix} --with-mpifftw-prefix=${prefix}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#    if {![mpi_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#        ui_error "+pfft requires an MPI variant. Choose +mpich, +mpich_devel, +openmpi, or +openmpi_devel."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#        return -code error "+pfft requires an MPI variant."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-#    }
</span> #    mpi.enforce_variant     pfft
 #}
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -158,11 +142,6 @@ variant arpack description {Build with ARPACK for complex-scaling calculations}
</span> variant parpack requires arpack description {Build with PARPACK for complex-scaling calculations} {
     configure.args-delete   --with-parpack=no
     configure.args-append   --with-parpack=${prefix}/lib/libparpack.dylib
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {![mpi_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        ui_error "+parpack requires an MPI variant. Choose +mpich, +mpich_devel, +openmpi, or +openmpi_devel."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        return -code error "+parpack requires an MPI variant."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span>     mpi.enforce_variant     parpack
 }
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -173,11 +152,6 @@ variant parmetis description {Build with ParMETIS for parallel domain decomposit
</span>     configure.args-delete   --with-metis-prefix=no
     configure.args-delete   --with-parmetis-prefix=no
     configure.args-append   --with-metis-prefix=${prefix} --with-parmetis-prefix=${prefix}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {![mpi_variant_isset]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        ui_error "+parmetis requires an MPI variant. Choose +mpich, +mpich_devel, +openmpi, or +openmpi_devel."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        return -code error "+parmetis requires an MPI variant."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span>     mpi.enforce_variant     parmetis
 }
 
</pre><pre style='margin:0'>

</pre>