morandi 0.100.0 → 0.101.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ca1f4a637e59ee90de4f309dfecc40eae27b103f416ea301af36e8bd04c555f
-  data.tar.gz: 8ebeda3024e6275ea7e8b75c8357339523a59a24987e1ebd07142cba445edce7
+  metadata.gz: 82a7c6e28aeb2aae50257fa51f5eef82c7ceba3ee28604f53d875803363e582b
+  data.tar.gz: d7d80b7c3ba2f279b3650e9fdc105e41cd62632098e35ae66cb615c91ec8daad
 SHA512:
-  metadata.gz: 55308d1a2ae4626a81bf1faa9a6ae0535f816d777e31d25266e36e563bdc1dc60c55ef588ade477ce113a13bd20f80aefd046a52c1e4aaccffdc103e48d75cac
-  data.tar.gz: c10ab8cbe21121d50a2c0a10cdb6c0fefe595962d11cf936adeaef78e8291bd4688b37bb464577c7f455d8aa02e04def65d46930a86fb3cbf9c1aed4e9abf972
+  metadata.gz: 4ffda2485c2c6253064444eac78de3aacaab5aa46ed40998af6f37b578d98ef1883418cad26fac045fdbc251769fd60822c977bbdde6d3f536f0e1bcbd906f01
+  data.tar.gz: 8b785f03ee1ae2ebcfc4eb3b2b5e79c291b8596ab176208f18f7438cd57e400471847ff22949abdf35276f2b4acfb01d5771780f9d44b471d1fc4935033d457b

data/CHANGELOG.md

@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [0.101.0] 14.01.2026
+### Added
+- [BREAKING] introduced automated cleanup of srgb files after processing
+
+### Removed
+- [BREAKING] support for custom srgb file path
+
 ## [0.100.0] 17.01.2024
 ### Added
 - Vips image processor (resizing)

data/lib/morandi/image_processor.rb

@@ -126,7 +126,7 @@ module Morandi
         @pb = MorandiNative::PixbufUtils.brightness(@pb, brighten)
       end
 
-      if options['gamma'] && not_equal_to_one(options['gamma'])
+      if options['gamma'] && not_equal_to_one?(options['gamma'])
         @pb = MorandiNative::PixbufUtils.gamma(@pb,
                                                options['gamma'])
       end
@@ -197,7 +197,7 @@ module Morandi
       # can't crop, won't crop
       return if @width.nil? && @height.nil? && crop.nil?
 
-      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one(@scale)
+      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one?(@scale)
 
       crop ||= Morandi::CropUtils.autocrop_coords(@pb.width, @pb.height, @width, @height)
 
@@ -226,7 +226,7 @@ module Morandi
       colour ||= 'black'
 
       crop = options['crop']
