bootsnap 1.22.0 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ab247855b4e6a35b2db80b44f2d3da184fd9780e9a4faa3d35d952e15398839
-  data.tar.gz: c8fd54ed988e11df7e934e9ff4f30f7dfb6440b4b9a2ecd2eaf3dbf9aefa61fd
+  metadata.gz: 4e0ad269f816c24dc901b6544acb59c5ebd65b22432f39ee69bc3387b9c2b04d
+  data.tar.gz: 700f525d90d4e77421ce83e38f8731981a48cc82a8b9a937c29f661972f78d7f
 SHA512:
-  metadata.gz: 4a7225b866cbe788281af91aa690c75191e19bc09f4d8695a58557e024193d4b0924bbf9892cbb84dad93658eda9ff46c609b113c0b588660ac318e0b9176f1f
-  data.tar.gz: cadb844c862e9185befec65d7fa53beb6940945ebbddbea48ab30fd22b88f19aa773baa856c414522f5acf1a1c2a8aeafa75d72fb5a86cde5ebc8557172d049c
+  metadata.gz: f5859d7fd4b81f1969c55e12b33a957b34b1dc2825d03bc3f7ab60a57d34153cbcdda7a60626ea758fd66877b1b74a58d17b276aa2055f41086488d6ced5317b
+  data.tar.gz: c09f2cd48da0ba3bb5e9d3c995ffb86b8b16aec987c311162fc79ca648a66f355e4051d6ae4c5ee486d5ad2d2bc3af32090e15776e1f45129c1ea8c8e6c811ed

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Unreleased
 
+# 1.23.0
+
+* Require Ruby 2.7.
+* Fix support for absolute paths in `BOOTSNAP_IGNORE_DIRECTORIES`.
+
 # 1.22.0
 
 * Better fix for the `opendir` crash.

data/ext/bootsnap/bootsnap.c

@@ -194,8 +194,12 @@ bs_rb_scan_dir(VALUE self, VALUE abspath)
     struct stat st;
     int dfd = -1;
 
