From 8a0a88705676d1ad03b8ce8874e81befda5a4c0c Mon Sep 17 00:00:00 2001 From: Adrian Ho Date: Sun, 20 Jul 2025 10:24:08 +0800 Subject: [PATCH] env_script_all_files: don't overwrite existing stuff Overwriting upstream content potentially causes major confusion at best, major breakage at worst. --- Library/Homebrew/extend/pathname.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Library/Homebrew/extend/pathname.rb b/Library/Homebrew/extend/pathname.rb index 111a464804..f65ac07a07 100644 --- a/Library/Homebrew/extend/pathname.rb +++ b/Library/Homebrew/extend/pathname.rb @@ -340,8 +340,10 @@ class Pathname Pathname.glob("#{self}/*") do |file| next if file.directory? - dst.install(file) new_file = dst.join(file.basename) + raise Errno::EEXIST, new_file if new_file.exist? + + dst.install(file) file.write_env_script(new_file, env) end end