fontist 1.13.2 → 1.14.0

data/lib/fontist/fontconfig.rb

@@ -0,0 +1,87 @@
+module Fontist
+  class Fontconfig
+    def self.update
+      new.update
+    end
+
+    def self.remove(options = {})
+      new(options).remove
+    end
+
+    def initialize(options = {})
+      @options = options
+    end
+
+    def update
+      ensure_fontconfig_installed
+      create_config
+      regenerate_fontconfig_cache
+    end
+
+    def remove
+      return handle_file_not_found unless config_exists?
+
+      regenerate_fontconfig_cache if fontconfig_installed?
+      remove_config
+    end
+
+    private
+
+    def ensure_fontconfig_installed
+      raise Errors::FontconfigNotFoundError unless fontconfig_installed?
+    end
+
+    def fontconfig_installed?
+      Utils::System.fontconfig_installed?
+    end
+
+    def create_config
+      return if File.exist?(config_path)
+
+      FileUtils.mkdir_p(File.dirname(config_path))
+      File.write(config_path, config_content)
+    end
+
+    def config_path
+      File.join(xdg_config_home, "fontconfig", "conf.d", "10-fontist.conf")
+    end
+
+    def xdg_config_home
+      ENV["XDG_CONFIG_HOME"] || File.join(Dir.home, ".config")
+    end
+
+    def config_content
+      <<~CONTENT
+        <?xml version='1.0'?>
+        <!DOCTYPE fontconfig SYSTEM 'fonts.dtd'>
+        <fontconfig>
+          <dir>#{Fontist.fonts_path}</dir>
+        </fontconfig>
+      CONTENT
+    end
+
+    def regenerate_fontconfig_cache
+      Helpers.run("fc-cache -f")
+    end
+
+    def ensure_file_exists
+      return if @options[:force]
+
+      raise Errors::FontconfigFileNotFoundError unless File.exist?(config_path)
+    end
+
+    def config_exists?
+      File.exist?(config_path)
+    end
+
+    def handle_file_not_found
+      return if @options[:force]
+
+      raise Errors::FontconfigFileNotFoundError
+    end
+
+    def remove_config
+      FileUtils.rm(config_path)
+    end
+  end
+end

data/lib/fontist/fontconfig_cli.rb

@@ -0,0 +1,29 @@
+module Fontist
+  class FontconfigCLI < Thor
+    include CLI::ClassOptions
+
+    desc "update", "Update fontconfig configuration to use fontist fonts"
+    def update
+      handle_class_options(options)
+      Fontconfig.update
+      Fontist.ui.success("Fontconfig file has been successfully updated.")
+      CLI::STATUS_SUCCESS
+    rescue Errors::FontconfigNotFoundError => e
+      Fontist.ui.error(e.message)
+      CLI::STATUS_FONTCONFIG_NOT_FOUND
+    end
+
+    desc "remove", "Remove fontist file in fontconfig configuration"
+    option :force, type: :boolean, aliases: :f,
+                   desc: "Proceed even if does not exist"
+    def remove
+      handle_class_options(options)
+      Fontconfig.remove(options)
+      Fontist.ui.success("Fontconfig file has been successfully removed.")
+      CLI::STATUS_SUCCESS
+    rescue Errors::FontconfigFileNotFoundError => e
+      Fontist.ui.error(e.message)
+      CLI::STATUS_FONTCONFIG_FILE_NOT_FOUND
+    end
+  end
+end

data/lib/fontist/helpers.rb

@@ -3,5 +3,26 @@ module Fontist
     def self.parse_to_object(data)
       JSON.parse(data.to_json, object_class: OpenStruct)
     end
+
+    def self.run(command)
+      Fontist.ui.debug("Run `#{command}`")
+
+      result = `#{command}`
+      unless $CHILD_STATUS.to_i.zero?
+        raise Errors::BinaryCallError,
+              "Failed to run #{command}, status: #{$CHILD_STATUS}"
+      end
+
+      result
+    end
+
+    def self.silence_stream(stream)
+      old_stream = stream.dup
+      stream.reopen(RbConfig::CONFIG["host_os"] =~ /mswin|mingw/ ? "NUL:" : "/dev/null") # rubocop:disable Performance/RegexpMatch, Metrics/LineLength
+      stream.sync = true
+      yield
+    ensure
+      stream.reopen(old_stream)
+    end
   end
 end

