Merge pull request #14 from zkgan/use-mktmpdir

Use mktmpdir instead of tmpdir for temporary files created

Author: ricn

lib/libreconv.rb

@@ -16,7 +16,6 @@ class Converter
     def initialize(source, target, soffice_command = nil, convert_to = nil)
       @source = source
       @target = target
-      @target_path = Dir.tmpdir
       @soffice_command = soffice_command
       @convert_to = convert_to || "pdf"
       determine_soffice_command
@@ -30,11 +29,13 @@ def initialize(source, target, soffice_command = nil, convert_to = nil)
     def convert
       orig_stdout = $stdout.clone
       $stdout.reopen File.new('/dev/null', 'w')
-      pid = Spoon.spawnp(@soffice_command, "--headless", "--convert-to", @convert_to, @source, "--outdir", @target_path)
-      Process.waitpid(pid)
-      $stdout.reopen orig_stdout
-      target_tmp_file = "#{@target_path}/#{File.basename(@source, ".*")}.#{File.basename(@convert_to, ":*")}"
-      FileUtils.cp target_tmp_file, @target
+      Dir.mktmpdir { |target_path|
+        pid = Spoon.spawnp(@soffice_command, "--headless", "--convert-to", @convert_to, @source, "--outdir", target_path)
+        Process.waitpid(pid)
+        $stdout.reopen orig_stdout
+        target_tmp_file = "#{target_path}/#{File.basename(@source, ".*")}.#{File.basename(@convert_to, ":*")}"
+        FileUtils.cp target_tmp_file, @target
+      }
     end
 
     private
@@ -65,4 +66,4 @@ def check_source_type
       raise IOError, "Source (#{@source}) is neither a file nor an URL."
     end
   end
-end
+end