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

Add Ruby 3.1 support #1472

Merged
merged 3 commits into from
Apr 22, 2022
Merged

Add Ruby 3.1 support #1472

merged 3 commits into from
Apr 22, 2022

Conversation

deivid-rodriguez
Copy link
Contributor

No description provided.

@eregon
Copy link
Contributor

eregon commented Feb 16, 2022

I was wondering why the truffleruby job gets cancelled, and found https://github.com/deivid-rodriguez/mail/actions/runs/1853944108

ruby (flavorjones/truffleruby:stable)
The job running on runner Hosted Agent has exceeded the maximum execution time of 5 minutes.
ruby (flavorjones/truffleruby:stable)
The operation was canceled.

So it seems because of the rather low timeout-minutes.
There are also some upcoming truffleruby changes to speed it up in CI and use less memory.
Anyway, seems OK to remove from now, that Docker image was also picking a older TruffleRuby release it seems.

And potentially the CI could (later) run outside Docker now that it doesn't require such ancient Ruby versions like 1.8, which would make the CI faster as well.

.github/workflows/test.yml Outdated Show resolved Hide resolved
@deivid-rodriguez deivid-rodriguez force-pushed the ruby-31-support branch 3 times, most recently from 24cb453 to 30d1819 Compare February 17, 2022 12:43
@deivid-rodriguez
Copy link
Contributor Author

This is now green and I'm happy with it!

@wolak88
Copy link

wolak88 commented Mar 7, 2022

Is there anything else TBD before merging that to the gem itself?

Thanks for the great job!

@deivid-rodriguez
Copy link
Contributor Author

A maintainer of this gem needs to review the PR. From my side it's ready.

@simi
Copy link
Contributor

simi commented Mar 7, 2022

@mikel @jeremy any chance to get some time to check this? Is there anything I can help with to help maintaining this gem?

@eregon
Copy link
Contributor

eregon commented Mar 10, 2022

@deivid-rodriguez FWIW, I think there is no need to depend on the io-wait gem, is there?
Also for context: https://bugs.ruby-lang.org/issues/18591#note-7

@deivid-rodriguez deivid-rodriguez force-pushed the ruby-31-support branch 3 times, most recently from 316d8c9 to 508f432 Compare March 10, 2022 14:32
Gemfile Outdated Show resolved Hide resolved
@mikel
Copy link
Owner

mikel commented Apr 19, 2022

Let's get #1476 passing CI and merged first, then we can rebase this and move forward @deivid-rodriguez @simi and @voxik

@deivid-rodriguez deivid-rodriguez changed the title Add Ruby 3.1 support, require Ruby >= 2.5, and get CI green Add Ruby 3.1 support Apr 20, 2022
@simi
Copy link
Contributor

simi commented Apr 21, 2022

🙏 time to rebase @deivid-rodriguez

The default RubyGems version on Ruby 2.5 has issues with it.
The released version of `truffleruby` does not work with old versions of
Bundler. However, it's not possible to specify a custom version of
Bundler in truffleruby, because the `ruby/setup-ruby` will ignore that
in the case of `truffleruby-22.0` (the current release) That's in my
opinion an issue with `ruby/setup-ruby` because it should respect user's
choice when it's explicit. But it's not possible to change that because
it was an explicit choice of that library.

So as an alternative solution, use `truffleruby-head`, which apparently
does not have the issue.
@deivid-rodriguez deivid-rodriguez force-pushed the ruby-31-support branch 2 times, most recently from 5c24652 to 63a749c Compare April 21, 2022 12:49
@deivid-rodriguez
Copy link
Contributor Author

This should be ready now!

Gemfile Outdated Show resolved Hide resolved
ro3t pushed a commit to bfpi/klarschiff-backoffice that referenced this pull request Jun 22, 2022
* Update Ruby 3.1.2

* Gem 'net-smtp' als temporäre Abhängigkeit ergänzt

