check spec name
Signed-off-by: Todd Gamblin <tgamblin@llnl.gov>
This commit is contained in:
parent
795335d56e
commit
b14f172a61
@ -12,6 +12,7 @@
|
|||||||
from shutil import copy
|
from shutil import copy
|
||||||
from typing import Dict, List, Optional
|
from typing import Dict, List, Optional
|
||||||
|
|
||||||
|
import llnl.util.filesystem as fs
|
||||||
from llnl.util.lang import dedupe
|
from llnl.util.lang import dedupe
|
||||||
|
|
||||||
from spack.build_environment import dso_suffix, stat_suffix
|
from spack.build_environment import dso_suffix, stat_suffix
|
||||||
@ -49,6 +50,9 @@ def find_python_in_prefix(spec: "spack.spec.Spec", prefix: Optional[str] = None)
|
|||||||
Executable: the Python command
|
Executable: the Python command
|
||||||
|
|
||||||
"""
|
"""
|
||||||
|
if not spec.name == "python":
|
||||||
|
raise ValueError("find_python_in_prefix() only works on Python installations.")
|
||||||
|
|
||||||
prefix = Prefix(prefix or spec.prefix)
|
prefix = Prefix(prefix or spec.prefix)
|
||||||
|
|
||||||
# We need to be careful here. If the user is using an externally
|
# We need to be careful here. If the user is using an externally
|
||||||
@ -67,7 +71,7 @@ def find_python_in_prefix(spec: "spack.spec.Spec", prefix: Optional[str] = None)
|
|||||||
patterns = [f"python{ver}{file_extension}" for ver in suffixes]
|
patterns = [f"python{ver}{file_extension}" for ver in suffixes]
|
||||||
root = prefix.bin if sys.platform != "win32" else prefix
|
root = prefix.bin if sys.platform != "win32" else prefix
|
||||||
|
|
||||||
path = find_first(root, files=patterns)
|
path = fs.find_first(root, files=patterns)
|
||||||
if path is not None:
|
if path is not None:
|
||||||
return Executable(path)
|
return Executable(path)
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user