Merge pull request #358 from metaskills/ReadOnlyFilesystem

Ignore Read-Only Filesystems
Shopify · Apr 27, 2021 · 2d40bd3 · 2d40bd3
2 parents a7b3628 + 4ccccfc
commit 2d40bd3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/lib/bootsnap/load_path_cache/store.rb b/lib/bootsnap/load_path_cache/store.rb
@@ -91,6 +91,7 @@ def dump_data
         FileUtils.mv(tmp, @store_path)
       rescue Errno::EEXIST
         retry
+      rescue SystemCallError
       end
     end
   end

diff --git a/test/load_path_cache/store_test.rb b/test/load_path_cache/store_test.rb
@@ -74,6 +74,12 @@ def test_retry_on_collision
 
         store.transaction { store.set('a', 1) }
       end
+
+      def test_ignore_read_only_filesystem
+        MessagePack.expects(:dump).raises(Errno::EROFS.new("Read-only file system @ rb_sysopen"))
+        store.transaction { store.set('a', 1) }
+        refute(File.exist?(@path))
+      end
     end
   end
 end