Skip to content

chore: make armv7-unknown-linux-musleabihf optional#85

Merged
socksy merged 1 commit intodevelopfrom
ben/ignore-armv7-unknown-linux-musleabihf
Sep 2, 2025
Merged

chore: make armv7-unknown-linux-musleabihf optional#85
socksy merged 1 commit intodevelopfrom
ben/ignore-armv7-unknown-linux-musleabihf

Conversation

@socksy
Copy link
Contributor

@socksy socksy commented Sep 1, 2025

armv7-unknown-linux-musleabihf is a Tier 2 without Host Tools target. That means that we should expect it to not always build, as they do not provide any guarantees that it will.

Since there's already a arm-unknown-linux-gnueabihf target, I don't think it should be the end of the world if the LLVM version doesn't build - at least there's still a binary for that platform. So in order to deal with the intermittent builds, I'm making this job not block the CI.

armv7-unknown-linux-musleabihf is a [Tier 2 without Host Tools](https://doc.rust-lang.org/beta/rustc/platform-support.html#tier-2-without-host-tools)
target. That means that we should expect it to not always build, as they
do not provide any guarantees that it will.

Since there's already a arm-unknown-linux-gnueabihf target, I don't
think it should be the end of the world if the LLVM version doesn't
build - at least there's still a binary for that platform. So in
order to deal with the intermittent builds, I'm making this job not block
@socksy socksy requested a review from Copilot September 1, 2025 16:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR makes the armv7-unknown-linux-musleabihf target optional in the CI build process to prevent intermittent build failures from blocking the entire CI pipeline. Since this is a Tier 2 without Host Tools target that doesn't guarantee successful builds, and there's already an alternative arm-unknown-linux-gnueabihf target available, allowing this specific target to fail gracefully improves CI reliability.

Key changes:

  • Added conditional continue-on-error flag for the armv7-unknown-linux-musleabihf target in the musllinux-cross job

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@socksy socksy changed the base branch from main to develop September 1, 2025 16:40
@tower tower deleted a comment from github-actions bot Sep 1, 2025
@socksy
Copy link
Contributor Author

socksy commented Sep 1, 2025

Of course when I make a PR specifically for this, the job passes 🙄

@socksy socksy merged commit f3426e9 into develop Sep 2, 2025
19 checks passed
@socksy socksy deleted the ben/ignore-armv7-unknown-linux-musleabihf branch September 2, 2025 08:53
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.

3 participants