fontist 1.8.3 → 1.8.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b8f20509f7376c4f88906c9f5d95d02f965351433b2b9f47c46d7e9e5afbb1e
-  data.tar.gz: e4c71f47525f2c82f0d098eca47629bc808bd1bbff7c83b39f4ede42fd507c05
+  metadata.gz: 7cd7962408633c39bb2cfff2ef8c3c0500842033f80ed7211474eefe98845327
+  data.tar.gz: 4630d28fbe5afad9865aeaebfa46e50215eb99cc8e2580466ae1fcb616117cfa
 SHA512:
-  metadata.gz: 82f8dc79feec65cd9a1ec567f6168dd9ff06fb7ba4ac7d8fbc58e450dba52380ba98d1d9f559b102ab4c997321565e2ee7fe04ac1b34100b7ce3c66078367b20
-  data.tar.gz: 2c80173b5683ed435c609f40bdfae52a0be258a1f491228fba93f2feedea1f28141461708b5c2a84385b5dde84af52792465c21dcf709dfd0f9a4f42d142cf57
+  metadata.gz: 57ac2451e5911694725ddd4f7f29bb8693e0be8e4397101f1f90c6a83ba353ea51fc7e2c9c0875e1bc0f4f17a466b9191d2c4017a47e971c9bb620bef0ea4a4c
+  data.tar.gz: 6aee6a31dd412af421191cb8cb5ca7793d5071bfcb0101666ef7d591abffcb98a7e9a3990978e7268d4e55fb1f0fa339711bffe801f272e08c133cd3c6783a9b

data/.github/workflows/rspec.yml

@@ -13,17 +13,17 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: [ '2.6', '2.5', '2.4' ]
+        ruby: [ '2.4', '2.5', '2.6', '2.7' ]
         os: [ ubuntu-latest, windows-latest, macos-latest ]
         experimental: [ false ]
         include:
-          - ruby: '2.7'
+          - ruby: '3.0'
             os: 'ubuntu-latest'
             experimental: true
-          - ruby: '2.7'
+          - ruby: '3.0'
             os: 'windows-latest'
             experimental: true
-          - ruby: '2.7'
+          - ruby: '3.0'
             os: 'macos-latest'
             experimental: true
 
@@ -37,8 +37,8 @@ jobs:
       - uses: actions/cache@v1
         with:
           path: vendor/bundle
-          key: bundle-${{ matrix.os }}-${{ matrix.ruby }}-${{ hashFiles('**/*.gemspec') }}
-          restore-keys: bundle-${{ matrix.os }}-${{ matrix.ruby }}
+          key: bundle-v2-${{ matrix.os }}-${{ matrix.ruby }}-${{ hashFiles('**/*.gemspec') }}
+          restore-keys: bundle-v2-${{ matrix.os }}-${{ matrix.ruby }}
 
       - if: matrix.os == 'macos-latest'
         run: brew install lcdf-typetools

data/fontist.gemspec

@@ -5,15 +5,15 @@ require "fontist/version"
 Gem::Specification.new do |spec|
   spec.name          = "fontist"
   spec.version       = Fontist::VERSION
-  spec.authors       = ["Ribose Inc.", "Abu Nashir"]
-  spec.email         = ["operations@ribose.com", "abunashir@gmail.com"]
+  spec.authors       = ["Ribose Inc."]
+  spec.email         = ["open.source@ribose.com"]
 
-  spec.summary       = %q{A libarary find or download fonts}
-  spec.description   = %q{A libarary find or download fonts}
+  spec.summary       = %q{Install openly-licensed fonts on Windows, Linux and Mac!}
+  spec.description   = %q{Install openly-licensed fonts on Windows, Linux and Mac!}
   spec.homepage      = "https://github.com/fontist/fontist"
   spec.license       = "BSD-2-Clause"
 
-  spec.post_install_message = "Please run `fontist update` to fetch formulas"
+  spec.post_install_message = "Please run `fontist update` to fetch formulas."
 
   spec.metadata["homepage_uri"] = spec.homepage
   spec.metadata["source_code_uri"] = "https://github.com/fontist/fontist"
