Skip to content

Fixes for Ruby 3.4-dev - #4411

Merged
kenhys merged 2 commits into
masterfrom
fix-ruby-3.4-dev
Mar 6, 2024
Merged

Fixes for Ruby 3.4-dev#4411
kenhys merged 2 commits into
masterfrom
fix-ruby-3.4-dev

Conversation

@ashie

@ashie ashie commented Feb 22, 2024

Copy link
Copy Markdown
Member

Which issue(s) this PR fixes:
N/A

What this PR does / why we need it:
CI is failed on Ruby 3.4-dev:
https://github.com/fluent/fluentd/actions/runs/7949585800

Docs Changes:
N/A

Release Note:
N/A

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
base64, csv and drb gems aren't part of the default gems since Ruby
3.4.0.

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
@ashie
ashie marked this pull request as ready for review March 6, 2024 03:07
@ashie
ashie requested a review from kenhys March 6, 2024 03:07

@kenhys kenhys left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@kenhys
kenhys merged commit c56f91e into master Mar 6, 2024
@kenhys
kenhys deleted the fix-ruby-3.4-dev branch March 6, 2024 03:11
@ashie ashie added this to the v1.17.0 milestone Apr 30, 2024
Watson1978 added a commit to Watson1978/fluentd that referenced this pull request Aug 15, 2024
logger gem was marked as bundled gem for Ruby 3.5.
Therefore, it will not work on Ruby 3.5 unless it is added as a dependency.
Ref. ruby/ruby@d7e558e

(Currently, this patch will suppress `warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.` message.)

Similar with fluentGH-4411

Signed-off-by: Watson <watson1978@gmail.com>
daipom pushed a commit that referenced this pull request Aug 15, 2024
logger gem was marked as bundled gem for Ruby 3.5.
Therefore, it will not work on Ruby 3.5 unless it is added as a dependency.
Ref. ruby/ruby@d7e558e

(Currently, this patch will suppress `warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.` message.)

Similar with GH-4411

Signed-off-by: Watson <watson1978@gmail.com>
@Watson1978

Watson1978 commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

Without this patch, it seems impossible to test the v1.16 branch with ruby 3.4 or later...

@Watson1978 Watson1978 added the backport to v1.16 We will backport this fix to the LTS branch label Dec 5, 2025
@Watson1978 Watson1978 added the backported "backport to LTS" is done label Dec 5, 2025
kenhys pushed a commit that referenced this pull request Dec 5, 2025
Backport #4411

**Which issue(s) this PR fixes**: 
N/A

**What this PR does / why we need it**: 
CI is failed on Ruby 3.4-dev:
https://github.com/fluent/fluentd/actions/runs/7949585800

**Docs Changes**:
N/A

**Release Note**: 
N/A

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Co-authored-by: Takuro Ashie <ashie@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to v1.16 We will backport this fix to the LTS branch backported "backport to LTS" is done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants