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

Update rack to 2.2.10 to fix error with Ruby 3.3 regexp. #48

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

jackorp
Copy link
Contributor

@jackorp jackorp commented Dec 3, 2024

Ruby 3.3's Regexp class removed the third argument for #new and similar calls

Update nio4r, it didn't compile with Ruby 3.3.6. After update it still compiles with Ruby 2.5.

Updated only patch version of the gems. Puma seems to be working.

Ruby 3.3's Regexp class removed the third argument for #new and similar calls

Update nio4r to compile with Ruby 3.3.6 as well.
@jackorp
Copy link
Contributor Author

jackorp commented Dec 3, 2024

Should supersede #47 and partially #37

@jackorp
Copy link
Contributor Author

jackorp commented Dec 3, 2024

Tested bundle install and bundle exec rackup for 2.5, 3.1, 3.3 ubi8 containers, app was up with no crashes.

@phracek
Copy link
Member

phracek commented Dec 3, 2024

@jackorp Can you please fix this as well https://github.com/sclorg/rails-ex?

Copy link
Member

@phracek phracek left a comment

Choose a reason for hiding this comment

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

LGTM.

@phracek phracek merged commit 589edb7 into sclorg:master Dec 3, 2024
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