hlrs-software-stack/spack
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Cgns fix no fortran config (#40241)

Browse source 
* SEACAS: Update package.py to handle new SEACAS project name

The base project name for the SEACAS project has changed from
"SEACASProj" to "SEACAS" as of @2022-10-14, so the package
needed to be updated to use the new project name when needed.

The refactor also changes several:
    "-DSome_CMAKE_Option:BOOL=ON"
to
   define("Some_CMAKE_Option", True)

* CGNS: If fortran not enabled, do not specify parallel fortran compiler

* Update package formatting as suggested by black

* Accept suggested change
This commit is contained in:
Greg Sjaardema 2023-10-02 01:03:47 -06:00 committed by GitHub
parent 71c5b948d0
commit e60bbd1bfc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
var/spack/repos/builtin/packages/cgns/package.py
View file

@ -87,6 +87,7 @@ def cmake_args(self):
self.define_from_variant("CGNS_ENABLE_LEGACY", "legacy"),
self.define_from_variant("CGNS_ENABLE_MEM_DEBUG", "mem_debug"),
self.define_from_variant("CMAKE_POSITION_INDEPENDENT_CODE", "pic"),
self.define_from_variant("CGNS_ENABLE_64BIT", "int64"),
]
)
@ -95,11 +96,10 @@ def cmake_args(self):
[
"-DCMAKE_C_COMPILER=%s" % spec["mpi"].mpicc,
"-DCMAKE_CXX_COMPILER=%s" % spec["mpi"].mpicxx,
"-DCMAKE_Fortran_COMPILER=%s" % spec["mpi"].mpifc,
]
)
options.append(self.define_from_variant("CGNS_ENABLE_64BIT", "int64"))
if "+fortran" in spec:
options.append(self.define("CMAKE_Fortran_COMPILER", spec["mpi"].mpifc))
if "+hdf5" in spec:
options.extend(