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 missing dependency to base64 #4425

Closed
wants to merge 1 commit into from

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Mar 6, 2024

Which issue(s) this PR fixes:

Fixes #

What this PR does / why we need it:

Since ruby 3.4.0, base64 is changed to non-default gems. It need to be installed explicitly now.
(service discovery and out forward plugin use it)

At least base64 >= 0.1.0 should be installed.

FYI:

  • ruby 3.0.0 (base64 0.1.0)
  • ruby 3.0.6 (base64 0.1.0)
  • ruby 3.1.4 (base64 0.1.1)
  • ruby 3.2.3 (base64 0.1.1)
  • ruby 3.3.0 (base64 0.2.0)

It seems that ruby-2.7.8 bundles base64 0.1.0 equivalent version.

e.g. It fixes the following LoadError.

warning: base64 was loaded from the standard library, but is not part
of the default gems since Ruby 3.4.0. Add base64 to your Gemfile or
gemspec.
/home/runner/.rubies/ruby-head/lib/ruby/3.4.0+0/bundled_gems.rb:74:in
'Kernel.require': cannot load such file -- base64 (LoadError)

Docs Changes:

N/A

Release Note:

N/A

Since ruby 3.4.0, base64 is changed to non-default
gems. It need to be installed explicitly now.
(service discovery and out forward plugin use it)

At least base64 >= 0.1.0 should be installed.

FYI:

* ruby 3.0.0 (base64 0.1.0)
* ruby 3.0.6 (base64 0.1.0)
* ruby 3.1.4 (base64 0.1.1)
* ruby 3.2.3 (base64 0.1.1)
* ruby 3.3.0 (base64 0.2.0)

It seems that ruby-2.7.8 bundles base64 0.1.0 equivalent version.

e.g. It fixes the following LoadError.

 warning: base64 was loaded from the standard library, but is not part
 of the default gems since Ruby 3.4.0. Add base64 to your Gemfile or
 gemspec.
 /home/runner/.rubies/ruby-head/lib/ruby/3.4.0+0/bundled_gems.rb:74:in
 'Kernel.require': cannot load such file -- base64 (LoadError)

Signed-off-by: Kentaro Hayashi <[email protected]>
@ashie
Copy link
Member

ashie commented Mar 6, 2024

I'm working on it at #4411

@ashie ashie closed this Mar 6, 2024
@kenhys
Copy link
Contributor Author

kenhys commented Mar 6, 2024

I've overlooked it...

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