From f1fbcfa9713b78792223c808607483eb61ef2a24 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Fri, 22 Sep 2017 21:14:53 +0200 Subject: plumed: fixed possible linking issue (#5425) Due to the logic in configure.ac, we may be using both the system linker (LD_RO='ld -r -o') and the linker tied to the compiler (LD=$CXX). This may produce weird link-time errors when the two executables are not the same. The PR works around the issue disabling 'ld -r' --- var/spack/repos/builtin/packages/plumed/package.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'var') diff --git a/var/spack/repos/builtin/packages/plumed/package.py b/var/spack/repos/builtin/packages/plumed/package.py index 6896cbeb77..078af3a24a 100644 --- a/var/spack/repos/builtin/packages/plumed/package.py +++ b/var/spack/repos/builtin/packages/plumed/package.py @@ -148,7 +148,12 @@ class Plumed(AutotoolsPackage): # with MPI you should use: # # > ./configure CXX="$MPICXX" - configure_opts = [] + + # The configure.ac script may detect the wrong linker for + # LD_RO which causes issues at link time. Here we work around + # the issue saying we have no LD_RO executable. + configure_opts = ['--disable-ld-r'] + # If using MPI then ensure the correct compiler wrapper is used. if '+mpi' in spec: configure_opts.extend([ -- cgit v1.2.3-70-g09d2