Dedup .pryrc across symlinks too

lib/pry.rb

@@ -212,6 +212,7 @@ require 'coderay'
 require 'slop'
 require 'rbconfig'
 require 'tempfile'
+require 'pathname'
 
 begin
   require 'readline'

lib/pry/pry_class.rb

@@ -68,26 +68,33 @@ class Pry
 
   # Load the given file in the context of `Pry.toplevel_binding`
   # @param [String] file_name The unexpanded file path.
-  def self.load_file_at_toplevel(file_name)
-    full_name = File.expand_path(file_name)
-    begin
-      toplevel_binding.eval(File.read(full_name), full_name) if File.exists?(full_name)
+  def self.load_file_at_toplevel(file)
+    toplevel_binding.eval(File.read(file), file)
   rescue RescuableException => e
-      puts "Error loading #{file_name}: #{e}\n#{e.backtrace.first}"
-    end
+    puts "Error loading #{file}: #{e}\n#{e.backtrace.first}"
   end
 
-  # Load the rc files given in the `Pry::RC_FILES` array.
+  # Load HOME_RC_FILE and LOCAL_RC_FILE if appropriate
   # This method can also be used to reload the files if they have changed.
-  def self.load_rc
-    load_file_at_toplevel(HOME_RC_FILE)
+  def self.load_rc_files
+    rc_files_to_load.each do |file|
+      load_file_at_toplevel(file)
+    end
   end
 
   # Load the local RC file (./.pryrc)
-  def self.load_local_rc
-    unless File.expand_path(HOME_RC_FILE) == File.expand_path(LOCAL_RC_FILE)
-      load_file_at_toplevel(LOCAL_RC_FILE)
+  def self.rc_files_to_load
+    files = []
+    files << HOME_RC_FILE if Pry.config.should_load_rc
+    files << LOCAL_RC_FILE if Pry.config.should_load_local_rc
+    files.map { |file| real_path_to(file) }.compact.uniq
   end
+
+  # Expand a file to its canonical name (following symlinks as appropriate)
+  def self.real_path_to(file)
+    Pathname.new(File.expand_path(file)).realpath.to_s
+  rescue Errno::ENOENT
+    nil
   end
 
   # Load any Ruby files specified with the -r flag on the command line.
@@ -112,8 +119,7 @@ class Pry
 
     # note these have to be loaded here rather than in pry_instance as
     # we only want them loaded once per entire Pry lifetime.
-    load_rc if Pry.config.should_load_rc
-    load_local_rc if Pry.config.should_load_local_rc
+    load_rc_files
     load_plugins if Pry.config.should_load_plugins
     load_requires if Pry.config.should_load_requires
     load_history if Pry.config.history.should_load

spec/fixtures/testlinkrc

@@ -0,0 +1 @@
+testrc

spec/pry_spec.rb

@@ -281,90 +281,6 @@ describe Pry do
         end
       end
 
-      describe "test loading rc files" do
-        before do
-          Pry::HOME_RC_FILE.replace "spec/fixtures/testrc"
-          Pry::LOCAL_RC_FILE.replace "spec/fixtures/testrc/../testrc"
-          Pry.instance_variable_set(:@initial_session, true)
-        end
-
-        after do
-          Pry::HOME_RC_FILE.replace "~/.pryrc"
-          Pry::LOCAL_RC_FILE.replace "./.pryrc"
-          Pry.config.should_load_rc = false
-          Object.remove_const(:TEST_RC) if defined?(TEST_RC)
-        end
-
-        it "should never run the rc file twice" do
-          Pry.config.should_load_rc = true
-
-          Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
-          TEST_RC.should == [0]
-
-          Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
-          TEST_RC.should == [0]
-        end
-
-        it "should not load the pryrc if it cannot expand ENV[HOME]" do
-          old_home = ENV['HOME']
-          old_rc = Pry.config.should_load_rc
-          ENV['HOME'] = nil
-          Pry.config.should_load_rc = true
-          lambda { Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new) }.should.not.raise
-
-          ENV['HOME'] = old_home
-          Pry.config.should_load_rc = old_rc
-        end
-
-        it "should not run the rc file at all if Pry.config.should_load_rc is false" do
-          Pry.config.should_load_rc = false
-          Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
-          Object.const_defined?(:TEST_RC).should == false
-        end
-
-        describe "that raise exceptions" do
-          before do
-            Pry::HOME_RC_FILE = "spec/fixtures/testrcbad"
-            Pry.config.should_load_rc = true
-            Pry.config.should_load_local_rc = false
-
-            putsed = nil
-
-            # YUCK! horrible hack to get round the fact that output is not configured
-            # at the point this message is printed.
-            (class << Pry; self; end).send(:define_method, :puts) { |str|
-              putsed = str
-            }
-
-            @doing_it = lambda{
-              Pry.start(self, :input => StringIO.new("Object::TEST_AFTER_RAISE=1\nexit-all\n"), :output => StringIO.new)
-              putsed
-            }
-          end
-
-          after do
-            Object.remove_const(:TEST_BEFORE_RAISE)
-            Object.remove_const(:TEST_AFTER_RAISE)
-            (class << Pry; undef_method :puts; end)
-          end
-
-          it "should not raise exceptions" do
-            @doing_it.should.not.raise
-          end
-
-          it "should continue to run pry" do
-            @doing_it[]
-            Object.const_defined?(:TEST_BEFORE_RAISE).should == true
-            Object.const_defined?(:TEST_AFTER_RAISE).should == true
-          end
-
-          it "should output an error" do
-            @doing_it.call.split("\n").first.should ==
-              "Error loading spec/fixtures/testrcbad: messin with ya"
-          end
-        end
-      end
-
       describe "nesting" do
         after do
           Pry.reset_defaults

