Revert ruby/rubygems@d74fa0f

Author: hsbt

spec/bundler/bin/parallel_rspec

@@ -95,6 +95,8 @@ def self.ruby=(ruby)
 
     extend(Spec::Builders)
 
+    check_test_gems!
+
     build_repo1
 
     reset_paths!
@@ -117,4 +119,8 @@ def self.ruby=(ruby)
   ensure
     reset!
   end
+
+  config.after :suite do
+    FileUtils.rm_r Spec::Path.pristine_system_gem_path
+  end
 end

spec/bundler/spec_helper.rb

@@ -276,6 +276,14 @@ def build_repo(path, **kwargs, &blk)
       update_repo(path,**kwargs, &blk)
     end
 
+    def check_test_gems!
+      if rake_path.nil?
+        Spec::Rubygems.install_test_deps
+      end
+
+      Helpers.install_dev_bundler unless pristine_system_gem_path.exist?
+    end
+
     def update_repo(path, build_compact_index: true)
       exempted_caller = Gem.ruby_version >= Gem::Version.new("3.4.0.dev") ? "#{Module.nesting.first}#build_repo" : "build_repo"
       if path == gem_repo1 && caller_locations(1, 1).first.label != exempted_caller
@@ -445,7 +453,6 @@ def _build(options = {})
         build_path = @context.tmp + full_name
         bundler_path = build_path + "#{full_name}.gem"
 
-        require "fileutils"
         FileUtils.mkdir_p build_path
 
         @context.shipped_files.each do |shipped_file|
@@ -454,7 +461,7 @@ def _build(options = {})
           target_shipped_file = build_path + target_shipped_file
           target_shipped_dir = File.dirname(target_shipped_file)
           FileUtils.mkdir_p target_shipped_dir unless File.directory?(target_shipped_dir)
-          FileUtils.cp File.expand_path(shipped_file, @context.source_root), target_shipped_file, preserve: true
+          FileUtils.cp shipped_file, target_shipped_file, preserve: true
         end
 
         @context.replace_version_file(@version, dir: build_path)

spec/bundler/support/builders.rb

@@ -25,8 +25,12 @@ def relative_gemspec
       @relative_gemspec ||= ruby_core? ? "lib/bundler/bundler.gemspec" : "bundler.gemspec"
     end
 
+    def gemspec_dir
+      @gemspec_dir ||= gemspec.parent
+    end
+
     def loaded_gemspec
-      @loaded_gemspec ||= Dir.chdir(source_root) { Gem::Specification.load(gemspec.to_s) }
+      @loaded_gemspec ||= Gem::Specification.load(gemspec.to_s)
     end
 
     def test_gemfile
@@ -98,11 +102,11 @@ def man_tracked_files
     end
 
     def tmp(*path)
-      tmp_root.join("#{test_env_version}.#{scope}").join(*path)
+      tmp_root(scope).join(*path)
     end
 
-    def tmp_root
-      source_root.join("tmp")
+    def tmp_root(scope)
+      source_root.join("tmp", "#{test_env_version}.#{scope}")
     end
 
     # Bump this version whenever you make a breaking change to the spec setup
@@ -176,15 +180,15 @@ def base_system_gem_path
     end
 
     def base_system_gems
-      tmp_root.join("gems/base")
+      tmp("gems/base")
     end
 
     def rubocop_gems
-      tmp_root.join("gems/rubocop")
+      tmp("gems/rubocop")
     end
 
     def standard_gems
-      tmp_root.join("gems/standard")
+      tmp("gems/standard")
     end
 
     def file_uri_for(path)
@@ -223,7 +227,7 @@ def system_gem_path(*path)
     end
 
     def pristine_system_gem_path
-      tmp_root.join("gems/pristine_system")
+      tmp("gems/base_system")
     end
 
     def local_gem_path(*path, base: bundled_app)
@@ -277,7 +281,7 @@ def git_root
     end
 
     def rake_path
-      Dir["#{base_system_gems}/*/*/**/rake*.gem"].first
+      Dir["#{base_system_gems}/#{Bundler.ruby_scope}/**/rake*.gem"].first
     end
 
     def sinatra_dependency_paths

spec/bundler/support/path.rb

@@ -10,6 +10,10 @@ module Spec
   module Rubygems
     extend self
 
+    def dev_setup
+      install_gems(dev_gemfile)
+    end
+
     def gem_load(gem_name, bin_container)
       require_relative "switch_rubygems"
 
@@ -46,19 +50,41 @@ def test_setup
       Gem::DefaultUserInteraction.ui = Gem::SilentUI.new
     end
 