@@ -28,20 +28,17 @@ Gem::Specification.new do |spec|
   spec.test_files    = `git ls-files -- {spec}/*`.split("\n")
 
   spec.add_runtime_dependency "down", "~> 5.0"
-  spec.add_runtime_dependency "libmspack", "~> 0.1.0"
-  spec.add_runtime_dependency "rubyzip", "~> 2.3.0"
-  spec.add_runtime_dependency "seven_zip_ruby", "~> 1.0"
-  spec.add_runtime_dependency "ruby-ole", "~> 1.0"
   spec.add_runtime_dependency "thor", "~> 1.0.1"
   spec.add_runtime_dependency "git", "~> 1.0"
   spec.add_runtime_dependency "ttfunk", "~> 1.0"
+  spec.add_runtime_dependency "excavate", "~> 0.1"
 
   spec.add_development_dependency "extract_ttc", "~> 0.1"
   spec.add_development_dependency "pry"
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "gem-release"
   spec.add_development_dependency "nokogiri", "~> 1.0"
-  spec.add_development_dependency "rake", "~> 12.3.3"
+  spec.add_development_dependency "rake", "~> 13"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency "rubocop", "0.75.0"
   spec.add_development_dependency "rubocop-rails"

data/lib/fontist.rb

@@ -62,4 +62,8 @@ module Fontist
   def self.formula_index_path
     Fontist.formulas_repo_path.join("index.yml")
   end
+
+  def self.formula_filename_index_path
+    Fontist.formulas_repo_path.join("filename_index.yml")
+  end
 end

data/lib/fontist/cli.rb

@@ -29,12 +29,14 @@ module Fontist
     desc "install FONT", "Install font"
     option :force, type: :boolean, aliases: :f,
                    desc: "Install even if it's already installed in system"
-    option :confirm_license, type: :boolean, desc: "Confirm license agreement"
+    option :accept_all_licenses, type: :boolean, aliases: "--confirm-license", desc: "Accept all license agreements"
+    option :hide_licenses, type: :boolean, desc: "Hide license texts"
     def install(font)
       Fontist::Font.install(
         font,
         force: options[:force],
-        confirmation: options[:confirm_license] ? "yes" : "no"
+        confirmation: options[:accept_all_licenses] ? "yes" : "no",
+        hide_licenses: options[:hide_licenses]
       )
       success
     rescue Fontist::Errors::GeneralError => e
@@ -52,12 +54,11 @@ module Fontist
     end
     map remove: :uninstall
 
-    desc "status [FONT]", "Show status of FONT or all fonts in fontist"
+    desc "status [FONT]", "Show paths of FONT or all fonts"
     def status(font = nil)
-      formulas = Fontist::Font.status(font)
-      return error("No font is installed.", STATUS_MISSING_FONT_ERROR) if formulas.empty?
+      paths = Fontist::Font.status(font)
+      return error("No font is installed.", STATUS_MISSING_FONT_ERROR) if paths.empty?
 
-      print_formulas(formulas)
       success
     rescue Fontist::Errors::GeneralError => e
       handle_error(e)
@@ -90,11 +91,13 @@ module Fontist
     end
 
     desc "manifest-install MANIFEST", "Install fonts from MANIFEST (yaml)"
-    option :confirm_license, type: :boolean, desc: "Confirm license agreement"
+    option :accept_all_licenses, type: :boolean, aliases: "--confirm-license", desc: "Accept all license agreements"
+    option :hide_licenses, type: :boolean, desc: "Hide license texts"
     def manifest_install(manifest)
       paths = Fontist::Manifest::Install.from_file(
         manifest,
-        confirmation: options[:confirm_license] ? "yes" : "no"
+        confirmation: options[:accept_all_licenses] ? "yes" : "no",
+        hide_licenses: options[:hide_licenses]
       )
 
       print_yaml(paths)
@@ -155,20 +158,6 @@ module Fontist
       Fontist.ui.say(YAML.dump(object))
     end
 
-    def print_formulas(formulas)
-      formulas.each do |formula, fonts|
-        Fontist.ui.success(formula.key)
-
-        fonts.each do |font, styles|
-          Fontist.ui.success(" #{font.name}")
-
-          styles.each do |style, path|
-            Fontist.ui.success("  #{style.type} (#{path})")
-          end
-        end
-      end
-    end
-
     # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
     def print_list(formulas)
       formulas.each do |formula, fonts|

