mini_magick 4.13.2 → 5.1.2

data/lib/mini_magick/image/info.rb

@@ -17,8 +17,6 @@ module MiniMagick
           cheap_info(value)
         when "colorspace"
           colorspace
-        when "mime_type"
-          mime_type
         when "resolution"
           resolution(*args)
         when "signature"
@@ -27,8 +25,6 @@ module MiniMagick
           raw_exif(value)
         when "exif"
           exif
-        when "details"
-          details
         when "data"
           data
         else
@@ -76,12 +72,6 @@ module MiniMagick
         @info["colorspace"] ||= self["%r"]
       end
 
-      def mime_type
-        warn "[MiniMagick] MiniMagick::Image#mime_type has been deprecated, because it wasn't returning correct result for all formats ImageMagick supports. Unfortunately, returning the correct MIME type would be very slow, because it would require ImageMagick to read the whole file. It's better to use Marcel and MimeMagic gems, which are able to determine the MIME type just from the image header."
-
-        "image/#{self["format"].downcase}"
-      end
-
       def resolution(unit = nil)
         output = identify do |b|
           b.units unit if unit
@@ -102,20 +92,12 @@ module MiniMagick
           output.each_line do |line|
             line = line.chomp("\n")
 
-            case MiniMagick.cli
-            when :imagemagick, :imagemagick7
-              if match = line.match(/^exif:/)
-                key, value = match.post_match.split("=", 2)
-                value = decode_comma_separated_ascii_characters(value) if ASCII_ENCODED_EXIF_KEYS.include?(key)
-                hash[key] = value
-              else
-                hash[hash.keys.last] << "\n#{line}"
-              end
-            when :graphicsmagick
-              next if line == "unknown"
-              key, value = line.split("=", 2)
-              value.gsub!("\\012", "\n") # convert "\012" characters to newlines
+            if match = line.match(/^exif:/)
+              key, value = match.post_match.split("=", 2)
+              value = decode_comma_separated_ascii_characters(value) if ASCII_ENCODED_EXIF_KEYS.include?(key)
               hash[key] = value
+            else
+              hash[hash.keys.last] << "\n#{line}"
             end
           end
 
@@ -131,44 +113,9 @@ module MiniMagick
         @info["signature"] ||= self["%#"]
       end
 
-      def details
-        warn "[MiniMagick] MiniMagick::Image#details has been deprecated, as it was causing too many parsing errors. You should use MiniMagick::Image#data instead, which differs in a way that the keys are in camelcase." if MiniMagick.imagemagick? || MiniMagick.imagemagick7?
-
-        @info["details"] ||= (
-          details_string = identify(&:verbose)
-          key_stack = []
-          details_string.lines.to_a[1..-1].each_with_object({}) do |line, details_hash|
-            next if !line.valid_encoding? || line.strip.length.zero?
-
-            level = line[/^\s*/].length / 2 - 1
-            if level >= 0
-              key_stack.pop until key_stack.size <= level
-            else
-              # Some metadata, such as SVG clipping paths, will be saved without
-              # indentation, resulting in a level of -1
-              last_key = details_hash.keys.last
-              details_hash[last_key] = '' if details_hash[last_key].empty?
-              details_hash[last_key] << line
-              next
-            end
-
-            key, _, value = line.partition(/:[\s]/).map(&:strip)
-            hash = key_stack.inject(details_hash) { |_hash, _key| _hash.fetch(_key) }
-            if value.empty?
-              hash[key] = {}
-              key_stack.push key
-            else
-              hash[key] = value
-            end
-          end
-        )
-      end
-
       def data
