Skip to content

fix(cli): download ic-admin from GitHub releases#2039

Draft
r-birkner wants to merge 3 commits into
mainfrom
rjb/fix-ic-admin-download
Draft

fix(cli): download ic-admin from GitHub releases#2039
r-birkner wants to merge 3 commits into
mainfrom
rjb/fix-ic-admin-download

Conversation

@r-birkner

Copy link
Copy Markdown
Contributor

Summary

  • download.dfinity.systems no longer serves ic-admin binaries; switch to GitHub Releases on dfinity/ic
  • Add find_github_release_tag_for_commit which resolves a commit SHA to a release tag by querying the GitHub matching-refs API (searches from current year back to 2024, with pagination)
  • Update asset naming to match new conventions: ic-admin-{arm64,x86_64}-{darwin,linux}.gz
  • Add proper ARM64 support for both Linux and macOS (Apple Silicon)
  • Update FALLBACK_IC_ADMIN_VERSION to b95f4a32b41798de115aac9298b51dd1662f1da5 (release-2026-06-04_04-52-base), since the old fallback has no GitHub release

Test plan

  • Run dre on a macOS (Apple Silicon) machine and verify ic-admin is downloaded from GitHub
  • Run dre on an x86_64 Linux machine and verify ic-admin is downloaded from GitHub
  • Verify the fallback version resolves correctly to a GitHub release tag

🤖 Generated with Claude Code

r-birkner and others added 3 commits June 8, 2026 10:21
…dfinity.systems

The download.dfinity.systems host no longer serves ic-admin binaries.
Switch to GitHub releases (dfinity/ic), resolving the release tag from
the commit hash via the GitHub matching-refs API. Update asset naming
to match new conventions (ic-admin-{arch}-{os}.gz) and add ARM support.
Update fallback version to one that has a GitHub release.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
rls was removed from Rust in newer versions. rust-analyzer is the
current replacement.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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