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

Bump rake-compiler-dock images to v1.8.0 with Ruby patch versions #489

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

stanhu
Copy link
Contributor

@stanhu stanhu commented Jan 14, 2025

This reflects the changes made in rake-compiler/rake-compiler-dock#135.

While this requires packages to update their RUBY_CC_VERSION to use Ruby patches across the board, this is already needed for Ruby 3.3.5. This change at least makes this consistent.

See https://github.com/rake-compiler/rake-compiler-dock/releases/tag/v1.8.0.

@stanhu stanhu changed the title Bump rake-compiler-dock images to v1.8.0 and with Ruby patch versions Bump rake-compiler-dock images to v1.8.0 with Ruby patch versions Jan 14, 2025
This reflects the changes made in
rake-compiler/rake-compiler-dock#135.

While this requires packages to update their `RUBY_CC_VERSION` to use
Ruby patches across the board, this is already needed for Ruby 3.3.5.
This change at least makes this consistent.

See
https://github.com/rake-compiler/rake-compiler-dock/releases/tag/v1.8.0.
@ianks
Copy link
Collaborator

ianks commented Jan 16, 2025

Thank you so much! Running a build to check that it works, then will merge ❤️

https://github.com/oxidize-rb/rb-sys/actions/runs/12814269578

@ianks ianks merged commit ac7c48c into oxidize-rb:main Jan 16, 2025
65 checks passed
@ianks
Copy link
Collaborator

ianks commented Jan 16, 2025

Ahh, forgot that we need to make rb-sys-dock aware of the patch versions. Can you add them here?

(also, it's a little janky since we have to define the same string everywhere, open to ideas for improving as well)

@stanhu
Copy link
Contributor Author

stanhu commented Jan 16, 2025

Yes, I'll add them now.

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.

2 participants