[pull] master from ruby:master#1152
Merged
Merged
Conversation
…ministic The test ran a real `bundle install` under a 32M JVM heap and expected Bundler to catch the OOM. At that heap size the JVM sometimes dies during boot and the JRuby launcher prints its own message before Bundler's rescue runs, so the spec flaked on Ubuntu JRuby CI. Drive the handler directly like the sibling log_error examples instead. ruby/rubygems@659fac7ba3 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
When the same Gem::SecureRandom copy is shipped in two gems (RubyGems and Bundler) and required from different paths, RubyGems' unguarded require_relative reloads it and emits "already initialized constant" warnings. Add an idempotent guard at the top of the vendored securerandom.rb so a second load from any path short-circuits. This prepares Bundler to reuse RubyGems' copy. ruby/rubygems@0acdd1d8a9 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add lib/rubygems/vendor/securerandom to the gem's file list so the Bundler gem carries its own copy of Gem::SecureRandom. This keeps Bundler self-contained on RubyGems versions that predate the vendored copy, the same way the vendored URI is shipped. ruby/rubygems@6810522806 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Bundler now reuses RubyGems' Gem::SecureRandom directly instead of keeping its own vendored copy. The shim prefers an already-loaded Gem::SecureRandom, then the copy shipped in the gem under lib/rubygems/vendor, and only falls back to the stdlib when neither is present. Drop the Bundler vendored tree and its automatiek entry. ruby/rubygems@73478ae41d Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…nvariant ruby/rubygems@08ebcf2e97 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )