diff options
author | Harmen Stoppels <harmenstoppels@gmail.com> | 2022-11-08 20:05:05 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-11-08 11:05:05 -0800 |
commit | 4d28a6466188ea9fe3b55a4c3d7690dd66e0dc8f (patch) | |
tree | f1e1cf3f466e5ee21f109c5583ff481c8aace294 /var | |
parent | 4a5e68816bdb83d7b614e586042a1927b7267576 (diff) | |
download | spack-4d28a6466188ea9fe3b55a4c3d7690dd66e0dc8f.tar.gz spack-4d28a6466188ea9fe3b55a4c3d7690dd66e0dc8f.tar.bz2 spack-4d28a6466188ea9fe3b55a4c3d7690dd66e0dc8f.tar.xz spack-4d28a6466188ea9fe3b55a4c3d7690dd66e0dc8f.zip |
fix racy sbang (#33549)
Spack currently creates a temporary sbang that is moved "atomically" in place,
but this temporary causes races when multiple processes start installing sbang.
Let's just stick to an idempotent approach. Notice that we only re-install sbang
if Spack updates it (since we do file compare), and sbang was only touched
18 times in the past 6 years, whereas we hit the sbang tempfile issue
frequently with parallel install on a fresh spack instance in CI.
Also fixes a bug where permissions weren't updated if config changed but
the latest version of the sbang file was already installed.
Diffstat (limited to 'var')
0 files changed, 0 insertions, 0 deletions