From ceda5fb46ccae8b2942d962c58a9011a56742e6e Mon Sep 17 00:00:00 2001 From: Harmen Stoppels Date: Thu, 11 Aug 2022 16:33:08 +0200 Subject: Don't set `LD_LIBRARY_PATH` by default on Linux (#28354) `LD_LIBRARY_PATH` can break system executables (e.g., when an enviornment is loaded) and isn't necessary thanks to `RPATH`s. Packages that require `LD_LIBRARY_PATH` can set this in `setup_run_environment`. - [x] Prefix inspections no longer set `LD_LIBRARY_PATH` by default - [x] Document changes and workarounds for people who want `LD_LIBRARY_PATH` --- etc/spack/defaults/cray/modules.yaml | 7 +------ etc/spack/defaults/linux/modules.yaml | 7 +------ 2 files changed, 2 insertions(+), 12 deletions(-) (limited to 'etc') diff --git a/etc/spack/defaults/cray/modules.yaml b/etc/spack/defaults/cray/modules.yaml index a86a4794f1..a80f87b16a 100644 --- a/etc/spack/defaults/cray/modules.yaml +++ b/etc/spack/defaults/cray/modules.yaml @@ -13,9 +13,4 @@ # Per-user settings (overrides default and site settings): # ~/.spack/modules.yaml # ------------------------------------------------------------------------- -modules: - prefix_inspections: - lib: - - LD_LIBRARY_PATH - lib64: - - LD_LIBRARY_PATH +modules: {} diff --git a/etc/spack/defaults/linux/modules.yaml b/etc/spack/defaults/linux/modules.yaml index a86a4794f1..a80f87b16a 100644 --- a/etc/spack/defaults/linux/modules.yaml +++ b/etc/spack/defaults/linux/modules.yaml @@ -13,9 +13,4 @@ # Per-user settings (overrides default and site settings): # ~/.spack/modules.yaml # ------------------------------------------------------------------------- -modules: - prefix_inspections: - lib: - - LD_LIBRARY_PATH - lib64: - - LD_LIBRARY_PATH +modules: {} -- cgit v1.2.3-60-g2f50