Skip to content

Conversation

@deivid-rodriguez
Copy link
Contributor

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

This is a regression of #7795.

Since that PR, the generic Ruby platform was getting unconditionally added in truffleruby, preventing resolution in situations where there's no generic ruby version (sorbet-static).

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

Consider the generic platform per dependency, not globally, so that when gems actually support platform specific gems in truffleruby like sorbet-static, that's actually respected.

Make sure the following tasks are checked

@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/truffleruby-issues branch from 426f3b5 to b63ab49 Compare July 25, 2025 17:00
@deivid-rodriguez deivid-rodriguez marked this pull request as ready for review July 25, 2025 17:37
…file

The generic Ruby platform was getting unconditionally added in
truffleruby, preventing resolution in situations where there's no
generic ruby version (sorbet-static). Instead, the generic platform
should be considered per dependency, not globally.
@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/truffleruby-issues branch from b63ab49 to a96afc5 Compare July 25, 2025 19:31
@deivid-rodriguez deivid-rodriguez merged commit ad26cd4 into master Jul 29, 2025
76 checks passed
@deivid-rodriguez deivid-rodriguez deleted the deivid-rodriguez/truffleruby-issues branch July 29, 2025 05:26
deivid-rodriguez added a commit that referenced this pull request Sep 4, 2025
…ssues

Fix bundler failing to install sorbet-static in truffleruby when there's no lockfile

(cherry picked from commit ad26cd4)
deivid-rodriguez added a commit that referenced this pull request Sep 9, 2025
…ssues

Fix bundler failing to install sorbet-static in truffleruby when there's no lockfile

(cherry picked from commit ad26cd4)
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.

2 participants