From f223fef21882c2e7f0a7d723736de0006b20e210 Mon Sep 17 00:00:00 2001
From: Michael Cho <michael@michaelcho.dev>
Date: Fri, 11 Oct 2024 13:21:43 -0400
Subject: [PATCH] unpack_strategy: fix unpack Dir.mktmpdir group

Otherwise the files will be unpacked with group wheel on macOS which
gets copied over when running `cp -al`.
---
 Library/Homebrew/unpack_strategy.rb | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/Library/Homebrew/unpack_strategy.rb b/Library/Homebrew/unpack_strategy.rb
index e234442d9ecff..8f102c7509a2d 100644
--- a/Library/Homebrew/unpack_strategy.rb
+++ b/Library/Homebrew/unpack_strategy.rb
@@ -160,6 +160,27 @@ def extract_nestedly(to: nil, basename: nil, verbose: false, prioritize_extensio
     Dir.mktmpdir("homebrew-unpack", HOMEBREW_TEMP) do |tmp_unpack_dir|
       tmp_unpack_dir = Pathname(tmp_unpack_dir)
 
+      # Make sure files inside the temporary directory have the same group as the brew instance.
+      #
+      # @see https://github.com/Homebrew/brew/blob/4.4.0/Library/Homebrew/mktemp.rb#L57-L72
+      group_id = if HOMEBREW_BREW_FILE.grpowned?
+        HOMEBREW_BREW_FILE.stat.gid
+      else
+        Process.gid
+      end
+      begin
+        tmp_unpack_dir.chown(nil, group_id)
+      rescue Errno::EPERM
+        require "etc"
+        group_name = begin
+          Etc.getgrgid(group_id)&.name
+        rescue ArgumentError
+          # Cover for misconfigured NSS setups
+          nil
+        end
+        opoo "Failed setting group \"#{group_name || group_id}\" on #{tmp_unpack_dir}"
+      end
+
       extract(to: tmp_unpack_dir, basename:, verbose:)
 
       children = tmp_unpack_dir.children