ruby-changes:27545
From: drbrain <ko1@a...>
Date: Tue, 5 Mar 2013 12:33:55 +0900 (JST)
Subject: [ruby-changes:27545] drbrain:r39597 (trunk): * lib/rubygems.rb: Bump version to 2.0.1 for upcoming bugfix release
drbrain 2013-03-05 12:31:28 +0900 (Tue, 05 Mar 2013) New Revision: 39597 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=39597 Log: * lib/rubygems.rb: Bump version to 2.0.1 for upcoming bugfix release * lib/rubygems/ext/ext_conf_builder.rb: Restore ruby 1.8 compatibility for [Bug #9698] * test/rubygems/test_gem_installer.rb: Ditto. * lib/rubygems/package.rb: Restore ruby 1.8 compatibility. * test/rubygems/test_gem_dependency_installer.rb: Fix warnings Modified files: trunk/ChangeLog trunk/lib/rubygems/ext/ext_conf_builder.rb trunk/lib/rubygems/package.rb trunk/lib/rubygems.rb trunk/test/rubygems/test_gem_dependency_installer.rb trunk/test/rubygems/test_gem_installer.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 39596) +++ ChangeLog (revision 39597) @@ -1,3 +1,15 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Tue Mar 5 12:30:55 2013 Eric Hodel <drbrain@s...> + + * lib/rubygems.rb: Bump version to 2.0.1 for upcoming bugfix release + + * lib/rubygems/ext/ext_conf_builder.rb: Restore ruby 1.8 compatibility + for [Bug #9698] + * test/rubygems/test_gem_installer.rb: Ditto. + + * lib/rubygems/package.rb: Restore ruby 1.8 compatibility. + + * test/rubygems/test_gem_dependency_installer.rb: Fix warnings + Tue Mar 5 12:24:23 2013 Eric Hodel <drbrain@s...> * enumerator.c (enumerator_with_index): Restore handling of a nil memo Index: lib/rubygems/ext/ext_conf_builder.rb =================================================================== --- lib/rubygems/ext/ext_conf_builder.rb (revision 39596) +++ lib/rubygems/ext/ext_conf_builder.rb (revision 39597) @@ -13,41 +13,45 @@ class Gem::Ext::ExtConfBuilder < Gem::Ex https://github.com/ruby/ruby/blob/trunk/lib/rubygems/ext/ext_conf_builder.rb#L13 FileEntry = FileUtils::Entry_ # :nodoc: def self.build(extension, directory, dest_path, results, args=[]) - tmp_dest = (Dir.mktmpdir(".gem.", ".") if File.identical?(dest_path, ".")) + tmp_dest = Dir.mktmpdir(".gem.", ".") if File.identical?(dest_path, ".") - siteconf = Tempfile.open(%w"siteconf .rb", ".") do |f| - f.puts "require 'rbconfig'" - f.puts "dest_path = #{(tmp_dest || dest_path).dump}" + Tempfile.open %w"siteconf .rb", "." do |siteconf| + siteconf.puts "require 'rbconfig'" + siteconf.puts "dest_path = #{(tmp_dest || dest_path).dump}" %w[sitearchdir sitelibdir].each do |dir| - f.puts "RbConfig::MAKEFILE_CONFIG['#{dir}'] = dest_path" - f.puts "RbConfig::CONFIG['#{dir}'] = dest_path" + siteconf.puts "RbConfig::MAKEFILE_CONFIG['#{dir}'] = dest_path" + siteconf.puts "RbConfig::CONFIG['#{dir}'] = dest_path" end - f - end - rubyopt = ENV["RUBYOPT"] - ENV["RUBYOPT"] = ["-r#{siteconf.path}", rubyopt].compact.join(' ') - cmd = [Gem.ruby, File.basename(extension), *args].join ' ' + siteconf.flush + + rubyopt = ENV["RUBYOPT"] + destdir = ENV["DESTDIR"] + + begin + ENV["RUBYOPT"] = ["-r#{siteconf.path}", rubyopt].compact.join(' ') + cmd = [Gem.ruby, File.basename(extension), *args].join ' ' - run cmd, results + run cmd, results - destdir = ENV["DESTDIR"] - ENV["DESTDIR"] = nil + ENV["DESTDIR"] = nil - make dest_path, results + make dest_path, results - if tmp_dest - FileEntry.new(tmp_dest).traverse do |ent| - destent = ent.class.new(dest_path, ent.rel) - destent.exist? or File.rename(ent.path, destent.path) + if tmp_dest + FileEntry.new(tmp_dest).traverse do |ent| + destent = ent.class.new(dest_path, ent.rel) + destent.exist? or File.rename(ent.path, destent.path) + end + end + + results + ensure + ENV["RUBYOPT"] = rubyopt + ENV["DESTDIR"] = destdir end end - - results ensure - ENV["RUBYOPT"] = rubyopt - ENV["DESTDIR"] = destdir - siteconf.close(true) if siteconf FileUtils.rm_rf tmp_dest if tmp_dest end Index: lib/rubygems/package.rb =================================================================== --- lib/rubygems/package.rb (revision 39596) +++ lib/rubygems/package.rb (revision 39597) @@ -387,8 +387,9 @@ EOM https://github.com/ruby/ruby/blob/trunk/lib/rubygems/package.rb#L387 @spec = Gem::Specification.from_yaml entry.read when 'metadata.gz' then args = [entry] - args << { :external_encoding => Encoding::UTF_8 } unless - Zlib::GzipReader.method(:wrap).arity == 1 + args << { :external_encoding => Encoding::UTF_8 } if + Object.const_defined?(:Encoding) && + Zlib::GzipReader.method(:wrap).arity != 1 Zlib::GzipReader.wrap(*args) do |gzio| @spec = Gem::Specification.from_yaml gzio.read Index: lib/rubygems.rb =================================================================== --- lib/rubygems.rb (revision 39596) +++ lib/rubygems.rb (revision 39597) @@ -8,7 +8,7 @@ https://github.com/ruby/ruby/blob/trunk/lib/rubygems.rb#L8 require 'rbconfig' module Gem - VERSION = '2.0.0' + VERSION = '2.0.1' end # Must be first since it unloads the prelude from 1.9.2 Index: test/rubygems/test_gem_dependency_installer.rb =================================================================== --- test/rubygems/test_gem_dependency_installer.rb (revision 39596) +++ test/rubygems/test_gem_dependency_installer.rb (revision 39597) @@ -37,7 +37,7 @@ class TestGemDependencyInstaller < Gem:: https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_dependency_installer.rb#L37 def test_available_set_for_name util_setup_gems - p1a, gem = util_gem 'a', '10.a' + p1a, = util_gem 'a', '10.a' util_setup_spec_fetcher p1a, @a1, @a1_pre inst = Gem::DependencyInstaller.new @@ -49,7 +49,7 @@ class TestGemDependencyInstaller < Gem:: https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_dependency_installer.rb#L49 def test_available_set_for_name_prerelease util_setup_gems - p1a, gem = util_gem 'a', '10.a' + p1a, = util_gem 'a', '10.a' util_setup_spec_fetcher p1a, @a1, @a1_pre inst = Gem::DependencyInstaller.new :prerelease => true @@ -62,7 +62,7 @@ class TestGemDependencyInstaller < Gem:: https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_dependency_installer.rb#L62 def test_available_set_for_dep util_setup_gems - p1a, gem = util_gem 'a', '10.a' + p1a, = util_gem 'a', '10.a' util_setup_spec_fetcher p1a, @a1, @a1_pre inst = Gem::DependencyInstaller.new @@ -76,7 +76,7 @@ class TestGemDependencyInstaller < Gem:: https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_dependency_installer.rb#L76 def test_available_set_for_dep_prerelease util_setup_gems - p1a, gem = util_gem 'a', '10.a' + p1a, = util_gem 'a', '10.a' util_setup_spec_fetcher p1a, @a1, @a1_pre inst = Gem::DependencyInstaller.new :prerelease => true Index: test/rubygems/test_gem_installer.rb =================================================================== --- test/rubygems/test_gem_installer.rb (revision 39596) +++ test/rubygems/test_gem_installer.rb (revision 39597) @@ -1060,6 +1060,7 @@ gem 'other', version https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_installer.rb#L1060 end def test_install_extension_flat + skip '1.8 mkmf.rb does not create TOUCH' if RUBY_VERSION < '1.9' @spec.require_paths = ["."] @spec.extensions << "extconf.rb" @@ -1089,6 +1090,13 @@ gem 'other', version https://github.com/ruby/ruby/blob/trunk/test/rubygems/test_gem_installer.rb#L1090 @installer.install end assert File.exist?(so) + rescue + puts '-' * 78 + puts File.read File.join(@gemhome, 'gems', 'a-2', 'Makefile') + puts '-' * 78 + puts File.read File.join(@gemhome, 'gems', 'a-2', 'gem_make.out') + puts '-' * 78 + raise end def test_installation_satisfies_dependency_eh -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/