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

DRb has been extracted from stdlib to a gem #23232

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

agrare
Copy link
Member

@agrare agrare commented Oct 14, 2024

Add DRb to the Gemfile as a gem as it has been extracted from stdlib to a standalone gem.

lib/miq_automation_engine/miq_ae_exception.rb:1: warning: lib/ruby/3.3.0/drb.rb was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add drb to your Gemfile or gemspec to silence this warning.

@agrare agrare requested a review from Fryguy as a code owner October 14, 2024 17:27
Gemfile Outdated
@@ -38,6 +38,7 @@ gem "connection_pool", :require => false #
gem "dalli", "~>3.2.3", :require => false
gem "default_value_for", "~>4.0"
gem "docker-api", "~>1.33.6", :require => false
gem "drb", "~>2.1.1", :require => false
Copy link
Member Author

@agrare agrare Oct 14, 2024

Choose a reason for hiding this comment

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

NOTE ruby 3.1.0 has drb version 2.1.1 so that's what I used here, the latest (and what is in 3.3) is 2.2.1.

Though looking at the changes (https://my.diffend.io/gems/drb/2.1.1/2.2.0 and https://my.diffend.io/gems/drb/2.2.0/2.2.1) it is probably safe to use 2.2.1 on any of our current ruby versions

Copy link
Member

Choose a reason for hiding this comment

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

I'm leaning towards 2.2.1.
We will be changing this in the short term anyway

@agrare
Copy link
Member Author

agrare commented Oct 14, 2024

@miq-bot cross-repo-tests /all

miq-bot pushed a commit to ManageIQ/manageiq-cross_repo-tests that referenced this pull request Oct 14, 2024
@kbrock kbrock self-assigned this Oct 14, 2024
@kbrock kbrock added the ruby3.3 label Oct 14, 2024
Gemfile Outdated
@@ -38,6 +38,7 @@ gem "connection_pool", :require => false #
gem "dalli", "~>3.2.3", :require => false
gem "default_value_for", "~>4.0"
gem "docker-api", "~>1.33.6", :require => false
gem "drb", "~>2.2.1", :require => false
Copy link
Member

Choose a reason for hiding this comment

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

I'm wondering if we should be more open - maybe ">=2.2.1" or "~>2.2" ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure ~>2.2 would work, don't want to accidentally pull in a breaking change

Add DRb to the Gemfile as a gem as it has been extracted from stdlib to
a standalone gem.

```
lib/miq_automation_engine/miq_ae_exception.rb:1: warning: lib/ruby/3.3.0/drb.rb was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add drb to your Gemfile or gemspec to silence this warning.
```
@agrare
Copy link
Member Author

agrare commented Oct 14, 2024

Okay changed to ~>2.2, cross repo was green with ~>2.2.1 so I don't think we need to kick it again

@miq-bot
Copy link
Member

miq-bot commented Oct 14, 2024

Checked commit agrare@5103e69 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 👍

@Fryguy Fryguy merged commit 02c20ec into ManageIQ:master Oct 14, 2024
12 checks passed
@Fryguy Fryguy assigned Fryguy and unassigned kbrock Oct 14, 2024
@agrare agrare deleted the add_drb_to_gemfile branch October 14, 2024 22:02
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.

4 participants