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

ruby-changes:57389

From: Kazuhiro <ko1@a...>
Date: Fri, 30 Aug 2019 10:22:30 +0900 (JST)
Subject: [ruby-changes:57389] Kazuhiro NISHIYAMA: 500149709b (master): require 'pp' before use PP

https://git.ruby-lang.org/ruby.git/commit/?id=500149709b

From 500149709b92ccb36396589a0c39afb3ff77bfb6 Mon Sep 17 00:00:00 2001
From: Kazuhiro NISHIYAMA <zn@m...>
Date: Fri, 30 Aug 2019 10:18:32 +0900
Subject: require 'pp' before use PP

`Kernel#pp` has wrapper, but `PP` does not.

https://github.com/ruby/ruby/runs/207405029#step:10:141
`NameError: uninitialized constant Bundler::Molinillo::Resolver::Resolution::PP`

diff --git a/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb b/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb
index b49acd4..ab6529e 100644
--- a/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb
+++ b/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb
@@ -1,4 +1,5 @@ https://github.com/ruby/ruby/blob/trunk/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb#L1
 # frozen_string_literal: true
+require 'pp'
 
 module Bundler::Molinillo
   class Resolver
diff --git a/lib/rubygems/resolver.rb b/lib/rubygems/resolver.rb
index f749957..ea9687b 100644
--- a/lib/rubygems/resolver.rb
+++ b/lib/rubygems/resolver.rb
@@ -125,7 +125,10 @@ class Gem::Resolver https://github.com/ruby/ruby/blob/trunk/lib/rubygems/resolver.rb#L125
 
     data = yield
     $stderr.printf "%10s (%d entries)\n", stage.to_s.upcase, data.size
-    PP.pp data, $stderr unless data.empty?
+    unless data.empty?
+      require 'pp'
+      PP.pp data, $stderr
+    end
   end
 
   ##
-- 
cgit v0.10.2


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

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