file-find 0.4.2 → 0.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d8c5c8fe59b53934405e4cb077bceddb29f988518ccf79f7083ad40357b25fc
-  data.tar.gz: 9bad3ffcce8d90697e66b4e445bf54d5a4b899a8449c6a589ca6a558fea4a84c
+  metadata.gz: 97856aefde25218603e2094d0034a491441291c277fc88b11ae39e811ac31ab6
+  data.tar.gz: f3b1061dcdc66c673983caa648e8c580d84957522ac27476a09f2494c035ca9c
 SHA512:
-  metadata.gz: 4d0ba5a6cefdb49e55bbec64b6e6105c6e95fac8a73206039d74656b4681795a1c09726c19b5150937037dc1f433eef6144c5be5e0704d9c7b82aede164545c0
-  data.tar.gz: 190060624d7f5e47846ae7a69c5f3cdaf18d1ce0cb44037cea24951c67579dd40a492426ba0720e3186c710bd1b77f1be109c7658472e7a05d23ad37acbb3773
+  metadata.gz: 4e22e2d823397614eb4341875b3098e049c8cea6107aaf7f5b000eb2df18856f5a2aef95ee7c649f1342e1233569f76cc9ab83dc80721a31fb0b08f58f6706ff
+  data.tar.gz: a8d03101ccd31356d8e735fad4a82e0abbde94d1d82c42ea17cdcc39fe935be4063818addb9f6275cf04b992bdb69d6676f177da395de095c577683543a20df1

data/CHANGES

@@ -1,3 +1,6 @@
+== 0.4.3 - 8-Jun-2020
+* Fixed a bug with ELOOP handling. Thanks go to joast for the spot and fix.
+
 == 0.4.2 - 6-May-2020
 * Added LICENSE file as required by the Apache-2.0 license.
 

data/file-find.gemspec

@@ -2,7 +2,7 @@ require 'rubygems'
 
 Gem::Specification.new do |spec|
   spec.name       = 'file-find'
-  spec.version    = '0.4.2'
+  spec.version    = '0.4.3'
   spec.author     = 'Daniel Berger'
   spec.license    = 'Apache-2.0'
   spec.summary    = 'A better way to find files'

data/lib/file/find.rb

@@ -9,7 +9,7 @@ end
 
 class File::Find
   # The version of the file-find library
-  VERSION = '0.4.2'.freeze
+  VERSION = '0.4.3'.freeze
 
   # :stopdoc:
   VALID_OPTIONS = %w[
@@ -235,8 +235,10 @@ class File::Find
           rescue Errno::ENOENT, Errno::EACCES
             next
           rescue Errno::ELOOP
-            stat_method = :lstat # Handle recursive symlinks
-            retry if stat_method.to_s != 'lstat'
+            if stat_method.to_s != 'lstat'
+              stat_method = :lstat # Handle recursive symlinks
+              retry
+            end
           end
 
           # We need to escape any brackets in the directory name.

data/test/test_file_find.rb

@@ -8,6 +8,7 @@ require 'test-unit'
 require 'fileutils'
 require 'file/find'
 require 'sys/admin'
+require 'tmpdir'
 
 if File::ALT_SEPARATOR
   require 'win32/file'
@@ -68,7 +69,7 @@ class TC_File_Find < Test::Unit::TestCase
   end
 
   test "version constant is set to expected value" do
-    assert_equal('0.4.2', File::Find::VERSION)
+    assert_equal('0.4.3', File::Find::VERSION)
     assert_true(File::Find::VERSION.frozen?)
   end
 
@@ -538,6 +539,26 @@ class TC_File_Find < Test::Unit::TestCase
     assert_raise(ArgumentError){ File::Find.new(:bogus? => true) }
   end
 
+  # TODO: Update test for Windows
+  test 'eloop handling works as expected' do
+    omit_if(@@windows, 'eloop handling test skipped on MS Windows')
+
+    begin
+      dir_eloop = ::Dir.mktmpdir
+
+      Dir.chdir(dir_eloop) do
+        File.symlink('eloop0', 'eloop1')
+        File.symlink('eloop1', 'eloop0')
+        expecting = ['./eloop0', './eloop1']
+
+        results = File::Find.new(:path => '.', :follow => true).find
+        assert_equal(expecting, results.sort)
+      end
+    ensure
+      rm_rf(dir_eloop)
+    end
+  end
+
   def teardown
     rm_rf(@file_rb)
     rm_rf(@file_txt1)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: file-find
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.3
 platform: ruby
 authors:
 - Daniel Berger
@@ -131,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: A better way to find files