diff options
author | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-03-17 14:27:00 +0000 |
---|---|---|
committer | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-03-17 14:27:00 +0000 |
commit | b28cb4fa85e141970457a2e9f9411d55a2e13bf6 (patch) | |
tree | 18407178872dcb61b58aa37ca5e4fb351033a5d6 /lib | |
parent | 8ff247aee71e590e05b0c85a352b0af66aa97578 (diff) |
merge revision(s) 39579: [Backport #7991]
* lib/rubygems/ext/ext_conf_builder.rb (Gem::Ext::ExtConfBuilder.build):
fix for unusal cases again. install to a temporary directory once
and move instaled files to the destination directory, if it is same
as the current directory. [Bug #7698]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@39791 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'lib')
-rw-r--r-- | lib/rubygems/ext/ext_conf_builder.rb | 30 |
1 files changed, 12 insertions, 18 deletions
diff --git a/lib/rubygems/ext/ext_conf_builder.rb b/lib/rubygems/ext/ext_conf_builder.rb index 7babf409a0..2977cf0207 100644 --- a/lib/rubygems/ext/ext_conf_builder.rb +++ b/lib/rubygems/ext/ext_conf_builder.rb @@ -10,26 +10,14 @@ require 'fileutils' require 'tempfile' class Gem::Ext::ExtConfBuilder < Gem::Ext::Builder - - def self.hack_for_obsolete_style_gems(directory) - return unless directory and File.identical?(directory, ".") - mf = Gem.read_binary 'Makefile' - changed = false - changed |= mf.gsub!(/^(install-rb-default:)(.*)/) { - "#$1#{$2.gsub(/(?:^|\s+)\$\(RUBY(?:ARCH|LIB)DIR\)\/\S+(?=\s|$)/, '')}" - } - changed |= mf.gsub!(/^(install-so:.*DLLIB.*\n)((?:\t.*\n)+)/) { - "#$1#{$2.gsub(/.*INSTALL.*DLLIB.*\n/, '')}" - } - if changed - File.open('Makefile', 'wb') {|f| f.print mf} - end - end + FileEntry = FileUtils::Entry_ # :nodoc: def self.build(extension, directory, dest_path, results, args=[]) + 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 = #{dest_path.dump}" + f.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" @@ -43,14 +31,20 @@ class Gem::Ext::ExtConfBuilder < Gem::Ext::Builder run cmd, results - hack_for_obsolete_style_gems directory - 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) + end + end + results ensure ENV["RUBYOPT"] = rubyopt siteconf.close(true) if siteconf + FileUtils.rm_rf tmp_dest if tmp_dest end end |