diff options
| author | Andre Arko <andre@arko.net> | 2017-11-26 23:11:03 -0800 |
|---|---|---|
| committer | Andre Arko <andre@arko.net> | 2017-11-26 23:11:03 -0800 |
| commit | 4dfb32241a4af5d115f65ba8b17aa9bfcde19c57 (patch) | |
| tree | 525c849cf153f5474554ce37e6f929fa55872407 /lib/bundler/source | |
| parent | 914a4a8b8d0cf1a79dbc7b334fbb5c2db1ecdc16 (diff) | |
| download | bundler-4dfb32241a4af5d115f65ba8b17aa9bfcde19c57.tar.gz | |
Stop overriding specs during double-checks
This fixes my test-case reproduction of #6072. I’m pretty sure it’s
because the double-check was overriding the locally installed index
specs with specs fetched from the RubyGems API, causing Bundler to
conclude that the gems aren’t installed, even though they are.
Diffstat (limited to 'lib/bundler/source')
| -rw-r--r-- | lib/bundler/source/rubygems.rb | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/bundler/source/rubygems.rb b/lib/bundler/source/rubygems.rb index 45ff3d17b3..3ba7641f84 100644 --- a/lib/bundler/source/rubygems.rb +++ b/lib/bundler/source/rubygems.rb @@ -258,10 +258,8 @@ module Bundler end end - def double_check_for(unmet_dependency_names, override_dupes = false, index = specs) + def double_check_for(unmet_dependency_names) return unless @allow_remote - raise ArgumentError, "missing index" unless index - return unless api_fetchers.any? unmet_dependency_names = unmet_dependency_names.call @@ -276,7 +274,7 @@ module Bundler Bundler.ui.debug "Double checking for #{unmet_dependency_names || "all specs (due to the size of the request)"} in #{self}" - fetch_names(api_fetchers, unmet_dependency_names, index, override_dupes) + fetch_names(api_fetchers, unmet_dependency_names, specs, false) end def dependency_names_to_double_check |