diff options
author | Ben Gamari <ben@smart-cactus.org> | 2022-08-08 09:53:54 -0400 |
---|---|---|
committer | Marge Bot <ben+marge-bot@smart-cactus.org> | 2022-08-09 02:31:49 -0400 |
commit | 681aa076259c05c626266cf516de7e7c5524eadb (patch) | |
tree | 09e4cf216588919c485769086512b4a45f9b1e8a /hadrian | |
parent | 1c582f44e41f534a8506a76618f6cffe5d71ed42 (diff) | |
download | haskell-681aa076259c05c626266cf516de7e7c5524eadb.tar.gz |
hadrian: Fix access mode of installed package registration files
Previously hadrian's bindist Makefile would modify package
registrations placed by `install` via a shell pipeline and `mv`.
However, the use of `mv` means that if umask is set then the user may
otherwise end up with package registrations which are inaccessible.
Fix this by ensuring that the mode is 0644.
Diffstat (limited to 'hadrian')
-rw-r--r-- | hadrian/bindist/Makefile | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/hadrian/bindist/Makefile b/hadrian/bindist/Makefile index 9bf9ad119f..5159558c5f 100644 --- a/hadrian/bindist/Makefile +++ b/hadrian/bindist/Makefile @@ -46,6 +46,8 @@ define patchpackageconf \ ((echo "$1" | grep rts) && (cat '$2.copy' | sed 's|haddock-.*||' > '$2.copy.copy')) || (cat '$2.copy' > '$2.copy.copy') # We finally replace the original file. mv '$2.copy.copy' '$2' + # Fix the mode, in case umask is set + chmod 644 '$2' endef # QUESTION : should we use shell commands? |