diff options
author | Bundlerbot <bot@bundler.io> | 2019-12-17 22:32:52 +0000 |
---|---|---|
committer | Bundlerbot <bot@bundler.io> | 2019-12-17 22:32:52 +0000 |
commit | 8bd21aa0f5dc92fce4df277715f4eab546d30a7f (patch) | |
tree | b2ab1e41a1f24405d4fbf30e25e2daf06eaf7c8a | |
parent | 683fe9799e37bff81e53f1a7e97d3a19896b8fc9 (diff) | |
parent | 0a5db7e274cac1198d7e22d4e302dd9002342eae (diff) | |
download | bundler-8bd21aa0f5dc92fce4df277715f4eab546d30a7f.tar.gz |
Merge #7502v2.1.1
7502: Release 2.1.1 r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was I want to get regression fixed as soon as possible to reduce impact.
### What is your fix for the problem, implemented in this PR?
My fix is to release what we have already fixed as soon as possible.
Co-authored-by: Bundlerbot <bot@bundler.io>
Co-authored-by: David RodrÃguez <deivid.rodriguez@riseup.net>
-rw-r--r-- | .travis.yml | 3 | ||||
-rw-r--r-- | CHANGELOG.md | 7 | ||||
-rw-r--r-- | Rakefile | 9 | ||||
-rw-r--r-- | lib/bundler/cli/exec.rb | 15 | ||||
-rw-r--r-- | lib/bundler/rubygems_integration.rb | 8 | ||||
-rw-r--r-- | lib/bundler/setup.rb | 5 | ||||
-rw-r--r-- | lib/bundler/version.rb | 2 | ||||
-rw-r--r-- | spec/commands/exec_spec.rb | 6 |
8 files changed, 41 insertions, 14 deletions
diff --git a/.travis.yml b/.travis.yml index 47e775cba7..e7828fd4af 100644 --- a/.travis.yml +++ b/.travis.yml @@ -48,6 +48,9 @@ jobs: - rvm: 2.6.5 script: bin/rake rubocop stage: linting + - rvm: 2.6.5 + script: bin/rake check_rvm_integration + stage: linting # Ruby 2.3 also tested in 2.x mode - rvm: 2.3.8 env: RGV=master diff --git a/CHANGELOG.md b/CHANGELOG.md index 1393d19942..ec13c08754 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,10 @@ +## 2.1.1 (December 17, 2019) + +Bugfixes: + + - Fix some cases of shelling out to `rubygems` still being silent [#7493](https://github.com/bundler/bundler/pull/7493) + - Restore compatibility with `rubygems-bundler` so that binstubs work under `RVM` [#7498](https://github.com/bundler/bundler/pull/7498) + ## 2.1.0 (December 15, 2019) Features: @@ -1,6 +1,5 @@ # frozen_string_literal: true -$:.unshift File.expand_path("../lib", __FILE__) require "benchmark" require_relative "spec/support/rubygems_ext" @@ -123,6 +122,14 @@ task :rubocop do sh("bin/rubocop --parallel") end +desc "Check RVM integration" +task :check_rvm_integration do + # The rubygems-bundler gem is installed by RVM by default and it could easily + # break when we change bundler. Make sure that binstubs still run with it + # installed. + sh("bin/rake install && gem install rubygems-bundler && rake -T") +end + namespace :man do if RUBY_ENGINE == "jruby" task(:build) {} diff --git a/lib/bundler/cli/exec.rb b/lib/bundler/cli/exec.rb index 2bace6e77d..0a1edbdbbd 100644 --- a/lib/bundler/cli/exec.rb +++ b/lib/bundler/cli/exec.rb @@ -25,12 +25,12 @@ module Bundler SharedHelpers.set_bundle_environment if bin_path = Bundler.which(cmd) if !Bundler.settings[:disable_exec_load] && ruby_shebang?(bin_path) - return with_verbose_rubygems { kernel_load(bin_path, *args) } + return kernel_load(bin_path, *args) end - with_verbose_rubygems { kernel_exec(bin_path, *args) } + kernel_exec(bin_path, *args) else # exec using the given command - with_verbose_rubygems { kernel_exec(cmd, *args) } + kernel_exec(cmd, *args) end end @@ -89,14 +89,5 @@ module Bundler first_line = File.open(file, "rb") {|f| f.read(possibilities.map(&:size).max) } possibilities.any? {|shebang| first_line.start_with?(shebang) } end - - def with_verbose_rubygems - old_ui = Gem::DefaultUserInteraction.ui - Gem::DefaultUserInteraction.ui = nil - - yield - ensure - Gem::DefaultUserInteraction.ui = old_ui - end end end diff --git a/lib/bundler/rubygems_integration.rb b/lib/bundler/rubygems_integration.rb index bb63209632..88fcd4d9e0 100644 --- a/lib/bundler/rubygems_integration.rb +++ b/lib/bundler/rubygems_integration.rb @@ -525,6 +525,14 @@ module Bundler end end + def plain_specs + Gem::Specification._all + end + + def plain_specs=(specs) + Gem::Specification.all = specs + end + def fetch_specs(remote, name) path = remote.uri.to_s + "#{name}.#{Gem.marshal_version}.gz" fetcher = gem_remote_fetcher diff --git a/lib/bundler/setup.rb b/lib/bundler/setup.rb index 70a0968f4b..27911dc1ad 100644 --- a/lib/bundler/setup.rb +++ b/lib/bundler/setup.rb @@ -19,4 +19,9 @@ if Bundler::SharedHelpers.in_bundle? else Bundler.ui.silence { Bundler.setup } end + + # We might be in the middle of shelling out to rubygems + # (RUBYOPT=-rbundler/setup), so we need to give rubygems the opportunity of + # not being silent. + Gem::DefaultUserInteraction.ui = nil end diff --git a/lib/bundler/version.rb b/lib/bundler/version.rb index 417b21f18a..06d6a0f255 100644 --- a/lib/bundler/version.rb +++ b/lib/bundler/version.rb @@ -1,7 +1,7 @@ # frozen_string_literal: false module Bundler - VERSION = "2.1.0".freeze + VERSION = "2.1.1".freeze def self.bundler_major_version @bundler_major_version ||= VERSION.split(".").first.to_i diff --git a/spec/commands/exec_spec.rb b/spec/commands/exec_spec.rb index c1d6e4c25a..42f760ab12 100644 --- a/spec/commands/exec_spec.rb +++ b/spec/commands/exec_spec.rb @@ -61,6 +61,12 @@ RSpec.describe "bundle exec" do expect(out).to eq(Gem::VERSION) end + it "works when exec'ing to rubygems through sh -c" do + install_gemfile 'gem "rack"' + bundle "exec sh -c '#{gem_cmd} --version'" + expect(out).to eq(Gem::VERSION) + end + it "respects custom process title when loading through ruby" do script_that_changes_its_own_title_and_checks_if_picked_up_by_ps_unix_utility = <<~'RUBY' Process.setproctitle("1-2-3-4-5-6-7-8-9-10-11-12-13-14-15") |