bootsnap 0.2.10 → 0.2.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9899f0b2bee1a4e655277b00c7e31aa9b9bb11de
-  data.tar.gz: 10d657b77486731b2dc6cb0c21b6b9cd59dcb021
+  metadata.gz: cd947bf62b1cc1309c2d1b41a8b712027435517c
+  data.tar.gz: d15f1bbd5ddeadded26f2f3884a29ddbe673817f
 SHA512:
-  metadata.gz: 345a4bf40d80ca9b04d03a8c5088758f59dddf6ff8272637bc6cd247f8047ee4dfbccf9ff2d6866d6220361c110d60d6ba08addb76a43e6d7fd36f04fb643c5a
-  data.tar.gz: 5ec8b6b63ef27265355d6ff879c710d3720a3fb7671e78e07326ddaeca7d77af800ddc86e13ddc4c1079449e9bbc79d7fedf1efe4a76c68dab6453f041d5fbac
+  metadata.gz: 572cf3a924e492cde28be986f2e466a4fcc5697f8e6dbc7eb2fff4b45c6a30a51d6e7777d6e300ceb8ddc5d7c876eeaa4ed3291011e5f4fb319734bbdf66daa6
+  data.tar.gz: 3baa2139ae74322a885124f7f5f02e6cb74803d26c609d811048abbd25c0d651c952c2cfef8b861d5310049e8f9449a2c7cc200a06143071e6205d43880ab86f

data/dev.yml

@@ -1,3 +1,6 @@
+env:
+  BOOTSNAP_PEDANTIC: '1'
+
 up:
   - ruby: 2.3.3
   - bundler

data/ext/bootsnap/extconf.rb

@@ -1,6 +1,17 @@
 require "mkmf"
-$CFLAGS << ' -O3 -std=c99'
-$CFLAGS << ' -Wall -Wextra -Wpedantic -Werror'
-$CFLAGS << ' -Wno-unused-parameter' # VALUE self has to be there but we don't care what it is.
-$CFLAGS << ' -Wno-keyword-macro' # hiding return
+$CFLAGS << ' -O3 '
+$CFLAGS << ' -std=c99'
+$CFLAGS << ' -Wall'
+$CFLAGS << ' -Werror'
+
+# ruby.h has some -Wpedantic fails in some cases
+# (e.g. https://github.com/Shopify/bootsnap/issues/15)
+unless ['0', '', nil].include?(ENV['BOOTSNAP_PEDANTIC'])
+  $CFLAGS << ' -Wextra'
+  $CFLAGS << ' -Wpedantic'
+
+  $CFLAGS << ' -Wno-unused-parameter' # VALUE self has to be there but we don't care what it is.
+  $CFLAGS << ' -Wno-keyword-macro' # hiding return
+end
+
 create_makefile("bootsnap/bootsnap")

data/lib/bootsnap/load_path_cache/cache.rb

@@ -120,6 +120,7 @@ module Bootsnap
         @store.transaction do
           paths.map(&:to_s).each do |path|
             p = Path.new(path)
+            next if p.non_directory?
             entries, dirs = p.entries_and_dirs(@store)
             # push -> low precedence -> set only if unset
             dirs.each    { |dir| @dirs[dir]  ||= true }
@@ -132,6 +133,7 @@ module Bootsnap
         @store.transaction do
           paths.map(&:to_s).reverse.each do |path|
             p = Path.new(path)
+            next if p.non_directory?
             entries, dirs = p.entries_and_dirs(@store)
             # unshift -> high precedence -> unconditional set
             dirs.each    { |dir| @dirs[dir]  = true }

data/lib/bootsnap/load_path_cache/path.rb

@@ -23,6 +23,13 @@ module Bootsnap
         @path = path.to_s
       end
 
+      # True if the path exists, but represents a non-directory object
+      def non_directory?
+        !File.stat(path).directory?
+      rescue Errno::ENOENT
+        false
+      end
+
       # Return a list of all the requirable files and all of the subdirectories
       # of this +Path+.
       def entries_and_dirs(store)

data/lib/bootsnap/version.rb

@@ -1,3 +1,3 @@
 module Bootsnap
-  VERSION = "0.2.10"
+  VERSION = "0.2.11"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bootsnap
 version: !ruby/object:Gem::Version
-  version: 0.2.10
+  version: 0.2.11
 platform: ruby
 authors:
 - Burke Libbey
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-04-25 00:00:00.000000000 Z
+date: 2017-04-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler