Make sure perl doesn't run into recompile with fPIC linker errors (#25834)

1 files changed, 6 insertions, 2 deletions

diff --git a/var/spack/repos/builtin/packages/perl/package.py b/var/spack/repos/builtin/packages/perl/package.py
index 7ca4b70fcc..670bc0f626 100644
--- a/var/spack/repos/builtin/packages/perl/package.py
+++ b/var/spack/repos/builtin/packages/perl/package.py
@@ -68,8 +68,8 @@ class Perl(Package):  # Perl doesn't use Autotools, it should subclass Package
     # :5.28 needs gdbm@:1:14.1: https://rt-archive.perl.org/perl5/Ticket/Display.html?id=133295
     depends_on('gdbm@:1.14.1', when='@:5.28.0')
     depends_on('berkeley-db')
-    depends_on('bzip2+shared')
-    depends_on('zlib+shared')
+    depends_on('bzip2')
+    depends_on('zlib')
     # :5.24.1 needs zlib@:1.2.8: https://rt.cpan.org/Public/Bug/Display.html?id=120134
     depends_on('zlib@:1.2.8', when='@5.20.3:5.24.1')
 
@@ -102,6 +102,10 @@ class Perl(Package):  # Perl doesn't use Autotools, it should subclass Package
     conflicts('@5.32.0:', when='%nvhpc@:20.11',
               msg='The NVIDIA compilers are incompatible with version 5.32 and later')
 
+    # Make sure we don't get "recompile with -fPIC" linker errors when using static libs
+    conflicts('^zlib~shared~pic', msg='Needs position independent code when using static zlib')
+    conflicts('^bzip2~shared~pic', msg='Needs position independent code when using static bzip2')
+
     # Installing cpanm alongside the core makes it safe and simple for
     # people/projects to install their own sets of perl modules.  Not
     # having it in core increases the "energy of activation" for doing