-      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one(@scale)
+      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one?(@scale)
 
       op = Morandi::Operation::ImageBorder.new_from_hash(
         'style' => style,
@@ -243,7 +243,7 @@ module Morandi
 
     private
 
-    def not_equal_to_one(float)
+    def not_equal_to_one?(float)
       (float - 1.0).abs >= Float::EPSILON
     end
   end

data/lib/morandi/operation/image_border.rb

@@ -16,7 +16,7 @@ module Morandi
         return pixbuf unless %w[square retro].include? @style
 
         create_pixbuf_from_image_surface(:rgb24, pixbuf.width, pixbuf.height) do |cr|
-          if @crop && ((@crop[0]).negative? || (@crop[1]).negative?)
+          if @crop && (@crop[0].negative? || @crop[1].negative?)
             img_width = size[0]
             img_height = size[1]
           else
@@ -38,7 +38,7 @@ module Morandi
           # Should be less than 1
           pb_scale = (longest_side - (border_width * 2)) / longest_side
 
-          if @crop && ((@crop[0]).negative? || (@crop[1]).negative?)
+          if @crop && (@crop[0].negative? || @crop[1].negative?)
             x -= @crop[0]
             y -= @crop[1]
           end
@@ -77,7 +77,7 @@ module Morandi
 
       def draw_background(cr, img_height, img_width)
         cr.save do
-          cr.translate(-@crop[0], -@crop[1]) if @crop && ((@crop[0]).negative? || (@crop[1]).negative?)
+          cr.translate(-@crop[0], -@crop[1]) if @crop && (@crop[0].negative? || @crop[1].negative?)
 
           cr.save do
             cr.set_operator :source

data/lib/morandi/profiled_pixbuf.rb

@@ -8,22 +8,23 @@ module Morandi
   # It attempts to load an image using jpegicc/littlecms to ensure that it is sRGB.
   # NOTE: pixbuf supports colour profiles, but it requires an explicit icc-profile option to embed it when saving file
   class ProfiledPixbuf < GdkPixbuf::Pixbuf
-    def initialize(path, local_options, max_size_px = nil)
-      @local_options = local_options
-
-      path = srgb_path(path) || path
+    def initialize(path, _local_options, max_size_px = nil)
+      srgb_converted_file_path = srgb_path(path)
+      path = srgb_converted_file_path || path
 
       if max_size_px
         super(file: path, width: max_size_px, height: max_size_px)
       else
         super(file: path)
       end
+    ensure
+      FileUtils.rm_f(srgb_converted_file_path) if srgb_converted_file_path
     end
 
     private
 
     def srgb_path(original_path)
-      Morandi::SrgbConversion.perform(original_path, target_path: @local_options['path.icc'])
+      Morandi::SrgbConversion.perform(original_path)
     end
   end
 end

data/lib/morandi/srgb_conversion.rb

@@ -7,15 +7,16 @@ module Morandi
   class SrgbConversion
     # Performs a conversion to srgb colour space if possible
     # Returns a path to converted file on success or nil on failure
-    def self.perform(path, target_path: nil)
+    def self.perform(path)
       return unless suitable_for_jpegicc?(path)
 
-      icc_file_path = target_path || default_icc_path(path)
-      return icc_file_path if valid_jpeg?(icc_file_path)
-
+      icc_file_path = default_icc_path(path)
       system('jpgicc', '-q97', path, icc_file_path, out: '/dev/null', err: '/dev/null')
 
-      return unless valid_jpeg?(icc_file_path)
+      unless valid_jpeg?(icc_file_path)
+        FileUtils.rm_f(icc_file_path) # jpgicc likes to leave an empty file after failing
+        return
+      end
 
       icc_file_path
     end

data/lib/morandi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Morandi
-  VERSION = '0.100.0'
+  VERSION = '0.101.0'
 end

data/lib/morandi/vips_image_processor.rb

@@ -55,7 +55,7 @@ module Morandi
     end
 
     def process!
-      source_file_path = Morandi::SrgbConversion.perform(@path) || @path
+      source_file_path = @path
       begin
         @img = Vips::Image.new_from_file(source_file_path)
       rescue Vips::Error => e
@@ -68,7 +68,7 @@ module Morandi
       end
       if @size_limit_on_load_px
         @scale = @size_limit_on_load_px.to_f / [@img.width, @img.height].max
-        @img = @img.resize(@scale) if not_equal_to_one(@scale)
+        @img = @img.resize(@scale) if not_equal_to_one?(@scale)
       else
         @scale = 1.0
       end
@@ -113,7 +113,7 @@ module Morandi
     end
 
     def apply_gamma!
-      return unless @options['gamma'] && not_equal_to_one(@options['gamma'])
+      return unless @options['gamma'] && not_equal_to_one?(@options['gamma'])
 
       @img = @img.gamma(exponent: @options['gamma'])
     end
@@ -152,7 +152,7 @@ module Morandi
       # can't crop, won't crop
       return if @output_width.nil? && @output_height.nil? && crop.nil?
 
-      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one(@scale)
+      crop = crop.map { |s| (s.to_f * @scale).floor } if crop && not_equal_to_one?(@scale)
       crop ||= Morandi::CropUtils.autocrop_coords(@img.width, @img.height, @output_width, @output_height)
       @img = Morandi::CropUtils.apply_crop_vips(@img, crop[0], crop[1], crop[2], crop[3])
     end
@@ -183,7 +183,7 @@ module Morandi
       @img = @img.linear(1.0, colour_filter_modifier)
     end
 
-    def not_equal_to_one(float)
+    def not_equal_to_one?(float)
       (float - 1.0).abs >= Float::EPSILON
     end
 

data/lib/morandi.rb

@@ -43,7 +43,6 @@ module Morandi
   #                                                              size of the longer edge (ignoring shorter dimension!)
   # @param target_path [String] target location for image
   # @param local_options [Hash] Hash of options other than desired transformations
-  # @option local_options [String] 'path.icc' A path to store the input after converting to sRGB colour space
   # @option local_options [String] 'processor' ('pixbuf') Name of the image processing library ('pixbuf', 'vips')
   #                                                       NOTE: vips processor only handles subset of operations,
   #                                                       see `Morandi::VipsImageProcessor.supports?` for details
@@ -58,7 +57,10 @@ module Morandi
       cache_max = 0
       concurrency = 2 # Hardcoding to 2 for now to maintain some balance between resource usage and performance
       VipsImageProcessor.with_global_options(cache_max: cache_max, concurrency: concurrency) do
-        VipsImageProcessor.new(source, options).write_to_jpeg(target_path)
+        srgb_converted_file_path = Morandi::SrgbConversion.perform(source)
+        VipsImageProcessor.new(srgb_converted_file_path || source, options).write_to_jpeg(target_path)
+      ensure
+        FileUtils.rm_f(srgb_converted_file_path) if srgb_converted_file_path
       end
     else
       ImageProcessor.new(source, options, local_options).tap(&:result).write_to_jpeg(target_path)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: morandi
 version: !ruby/object:Gem::Version
-  version: 0.100.0
+  version: 0.101.0
 platform: ruby
 authors:
 - |+
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-01-17 00:00:00.000000000 Z
+date: 2026-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: atk
@@ -171,7 +171,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.5.3
 signing_key:
 specification_version: 4
 summary: Simple Image Edits