admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Remove atomic_write workaround.

Browse Source
This commit is contained in:
Markus Reiter 2018-11-22 23:37:57 +01:00
parent 7129a4852a
commit bcf8cfc97c
1 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Library/Homebrew/extend/pathname.rb
View File

@ -162,18 +162,8 @@ class Pathname
# NOTE: This always overwrites. # NOTE: This always overwrites.
def atomic_write(content) def atomic_write(content)
# The enclosing `mktmpdir` and the `chmod` are a workaround File.atomic_write(self) do |file|
# for https://github.com/rails/rails/pull/34037. file.write(content)
Dir.mktmpdir(".d", dirname) do |tmpdir|
should_fix_sticky_bit = dirname.world_writable? && !dirname.sticky?
FileUtils.chmod "+t", dirname if should_fix_sticky_bit
begin
File.atomic_write(self, tmpdir) do |file|
file.write(content)
end
ensure
FileUtils.chmod "-t", dirname if should_fix_sticky_bit
end
end end
end end