From 75e86a61fef9ad020839e49ff7785bb1bc41f5af Mon Sep 17 00:00:00 2001 From: Sergey Kosukhin Date: Wed, 14 Nov 2018 16:40:16 +0100 Subject: [PATCH] netcdf-fortran: add LDFLAGS to fix contents of libnetcdff.la. (#9647) --- .../repos/builtin/packages/netcdf-fortran/package.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/var/spack/repos/builtin/packages/netcdf-fortran/package.py b/var/spack/repos/builtin/packages/netcdf-fortran/package.py index d622ef025a3..92572bb5aa7 100644 --- a/var/spack/repos/builtin/packages/netcdf-fortran/package.py +++ b/var/spack/repos/builtin/packages/netcdf-fortran/package.py @@ -28,9 +28,16 @@ def flag_handler(self, name, flags): if name in ['cflags', 'fflags'] and '+pic' in self.spec: flags.append(self.compiler.pic_flag) elif name == 'cppflags': - flags.append('-I' + self.spec['netcdf'].prefix.include) + flags.append(self.spec['netcdf'].headers.cpp_flags) + elif name == 'ldflags': + # We need to specify LDFLAGS to get correct dependency_libs + # in libnetcdff.la, so packages that use libtool for linking + # could correctly link to all the dependencies even when the + # building takes place outside of Spack environment, i.e. + # without Spack's compiler wrappers. + flags.append(self.spec['netcdf'].libs.search_flags) - return (None, None, flags) + return None, None, flags @property def libs(self):