Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert addition of support for musl variants to restore graceful fallback on Alpine #4434

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

deivid-rodriguez
Copy link
Member

What was the end-user or developer problem that led to this PR?

Adding support for non-gnu libc linux platforms caused issues for gems where the gnu variant was working just fine.

What is your fix for the problem, implemented in this PR?

For now this reverts commit ce4e3f5, reversing changes made to 8a712a0.

We plan to introduce this improvement, but with a better fallback behavior where linux-musl variants are never installed on standard linux, and linux variants are picked up as a fallback on Alpine.

Closes #4425.
Reopens #3174.

Make sure the following tasks are checked

This reverts commit ce4e3f5, reversing
changes made to 8a712a0.

We plan to introduce this improvement, but with a better fallback
behavior where `linux-musl` variants are never installed on standard linux,
and `linux` variants are picked up as a fallback on Alpine.
@deivid-rodriguez deivid-rodriguez changed the title Revert addition of support for musl variants Revert addition of support for musl variants to restore graceful fallback on Alpine Mar 8, 2021
@deivid-rodriguez
Copy link
Member Author

@lloeki agreed with this approach, and we'll try to reintroduce this soon! For now restoring the previous behaviour.

@deivid-rodriguez deivid-rodriguez merged commit 4f42ac8 into master Mar 8, 2021
@deivid-rodriguez deivid-rodriguez deleted the revert_libc_changes branch March 8, 2021 15:27
deivid-rodriguez added a commit that referenced this pull request Mar 8, 2021
Revert addition of support for `musl` variants to restore graceful fallback on Alpine

(cherry picked from commit 4f42ac8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rubygems-update 3.2.13 breaks sorbet installation on alpine
2 participants