Skip to content

CI: Use latest Bundler to support Ruby head#421

Merged
kenhys merged 1 commit into
masterfrom
ci
May 11, 2026
Merged

CI: Use latest Bundler to support Ruby head#421
kenhys merged 1 commit into
masterfrom
ci

Conversation

@Watson1978

@Watson1978 Watson1978 commented May 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Ruby 4.1.0dev (head) removed Pathname::SEPARATOR_PAT, which caused Bundler 2.4.22 (specified in Gemfile.lock) to fail with a NameError.

NameError: private constant Pathname::SEPARATOR_PAT referenced
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/source/path.rb:228:in 'block in Bundler::Source::Path#generate_bin'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/source/path.rb:227:in 'Array#map'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/source/path.rb:227:in 'Bundler::Source::Path#generate_bin'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/source/path.rb:88:in 'Bundler::Source::Path#install'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/installer/gem_installer.rb:54:in 'Bundler::GemInstaller#install'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/installer/gem_installer.rb:16:in 'Bundler::GemInstaller#install_from_spec'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/installer/parallel_installer.rb:130:in 'Bundler::ParallelInstaller#do_install'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/installer/parallel_installer.rb:121:in 'block in Bundler::ParallelInstaller#worker_pool'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/worker.rb:62:in 'Bundler::Worker#apply_func'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/worker.rb:57:in 'block in Bundler::Worker#process_queue'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/worker.rb:54:in 'Kernel#loop'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/worker.rb:54:in 'Bundler::Worker#process_queue'
  /home/runner/.rubies/ruby-head/lib/ruby/gems/4.1.0+1/gems/bundler-2.4.22/lib/bundler/worker.rb:90:in 'block (2 levels) in Bundler::Worker#create_threads'

Ref. https://github.com/fluent-plugins-nursery/fluent-plugin-kubernetes_metadata_filter/actions/runs/25237069015

By removing BUNDLED WITH, setup-ruby will use a version of Bundler compatible with the running Ruby version, resolving the incompatibility in the head build while maintaining Ruby 2.7 support.

@Watson1978 Watson1978 force-pushed the ci branch 5 times, most recently from 956acb9 to 9a965b3 Compare May 11, 2026 04:55
@Watson1978 Watson1978 requested a review from kenhys May 11, 2026 04:59
@Watson1978
CI: Use latest Bundler to support Ruby head
3a1845f 
Ruby 4.1.0dev (head) removed Pathname::SEPARATOR_PAT, which caused
Bundler 2.4.22 (specified in Gemfile.lock) to fail with a NameError.

By setting 'bundler: latest', setup-ruby will install a version of
Bundler compatible with the running Ruby version, resolving the
incompatibility in the head build while maintaining Ruby 2.7 support.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
kenhys
kenhys approved these changes May 11, 2026
View reviewed changes

@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 9a1aa84 into master May 11, 2026
18 of 19 checks passed
@kenhys kenhys deleted the ci branch May 11, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kenhys kenhys kenhys approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Watson1978 @kenhys