diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | common.mk | 4 | ||||
-rw-r--r-- | gem_prelude.rb | 120 | ||||
-rw-r--r-- | lib/rubygems.rb | 25 | ||||
-rw-r--r-- | lib/rubygems/custom_require.rb | 17 | ||||
-rw-r--r-- | test/rubygems/gems/current/gems/rubygems-bug-child-1.0/lib/rubygems-bug-child.rb | 1 | ||||
-rw-r--r-- | test/rubygems/gems/current/gems/rubygems-bug-child-1.1/lib/rubygems-bug-child.rb | 1 | ||||
-rw-r--r-- | test/rubygems/gems/current/gems/rubygems-bug-parent-1.0/lib/rubygems-bug-parent.rb | 1 | ||||
-rw-r--r-- | test/rubygems/gems/current/specifications/rubygems-bug-child-1.0.gemspec | 29 | ||||
-rw-r--r-- | test/rubygems/gems/current/specifications/rubygems-bug-child-1.1.gemspec | 29 | ||||
-rw-r--r-- | test/rubygems/gems/current/specifications/rubygems-bug-parent-1.0.gemspec | 32 | ||||
-rw-r--r-- | test/rubygems/test_gem_activation.rb | 20 |
12 files changed, 171 insertions, 113 deletions
@@ -1,3 +1,8 @@ +Tue Jun 8 06:27:09 2010 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * gem_prelude.rb: load full rubygems at LoadError for activation + check. [ruby-core:29486] + Tue Jun 8 06:04:49 2010 Nobuyoshi Nakada <nobu@ruby-lang.org> * io.c (parse_mode_enc): set set_by_bom bit. [ruby-core:30641] @@ -734,7 +734,9 @@ known_errors.inc: $(srcdir)/template/known_errors.inc.tmpl $(srcdir)/defs/known_ miniprelude.c: $(srcdir)/tool/compile_prelude.rb $(srcdir)/prelude.rb $(BASERUBY) -I$(srcdir) $(srcdir)/tool/compile_prelude.rb $(srcdir)/prelude.rb $@ -prelude.c: $(srcdir)/tool/compile_prelude.rb $(RBCONFIG) $(srcdir)/lib/rubygems/defaults.rb $(PRELUDE_SCRIPTS) $(PREP) +prelude.c: $(srcdir)/tool/compile_prelude.rb $(RBCONFIG) \ + $(srcdir)/lib/rubygems/defaults.rb $(srcdir)/lib/rubygems/custom_require.rb \ + $(PRELUDE_SCRIPTS) $(PREP) $(COMPILE_PRELUDE) $(PRELUDE_SCRIPTS) $@ golf_prelude.c: $(srcdir)/tool/compile_prelude.rb $(RBCONFIG) $(srcdir)/prelude.rb $(srcdir)/golf_prelude.rb $(PREP) diff --git a/gem_prelude.rb b/gem_prelude.rb index 71f30bd9ce..3e1b43f74b 100644 --- a/gem_prelude.rb +++ b/gem_prelude.rb @@ -1,4 +1,3 @@ -# depends on: array.rb dir.rb env.rb file.rb hash.rb module.rb regexp.rb # vim: filetype=ruby # NOTICE: Ruby is during initialization here. @@ -13,7 +12,8 @@ if defined?(Gem) then module Kernel def gem(gem_name, *version_requirements) - Gem.push_gem_version_on_load_path(gem_name, *version_requirements) + Gem::QuickLoader.load_full_rubygems_library + gem gem_name, *version_requirements end private :gem end @@ -133,29 +133,27 @@ if defined?(Gem) then end module QuickLoader - - @loaded_full_rubygems_library = false - def self.load_full_rubygems_library - return if @loaded_full_rubygems_library + return @full_rubygems_library if defined?(@full_rubygems_library) - @loaded_full_rubygems_library = true + @full_rubygems_library = false class << Gem undef_method(*Gem::GEM_PRELUDE_METHODS) - undef_method :const_missing - undef_method :method_missing - end - - Kernel.module_eval do - undef_method :gem if method_defined? :gem end $".delete path_to_full_rubygems_library if $".any? {|path| path.end_with?('/rubygems.rb')} raise LoadError, "another rubygems is already loaded from #{path}" end + + verbose, debug = $VERBOSE, $DEBUG + $VERBOSE = $DEBUG = nil + require 'rubygems' + @full_rubygems_library = true + ensure + $VERBOSE, $DEBUG = verbose, debug end def self.fake_rubygems_as_loaded @@ -177,95 +175,6 @@ if defined?(Gem) then end end - GemPaths = {} - GemVersions = {} - - def push_gem_version_on_load_path(gem_name, *version_requirements) - if version_requirements.empty? - unless GemPaths.has_key?(gem_name) then - raise Gem::LoadError, "Could not find RubyGem #{gem_name} (>= 0)\n" - end - - # highest version gems already active - return false - else - if version_requirements.length > 1 then - QuickLoader.load_full_rubygems_library - return gem(gem_name, *version_requirements) - end - - requirement, version = version_requirements[0].split - requirement.strip! - - if loaded_version = GemVersions[gem_name] then - case requirement - when ">", ">=" then - return false if - (loaded_version <=> Gem.integers_for(version)) >= 0 - when "~>" then - required_version = Gem.integers_for version - - return false if loaded_version.first == required_version.first - end - end - - QuickLoader.load_full_rubygems_library - gem gem_name, *version_requirements - end - end - - def integers_for(gem_version) - numbers = gem_version.split(".").collect {|n| n.to_i} - numbers.pop while numbers.last == 0 - numbers << 0 if numbers.empty? - numbers - end - - def push_all_highest_version_gems_on_load_path - Gem.path.each do |path| - gems_directory = File.join(path, "gems") - - if File.exist?(gems_directory) then - Dir.entries(gems_directory).each do |gem_directory_name| - next if gem_directory_name == "." || gem_directory_name == ".." - - next unless gem_name = gem_directory_name[/(.*)-(.*)/, 1] - new_version = integers_for($2) - current_version = GemVersions[gem_name] - - if !current_version or (current_version <=> new_version) < 0 then - GemVersions[gem_name] = new_version - GemPaths[gem_name] = File.join(gems_directory, gem_directory_name) - end - end - end - end - - require_paths = [] - - GemPaths.each_value do |path| - if File.exist?(file = File.join(path, ".require_paths")) then - paths = File.read(file).split.map do |require_path| - File.join path, require_path - end - - require_paths.concat paths - else - require_paths << file if File.exist?(file = File.join(path, "bin")) - require_paths << file if File.exist?(file = File.join(path, "lib")) - end - end - - # "tag" the first require_path inserted into the $LOAD_PATH to enable - # indexing correctly with rubygems proper when it inserts an explicitly - # gem version - unless require_paths.empty? then - require_paths.first.instance_variable_set(:@gem_prelude_index, true) - end - # gem directories must come after -I and ENV['RUBYLIB'] - $:[$:.index{|e|e.instance_variable_defined?(:@gem_prelude_index)}||-1,0] = require_paths - end - def const_missing(constant) QuickLoader.load_full_rubygems_library @@ -285,10 +194,15 @@ if defined?(Gem) then extend QuickLoader + def self.try_activate(path) # :nodoc: + QuickLoader.load_full_rubygems_library + end + end begin - Gem.push_all_highest_version_gems_on_load_path + require 'lib/rubygems/custom_require.rb' + Gem::QuickLoader.fake_rubygems_as_loaded rescue Exception => e puts "Error loading gem paths on load path in gem_prelude" diff --git a/lib/rubygems.rb b/lib/rubygems.rb index d2e0f2cb5d..1dee815a15 100644 --- a/lib/rubygems.rb +++ b/lib/rubygems.rb @@ -1020,8 +1020,6 @@ end module Kernel - undef gem if respond_to? :gem # defined in gem_prelude.rb on 1.9 - ## # Use Kernel#gem to activate a specific version of +gem_name+. # @@ -1098,13 +1096,34 @@ end require 'rubygems/config_file' +class << Gem + verbose, debug = $VERBOSE, $DEBUG + $VERBOSE = $DEBUG = nil + + ## + # + # Called from the custom_require to attempt to activate +path+ + # Internal use only. + + def try_activate(path) # :doc: + spec = Gem.searcher.find(path) + return false unless spec + + Gem.activate(spec.name, "= #{spec.version}") + return true + end + +ensure + $VERBOSE, $DEBUG = verbose, debug +end + ## # Enables the require hook for RubyGems. # # Ruby 1.9 allows --disable-gems, so we require it when we didn't detect a Gem # constant at rubygems.rb load time. -require 'rubygems/custom_require' if gem_disabled or RUBY_VERSION < '1.9' +require 'rubygems/custom_require' Gem.clear_paths diff --git a/lib/rubygems/custom_require.rb b/lib/rubygems/custom_require.rb index 43b3136b60..36e3c78911 100644 --- a/lib/rubygems/custom_require.rb +++ b/lib/rubygems/custom_require.rb @@ -4,8 +4,6 @@ # See LICENSE.txt for permissions. #++ -require 'rubygems' - module Kernel ## @@ -31,10 +29,17 @@ module Kernel gem_original_require path rescue LoadError => load_error if load_error.message.end_with?(path) and - spec = Gem.searcher.find(path) then - Gem.activate(spec.name, "= #{spec.version}") - gem_original_require path + begin + Gem.try_activate(path) + rescue Gem::LoadError => load_error + pat = "#{__FILE__}:#{__LINE__-2}:in " + bt = load_error.backtrace + num = bt.index {|e| e.start_with?(pat)} and bt.shift(num+2) + raise load_error + end + retry else + load_error.backtrace.shift(2) raise load_error end end @@ -42,5 +47,5 @@ module Kernel private :require private :gem_original_require -end +end unless Kernel.private_method_defined?(:gem_original_require) diff --git a/test/rubygems/gems/current/gems/rubygems-bug-child-1.0/lib/rubygems-bug-child.rb b/test/rubygems/gems/current/gems/rubygems-bug-child-1.0/lib/rubygems-bug-child.rb new file mode 100644 index 0000000000..4544e24d99 --- /dev/null +++ b/test/rubygems/gems/current/gems/rubygems-bug-child-1.0/lib/rubygems-bug-child.rb @@ -0,0 +1 @@ +$bug_3140 = "1.0" diff --git a/test/rubygems/gems/current/gems/rubygems-bug-child-1.1/lib/rubygems-bug-child.rb b/test/rubygems/gems/current/gems/rubygems-bug-child-1.1/lib/rubygems-bug-child.rb new file mode 100644 index 0000000000..92edd5f91c --- /dev/null +++ b/test/rubygems/gems/current/gems/rubygems-bug-child-1.1/lib/rubygems-bug-child.rb @@ -0,0 +1 @@ +$bug_3140 = "1.1" diff --git a/test/rubygems/gems/current/gems/rubygems-bug-parent-1.0/lib/rubygems-bug-parent.rb b/test/rubygems/gems/current/gems/rubygems-bug-parent-1.0/lib/rubygems-bug-parent.rb new file mode 100644 index 0000000000..2a8eaba1f6 --- /dev/null +++ b/test/rubygems/gems/current/gems/rubygems-bug-parent-1.0/lib/rubygems-bug-parent.rb @@ -0,0 +1 @@ +require "rubygems-bug-child" diff --git a/test/rubygems/gems/current/specifications/rubygems-bug-child-1.0.gemspec b/test/rubygems/gems/current/specifications/rubygems-bug-child-1.0.gemspec new file mode 100644 index 0000000000..4dc6cfdb31 --- /dev/null +++ b/test/rubygems/gems/current/specifications/rubygems-bug-child-1.0.gemspec @@ -0,0 +1,29 @@ +# -*- encoding: utf-8 -*- + +Gem::Specification.new do |s| + s.name = %q{rubygems-bug-child} + s.version = "1.0" + + s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version= + s.authors = ["Yehuda Katz"] + s.date = %q{2010-04-12} + s.description = %q{Demonstrates a rubygems bug that exists in 1.9 because of gem prelude but not 1.8} + s.email = %q{wycats@gmail.com} + s.files = ["lib/rubygems-bug-child.rb"] + s.homepage = %q{http://www.yehudakatz.com} + s.require_paths = ["lib"] + s.required_ruby_version = Gem::Requirement.new(">= 1.8.7") + s.rubyforge_project = %q{rubygems-bug} + s.rubygems_version = %q{1.3.6} + s.summary = %q{Demonstrates a rubygems bug} + + if s.respond_to? :specification_version then + current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION + s.specification_version = 3 + + if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then + else + end + else + end +end diff --git a/test/rubygems/gems/current/specifications/rubygems-bug-child-1.1.gemspec b/test/rubygems/gems/current/specifications/rubygems-bug-child-1.1.gemspec new file mode 100644 index 0000000000..6428ee04c8 --- /dev/null +++ b/test/rubygems/gems/current/specifications/rubygems-bug-child-1.1.gemspec @@ -0,0 +1,29 @@ +# -*- encoding: utf-8 -*- + +Gem::Specification.new do |s| + s.name = %q{rubygems-bug-child} + s.version = "1.1" + + s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version= + s.authors = ["Yehuda Katz"] + s.date = %q{2010-04-12} + s.description = %q{Demonstrates a rubygems bug that exists in 1.9 because of gem prelude but not 1.8} + s.email = %q{wycats@gmail.com} + s.files = ["lib/rubygems-bug-child.rb"] + s.homepage = %q{http://www.yehudakatz.com} + s.require_paths = ["lib"] + s.required_ruby_version = Gem::Requirement.new(">= 1.8.7") + s.rubyforge_project = %q{rubygems-bug} + s.rubygems_version = %q{1.3.6} + s.summary = %q{Demonstrates a rubygems bug} + + if s.respond_to? :specification_version then + current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION + s.specification_version = 3 + + if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then + else + end + else + end +end diff --git a/test/rubygems/gems/current/specifications/rubygems-bug-parent-1.0.gemspec b/test/rubygems/gems/current/specifications/rubygems-bug-parent-1.0.gemspec new file mode 100644 index 0000000000..0f714c0624 --- /dev/null +++ b/test/rubygems/gems/current/specifications/rubygems-bug-parent-1.0.gemspec @@ -0,0 +1,32 @@ +# -*- encoding: utf-8 -*- + +Gem::Specification.new do |s| + s.name = %q{rubygems-bug-parent} + s.version = "1.0" + + s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version= + s.authors = ["Yehuda Katz"] + s.date = %q{2010-04-12} + s.description = %q{Demonstrates a rubygems bug that exists in 1.9 because of gem prelude but not 1.8} + s.email = %q{wycats@gmail.com} + s.files = ["lib/rubygems-bug-parent.rb"] + s.homepage = %q{http://www.yehudakatz.com} + s.require_paths = ["lib"] + s.required_ruby_version = Gem::Requirement.new(">= 1.8.7") + s.rubyforge_project = %q{rubygems-bug} + s.rubygems_version = %q{1.3.6} + s.summary = %q{Demonstrates a rubygems bug} + + if s.respond_to? :specification_version then + current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION + s.specification_version = 3 + + if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then + s.add_runtime_dependency(%q<rubygems-bug-child>, ["= 1.0.0"]) + else + s.add_dependency(%q<rubygems-bug-child>, ["= 1.0.0"]) + end + else + s.add_dependency(%q<rubygems-bug-child>, ["= 1.0.0"]) + end +end diff --git a/test/rubygems/test_gem_activation.rb b/test/rubygems/test_gem_activation.rb new file mode 100644 index 0000000000..d39f9b3c59 --- /dev/null +++ b/test/rubygems/test_gem_activation.rb @@ -0,0 +1,20 @@ +require_relative '../ruby/envutil' +require 'test/unit' + +class TestGemActivation < Test::Unit::TestCase + def test_activation + bug3140 = '[ruby-core:29486]' + src = %{begin + require 'rubygems-bug-parent' +rescue Gem::LoadError + puts $! +else + puts $bug_3140 +end} + basedir = File.expand_path("../gems/current", __FILE__) + env = {"HOME"=>basedir, "GEM_HOME"=>basedir, "GEM_PATH"=>basedir} + assert_in_out_err([env, "-rrubygems-bug-child", "-e", src], "", + /can't activate rubygems-bug-child.*already activated rubygems-bug-child-1\.1/, [], + $bug_3140) + end +end if defined?(::Gem) |