Skip to content

[vcpkg-script] fix vcpkg bootstrap failing on Musl Arm64#44875

Merged
BillyONeal merged 1 commit intomicrosoft:masterfrom
aminya:alpine-bootstrap
Apr 7, 2025
Merged

[vcpkg-script] fix vcpkg bootstrap failing on Musl Arm64#44875
BillyONeal merged 1 commit intomicrosoft:masterfrom
aminya:alpine-bootstrap

Conversation

@aminya
Copy link
Contributor

@aminya aminya commented Apr 7, 2025

After #44587, Alpine Arm64 fails to bootstrap vcpkg because it tries to run the glibc binaries on the Musl runtime instead of building from the source. This fixes the issue, making sure the Arm64 binaries are only used on Glibc runtime

Here's the failure:
https://github.com/aminya/setup-cpp/actions/runs/14304315923/job/40084748302#step:6:474

@aminya aminya force-pushed the alpine-bootstrap branch from 640d0ec to ece947a Compare April 7, 2025 08:42
@Cheney-W Cheney-W added the category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly label Apr 7, 2025
@Cheney-W Cheney-W added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 7, 2025
@Cheney-W Cheney-W changed the title [musl] fix vcpkg bootstrap failing on Musl Arm64 [vcpkg-script] fix vcpkg bootstrap failing on Musl Arm64 Apr 7, 2025
@BillyONeal BillyONeal merged commit 5bb626a into microsoft:master Apr 7, 2025
18 checks passed
@BillyONeal
Copy link
Member

Thanks for the fix!

Copy link
Contributor

@dg0yt dg0yt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if an autotoolish approach would help in the long term:

case "$ARCH-$UNAME-$CRT" in
arm64-linux-muslc)
   ...
   ;;
esac

Well, no change requested. Just a note for another PR.

@BillyONeal
Copy link
Member

I wonder if an autotoolish approach would help in the long term:
Well, no change requested. Just a note for another PR.

I did want separate variables for 'choose the arm64 one' but the 'red code' already didn't have them so I didn't argue it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants