From 4eb3558d203f39a7cf7a1f5847163f51fe91cd28 Mon Sep 17 00:00:00 2001
From: Robert Underwood <robertu94@users.noreply.github.com>
Date: Tue, 4 Aug 2020 10:22:53 -0700
Subject: [PATCH] Prefer dynamic linking for Python in vim when +python
(#17870)
Previously the python package for vim used static linking, and depending
on what system libraries were available and linked against could cause
symbol conflicts for python leading to segfaults in loading c modules in
the standard library (i.e. heapq). This patch address this issue by
dynamically linking them.
---
var/spack/repos/builtin/packages/vim/package.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/var/spack/repos/builtin/packages/vim/package.py b/var/spack/repos/builtin/packages/vim/package.py
index d1533cfe52..722fe46a67 100644
--- a/var/spack/repos/builtin/packages/vim/package.py
+++ b/var/spack/repos/builtin/packages/vim/package.py
@@ -92,11 +92,11 @@ def configure_args(self):
if '+python' in spec:
if 'python@3:' in self.spec:
- configure_args.append("--enable-python3interp=yes")
+ configure_args.append("--enable-python3interp=dynamic")
configure_args.append("--enable-pythoninterp=no")
else:
configure_args.append("--enable-python3interp=no")
- configure_args.append("--enable-pythoninterp=yes")
+ configure_args.append("--enable-pythoninterp=dynamic")
else:
configure_args.append("--enable-python3interp=no")
--
GitLab