data/lib/fontist/import/create_formula.rb

@@ -55,12 +55,32 @@ module Fontist
       end
 
       def save(builder)
-        filename = Import.name_to_filename(builder.name)
-        path = @options[:formula_dir] ? File.join(@options[:formula_dir], filename) : filename
+        path = vacant_path
         yaml = YAML.dump(Helpers::HashHelper.stringify_keys(builder.formula))
         File.write(path, yaml)
         path
       end
+
+      def vacant_path
+        path = path_from_name
+        return path unless @options[:keep_existing] && File.exist?(path)
+
+        2.upto(9) do |i|
+          candidate = path.sub(/\.yml$/, "#{i}.yml")
+          return candidate unless File.exist?(candidate)
+        end
+
+        raise Errors::GeneralError, "Formula #{path} already exists."
+      end
+
+      def path_from_name
+        filename = Import.name_to_filename(builder.name)
+        if @options[:formula_dir]
+          File.join(@options[:formula_dir], filename)
+        else
+          filename
+        end
+      end
     end
   end
 end

data/lib/fontist/import/files/collection_file.rb

@@ -26,9 +26,9 @@ module Fontist
 
         def read
           switch_to_temp_dir do |tmp_dir|
-            extract_ttfs(tmp_dir).map do |path|
-              Otf::FontFile.new(path)
-            end
+            extract_ttfs(tmp_dir)
+              .map { |path| Otf::FontFile.new(path) }
+              .reject { |font_file| hidden_or_pua_encoded?(font_file) }
           end
         end
 
@@ -47,6 +47,10 @@ module Fontist
           end
         end
 
+        def hidden_or_pua_encoded?(font_file)
+          font_file.family_name.start_with?(".")
+        end
+
         def detect_extension
           base_extension = "ttc"
 

data/lib/fontist/import/formula_builder.rb

@@ -4,9 +4,10 @@ require_relative "text_helper"
 module Fontist
   module Import
     class FormulaBuilder
-      FORMULA_ATTRIBUTES = %i[description homepage resources
+      FORMULA_ATTRIBUTES = %i[platforms description homepage resources
                               font_collections fonts extract copyright
-                              license_url open_license digest command].freeze
+                              license_url requires_license_agreement
+                              open_license digest command].freeze
 
       attr_writer :archive,
                   :url,
@@ -28,9 +29,15 @@ module Fontist
       def name
         return @options[:name] if @options[:name]
 
-        unique_names = both_fonts.map(&:family_name).uniq
-        TextHelper.longest_common_prefix(unique_names) ||
-          both_fonts.first.family_name
+        common = %i[family_name type]
+          .map { |attr| both_fonts.map(&attr).uniq }
+          .map { |names| TextHelper.longest_common_prefix(names) }
+          .map { |prefix| prefix unless prefix == "Regular" }
+          .compact
+          .join(" ")
+        return common unless common.empty?
+
+        both_fonts.map(&:family_name).first
       end
 
       private
@@ -50,6 +57,10 @@ module Fontist
         files
       end
 
+      def platforms
+        @options[:platforms]
+      end
+
       def description
         name
       end
@@ -175,12 +186,17 @@ module Fontist
         both_fonts.map(&:license_url).compact.first
       end
 
+      def requires_license_agreement
+        @options[:requires_license_agreement]
+      end
+
       def open_license
-        unless @license_text
+        unless @license_text || requires_license_agreement
           Fontist.ui.error("WARN: please add license manually")
-          return
         end
 
+        return unless @license_text
+
         Fontist.ui.error("WARN: ensure it's an open license, otherwise " \
                          "change the 'open_license' attribute to " \
                          "'requires_license_agreement'")

data/lib/fontist/import/helpers/system_helper.rb

@@ -4,15 +4,7 @@ module Fontist
       module SystemHelper
         class << self
           def run(command)
-            Fontist.ui.debug("Run `#{command}`")
-
-            result = `#{command}`
-            unless $CHILD_STATUS.to_i.zero?
-              raise Errors::BinaryCallError,
-                    "Failed to run #{command}, status: #{$CHILD_STATUS}"
-            end
-
-            result
+            Fontist::Helpers.run(command)
           end
         end
       end