[前][次][番号順一覧][スレッド一覧]

ruby-changes:34065

From: nobu <ko1@a...>
Date: Mon, 26 May 2014 23:36:44 +0900 (JST)
Subject: [ruby-changes:34065] nobu:r46146 (trunk): ext_conf_builder.rb: fix tempfile leak

nobu	2014-05-26 23:36:36 +0900 (Mon, 26 May 2014)

  New Revision: 46146

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=46146

  Log:
    ext_conf_builder.rb: fix tempfile leak
    
    * lib/rubygems/ext/ext_conf_builder.rb (build): ensure to remove
      temporary siteconf script.

  Modified files:
    trunk/lib/rubygems/ext/ext_conf_builder.rb
Index: lib/rubygems/ext/ext_conf_builder.rb
===================================================================
--- lib/rubygems/ext/ext_conf_builder.rb	(revision 46145)
+++ lib/rubygems/ext/ext_conf_builder.rb	(revision 46146)
@@ -38,11 +38,11 @@ class Gem::Ext::ExtConfBuilder < Gem::Ex https://github.com/ruby/ruby/blob/trunk/lib/rubygems/ext/ext_conf_builder.rb#L38
           run cmd, results
         ensure
           FileUtils.mv 'mkmf.log', dest_path if File.exist? 'mkmf.log'
+          siteconf.unlink
         end
 
         ENV["DESTDIR"] = nil
         ENV["RUBYOPT"] = rubyopt
-        siteconf.unlink
 
         make dest_path, results
 

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]