Mit Ruby 3.1 ist u.a. 'net-smtp' aus den Standardabhängigkeiten
entfallen. 'mail' benötigt diese Abhängigkeit und löst diese jedoch erst
ab Version 2.8.0 selbst auf (mikel/mail#1472),
das bisher noch nicht veröffentlicht wurde (nur als RC1).

* Update Bundler v2.3.3

Fix: "Calling DidYouMean::SPELL_CHECKERS.merge!(error_name => spell_checker)' has been deprecated. Please call DidYouMean.correct_error(error_name, spell_checker)' instead."

rubygems/rubygems#5234
ro3t pushed a commit to bfpi/klarschiff-backoffice that referenced this pull request Jun 22, 2022
Mit Ruby 3.1 ist u.a. 'net-smtp' aus den Standardabhängigkeiten
entfallen. 'mail' benötigt diese Abhängigkeit und löst diese jedoch erst
ab Version 2.8.0 selbst auf (mikel/mail#1472),
das bisher noch nicht veröffentlicht wurde (nur als RC1).
ro3t pushed a commit to bfpi/klarschiff-backoffice that referenced this pull request Jun 22, 2022
Mit Ruby 3.1 ist u.a. 'net-smtp' aus den Standardabhängigkeiten
entfallen. 'mail' benötigt diese Abhängigkeit und löst diese jedoch erst
ab Version 2.8.0 selbst auf (mikel/mail#1472),
das bisher noch nicht veröffentlicht wurde (nur als RC1).
@johnnyshields
Copy link
Contributor

johnnyshields commented Jun 24, 2022

@mikel Please do the final release of 2.8.0? This issue is causing a problem for many people as you can see in all the references above.

nbennke added a commit to bfpi/klarschiff-backoffice that referenced this pull request Jun 30, 2022
* Update Ruby 3.1.2

Mit Ruby 3.1 ist u.a. 'net-smtp' aus den Standardabhängigkeiten
entfallen. 'mail' benötigt diese Abhängigkeit und löst diese jedoch erst
ab Version 2.8.0 selbst auf (mikel/mail#1472),
das bisher noch nicht veröffentlicht wurde (nur als RC1).

Co-authored-by: Niels Bennke <bennke@bfpi.de>
@ghepting
Copy link

Is there a stable release with this out yet or just the rc?

@ashishra0
Copy link

Folks, I recently upgraded to ruby3.1.1 in my project which sends mail to users and hit a blocker, and to resolve that I need to add the net/smtp gem. Now my question is, do I still need to include that gem in my gemfile?

@deivid-rodriguez
Copy link
Contributor Author

Little has changed, unfortunately. Previously to the rc release, you needed to include net/smtp in your Gemfile. Now you need to either use that workaround or force the rc to be picked up by Bundler (with gem "mail", ">= 2.8.0.rc1" or similar).

Hopefully @mikel will release the final version soon.

@johnnyshields
Copy link
Contributor

@mikel any update on this?

kevindew added a commit to alphagov/gds-sso that referenced this pull request Oct 19, 2022
Ruby 3.1 removed net-smtp from stdlib which causes a problem when Rails
6 loads the mail gem [1]. Rails 7 has a fix for this [2].

I expect it won't be too long until we drop support for Rails 6 and can
remove this.

[1]: mikel/mail#1472
[2]: rails/rails@cb44df7
kevindew added a commit to alphagov/gds-sso that referenced this pull request Oct 19, 2022
Ruby 3.1 removed net-smtp from stdlib which causes a problem when Rails
6 loads the mail gem [1]. Rails 7 has a fix for this [2].

I expect it won't be too long until we drop support for Rails 6 and can
remove this.

[1]: mikel/mail#1472
[2]: rails/rails@cb44df7
kevindew added a commit to alphagov/gds-sso that referenced this pull request Oct 19, 2022
Ruby 3.1 removed net-smtp from stdlib which causes a problem when Rails
6 loads the mail gem [1]. Rails 7 has a fix for this [2].

I expect it won't be too long until we drop support for Rails 6 and can
remove this.

[1]: mikel/mail#1472
[2]: rails/rails@cb44df7
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Oct 26, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Oct 26, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Oct 28, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Nov 1, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Nov 2, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Nov 4, 2022
Pauloparakleto added a commit to Pauloparakleto/conference that referenced this pull request Nov 6, 2022
graalvmbot pushed a commit to oracle/truffleruby that referenced this pull request Nov 7, 2022
john-heinnickel pushed a commit to thermofisher-jch/truffleruby that referenced this pull request Aug 16, 2023
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.

None yet