-    errno = 0;
-    while ((entry = readdir(dirp))) {
+    while (1) {
+        errno = 0;
+
+        entry = readdir(dirp);
+        if (entry == NULL) break;
+
         if (entry->d_name[0] == '.') continue;
 
         if (RB_UNLIKELY(entry->d_type == DT_UNKNOWN || entry->d_type == DT_LNK)) {
@@ -212,11 +216,8 @@ bs_rb_scan_dir(VALUE self, VALUE abspath)
             }
 
             if (fstatat(dfd, entry->d_name, &st, 0)) {
-                if (errno == ENOENT) {
-                    // Broken symlinK
-                    errno = 0;
-                    continue;
-                }
+                if (errno == ENOENT) continue; // Broken symlink
+
                 int err = errno;
                 closedir(dirp);
                 bs_syserr_fail_dir_entry("fstatat", err, abspath, entry->d_name);
@@ -249,10 +250,15 @@ bs_rb_scan_dir(VALUE self, VALUE abspath)
         }
     }
 
-    if (closedir(dirp)) {
+    if (errno) {
+      int err = errno;
+      closedir(dirp);
+      bs_syserr_fail_path("readdir", err, abspath);
+    } else if (closedir(dirp)) {
         bs_syserr_fail_path("closedir", errno, abspath);
         return Qundef;
     }
+
     return result;
 }
 #endif

data/lib/bootsnap/compile_cache/iseq.rb

@@ -97,7 +97,7 @@ module Bootsnap
         end
 
         def compile_option=(hash)
-          super(hash)
+          super
           Bootsnap::CompileCache::ISeq.compile_option_updated
         end
       end

data/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb

@@ -5,7 +5,7 @@ module Kernel
 
   alias_method :require, :require # Avoid method redefinition warnings
 
-  def require(path) # rubocop:disable Lint/DuplicateMethods
+  def require(path)
     return require_without_bootsnap(path) unless Bootsnap::LoadPathCache.enabled?
 
     string_path = Bootsnap.rb_get_path(path)

data/lib/bootsnap/load_path_cache/path_scanner.rb

@@ -18,21 +18,12 @@ module Bootsnap
       class << self
         attr_accessor :ignored_directories
 
-        def ruby_call(path)
-          path = File.expand_path(path.to_s).freeze
-          return [] unless File.directory?(path)
-
-          # If the bundle path is a descendent of this path, we do additional
-          # checks to prevent recursing into the bundle path as we recurse
-          # through this path. We don't want to scan the bundle path because
-          # anything useful in it will be present on other load path items.
-          #
-          # This can happen if, for example, the user adds '.' to the load path,
-          # and the bundle path is '.bundle'.
-          contains_bundle_path = BUNDLE_PATH.start_with?(path)
+        def ruby_call(root_path)
+          root_path, contains_bundle_path, ignored_abs_paths, ignored_dir_names = prepare_scan(root_path)
+          return [] unless File.directory?(root_path)
 
           requirables = []
-          walk(path, nil) do |relative_path, absolute_path, is_directory|
+          walk(root_path, nil, ignored_abs_paths, ignored_dir_names) do |relative_path, absolute_path, is_directory|
             if is_directory
               !contains_bundle_path || !absolute_path.start_with?(BUNDLE_PATH)
             elsif relative_path.end_with?(*REQUIRABLE_EXTENSIONS)
@@ -42,25 +33,6 @@ module Bootsnap
           requirables
         end
 
-        def walk(absolute_dir_path, relative_dir_path, &block)
-          Dir.foreach(absolute_dir_path) do |name|
-            next if name.start_with?(".")
-
-            relative_path = relative_dir_path ? File.join(relative_dir_path, name) : name
-
-            absolute_path = "#{absolute_dir_path}/#{name}"
-            if File.directory?(absolute_path)
-              next if ignored_directories.include?(name) || ignored_directories.include?(absolute_path)
-
-              if yield relative_path, absolute_path, true
-                walk(absolute_path, relative_path, &block)
-              end
-            else
-              yield relative_path, absolute_path, false
-            end
-          end
-        end
-
         if RUBY_ENGINE == "ruby" && RUBY_PLATFORM.match?(/darwin|linux|bsd|mswin|mingw|cygwin/)
           require "bootsnap/bootsnap"
         end
@@ -69,33 +41,35 @@ module Bootsnap
           def native_call(root_path)
             # NOTE: if https://bugs.ruby-lang.org/issues/21800 is accepted we should be able
             # to have similar performance with pure Ruby
-
-            # If the bundle path is a descendent of this path, we do additional
-            # checks to prevent recursing into the bundle path as we recurse
-            # through this path. We don't want to scan the bundle path because
-            # anything useful in it will be present on other load path items.
-            #
-            # This can happen if, for example, the user adds '.' to the load path,
-            # and the bundle path is '.bundle'.
-            contains_bundle_path = BUNDLE_PATH.start_with?(root_path)
+            root_path, contains_bundle_path, ignored_abs_paths, ignored_dir_names = prepare_scan(root_path)
 
             all_requirables, queue = Native.scan_dir(root_path)
             all_requirables.each(&:freeze)
 
             queue.reject! do |dir|
-              ignored_directories.include?(dir) ||
-                (contains_bundle_path && dir.start_with?(BUNDLE_PATH))
+              if ignored_dir_names&.include?(dir)
+                true
+              elsif ignored_abs_paths || contains_bundle_path
+                absolute_dir = File.join(root_path, dir)
+                ignored_abs_paths&.include?(absolute_dir) ||
+                  (contains_bundle_path && absolute_dir.start_with?(BUNDLE_PATH))
+              end
             end
 
-            while (path = queue.pop)
-              requirables, dirs = Native.scan_dir(File.join(root_path, path))
-              dirs.reject! { |dir| ignored_directories.include?(dir) }
-              dirs.map! { |f| File.join(path, f).freeze }
-              requirables.map! { |f| File.join(path, f).freeze }
-
-              if contains_bundle_path
-                dirs.reject! { |dir| dir.start_with?(BUNDLE_PATH) }
+            while (relative_path = queue.pop)
+              absolute_base = File.join(root_path, relative_path)
+              requirables, dirs = Native.scan_dir(absolute_base)
+              dirs.reject! do |dir|
+                if ignored_dir_names&.include?(dir)
+                  true
+                elsif ignored_abs_paths || contains_bundle_path
+                  absolute_dir = File.join(absolute_base, dir)
+                  ignored_abs_paths&.include?(absolute_dir) ||
+                    (contains_bundle_path && absolute_dir.start_with?(BUNDLE_PATH))
+                end
               end
+              dirs.map! { |f| File.join(relative_path, f).freeze }
+              requirables.map! { |f| File.join(relative_path, f).freeze }
 
               all_requirables.concat(requirables)
               queue.concat(dirs)
@@ -107,6 +81,46 @@ module Bootsnap
         else
           alias_method :call, :ruby_call
         end
+
+        private
+
+        def prepare_scan(root_path)
+          root_path = File.expand_path(root_path.to_s).freeze
+
+          # If the bundle path is a descendent of this path, we do additional
+          # checks to prevent recursing into the bundle path as we recurse
+          # through this path. We don't want to scan the bundle path because
+          # anything useful in it will be present on other load path items.
+          #
+          # This can happen if, for example, the user adds '.' to the load path,
+          # and the bundle path is '.bundle'.
+          contains_bundle_path = BUNDLE_PATH.start_with?(root_path)
+
+          ignored_abs_paths, ignored_dir_names = ignored_directories.partition { |p| File.absolute_path?(p) }
+          ignored_abs_paths = nil if ignored_abs_paths.empty?
+          ignored_dir_names = nil if ignored_dir_names.empty?
+
+          [root_path, contains_bundle_path, ignored_abs_paths, ignored_dir_names]
+        end
+
+        def walk(absolute_dir_path, relative_dir_path, ignored_abs_paths, ignored_dir_names, &block)
+          Dir.foreach(absolute_dir_path) do |name|
+            next if name.start_with?(".")
+
+            relative_path = relative_dir_path ? File.join(relative_dir_path, name) : name
+
+            absolute_path = File.join(absolute_dir_path, name)
+            if File.directory?(absolute_path)
+              next if ignored_dir_names&.include?(name) || ignored_abs_paths&.include?(absolute_path)
+
+              if yield relative_path, absolute_path, true
+                walk(absolute_path, relative_path, ignored_abs_paths, ignored_dir_names, &block)
+              end
+            else
+              yield relative_path, absolute_path, false
+            end
+          end
+        end
       end
     end
   end

data/lib/bootsnap/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bootsnap
-  VERSION = "1.22.0"
+  VERSION = "1.23.0"
 end

data/lib/bootsnap.rb

@@ -150,7 +150,7 @@ module Bootsnap
     end
 
     # Allow the C extension to redefine `rb_get_path` without warning.
-    alias_method :rb_get_path, :rb_get_path # rubocop:disable Lint/DuplicateMethods
+    alias_method :rb_get_path, :rb_get_path
 
     private
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bootsnap
 version: !ruby/object:Gem::Version
-  version: 1.22.0
+  version: 1.23.0
 platform: ruby
 authors:
 - Burke Libbey
@@ -73,14 +73,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Boot large ruby/rails apps faster
 test_files: []