GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — ci(release): pin newer cross NDK image for aarch64-linux-android by mikolalysenko · Pull Request #86 · SocketDev/socket-patch

Conversation

@mikolalysenko

@mikolalysenko Mikola Lysenko (mikolalysenko) commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Release builds for aarch64-linux-android were failing with ld: cannot find -lunwind. Modern rustc emits -lunwind for Android targets, but the NDK bundled in cross 0.2.5's default Android image doesn't ship libunwind on the linker path (Target aarch64-linux-android link failure: unwind. cross-rs/cross#1222, 0.17.7. briansmith/ring#1853).
  • Add a Cross.toml that overrides only the Android target to a digest-pinned cross main image (NDK r25+, libunwind available). All other targets keep their existing cross 0.2.5 images.
  • Pinned by sha256 digest, not by floating :main tag, to stay reproducible.

Test plan

  • Run the Release workflow in dry-run mode and confirm the aarch64-linux-android matrix job links cleanly.
  • Confirm no other matrix targets regressed (they should be untouched — the override is target-scoped).

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@mikolalysenko @claude
ci(release): pin newer cross NDK image for aarch64-linux-android
7f839c7
The default image bundled with cross 0.2.5 ships an NDK whose sysroot
lacks libunwind on the linker path. Modern rustc emits `-lunwind` for
Android targets, so the release build fails with
`ld: cannot find -lunwind`.

Override just the Android target to a digest-pinned cross main image,
which ships an NDK that has libunwind available. All other targets
keep their default cross 0.2.5 images.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@mikolalysenko
Mikola Lysenko (mikolalysenko) enabled auto-merge (squash) May 27, 2026 15:00
@mikolalysenko
Mikola Lysenko (mikolalysenko) disabled auto-merge May 27, 2026 15:16
@mikolalysenko
Mikola Lysenko (mikolalysenko) enabled auto-merge (squash) May 27, 2026 15:16
Wenxin-Jiang
Wenxin Jiang (Wenxin-Jiang) approved these changes May 27, 2026
View reviewed changes
@mikolalysenko
Mikola Lysenko (mikolalysenko) merged commit c20619b into main May 27, 2026
42 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@mikolalysenko
Mikola Lysenko (mikolalysenko) deleted the ci/android-cross-image-pin branch May 27, 2026 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Wenxin-Jiang Wenxin Jiang (Wenxin-Jiang) Wenxin-Jiang approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@mikolalysenko @Wenxin-Jiang
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.