-        raise Error, "MiniMagick::Image#data isn't supported on GraphicsMagick. Use MiniMagick::Image#details instead." if MiniMagick.graphicsmagick?
-
         @info["data"] ||= (
-          json = MiniMagick::Tool::Convert.new do |convert|
+          json = MiniMagick.convert do |convert|
             convert << path
             convert << "json:"
           end
@@ -180,7 +127,7 @@ module MiniMagick
       end
 
       def identify
-        MiniMagick::Tool::Identify.new do |builder|
+        MiniMagick.identify do |builder|
           yield builder if block_given?
           builder << path
         end

data/lib/mini_magick/image.rb

@@ -51,11 +51,11 @@ module MiniMagick
     #
     def self.import_pixels(blob, columns, rows, depth, map, format = 'png')
       # Create an image object with the raw pixel data string:
-      create(".dat", false) { |f| f.write(blob) }.tap do |image|
+      read(blob, ".dat").tap do |image|
         output_path = image.path.sub(/\.\w+$/, ".#{format}")
         # Use ImageMagick to convert the raw data file to an image file of the
         # desired format:
-        MiniMagick::Tool::Convert.new do |convert|
+        MiniMagick.convert do |convert|
           convert.size "#{columns}x#{rows}"
           convert.depth depth
           convert << "#{map}:#{image.path}"
@@ -79,33 +79,15 @@ module MiniMagick
     # @param options [Hash] Specify options for the open method
     # @return [MiniMagick::Image] The loaded image
     #
-    def self.open(path_or_url, ext = nil, options = {})
-      options, ext = ext, nil if ext.is_a?(Hash)
-
-      # Don't use Kernel#open, but reuse its logic
-      openable =
-        if path_or_url.respond_to?(:open)
-          path_or_url
-        elsif path_or_url.respond_to?(:to_str) &&
-              %r{\A[A-Za-z][A-Za-z0-9+\-\.]*://} =~ path_or_url &&
-              (uri = URI.parse(path_or_url)).respond_to?(:open)
-          uri
-        else
-          options = { binmode: true }.merge(options)
-          Pathname(path_or_url)
-        end
-
-      if openable.is_a?(URI::Generic)
-        ext ||= File.extname(openable.path)
-      else
-        ext ||= File.extname(openable.to_s)
-      end
-      ext.sub!(/:.*/, '') # hack for filenames or URLs that include a colon
-
-      if openable.is_a?(URI::Generic)
-        openable.open(options) { |file| read(file, ext) }
+    def self.open(path_or_url, ext = nil, **options)
+      if path_or_url.to_s =~ %r{\A(https?|ftp)://}
+        uri = URI(path_or_url)
+        ext ||= File.extname(uri.path).sub(/:.*/, '') # handle URL including a colon
+        uri.open(options) { |file| read(file, ext) }
       else
-        openable.open(**options) { |file| read(file, ext) }
+        pathname = Pathname(path_or_url)
+        ext ||= File.extname(pathname.to_s)
+        pathname.open(binmode: true, **options) { |file| read(file, ext) }
       end
     end
 
@@ -118,18 +100,14 @@ module MiniMagick
     # we have a good tempfile.
     #
     # @param ext [String] Specify the extension you want to read it as
-    # @param validate [Boolean] If false, skips validation of the created
-    #   image. Defaults to true.
     # @yield [Tempfile] You can #write bits to this object to create the new
     #   Image
     # @return [MiniMagick::Image] The created image
     #
-    def self.create(ext = nil, validate = MiniMagick.validate_on_create, &block)
+    def self.create(ext = nil, &block)
       tempfile = MiniMagick::Utilities.tempfile(ext.to_s.downcase, &block)
 
-      new(tempfile.path, tempfile).tap do |image|
-        image.validate! if validate
-      end
+      new(tempfile.path, tempfile)
     end
 
     ##
@@ -164,7 +142,7 @@ module MiniMagick
     # which creates a temporary file for you and protects your original.
     #
     # @param input_path [String, Pathname] The location of an image file
-    # @yield [MiniMagick::Tool::Mogrify] If block is given, {#combine_options}
+    # @yield [MiniMagick::Tool] If block is given, {#combine_options}
     #   is called.
     #
     def initialize(input_path, tempfile = nil, &block)
@@ -228,10 +206,6 @@ module MiniMagick
     #
     attribute :type, "format"
     ##
-    # @return [String]
-    #
-    attribute :mime_type
-    ##
     # @return [Integer]
     #
     attribute :width
@@ -286,17 +260,10 @@ module MiniMagick
     #
     attribute :signature
     ##
-    # Returns the information from `identify -verbose` in a Hash format, for
-    # ImageMagick.
+    # Returns the result of converting the image to JSON format.
     #
     # @return [Hash]
     attribute :data
-    ##
-    # Returns the information from `identify -verbose` in a Hash format, for
-    # GraphicsMagick.
-    #
-    # @return [Hash]
-    attribute :details
 
     ##
     # Use this method if you want to access raw Identify's format API.
@@ -367,7 +334,7 @@ module MiniMagick
     # @return [Array] Matrix of each color of each pixel
     def get_pixels(map="RGB")
       raise ArgumentError, "Invalid map value" unless ["RGB", "RGBA"].include?(map)
-      convert = MiniMagick::Tool::Convert.new
+      convert = MiniMagick.convert
       convert << path
       convert.depth(8)
       convert << "#{map}:-"
@@ -397,10 +364,10 @@ module MiniMagick
     # @example
     #   # It is given in readme.md file
     ##
-    def self.get_image_from_pixels(pixels, dimension, map, depth, mime_type)
+    def self.get_image_from_pixels(pixels, dimension, map, depth, format)
       pixels = pixels.flatten
       blob = pixels.pack('C*')
-      import_pixels(blob, *dimension, depth, map, mime_type)
+      import_pixels(blob, *dimension, depth, map, format)
     end
 
     ##
@@ -426,7 +393,7 @@ module MiniMagick
     #   will convert all pages.
     # @param read_opts [Hash] Any read options to be passed to ImageMagick
     #   for example: image.format('jpg', page, {density: '300'})
-    # @yield [MiniMagick::Tool::Convert] It optionally yields the command,
+    # @yield [MiniMagick::Tool] It optionally yields the command,
     #   if you want to add something.
     # @return [self]
     #
@@ -441,7 +408,7 @@ module MiniMagick
       input_path = path.dup
       input_path << "[#{page}]" if page && !layer?
 
-      MiniMagick::Tool::Convert.new do |convert|
+      MiniMagick.convert do |convert|
         read_opts.each do |opt, val|
           convert.send(opt.to_s, val)
         end
@@ -477,7 +444,7 @@ module MiniMagick
     #     c.background "blue"
     #   end
     #
-    # @yield [MiniMagick::Tool::Mogrify]
+    # @yield [MiniMagick::Command]
     # @see http://www.imagemagick.org/script/mogrify.php
     # @return [self]
     #
@@ -498,8 +465,11 @@ module MiniMagick
       end
     end
 
-    def respond_to_missing?(method_name, include_private = false)
-      MiniMagick::Tool::Mogrify.option_methods.include?(method_name.to_s)
+    ##
+    # Prevents ruby from calling `#to_ary` on the image when checking if it's a
+    # splattable data structure in certain cases.
+    def respond_to_missing?(name, include_all)
+      false
     end
 
     ##
@@ -514,7 +484,7 @@ module MiniMagick
       case output_to
       when String, Pathname
         if layer?
-          MiniMagick::Tool::Convert.new do |builder|
+          MiniMagick.convert do |builder|
             builder << path
             builder << output_to
           end
@@ -541,7 +511,7 @@ module MiniMagick
     def composite(other_image, output_extension = type.downcase, mask = nil)
       output_tempfile = MiniMagick::Utilities.tempfile(".#{output_extension}")
 
-      MiniMagick::Tool::Composite.new do |composite|
+      MiniMagick.composite do |composite|
         yield composite if block_given?
         composite << other_image.path
         composite << path
@@ -583,27 +553,21 @@ module MiniMagick
     #     b.verbose
     #   end # runs `identify -verbose image.jpg`
     # @return [String] Output from `identify`
-    # @yield [MiniMagick::Tool::Identify]
+    # @yield [MiniMagick::Tool]
     #
     def identify
-      MiniMagick::Tool::Identify.new do |builder|
+      MiniMagick.identify do |builder|
         yield builder if block_given?
         builder << path
       end
     end
 
-    # @private
-    def run_command(tool_name, *args)
-      MiniMagick::Tool.const_get(tool_name.capitalize).new do |builder|
-        args.each do |arg|
-          builder << arg
-        end
-      end
-    end
-
     def mogrify(page = nil)
-      MiniMagick::Tool::MogrifyRestricted.new do |builder|
+      MiniMagick.mogrify do |builder|
         yield builder if block_given?
+        if builder.args.include?("-format")
+          fail MiniMagick::Error, "you must call #format on a MiniMagick::Image directly"
+        end
         builder << (page ? "#{path}[#{page}]" : path)
       end
 

data/lib/mini_magick/shell.rb

@@ -1,4 +1,4 @@
-require "timeout"
+require "open3"
 require "benchmark"
 
 module MiniMagick
@@ -10,24 +10,30 @@ module MiniMagick
   #
   class Shell
 
-    def run(command, options = {})
-      stdout, stderr, status = execute(command, stdin: options[:stdin])
+    def run(command, errors: MiniMagick.errors, warnings: MiniMagick.warnings, **options)
+      stdout, stderr, status = execute(command, **options)
 
-      if status != 0 && options.fetch(:whiny, MiniMagick.whiny)
-        fail MiniMagick::Error, "`#{command.join(" ")}` failed with status: #{status.inspect} and error:\n#{stderr}"
+      if status != 0
+        if stderr.include?("time limit exceeded")
+          fail MiniMagick::TimeoutError, "`#{command.join(" ")}` has timed out"
+        elsif errors
+          fail MiniMagick::Error, "`#{command.join(" ")}` failed with status: #{status.inspect} and error:\n#{stderr}"
+        end
       end
 
-      stderr = stderr.lines[2..-1].join if stderr.start_with? %(WARNING: The convert command is deprecated in IMv7)
-      $stderr.print(stderr) unless options[:stderr] == false
+      $stderr.print(stderr) if warnings
 
       [stdout, stderr, status]
     end
 
-    def execute(command, options = {})
-      stdout, stderr, status =
-        log(command.join(" ")) do
-          send("execute_#{MiniMagick.shell_api.tr("-", "_")}", command, options)
-        end
+    def execute(command, stdin: "", timeout: MiniMagick.timeout)
+      env = {}
+      env.merge!(MiniMagick.cli_env)
+      env["MAGICK_TIME_LIMIT"] = timeout.to_s if timeout
+
+      stdout, stderr, status = log(command.join(" ")) do
+        Open3.capture3(env, *command, stdin_data: stdin)
+      end
 
       [stdout, stderr, status&.exitstatus]
     rescue Errno::ENOENT, IOError
@@ -36,39 +42,6 @@ module MiniMagick
 
     private
 
-    def execute_open3(command, options = {})
-      require "open3"
-
-      # We would ideally use Open3.capture3, but it wouldn't allow us to
-      # terminate the command after timing out.
-      Open3.popen3(*command) do |in_w, out_r, err_r, thread|
-        [in_w, out_r, err_r].each(&:binmode)
-        stdout_reader = Thread.new { out_r.read }
-        stderr_reader = Thread.new { err_r.read }
-        begin
-          in_w.write options[:stdin].to_s
-        rescue Errno::EPIPE
-        end
-        in_w.close
-
-        unless thread.join(MiniMagick.timeout)
-          Process.kill("TERM", thread.pid) rescue nil
-          Process.waitpid(thread.pid)      rescue nil
-          raise Timeout::Error, "MiniMagick command timed out: #{command}"
-        end
-
-        [stdout_reader.value, stderr_reader.value, thread.value]
-      end
-    end
-
-    def execute_posix_spawn(command, options = {})
-      require "posix-spawn"
-      child = POSIX::Spawn::Child.new(*command, input: options[:stdin].to_s, timeout: MiniMagick.timeout)
-      [child.out, child.err, child.status]
-    rescue POSIX::Spawn::TimeoutExceeded
-      raise Timeout::Error, "MiniMagick command timed out: #{command}"
-    end
-
     def log(command, &block)
       value = nil
       duration = Benchmark.realtime { value = block.call }

data/lib/mini_magick/tool.rb

@@ -2,15 +2,13 @@ require "mini_magick/shell"
 
 module MiniMagick
   ##
-  # Abstract class that wraps command-line tools. It shouldn't be used directly,
-  # but through one of its subclasses (e.g. {MiniMagick::Tool::Mogrify}). Use
-  # this class if you want to be closer to the metal and execute ImageMagick
-  # commands directly, but still with a nice Ruby interface.
+  # Class that wraps command-line tools directly, as opposed MiniMagick::Image
+  # which is more high-level.
   #
   # @example
-  #   MiniMagick::Tool::Mogrify.new do |builder|
-  #     builder.resize "500x500"
-  #     builder << "path/to/image.jpg"
+  #   MiniMagick.mogrify do |mogrify|
+  #     mogrify.resize "500x500"
+  #     mogrify << "path/to/image.jpg"
   #   end
   #
   class Tool
@@ -23,8 +21,7 @@ module MiniMagick
     # executes the command in the end.
     #
     # @example
-    #   version = MiniMagick::Tool::Identify.new { |b| b.version }
-    #   puts version
+    #   puts MiniMagick.identify(&:version)
     #
     # @return [MiniMagick::Tool, String] If no block is given, returns an
     #   instance of the tool, if block is given, returns the output of the
@@ -46,31 +43,31 @@ module MiniMagick
 
     # @param name [String]
     # @param options [Hash]
-    # @option options [Boolean] :whiny Whether to raise errors on non-zero
+    # @option options [Boolean] :errors Whether to raise errors on non-zero
     #   exit codes.
+    # @option options [Boolean] :warnings Whether to print warnings to stderrr.
+    # @option options [String] :stdin Content to send to standard input stream.
     # @example
-    #   MiniMagick::Tool::Identify.new(whiny: false) do |identify|
+    #   MiniMagick.identify(errors: false) do |identify|
     #     identify.help # returns exit status 1, which would otherwise throw an error
     #   end
-    def initialize(name, options = {})
-      warn "MiniMagick::Tool.new(false) is deprecated and will be removed in MiniMagick 5, use MiniMagick::Tool.new(whiny: false) instead." if !options.is_a?(Hash)
-
-      @name  = name
-      @args  = []
-      @whiny = options.is_a?(Hash) ? options.fetch(:whiny, MiniMagick.whiny) : options
+    def initialize(name, **options)
+      @name = name
+      @args = []
+      @options = options
     end
 
     ##
     # Executes the command that has been built up.
     #
     # @example
-    #   mogrify = MiniMagick::Tool::Mogrify.new
+    #   mogrify = MiniMagick.mogrify
     #   mogrify.resize("500x500")
     #   mogrify << "path/to/image.jpg"
     #   mogrify.call # executes `mogrify -resize 500x500 path/to/image.jpg`
     #
     # @example
-    #   mogrify = MiniMagick::Tool::Mogrify.new
+    #   mogrify = MiniMagick.mogrify
     #   # build the command
     #   mogrify.call do |stdout, stderr, status|
     #     # ...
@@ -80,16 +77,12 @@ module MiniMagick
     #
     # @return [String] Returns the output of the command
     #
-    def call(*args)
-      options = args[-1].is_a?(Hash) ? args.pop : {}
-      warn "Passing whiny to MiniMagick::Tool#call is deprecated and will be removed in MiniMagick 5, use MiniMagick::Tool.new(whiny: false) instead." if args.any?
-      whiny = args.fetch(0, @whiny)
-
-      options[:whiny] = whiny
-      options[:stderr] = MiniMagick.warnings && !block_given?
+    def call(**options)
+      options = @options.merge(options)
+      options[:warnings] = false if block_given?
 
       shell = MiniMagick::Shell.new
-      stdout, stderr, status = shell.run(command, options)
+      stdout, stderr, status = shell.run(command, **options)
       yield stdout, stderr, status if block_given?
 
       stdout.chomp("\n")
@@ -101,7 +94,7 @@ module MiniMagick
     # @return [Array<String>]
     #
     # @example
-    #   mogrify = MiniMagick::Tool::Mogrify.new
+    #   mogrify = MiniMagick.mogrify
     #   mogrify.resize "500x500"
     #   mogrify.contrast
     #   mogrify.command #=> ["mogrify", "-resize", "500x500", "-contrast"]
@@ -112,30 +105,26 @@ module MiniMagick
 
     ##
     # The executable used for this tool. Respects
-    # {MiniMagick::Configuration#cli}, {MiniMagick::Configuration#cli_path},
-    # and {MiniMagick::Configuration#cli_prefix}.
+    # {MiniMagick::Configuration#cli_prefix}.
     #
     # @return [Array<String>]
     #
     # @example
-    #   MiniMagick.configure { |config| config.cli = :graphicsmagick }
-    #   identify = MiniMagick::Tool::Identify.new
-    #   identify.executable #=> ["gm", "identify"]
+    #   identify = MiniMagick.identify
+    #   identify.executable #=> ["magick", "identify"]
     #
     # @example
     #   MiniMagick.configure do |config|
-    #     config.cli = :graphicsmagick
     #     config.cli_prefix = ['firejail', '--force']
     #   end
-    #   identify = MiniMagick::Tool::Identify.new
-    #   identify.executable #=> ["firejail", "--force", "gm", "identify"]
+    #   identify = MiniMagick.identify
+    #   identify.executable #=> ["firejail", "--force", "magick", "identify"]
     #
     def executable
       exe = [name]
+      exe.unshift "gm" if MiniMagick.graphicsmagick
       exe.unshift "magick" if MiniMagick.imagemagick7? && name != "magick"
-      exe.unshift "gm" if MiniMagick.graphicsmagick?
-      exe.unshift File.join(MiniMagick.cli_path, exe.shift) if MiniMagick.cli_path
-      Array(MiniMagick.cli_prefix).reverse_each { |p| exe.unshift p } if MiniMagick.cli_prefix
+      exe.unshift *Array(MiniMagick.cli_prefix)
       exe
     end
 
@@ -163,7 +152,7 @@ module MiniMagick
     # Changes the last operator to its "plus" form.
     #
     # @example
-    #   MiniMagick::Tool::Mogrify.new do |mogrify|
+    #   MiniMagick.mogrify do |mogrify|
     #     mogrify.antialias.+
     #     mogrify.distort.+("Perspective", "0,0,4,5 89,0,45,46")
     #   end
@@ -181,7 +170,7 @@ module MiniMagick
     # Create an ImageMagick stack in the command (surround.
     #
     # @example
-    #   MiniMagick::Tool::Magick.new do |convert|
+    #   MiniMagick.convert do |convert|
     #     convert << "wand.gif"
     #     convert.stack do |stack|
     #       stack << "wand.gif"
@@ -208,7 +197,7 @@ module MiniMagick
     # Adds ImageMagick's pseudo-filename `-` for standard input.
     #
     # @example
-    #   identify = MiniMagick::Tool::Identify.new
+    #   identify = MiniMagick.identify
     #   identify.stdin
     #   identify.call(stdin: image_content)
     #   # executes `identify -` with the given standard input
@@ -221,7 +210,7 @@ module MiniMagick
     # Adds ImageMagick's pseudo-filename `-` for standard output.
     #
     # @example
-    #   content = MiniMagick::Tool::Magick.new do |convert|
+    #   content = MiniMagick.convert do |convert|
     #     convert << "input.jpg"
     #     convert.auto_orient
     #     convert.stdout
@@ -273,35 +262,14 @@ module MiniMagick
       self
     end
 
-    def self.option_methods
-      @option_methods ||= (
-        tool = new(whiny: false)
-        tool << "-help"
-        help_page = tool.call(stderr: false)
-
-        cli_options = help_page.scan(/^\s+-[a-z\-]+/).map(&:strip)
-        if tool.name == "mogrify" && MiniMagick.graphicsmagick?
-          # These options were undocumented before 2015-06-14 (see gm bug 302)
-          cli_options |= %w[-box -convolve -gravity -linewidth -mattecolor -render -shave]
+    # deprecated tool subclasses
+    %w[animate compare composite conjure convert display identify import magick mogrify montage stream].each do |tool|
+      const_set(tool.capitalize, Class.new(self) {
+        define_method(:initialize) do |*args|
+          super(tool, *args)
         end
-
-        cli_options.map { |o| o[1..-1].tr('-','_') }
-      )
+      })
+      deprecate_constant(tool.capitalize)
     end
-
   end
 end
-
-require "mini_magick/tool/animate"
-require "mini_magick/tool/compare"
-require "mini_magick/tool/composite"
-require "mini_magick/tool/conjure"
-require "mini_magick/tool/convert"
-require "mini_magick/tool/display"
-require "mini_magick/tool/identify"
-require "mini_magick/tool/import"
-require "mini_magick/tool/magick"
-require "mini_magick/tool/mogrify"
-require "mini_magick/tool/mogrify_restricted"
-require "mini_magick/tool/montage"
-require "mini_magick/tool/stream"

data/lib/mini_magick/utilities.rb

@@ -24,12 +24,10 @@ module MiniMagick
     end
 
     def tempfile(extension)
-      Tempfile.new(["mini_magick", extension], MiniMagick.tmpdir).tap do |tempfile|
-        tempfile.binmode
-        yield tempfile if block_given?
-        tempfile.close
-      end
+      tempfile = Tempfile.new(["mini_magick", extension], MiniMagick.tmpdir, binmode: true)
+      yield tempfile if block_given?
+      tempfile.close
+      tempfile
     end
-
   end
 end

data/lib/mini_magick/version.rb

@@ -7,8 +7,8 @@ module MiniMagick
   end
 
   module VERSION
-    MAJOR = 4
-    MINOR = 13
+    MAJOR = 5
+    MINOR = 1
     TINY  = 2
     PRE   = nil