diff options
author | Daniel Gröber <dxld@darkboxed.org> | 2021-09-19 22:10:22 +0200 |
---|---|---|
committer | Marge Bot <ben+marge-bot@smart-cactus.org> | 2022-02-10 19:18:58 -0500 |
commit | 2a47ee9c65140ad5d72ff27949f3fc7948569d9e (patch) | |
tree | 3496c6ae1209ff7627ad7ae4156c45eed827933e /rules | |
parent | 2a6f2681ad53899869473343e845bee189a809c3 (diff) | |
download | haskell-2a47ee9c65140ad5d72ff27949f3fc7948569d9e.tar.gz |
ghc-boot: Simplify writePackageDb permissions handling
Commit ef8a3fbf1 ("ghc-boot: Fix metadata handling of writeFileAtomic")
introduced a somewhat over-engineered fix for #14017 by trying to preserve
the current permissions if the target file already exists.
The problem in the issue is simply that the package db cache file should be
world readable but isn't if umask is too restrictive. In fact the previous
fix only handles part of this problem. If the file isn't already there in a
readable configuration it wont make it so which isn't really ideal either.
Rather than all that we now simply always force all the read access bits to
allow access while leaving the owner at the system default as it's just not
our business to mess with it.
Diffstat (limited to 'rules')
0 files changed, 0 insertions, 0 deletions