Skip to content

fix git link: use CURL_LIBCURL for static deps#66

Merged
adeebshihadeh merged 3 commits intomasterfrom
fix-git-link-v2
Mar 24, 2026
Merged

fix git link: use CURL_LIBCURL for static deps#66
adeebshihadeh merged 3 commits intomasterfrom
fix-git-link-v2

Conversation

@adeebshihadeh
Copy link
Copy Markdown
Contributor

@adeebshihadeh adeebshihadeh commented Mar 24, 2026

No description provided.

adeebshihadeh and others added 3 commits March 23, 2026 20:34
git-remote-http was failing to link on ARM64 with undefined
references to dlopen/dlsym from libcrypto.a. The issue was that
CURL_LDFLAGS doesn't affect the CURL_LIBCURL variable that git
uses to link git-remote-http. Use CURL_LIBCURL directly so all
static deps (-lssl -lcrypto -lz -lpthread -ldl) are included.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Two issues:
1. ARM64: git-imap-send links openssl directly (not via curl), so
   CURL_LIBCURL alone isn't enough. Override OPENSSL_LIBSSL and
   LIB_4_CRYPTO to include transitive deps (-ldl, -lpthread).
2. macOS: autoreconf (needed by curl build) wasn't installed.
   Add autoconf/automake/libtool to brew install.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
curl on macOS uses SCDynamicStoreCopyProxies from the
SystemConfiguration framework for proxy detection.
Add -framework SystemConfiguration/Security/CoreFoundation
to the link flags on Darwin.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@adeebshihadeh adeebshihadeh merged commit 2f5b981 into master Mar 24, 2026
12 checks passed
@adeebshihadeh adeebshihadeh deleted the fix-git-link-v2 branch March 24, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant