warn if read-only filesystem

rubygems · Feb 10, 2017 · 212be05 · 212be05
1 parent 1f99317
commit 212be05
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/bundler.rb b/lib/bundler.rb
@@ -176,7 +176,7 @@ def tmp_home_path(login, warning)
         tmp_home_path.join(login).tap(&:mkpath)
       end
     rescue => e
-      raise "#{warning}\nBundler also failed to create a temporary home directory at `#{path}':\n#{e}"
+      Bundler.ui.warn "#{warning}\nBundler also failed to create a temporary home directory at `#{path}':\n#{e}"
     end
 
     def user_bundle_path

diff --git a/spec/bundler/bundler_spec.rb b/spec/bundler/bundler_spec.rb
@@ -208,5 +208,10 @@
       expect(File).to receive(:chmod).with(0o777, "/TMP/bundler/home")
       expect(Bundler.tmp_home_path("USER", "")).to eq(Pathname("/TMP/bundler/home/USER"))
     end
+
+    it "should warn on r/o filesystem" do
+      allow(SharedHelpers).to receive(:filesystem_access).and_raise(PermissionError.new('/tmp'))
+      expect(Bundler.ui).to receive(:warn).with("AAA\nBundler also failed to create a temporary home directory at `AAA':\nAAA")
+    end
   end
 end