Fixed to warn with shadowing outer local variable.

rubygems · Jul 30, 2019 · b0588a8 · b0588a8
1 parent 18c0e86
commit b0588a8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/rubygems/ext/ext_conf_builder.rb b/lib/rubygems/ext/ext_conf_builder.rb
@@ -45,12 +45,12 @@ def self.build(extension, dest_path, results, args=[], lib_dir=nil)
         cmd.push(*args)
 
         begin
-          run(cmd, results) do |status, results|
+          run(cmd, results) do |s, r|
             if File.exist? 'mkmf.log'
-              unless status.success?
-                results << "To see why this extension failed to compile, please check" \
+              unless s.success?
+                r << "To see why this extension failed to compile, please check" \
                   " the mkmf.log which can be found here:\n"
-                results << "  " + File.join(dest_path, 'mkmf.log') + "\n"
+                r << "  " + File.join(dest_path, 'mkmf.log') + "\n"
               end
               FileUtils.mv 'mkmf.log', dest_path
             end