data/lib/fontist/errors.rb

@@ -14,6 +14,7 @@ module Fontist
     class TamperedFileError < GeneralError; end
     class TimeoutError < GeneralError; end
     class UnknownFontTypeError < GeneralError; end
+    class UnknownArchiveError < GeneralError; end
 
     class FontError < GeneralError
       attr_reader :font, :style

data/lib/fontist/font.rb

@@ -1,10 +1,12 @@
 require "fontist/font_installer"
+require "fontist/font_path"
 
 module Fontist
   class Font
     def initialize(options = {})
       @name = options[:name]
       @confirmation = options[:confirmation] || "no"
+      @hide_licenses = options[:hide_licenses]
       @force = options[:force] || false
 
       check_or_create_fontist_path!
@@ -18,8 +20,8 @@ module Fontist
       new(name: name).find
     end
 
-    def self.install(name, confirmation: "no", force: false)
-      new(name: name, confirmation: confirmation, force: force).install
+    def self.install(name, options = {})
+      new(options.merge(name: name)).install
     end
 
     def self.uninstall(name)
@@ -47,9 +49,9 @@ module Fontist
     end
 
     def status
-      return installed_statuses unless @name
+      return installed_paths unless @name
 
-      font_status || downloadable_font || raise_non_supported_font
+      find_system_font || downloadable_font || raise_non_supported_font
     end
 
     def list
@@ -64,7 +66,7 @@ module Fontist
 
     private
 
-    attr_reader :name, :confirmation
+    attr_reader :name
 
     def find_system_font
       paths = Fontist::SystemFont.find(name)
@@ -73,9 +75,14 @@ module Fontist
         return
       end
 
+      print_paths(paths)
+    end
+
+    def print_paths(paths)
       Fontist.ui.say("Fonts found at:")
       paths.each do |path|
-        Fontist.ui.say("- #{path}")
+        font_path = FontPath.new(path)
+        Fontist.ui.say(font_path.to_s)
       end
     end
 
@@ -94,6 +101,10 @@ module Fontist
       @formula ||= Fontist::Formula.find(name)
     end
 
+    def formulas
+      @formulas ||= Fontist::Formula.find_many(name)
+    end
+
     def downloadable_font
       if formula
         raise Fontist::Errors::MissingFontError.new(name)
@@ -101,8 +112,10 @@ module Fontist
     end
 
     def download_font
-      if formula
-        check_and_confirm_required_license(formula)
+      return if formulas.empty?
+
+      formulas.flat_map do |formula|
+        confirmation = check_and_confirm_required_license(formula)
         paths = font_installer(formula).install(confirmation: confirmation)
 
         Fontist.ui.say("Fonts installed at:")
@@ -113,19 +126,24 @@ module Fontist
     end
 
     def check_and_confirm_required_license(formula)
-      if formula.license_required && !confirmation.casecmp("yes").zero?
-        @confirmation = show_license_and_ask_for_input(formula.license)
+      return @confirmation unless formula.license_required
 
-        unless confirmation&.casecmp?("yes")
-          raise Fontist::Errors::LicensingError.new(
-            "Fontist will not download these fonts unless you accept the terms."
-          )
-        end
-      end
+      show_license(formula.license) unless @hide_licenses
+      return @confirmation if @confirmation.casecmp?("yes")
+
+      confirmation = ask_for_agreement
+      return confirmation if confirmation&.casecmp?("yes")
+
+      raise Fontist::Errors::LicensingError.new(
+        "Fontist will not download these fonts unless you accept the terms."
+      )
     end
 
-    def show_license_and_ask_for_input(license)
+    def show_license(license)
       Fontist.ui.say(license_agrement_message(license))
