defog 0.9.0 → 0.9.1

data/README.md

@@ -254,6 +254,7 @@ plus appropriate rspec examples.)
 
 Release Notes:
 
+* 0.9.1 - Bug fix: don't fail when measuring cache if file gets deleted by another process
 * 0.9.0 - Expose Handle#md5_hash
 * 0.8.0 - Include prefix in proxy_path (thus allowing cache sharing)
 * 0.7.2 - Bug fix: don't fail when clearing cache if another process clears it first

data/lib/defog/proxy.rb

@@ -193,7 +193,15 @@ module Defog
 
       # find available space (not counting current proxy)
       available = max_cache_size
-      proxy_root.find { |path| available -= path.size if path.file? and path != proxy_path}
+      proxy_root.find { |path|
+        available -= begin
+                       path.size
+                     rescue Errno::ENOENT
+                       # some other process has snuck in and deleted the
+                       # file since the path.file? check.  has happened...
+                       0
+                     end if path.file? and path != proxy_path
+      }
       return if available >= want_size
 
       space_needed = want_size - available
@@ -226,7 +234,7 @@ module Defog
         begin
           candidate.unlink
         rescue Errno::ENOENT
-          # some other process has deleted the while we were looking at it.
+          # some other process has deleted the file while we were looking at it.
           # nothing to do.
         end
       end

data/lib/defog/version.rb

@@ -1,3 +1,3 @@
 module Defog
-  VERSION = "0.9.0"
+  VERSION = "0.9.1"
 end

data/spec/proxy_spec.rb

@@ -167,7 +167,16 @@ shared_examples "a proxy" do |args|
       other_proxy_path("c").should_not be_exist
     end
 
-    it "should not fail when proxies get deleted by another process" do
+    it "should not fail size check when proxies get deleted by another process" do
+      create_other_proxy("a", 10)
+      create_other_proxy("b", 30)
+      create_other_proxy("c", 40)
+      create_remote("x" * 80)
+      Pathname.any_instance.should_receive(:size).and_raise Errno::ENOENT
+      expect { @proxy.file(key, "r") do end }.to_not raise_error(Errno::ENOENT)
+    end
+
+    it "should not fail unlinking when proxies get deleted by another process" do
       create_other_proxy("a", 10)
       create_other_proxy("b", 30)
       create_other_proxy("c", 40)

metadata

@@ -1,125 +1,142 @@
 --- !ruby/object:Gem::Specification
 name: defog
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
+  prerelease: 
 platform: ruby
 authors:
 - ronen barzel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-03 00:00:00.000000000 Z
+date: 2013-06-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fog
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: hash_keyword_args
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: fastandand
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov-gem-adapter
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Wrapper to fog gem, proxying access to cloud files as local files.
@@ -151,26 +168,27 @@ files:
 - spec/support/helpers.rb
 homepage: http://github.com/ronen/defog
 licenses: []
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 1.8.25
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Wrapper to fog gem, proxying access to cloud files as local files.  Access
   can be read-only (local cache), write-only (upload), or read-write (mirror)
 test_files:

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 024349bd2d3f5def5bf676cd93b7a222500003ab
-  data.tar.gz: 1636cbfc52043124967d8cd0dd5538ae80b42e0e
-SHA512:
-  metadata.gz: 282ade57688944a3d6b0203faece002aab973f92ddb93c702548ba3de50ae1252932b45a38f608c6acc56d2a0ce8ab83ac3dfb6f8d2d6c48ae4567acbdf131ca
-  data.tar.gz: f4d0323c3ef34a697f7f7bf667f46dd1fb5fd57ec948ce33fa90199cedbe20c8e212bb91c3cc5b1a97cdb89d9ac46a8d6f15fee8d91f699d1b595e006c0f3660