ci: drop quarantine removal for macos perforce binaries

dilim/git

0 Fork 0

mirror of https://github.com/git/git.git synced 2026-11-06 13:28:06 +00:00

The xattr -d call that strips com.apple.quarantine from the macOS
perforce binaries has been doing nothing for years, and can simply
go.

5ed9fc3fc8 (ci: prevent `perforce` from being quarantined,
2020-02-27) added that call to stop p4d -V from hanging on a GUI
confirmation dialog that never comes on a CI box, back when perforce
was installed via Homebrew's --no-quarantine cask flag. When
49af448197 (ci: reintroduce prevention from perforce being
quarantined in macOS, 2022-05-12) replaced that with a plain wget
and tar download, it kept the xattr -d call as a precaution, while
noting that it already does nothing on GitHub Actions.

A file downloaded with wget never gets com.apple.quarantine set in
the first place, only Safari, Finder, and AirDrop apply that
attribute through the LaunchServices quarantine API. The same holds
on GitLab CI's macOS runners, so the call has been a silent no-op
everywhere since 2022.

Signed-off-by: Harald Nordgren <haraldnordgren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

This commit is contained in:

Harald Nordgren authored and

Junio C Hamano committed 2026-10-27 07:31:05 -07:00

1 parent a018953688

commit 03ddaacda7

1 file changed

-1

									
										ci/install-dependencies.sh
									
		-1
	
												View File
												
				@@ -122,7 +122,6 @@ macos-*)

					mkdir -p "$CUSTOM_PATH"

					wget -q "$P4WHENCE/bin.macosx12arm64/helix-core-server.tgz" &&

					tar -xf helix-core-server.tgz -C "$CUSTOM_PATH" p4 p4d &&

					sudo xattr -d com.apple.quarantine "$CUSTOM_PATH/p4" "$CUSTOM_PATH/p4d" 2>/dev/null || true

					rm helix-core-server.tgz

					case "$jobname" in