+    end
+
+    def ask_for_agreement
       Fontist.ui.ask(
         "\nDo you accept all presented font licenses, and want Fontist " \
         "to download these fonts for you? => TYPE 'Yes' or 'No':"
@@ -161,37 +179,14 @@ module Fontist
       Fontist::FontistFont.find(name)
     end
 
-    def installed_statuses
-      installed_styles(all_formulas)
+    def installed_paths
+      print_paths(SystemFont.font_paths)
     end
 
     def all_formulas
       Fontist::Formula.all
     end
 
-    def font_status
-      return unless formula
-
-      statuses = installed_styles([formula])
-      statuses.empty? ? nil : statuses
-    end
-
-    def installed_styles(formulas)
-      filter_blank(formulas) do |formula|
-        filter_blank(formula.fonts) do |font|
-          filter_blank(font.styles) do |style|
-            path(style)
-          end
-        end
-      end
-    end
-
-    def filter_blank(elements)
-      elements.map { |e| [e, yield(e)] }
-        .to_h
-        .reject { |_k, v| v.nil? || v.empty? }
-    end
-
     def path(style)
       font_paths.detect do |path|
         File.basename(path) == style.font

data/lib/fontist/font_installer.rb

@@ -1,12 +1,8 @@
 require "fontist/utils"
+require "excavate"
 
 module Fontist
   class FontInstaller
-    include Utils::ZipExtractor
-    include Utils::ExeExtractor
-    include Utils::MsiExtractor
-    include Utils::SevenZipExtractor
-
     def initialize(formula)
       @formula = formula
     end
@@ -16,7 +12,6 @@ module Fontist
         raise(Fontist::Errors::LicensingError)
       end
 
-      reinitialize
       install_font
     end
 
@@ -24,10 +19,6 @@ module Fontist
 
     attr_reader :formula
 
-    def reinitialize
-      @downloaded = false
-    end
-
     def install_font
       fonts_paths = run_in_temp_dir { extract }
       fonts_paths.empty? ? nil : fonts_paths
@@ -46,47 +37,39 @@ module Fontist
     end
 
     def extract
-      resource = @formula.resources.first
-
-      [@formula.extract].flatten.each do |operation|
-        resource = extract_by_operation(operation, resource)
-      end
-
-      fonts_paths = resource
+      archive = download_file(@formula.resources.first)
 
-      fonts_paths
+      install_fonts_from_archive(archive)
     end
 
-    def extract_by_operation(operation, resource)
-      method = "#{operation.format}_extract"
-      if operation.options
-        send(method, resource, **operation.options.to_h)
-      else
-        send(method, resource)
-      end
-    end
+    def install_fonts_from_archive(archive)
+      Fontist.ui.say(%(Installing font "#{@formula.key}".))
 
-    def fonts_path
-      Fontist.fonts_path
+      Array.new.tap do |fonts_paths|
+        Excavate::Archive.new(archive.path).files(recursive_packages: true) do |path|
+          fonts_paths << install_font_file(path) if font_file?(path)
+        end
+      end
     end
 
     def download_file(source)
       url = source.urls.first
       Fontist.ui.say(%(Downloading font "#{@formula.key}" from #{url}))
 
-      downloaded_file = Fontist::Utils::Downloader.download(
+      Fontist::Utils::Downloader.download(
         url,
         sha: source.sha256,
         file_size: source.file_size,
         progress_bar: true
       )
+    end
 
-      @downloaded = true
-      downloaded_file
+    def font_file?(path)
+      source_file?(path) && font_directory?(path)
     end
 
-    def font_file?(filename)
-      source_files.include?(filename)
+    def source_file?(path)
+      source_files.include?(File.basename(path))
     end
 
     def source_files
@@ -97,6 +80,27 @@ module Fontist
       end
     end
 
+    def font_directory?(path)
+      return true unless subdirectory_pattern
+
+      File.fnmatch?(subdirectory_pattern, File.dirname(path))
+    end
+
+    def subdirectory_pattern
+      @subdirectory_pattern ||= "*" + subdirectories.first.chomp("/") unless subdirectories.empty?
+    end
+
+    def subdirectories
+      @subdirectories ||= [@formula.extract].flatten.map(&:options).compact.map(&:fonts_sub_dir).compact
+    end
+
+    def install_font_file(source)
+      target = Fontist.fonts_path.join(target_filename(File.basename(source))).to_s
+      FileUtils.mv(source, target)
+
+      target
+    end
+
     def target_filename(source_filename)
       target_filenames[source_filename]
     end