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

Append -gnu to RUBY_TARGET for GNU Linux builds #511

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

stanhu
Copy link
Contributor

@stanhu stanhu commented Jan 30, 2025

rake-compiler-dock images now make a distinction between GNU and musl builds. Add a -gnu extension to RUBY_TARGET to make it possible to run:

rb-sys-dock --platform x86_64-linux-gnu --directory . --build

Closes #510

rake-compiler-dock images now make a distinction between GNU and
musl builds. Add a `-gnu` extension to `RUBY_TARGET` to make
it possible to run:

```
rb-sys-dock --platform x86_64-linux-gnu --directory . --build
```

Closes oxidize-rb#510
@ianks ianks merged commit 798fad1 into oxidize-rb:main Jan 30, 2025
56 checks passed
@ianks
Copy link
Collaborator

ianks commented Jan 30, 2025

note to self: check build result -> https://github.com/oxidize-rb/rb-sys/actions/runs/13057340828

@stanhu
Copy link
Contributor Author

stanhu commented Jan 30, 2025

Oh, sorry, the tests aren't passing because the image is producing oxi-test-0.1.0-x86_64-linux-gnu.gem, but it's expecting oxi-test-0.1.0-x86_64-linux.gem.

Perhaps we have to make this configurable then.

stanhu added a commit to stanhu/rb-sys that referenced this pull request Jan 30, 2025
ianks pushed a commit that referenced this pull request Jan 30, 2025
* Revert "Append -gnu to RUBY_TARGET for GNU Linux builds (#511)"

This reverts commit 798fad1.

* Make it possible to override RUBY_TARGET

Previously if you attempt to run `rb-sys-dock --platform x86_64-linux-gnu`
the `RUBY_TARGET` was hard-coded to `x86_64-linux`. Now this can
be overriden by the `rb-sys-dock` so that the `--platform` argument
can work with `x86_64-linux` or `x86_64-linux-gnu`.
stanhu added a commit to stanhu/rb-sys that referenced this pull request Jan 30, 2025
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.

RUBY_TARGET not set properly when using rb-sys-dock --platform x86_64-linux-gnu
2 participants