Merge pull request #19218 from Homebrew/artifact_installer_sudo

MikeMcQuaid · web-flow · commit 47ef1a3b0b74 · 2025-02-03T16:17:24.000Z
cask/artifact/installer: don't reset uid if sudo is used.
diff --git a/Library/Homebrew/cask/artifact/installer.rb b/Library/Homebrew/cask/artifact/installer.rb
@@ -30,7 +30,7 @@ def install_phase(command: nil, **_)
             env:       { "PATH" => PATH.new(
               HOMEBREW_PREFIX/"bin", HOMEBREW_PREFIX/"sbin", ENV.fetch("PATH")
             ) },
-            reset_uid: true,
+            reset_uid: !args[:sudo],
           )
         end
       end

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ def install_phase(command: nil, **_)`
`30`	`30`	`env: { "PATH" => PATH.new(`
`31`	`31`	`HOMEBREW_PREFIX/"bin", HOMEBREW_PREFIX/"sbin", ENV.fetch("PATH")`
`32`	`32`	`) },`
`33`		`- reset_uid: true,`
	`33`	`+ reset_uid: !args[:sudo],`
`34`	`34`	`)`
`35`	`35`	`end`
`36`	`36`	`end`