+    def install_parallel_test_deps
+      Gem.clear_paths
+
+      require "parallel"
+      require "fileutils"
+
+      install_test_deps
+
+      (2..Parallel.processor_count).each do |n|
+        source = Path.tmp_root("1")
+        destination = Path.tmp_root(n.to_s)
+
+        FileUtils.cp_r source, destination, remove_destination: true
+      end
+    end
+
     def setup_test_paths
+      Gem.clear_paths
+
       ENV["BUNDLE_PATH"] = nil
+      ENV["GEM_HOME"] = ENV["GEM_PATH"] = Path.base_system_gem_path.to_s
       ENV["PATH"] = [Path.system_gem_path("bin"), ENV["PATH"]].join(File::PATH_SEPARATOR)
       ENV["PATH"] = [Path.bindir, ENV["PATH"]].join(File::PATH_SEPARATOR) if Path.ruby_core?
     end
 
     def install_test_deps
-      dev_bundle("install", gemfile: test_gemfile, path: Path.base_system_gems.to_s)
-      dev_bundle("install", gemfile: rubocop_gemfile, path: Path.rubocop_gems.to_s)
-      dev_bundle("install", gemfile: standard_gemfile, path: Path.standard_gems.to_s)
+      Gem.clear_paths
+
+      install_gems(test_gemfile, Path.base_system_gems.to_s)
+      install_gems(rubocop_gemfile, Path.rubocop_gems.to_s)
+      install_gems(standard_gemfile, Path.standard_gems.to_s)
 
-      require_relative "helpers"
-      Helpers.install_dev_bundler
+      # For some reason, doing this here crashes on JRuby + Windows. So defer to
+      # when the test suite is running in that case.
+      Helpers.install_dev_bundler unless Gem.win_platform? && RUBY_ENGINE == "jruby"
     end
 
     def check_source_control_changes(success_message:, error_message:)
@@ -81,36 +107,6 @@ def check_source_control_changes(success_message:, error_message:)
       end
     end
 
-    def dev_bundle(*args, gemfile: dev_gemfile, path: nil)
-      old_gemfile = ENV["BUNDLE_GEMFILE"]
-      old_orig_gemfile = ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"]
-      ENV["BUNDLE_GEMFILE"] = gemfile.to_s
-      ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"] = nil
-
-      if path
-        old_path = ENV["BUNDLE_PATH"]
-        ENV["BUNDLE_PATH"] = path
-      else
-        old_path__system = ENV["BUNDLE_PATH__SYSTEM"]
-        ENV["BUNDLE_PATH__SYSTEM"] = "true"
-      end
-
-      require "shellwords"
-      # We don't use `Open3` here because it does not work on JRuby + Windows
-      output = `ruby #{File.expand_path("support/bundle.rb", Path.spec_dir)} #{args.shelljoin}`
-      raise output unless $?.success?
-      output
-    ensure
-      if path
-        ENV["BUNDLE_PATH"] = old_path
-      else
-        ENV["BUNDLE_PATH__SYSTEM"] = old_path__system
-      end
-
-      ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"] = old_orig_gemfile
-      ENV["BUNDLE_GEMFILE"] = old_gemfile
-    end
-
     private
 
     def gem_load_and_activate(gem_name, bin_container)
@@ -139,6 +135,34 @@ def gem_activate(gem_name)
       gem gem_name, gem_requirement
     end
 
+    def install_gems(gemfile, path = nil)
+      old_gemfile = ENV["BUNDLE_GEMFILE"]
+      old_orig_gemfile = ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"]
+      ENV["BUNDLE_GEMFILE"] = gemfile.to_s
+      ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"] = nil
+
+      if path
+        old_path = ENV["BUNDLE_PATH"]
+        ENV["BUNDLE_PATH"] = path
+      else
+        old_path__system = ENV["BUNDLE_PATH__SYSTEM"]
+        ENV["BUNDLE_PATH__SYSTEM"] = "true"
+      end
+
+      # We don't use `Open3` here because it does not work on JRuby + Windows
+      output = `#{Gem.ruby} #{File.expand_path("support/bundle.rb", Path.spec_dir)} install`
+      raise output unless $?.success?
+    ensure
+      if path
+        ENV["BUNDLE_PATH"] = old_path
+      else
+        ENV["BUNDLE_PATH__SYSTEM"] = old_path__system
+      end
+
+      ENV["BUNDLER_ORIG_BUNDLE_GEMFILE"] = old_orig_gemfile
+      ENV["BUNDLE_GEMFILE"] = old_gemfile
+    end
+
     def test_gemfile
       Path.test_gemfile
     end