Simplify TIOGA package (#33396)

* Update TIOGA package.

* Add comment.

* Remove cuda variant and MPI_ROOT.

* Style.
This commit is contained in:
Jon Rood 2022-10-19 14:33:01 -06:00 committed by GitHub
parent 4b0832d3bc
commit ae7999d7a1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -7,15 +7,17 @@
from spack.package import *
class Tioga(CMakePackage, CudaPackage):
class Tioga(CMakePackage):
"""Topology Independent Overset Grid Assembly (TIOGA)"""
homepage = "https://github.com/jsitaraman/tioga"
git = "https://github.com/jsitaraman/tioga.git"
maintainers = ["jsitaraman", "sayerhs"]
maintainers = ["jrood-nrel"]
version("develop", branch="exawind")
# The original TIOGA repo has possibly been abandoned,
# so work on TIOGA has continued in the Exawind project
version("develop", git="https://github.com/Exawind/tioga.git", branch="exawind")
version("master", branch="master")
variant("shared", default=sys.platform != "darwin", description="Build shared libraries")
@ -23,39 +25,16 @@ class Tioga(CMakePackage, CudaPackage):
variant("nodegid", default=True, description="Enable support for global Node IDs")
variant("timers", default=False, description="Enable timers")
variant("stats", default=False, description="Enable output of holecut stats")
variant(
"cxxstd", default="11", values=("11", "14"), multi=False, description="C++ standard to use"
)
depends_on("mpi")
depends_on("cuda@9.0.0:", when="+cuda")
# Tioga has the fortran module file problem with parallel builds
parallel = False
def cmake_args(self):
spec = self.spec
args = [
self.define_from_variant("BUILD_SHARED_LIBS", "shared"),
self.define_from_variant("CMAKE_POSITION_INDEPENDENT_CODE", "pic"),
self.define_from_variant("CMAKE_CXX_STANDARD", "cxxstd"),
self.define_from_variant("TIOGA_HAS_NODEGID", "nodegid"),
self.define_from_variant("TIOGA_ENABLE_TIMERS", "timers"),
self.define_from_variant("TIOGA_OUTPUT_STATS", "stats"),
self.define_from_variant("TIOGA_ENABLE_CUDA", "cuda"),
]
if "+cuda" in self.spec:
args.append(self.define("CMAKE_CUDA_SEPARABLE_COMPILATION", True))
# Currently TIOGA only supports one device arch during specialization
cuda_arch = self.spec.variants["cuda_arch"].value
if cuda_arch:
arch_sorted = list(sorted(cuda_arch, reverse=True))
args.append(self.define("TIOGA_CUDA_SM", arch_sorted[0]))
if "darwin" in spec.architecture:
args.append(self.define("CMAKE_MACOSX_RPATH", True))
return args