diff options
| author | Harmen Stoppels <harmenstoppels@gmail.com> | 2023-07-19 11:48:31 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-07-19 09:48:31 +0000 |
| commit | 886946395d33356e2073f4f2112514c366871958 (patch) | |
| tree | dc1a662e1a76fe5e2b4abde893e32ebc3ced74e5 /NOTICE | |
| parent | 57b69c97033431aa24e386d927f25c520dd87a4e (diff) | |
| download | spack-886946395d33356e2073f4f2112514c366871958.tar.gz spack-886946395d33356e2073f4f2112514c366871958.tar.bz2 spack-886946395d33356e2073f4f2112514c366871958.tar.xz spack-886946395d33356e2073f4f2112514c366871958.zip | |
drop redundant rpaths post install (#38976)
Spack heuristically adds `<install prefix>/lib` and `<install prefix>/lib64` as rpath entries, as it doesn't know what the install dir is going to be ahead of the build. This PR cleans up non-existing, absolute paths[^1], which
1. avoids redundant stat calls at runtime
2. drops redundant rpaths in `patchelf`, making it relocatable -- you don't need patchelf recursively then.
[^1]: It also removes relative paths not starting with `$` (so, `$ORIGIN/../lib` is retained -- we _could_ interpolate `$ORIGIN`, but that's hard to get right when symlinks have to be taken into account). Relative paths _are_ supported in glibc, but are relative to _the current working directory_, which is madness, and it would be better to drop those paths.
Diffstat (limited to 'NOTICE')
0 files changed, 0 insertions, 0 deletions