simple_rotate 1.1.1 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: be22a3c38d7b281549b2f6f2b782793ba8a6a3ab
-  data.tar.gz: d19711617196e08109a09c536010ce4296cd886f
+  metadata.gz: bf3e13b061ee59983bddd60110493b35b7be8fa2
+  data.tar.gz: cb2c4c5e947b82092798deec43d60e64d21e287a
 SHA512:
-  metadata.gz: fcb0a5ad487e2f07c82cf6225155958922d3afde9f2cb9fec950c427482843477a1d149c72fe096227d25bda0b56d8d60dae2c7ce91903629d5d759abef1fb38
-  data.tar.gz: c962cdad590764c25af7f8bbee41beb655b8cae1069890474133e6be1c32a44d9837e1b61466abced8b7a6562f091aa69eb5d7d2680e089af520c3503cda18f3
+  metadata.gz: 325ec9d77167f9a9e9002de74a954a930d7961d8564df404bc93b489c36db5a90e95561924bdd87aed57b7dd45832e63cac91819374a108a977ee8ca2120100d
+  data.tar.gz: cb1fe8acbab5443665cb84215467043e479753abab26dae55fd3720b5131cb2e08ef13e3843586a0a32a205532feb4bde9a249053b9c1ba5e8b8a1eae1fd882e

data/lib/simple_rotate/const.rb

@@ -1,6 +1,6 @@
 class SimpleRotate
     LIBS_NAME   = "simple_rotate"
-    VERSION     = "1.1.1"
+    VERSION     = "1.2.0"
     SUMMARY     = "Simple and safety logger."
     HOMEPAGE    = "http://khotta.github.io/"
     DESCRIPTION = "#{SUMMARY} #{HOMEPAGE}"

data/lib/simple_rotate/core.rb

@@ -78,7 +78,7 @@ class SimpleRotate
             @psync.unlock
         end
 
-        # if block given, colse IO
+        # if block is given, colse IO
         if defined? yield
             yield self
             e

data/lib/simple_rotate/internal/process_sync.rb

@@ -24,7 +24,9 @@ class SimpleRotate
             return self if file_name == nil
 
             @try_limit   = 3
-            @@tempf_name = File.dirname(file_name) + File::SEPARATOR + ".SimpleRotate_tempfile_#{File.basename($0)}"
+            @@tempf_name = File.dirname(file_name) + File::SEPARATOR + ".simple_rotate_tempfile_#{File.basename($0)}"
+            # replace whitespaces
+            @@tempf_name.gsub!(/\s/, '_')
 
             create_tempfile if @enable && !@@scheduled_del_lockfile
         end
@@ -32,19 +34,16 @@ class SimpleRotate
         # Create the temp file for locking
         private
         def create_tempfile
-            if File.exist?(@@tempf_name)
-                open_temp_file
-                return nil
-            end
-
             begin
-                @@tempf = File.open(@@tempf_name, File::RDWR|File::CREAT|File::EXCL)
+                if tempf_exists?
+                    set_delete_tempfile
+                else
+                    @@tempf = File.open(@@tempf_name, File::RDWR|File::CREAT|File::EXCL)
+                    set_delete_tempfile
+                end
 
             rescue
-                SimpleRotate::Error.warning("Couldn't create temp file => #{@@tempf_name}")
-
-            ensure
-                set_delete_tempfile
+                SimpleRotate::Error.warning("Failed to create temp file => #{@@tempf_name}")
             end
         end
 
@@ -58,7 +57,7 @@ class SimpleRotate
         def set_delete_tempfile
             return true if @@scheduled_del_lockfile
 
-            if File.exist?(@@tempf_name)
+            if tempf_exists?
                 # is it empty?
                 if File.size(@@tempf_name) == 0
                     delete_at_end
@@ -78,7 +77,7 @@ class SimpleRotate
                 begin
                     File.delete(@@tempf_name)
                 rescue
-                    #SimpleRotate::Error.warning("Couldn't delete temp file => #{@@tempf_name}")
+                    SimpleRotate::Error.warning("Failed to delete temp file => #{@@tempf_name}")
                 end
             end
         end
@@ -95,7 +94,7 @@ class SimpleRotate
                 begin
                     @@tempf = File.open(@@tempf_name, File::RDWR|File::CREAT|File::APPEND)
                 rescue
-                    SimpleRotate::Error.warning("Couldn't open temp file => #{@@tempf_name}")
+                    SimpleRotate::Error.warning("Failed to open temp file => #{@@tempf_name}")
                 end
             end
         end

data/lib/simple_rotate/internal/process_sync_mixin.rb

@@ -14,8 +14,6 @@ module ProcessSyncMixin
 
     # lock the temp file
     def lock
-        create_tempfile if !tempf_exists?
-
         reopen_temp_file
 
         cnt = 0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simple_rotate
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.2.0
 platform: ruby
 authors:
 - khotta
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-09 00:00:00.000000000 Z
+date: 2017-01-19 00:00:00.000000000 Z
 dependencies: []
 description: Simple and safety logger. http://khotta.github.io/
 email: