From 367abcb8013ad4798487dba753a82ae02d077a90 Mon Sep 17 00:00:00 2001 From: Tom Scogland Date: Mon, 6 Nov 2023 11:44:38 -0800 Subject: [PATCH] ignore missing python bindings in llvm 17+ The python bindings to llvm have been removed in version 17, but those in the clang portion of the project still exist. This ignores the now missing llvm bindings but retains the variant to control installing the clang bindings. --- var/spack/repos/builtin/packages/llvm/package.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/llvm/package.py b/var/spack/repos/builtin/packages/llvm/package.py index 383871353fe..d4f07caf392 100644 --- a/var/spack/repos/builtin/packages/llvm/package.py +++ b/var/spack/repos/builtin/packages/llvm/package.py @@ -977,7 +977,9 @@ def post_install(self): ninja() ninja("install") if "+python" in self.spec: - install_tree("llvm/bindings/python", python_platlib) + if spec.version < Version("17.0.0"): + # llvm bindings were removed in v17: https://releases.llvm.org/17.0.1/docs/ReleaseNotes.html#changes-to-the-python-bindings + install_tree("llvm/bindings/python", python_platlib) if "+clang" in self.spec: install_tree("clang/bindings/python", python_platlib)