spec/pryrc_spec.rb

@@ -0,0 +1,94 @@
+require 'helper'
+
+describe Pry do
+  describe 'loading rc files' do
+    before do
+      Pry::HOME_RC_FILE.replace "spec/fixtures/testrc"
+      Pry::LOCAL_RC_FILE.replace "spec/fixtures/testrc/../testrc"
+      Pry.instance_variable_set(:@initial_session, true)
+      Pry.config.should_load_rc = true
+      Pry.config.should_load_local_rc = true
+    end
+
+    after do
+      Pry::HOME_RC_FILE.replace "~/.pryrc"
+      Pry::LOCAL_RC_FILE.replace "./.pryrc"
+      Pry.config.should_load_rc = false
+      Object.remove_const(:TEST_RC) if defined?(TEST_RC)
+    end
+
+    it "should never run the rc file twice" do
+      Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
+      TEST_RC.should == [0]
+
+      Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
+      TEST_RC.should == [0]
+    end
+
+    it "should not load the rc file twice if it's symlinked differently" do
+      Pry::HOME_RC_FILE.replace "spec/fixtures/testrc"
+      Pry::LOCAL_RC_FILE.replace "spec/fixtures/testlinkrc"
+
+      Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
+
+      TEST_RC.should == [0]
+    end
+
+    it "should not load the pryrc if it cannot expand ENV[HOME]" do
+      old_home = ENV['HOME']
+      ENV['HOME'] = nil
+      Pry.config.should_load_rc = true
+      lambda { Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new) }.should.not.raise
+
+      ENV['HOME'] = old_home
+    end
+
+    it "should not run the rc file at all if Pry.config.should_load_rc is false" do
+      Pry.config.should_load_rc = false
+      Pry.config.should_load_local_rc = false
+      Pry.start(self, :input => StringIO.new("exit-all\n"), :output => StringIO.new)
+      Object.const_defined?(:TEST_RC).should == false
+    end
+
+    describe "that raise exceptions" do
+      before do
+        Pry::HOME_RC_FILE = "spec/fixtures/testrcbad"
+        Pry.config.should_load_local_rc = false
+
+        putsed = nil
+
+        # YUCK! horrible hack to get round the fact that output is not configured
+        # at the point this message is printed.
+        (class << Pry; self; end).send(:define_method, :puts) { |str|
+          putsed = str
+        }
+
+        @doing_it = lambda{
+          Pry.start(self, :input => StringIO.new("Object::TEST_AFTER_RAISE=1\nexit-all\n"), :output => StringIO.new)
+          putsed
+        }
+      end
+
+      after do
+        Object.remove_const(:TEST_BEFORE_RAISE)
+        Object.remove_const(:TEST_AFTER_RAISE)
+        (class << Pry; undef_method :puts; end)
+      end
+
+      it "should not raise exceptions" do
+        @doing_it.should.not.raise
+      end
+
+      it "should continue to run pry" do
+        @doing_it[]
+        Object.const_defined?(:TEST_BEFORE_RAISE).should == true
+        Object.const_defined?(:TEST_AFTER_RAISE).should == true
+      end
+
+      it "should output an error" do
+        @doing_it.call.split("\n").first.should =~
+          %r{Error loading .*spec/fixtures/testrcbad: messin with ya}
+      end
+    end
+  end
+end