diff options
author | Adrien Bernede <51493078+adrienbernede@users.noreply.github.com> | 2024-10-08 05:53:38 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-10-08 05:53:38 +0200 |
commit | a9e9b901d12a4139801b6ef48a8cf3aee334b6f3 (patch) | |
tree | 3a8db65c564b7e6ece0fe9db7bb3e9919f1a0853 | |
parent | 95b46dca3d73c77c13502a6360e552a65daba95a (diff) | |
download | spack-a9e9b901d12a4139801b6ef48a8cf3aee334b6f3.tar.gz spack-a9e9b901d12a4139801b6ef48a8cf3aee334b6f3.tar.bz2 spack-a9e9b901d12a4139801b6ef48a8cf3aee334b6f3.tar.xz spack-a9e9b901d12a4139801b6ef48a8cf3aee334b6f3.zip |
mfem: Apply minor changes (replace ' with ") (#46537)
-rw-r--r-- | var/spack/repos/builtin/packages/mfem/package.py | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/var/spack/repos/builtin/packages/mfem/package.py b/var/spack/repos/builtin/packages/mfem/package.py index df419c1423..b50af840f2 100644 --- a/var/spack/repos/builtin/packages/mfem/package.py +++ b/var/spack/repos/builtin/packages/mfem/package.py @@ -163,11 +163,11 @@ class Mfem(Package, CudaPackage, ROCmPackage): variant("static", default=True, description="Build static library") variant("shared", default=False, description="Build shared library") variant("mpi", default=True, sticky=True, description="Enable MPI parallelism") - # Can we make the default value for 'metis' to depend on the 'mpi' value? + # Can we make the default value for "metis" to depend on the "mpi" value? variant("metis", default=True, sticky=True, description="Enable METIS support") variant("openmp", default=False, description="Enable OpenMP parallelism") - # Note: '+cuda' and 'cuda_arch' variants are added by the CudaPackage - # Note: '+rocm' and 'amdgpu_target' variants are added by the ROCmPackage + # Note: "+cuda" and "cuda_arch" variants are added by the CudaPackage + # Note: "+rocm" and "amdgpu_target" variants are added by the ROCmPackage variant("occa", default=False, description="Enable OCCA backend") variant("raja", default=False, description="Enable RAJA backend") variant("libceed", default=False, description="Enable libCEED backend") @@ -417,7 +417,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): depends_on("occa@1.0.8:", when="@:4.1+occa") depends_on("occa@1.1.0", when="@4.2.0:+occa") depends_on("occa+cuda", when="+occa+cuda") - # TODO: propagate '+rocm' variant to occa when it is supported + # TODO: propagate "+rocm" variant to occa when it is supported depends_on("raja@0.7.0:0.9.0", when="@4.0.0+raja") depends_on("raja@0.10.0:0.12.1", when="@4.0.1:4.2.0+raja") @@ -579,8 +579,8 @@ class Mfem(Package, CudaPackage, ROCmPackage): "PREFIX=%s" % prefix, "MFEM_USE_MEMALLOC=YES", "MFEM_DEBUG=%s" % yes_no("+debug"), - # NOTE: env['CXX'] is the spack c++ compiler wrapper. The real - # compiler is defined by env['SPACK_CXX']. + # NOTE: env["CXX"] is the spack c++ compiler wrapper. The real + # compiler is defined by env["SPACK_CXX"]. "CXX=%s" % env["CXX"], "MFEM_USE_LIBUNWIND=%s" % yes_no("+libunwind"), "%s=%s" % (zlib_var, yes_no("+zlib")), @@ -764,7 +764,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): elif "^spectrum-mpi" in strumpack: sp_lib += [ld_flags_from_dirs([mpi.prefix.lib], ["mpi_ibm_mpifh"])] if "+openmp" in strumpack: - # The '+openmp' in the spec means strumpack will TRY to find + # The "+openmp" in the spec means strumpack will TRY to find # OpenMP; if not found, we should not add any flags -- how do # we figure out if strumpack found OpenMP? if not self.spec.satisfies("%apple-clang"): @@ -799,7 +799,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): ) sp_lib += [ld_flags_from_library_list(zfp_lib)] if "+cuda" in strumpack: - # assuming also ('+cuda' in spec) + # assuming also ("+cuda" in spec) sp_lib += ["-lcusolver", "-lcublas"] options += [ "STRUMPACK_OPT=%s" % " ".join(sp_opt), @@ -914,7 +914,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): headers = find_headers("libunwind", libunwind.prefix.include) headers.add_macro("-g") libs = find_optional_library("libunwind", libunwind.prefix) - # When mfem uses libunwind, it also needs 'libdl'. + # When mfem uses libunwind, it also needs "libdl". libs += LibraryList(find_system_libraries("libdl")) options += [ "LIBUNWIND_OPT=%s" % headers.cpp_flags, @@ -1300,7 +1300,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): @property def config_mk(self): """Export the location of the config.mk file. - This property can be accessed using spec['mfem'].package.config_mk + This property can be accessed using spec["mfem"].package.config_mk """ dirs = [self.prefix, self.prefix.share.mfem] for d in dirs: @@ -1312,7 +1312,7 @@ class Mfem(Package, CudaPackage, ROCmPackage): @property def test_mk(self): """Export the location of the test.mk file. - This property can be accessed using spec['mfem'].package.test_mk. + This property can be accessed using spec["mfem"].package.test_mk. In version 3.3.2 and newer, the location of test.mk is also defined inside config.mk, variable MFEM_TEST_MK. """ |