kotovalexarian-likes-github/ruby--ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00
Code Releases Activity

[rubygems/rubygems] We can now avoid source materialization work

Browse source 
No need to fill up missing names from sources anymore since they should
now be properly set up correctly from the beginning.

https://github.com/rubygems/rubygems/commit/706fd28681
This commit is contained in:
David Rodríguez 2021-07-24 18:17:53 +02:00 • committed by Hiroshi SHIBATA
parent 1b9b41472f
commit d37c2441cc
Notes: git 2021-08-31 19:07:25 +09:00
1 changed files with 3 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
lib/bundler/spec_set.rb
View file

@ -69,16 +69,9 @@ module Bundler
def materialize(deps)
materialized = self.for(deps, false, true)
materialized.group_by(&:source).each do |source, specs|
next unless specs.any?{|s| s.is_a?(LazySpecification) }
source.local!
names = -> { specs.map(&:name).uniq }
source.double_check_for(names)
end
materialized.map! do |s|
next s unless s.is_a?(LazySpecification)
s.source.local!
s.__materialize__ || s
end
SpecSet.new(materialized)
@ -88,17 +81,10 @@ module Bundler
# This is in contrast to how for does platform filtering (and specifically different from how `materialize` calls `for` only for the current platform)
# @return [Array<Gem::Specification>]
def materialized_for_all_platforms
@specs.group_by(&:source).each do |source, specs|
next unless specs.any?{|s| s.is_a?(LazySpecification) }
source.local!
source.remote!
names = -> { specs.map(&:name).uniq }
source.double_check_for(names)
end
@specs.map do |s|
next s unless s.is_a?(LazySpecification)
s.source.local!
s.source.remote!
spec = s.__materialize__
raise GemNotFound, "Could not find #{s.full_name} in any of the sources" unless spec
spec