fontist 1.7.2 → 1.8.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 035a1318895900d90f27d2b5db8f08b86c64e6eb5e172ef81038c55305258756
-  data.tar.gz: 5a748c9632995e504b448508d56d0e292a7429b1eaa86d08978bf6e54d67f9e7
+  metadata.gz: e12ad24d94f6220992c66f06c21872135932b1b64f7ae17c7aa414c2f80f12ad
+  data.tar.gz: 29665a7556a8fbeda7449616c688a90e694b8060c7f2b8158abcc41e049579c4
 SHA512:
-  metadata.gz: 1558eb54750ed4efaabe694b79e23a00ce3a51d6c026641ab58796737ff609cce23b92dd141524e90c8d7e55f11cc0537fe405ca849b9d0309e65e47d1b672fe
-  data.tar.gz: 371d9283fa520c1dd036e22f600a5804a75af10b3a2d90bfe775432036ab51c12466696bb33a6614954c4de974889087255cdf2705b91731b6a0afa7ce9bef88
+  metadata.gz: 92b8942a88a3f02c57fd9840060c8ba333f0378611477bf1ff9435296792a4b79f97616da3520ea8d6fbe1088498c6d15d9c885c02e740fdc4bc26811988b176
+  data.tar.gz: 41aab015006c2109db2a52ea6ace23aeaaa571e5153559c53e327c448be4684189c7d72d5785811ecf505b124a48fa5016f5cfe62cd0946f92d0d6aab6d9b870

data/.github/workflows/release.yml

@@ -0,0 +1,38 @@
+name: release
+
+on:
+  push:
+    tags:
+      - 'v*'
+
+jobs:
+  release:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v1
+
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.6'
+
+      - run: bundle config set path 'vendor/bundle'
+
+      - run: bundle install --jobs 4 --retry 3
+
+      - run: bundle exec rspec --tag ~dev
+        env:
+          TEST_ENV: CI
+
+      - name: Publish to rubygems.org
+        env:
+          RUBYGEMS_API_KEY: ${{secrets.FONTIST_CI_RUBYGEMS_API_KEY}}
+        run: |
+          gem install gem-release
+          touch ~/.gem/credentials
+          cat > ~/.gem/credentials << EOF
+          ---
+          :rubygems_api_key: ${RUBYGEMS_API_KEY}
+          EOF
+          chmod 0600 ~/.gem/credentials
+          git status
+          gem release

data/.github/workflows/rspec.yml

@@ -0,0 +1,58 @@
+name: rspec
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+
+jobs:
+  build:
+    name: Test on Ruby ${{ matrix.ruby }} ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    continue-on-error: ${{ matrix.experimental }}
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '2.6', '2.5', '2.4' ]
+        os: [ ubuntu-latest, windows-latest, macos-latest ]
+        experimental: [ false ]
+        include:
+          - ruby: '2.7'
+            os: 'ubuntu-latest'
+            experimental: true
+          - ruby: '2.7'
+            os: 'windows-latest'
+            experimental: true
+          - ruby: '2.7'
+            os: 'macos-latest'
+            experimental: true
+
+    steps:
+      - uses: actions/checkout@master
+
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+
+      - uses: actions/cache@v1
+        with:
+          path: vendor/bundle
+          key: bundle-${{ matrix.os }}-${{ matrix.ruby }}-${{ hashFiles('**/*.gemspec') }}
+          restore-keys: bundle-${{ matrix.os }}-${{ matrix.ruby }}
+
+      - if: matrix.os == 'macos-latest'
+        run: brew install lcdf-typetools
+
+      - run: bundle config set path 'vendor/bundle'
+
+      - run: bundle install --jobs 4 --retry 3
+
+      - if: matrix.os == 'macos-latest'
+        run: bundle exec rspec
+        env:
+          TEST_ENV: CI
+
+      - if: matrix.os != 'macos-latest'
+        run: bundle exec rspec --tag ~dev
+        env:
+          TEST_ENV: CI

data/README.md

@@ -44,7 +44,7 @@ can we find a font in your system.
 
 #### Finding a font
 
-The `Fontist::Fontist.find` interface can be used a find a font in your system.
+The `Fontist::Font.find` interface can be used a find a font in your system.
 It will look into the operating system specific font directories, and also the
 fontist specific `~/.fontist` directory.
 
@@ -136,20 +136,18 @@ operation you would do in any ruby object.
 
 #### Locations
 
-Fontist lets find font locations from a YAML manifest of the following format:
+Fontist lets find font locations from a manifest of the following format:
 
-```yml
-Segoe UI:
-- Regular
-- Bold
-Roboto Mono:
-- Regular
+```ruby
+{"Segoe UI"=>["Regular", "Bold"],
+ "Roboto Mono"=>["Regular"]}
 ```
 
-Calling the following code returns a nested hash with font paths.
+Calling the following code returns a nested hash with font paths and names.
+Font name is useful to choose a specific font in a font collection file (TTC).
 
 ```ruby
-Fontist::Manifest::Locations.call(manifest_path)
+Fontist::Manifest::Locations.from_hash(manifest)
 ```
 
 ```ruby
@@ -169,7 +167,7 @@ Fontist lets not only to get font locations but also to install fonts from the
 manifest:
 
 ```ruby
-Fontist::Manifest::Install.call(manifest, confirmation: "yes")
+Fontist::Manifest::Install.from_hash(manifest, confirmation: "yes")
 ```
 
 It will install fonts and return their locations:
@@ -185,6 +183,27 @@ It will install fonts and return their locations:
                "paths"=>["/Users/user/.fontist/fonts/RobotoMono-VariableFont_wght.ttf"]}}}
 ```
 
+#### Support of YAML format
+
+Both commands support a YAML file as an input with a `from_file` method. For
+example, if there is a `manifest.yml` file containing:
+
+```yaml
+Segoe UI:
+- Regular
+- Bold
+Roboto Mono:
+- Regular
+```
+
+Then the following calls would return font names and paths, as from the
+`from_hash` method (see [Locations](#locations) and [Install](#install)).
+
+```ruby
+Fontist::Manifest::Locations.from_file("manifest.yml")
+Fontist::Manifest::Install.from_file("manifest.yml", confirmation: "yes")
+```
+
 ### CLI
 
 These commands makes possible to operate with fonts via command line. The CLI
@@ -230,7 +249,7 @@ Prints installed font paths grouped by formula and font.
 
 ```
 $ fontist status "segoe ui"
-Fontist::Formulas::SegoeUIFont
+segoe_ui
  Segoe UI
   Regular (/Users/user/.fontist/fonts/SEGOEUI.TTF)
   Bold (/Users/user/.fontist/fonts/SEGOEUIB.TTF)
@@ -244,7 +263,7 @@ Lists installation status of fonts supported by Fontist.
 
 ```
 $ fontist list "segoe ui"
-Fontist::Formulas::SegoeUIFont
+segoe_ui
  Segoe UI
   Regular (installed)
   Bold (installed)
@@ -254,7 +273,7 @@ Fontist::Formulas::SegoeUIFont
 
 ```
 $ fontist list "roboto mono"
-Fontist::Formulas::RobotoMonoFont
+google/roboto_mono
  Roboto Mono
   Regular (uninstalled)
   Italic (uninstalled)
@@ -376,8 +395,22 @@ archive:
 ```sh
 fontist create-formula https://www.latofonts.com/download/lato2ofl-zip/
 cp lato.yml ~/.fontist/formulas/Formulas/
+```
+
+A formula index should be rebuild, when a new formula is generated or an
+existing one changed:
+
+```sh
+fontist rebuild-index
+```
+
+Then, both the formula and the updated index should be commited and pushed to
+the formula repository:
+
+```sh
 cd ~/.fontist/formulas
 git add Formulas/lato.yml
+git add index.yml
 git commit -m "Add Lato formula"
 ```
 
@@ -401,6 +434,36 @@ git commit -m "Google Fonts update"
 git push
 ```
 
+### Import of SIL fonts
+
+Fontist contains formulas of [SIL fonts](https://software.sil.org/fonts/). They
+can be updated with:
+
+```sh
+fontist import-sil
+cd ~/.fontist/formulas
+git add Formulas/sil
+git add index.yml
+git commit -m "SIL fonts update"
+git push
+```
+
+### Releasing
+
+Releasing is done automatically with GitHub Action. Just bump and tag with `gem-release`.
+
+For a patch release (0.0.x) use:
+
+```sh
+gem bump --version patch --tag --push
+```
+
+For a minor release (0.x.0) use:
+
+```sh
+gem bump --version minor --tag --push
+```
+
 ## Contributing
 
 First, thank you for contributing! We love pull requests from everyone. By

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"
@@ -21,12 +21,13 @@ Gem::Specification.new do |spec|
 
   spec.require_paths = ["lib"]
   spec.files         = `git ls-files -z`.split("\x0").reject do |f|
-    f.match(%r{^(test|spec|features)/})
+    f.match(%r{^(bin|test|spec|features)/})
   end
-  spec.bindir        = "bin"
+  spec.bindir        = "exe"
   spec.executables   = ["fontist"]
   spec.test_files    = `git ls-files -- {spec}/*`.split("\n")
 
+  spec.add_runtime_dependency "arr-pm", "~> 0.0.1"
   spec.add_runtime_dependency "down", "~> 5.0"
   spec.add_runtime_dependency "libmspack", "~> 0.1.0"
   spec.add_runtime_dependency "rubyzip", "~> 2.3.0"
@@ -39,6 +40,8 @@ Gem::Specification.new do |spec|
   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 "rspec", "~> 3.0"
   spec.add_development_dependency "rubocop", "0.75.0"

data/lib/fontist.rb

@@ -8,12 +8,11 @@ require "fontist/errors"
 require "fontist/version"
 
 require "fontist/font"
-require "fontist/registry"
-require "fontist/formulas"
 require "fontist/formula"
 require "fontist/system_font"
 require "fontist/fontist_font"
 require "fontist/manifest"
+require "fontist/helpers"
 
 module Fontist
   def self.ui
@@ -59,4 +58,8 @@ module Fontist
   def self.system_index_path
     Fontist.fontist_path.join("system_index.yml")
   end
+
+  def self.formula_index_path
+    Fontist.formulas_repo_path.join("index.yml")
+  end
 end

data/lib/fontist/cli.rb

@@ -3,7 +3,24 @@ require "thor"
 module Fontist
   class CLI < Thor
     STATUS_SUCCESS = 0
-    STATUS_ERROR = 1
+    STATUS_UNKNOWN_ERROR = 1
+    STATUS_NON_SUPPORTED_FONT_ERROR = 2
+    STATUS_MISSING_FONT_ERROR = 3
+    STATUS_LICENSING_ERROR = 4
+    STATUS_MANIFEST_COULD_NOT_BE_FOUND_ERROR = 5
+    STATUS_MANIFEST_COULD_NOT_BE_READ_ERROR = 6
+    STATUS_FONT_INDEX_CORRUPTED = 7
+
+    ERROR_TO_STATUS = {
+      Fontist::Errors::UnsupportedFontError => [STATUS_NON_SUPPORTED_FONT_ERROR],
+      Fontist::Errors::MissingFontError => [STATUS_MISSING_FONT_ERROR],
+      Fontist::Errors::LicensingError => [STATUS_LICENSING_ERROR],
+      Fontist::Errors::ManifestCouldNotBeFoundError => [STATUS_MANIFEST_COULD_NOT_BE_FOUND_ERROR,
+                                                        "Manifest could not be found."],
+      Fontist::Errors::ManifestCouldNotBeReadError => [STATUS_MANIFEST_COULD_NOT_BE_READ_ERROR,
+                                                       "Manifest could not be read."],
+      Fontist::Errors::FontIndexCorrupted => [STATUS_FONT_INDEX_CORRUPTED],
+    }.freeze
 
     def self.exit_on_failure?
       false
@@ -19,9 +36,9 @@ module Fontist
         force: options[:force],
         confirmation: options[:confirm_license] ? "yes" : "no"
       )
-      STATUS_SUCCESS
-    rescue Fontist::Errors::NonSupportedFontError
-      could_not_find_font(font)
+      success
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
 
     desc "uninstall/remove FONT", "Uninstall font by font or formula"
@@ -29,26 +46,21 @@ module Fontist
       fonts_paths = Fontist::Font.uninstall(font)
       Fontist.ui.success("These fonts are removed:")
       Fontist.ui.success(fonts_paths.join("\n"))
-      STATUS_SUCCESS
-    rescue Fontist::Errors::MissingFontError => e
-      Fontist.ui.error(e.message)
-      STATUS_ERROR
-    rescue Fontist::Errors::NonSupportedFontError
-      could_not_find_font(font)
+      success
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
     map remove: :uninstall
 
     desc "status [FONT]", "Show status of FONT or all fonts in fontist"
     def status(font = nil)
       formulas = Fontist::Font.status(font)
-      return error("No font is installed.") if formulas.empty?
+      return error("No font is installed.", STATUS_MISSING_FONT_ERROR) if formulas.empty?
 
       print_formulas(formulas)
       success
-    rescue Fontist::Errors::MissingFontError => e
-      error(e.message)
-    rescue Fontist::Errors::NonSupportedFontError
-      could_not_find_font(font)
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
 
     desc "list [FONT]", "List installation status of FONT or fonts in fontist"
@@ -56,75 +68,87 @@ module Fontist
       formulas = Fontist::Font.list(font)
       print_list(formulas)
       success
-    rescue Fontist::Errors::NonSupportedFontError
-      could_not_find_font(font)
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
 
     desc "update", "Update formulas"
     def update
-      Formulas.fetch_formulas
+      Formula.update_formulas_repo
       Fontist.ui.say("Formulas have been successfully updated")
-      STATUS_SUCCESS
+      success
     end
 
     desc "manifest-locations MANIFEST",
          "Get locations of fonts from MANIFEST (yaml)"
     def manifest_locations(manifest)
-      paths = Fontist::Manifest::Locations.call(manifest)
+      paths = Fontist::Manifest::Locations.from_file(manifest)
       print_yaml(paths)
       success
-    rescue Fontist::Errors::ManifestCouldNotBeFoundError
-      error("Manifest could not be found.")
-    rescue Fontist::Errors::ManifestCouldNotBeReadError
-      error("Manifest could not be read.")
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
 
     desc "manifest-install MANIFEST", "Install fonts from MANIFEST (yaml)"
     option :confirm_license, type: :boolean, desc: "Confirm license agreement"
     def manifest_install(manifest)
-      paths = Fontist::Manifest::Install.call(
+      paths = Fontist::Manifest::Install.from_file(
         manifest,
         confirmation: options[:confirm_license] ? "yes" : "no"
       )
 
       print_yaml(paths)
       success
-    rescue Fontist::Errors::ManifestCouldNotBeFoundError
-      error("Manifest could not be found.")
-    rescue Fontist::Errors::ManifestCouldNotBeReadError
-      error("Manifest could not be read.")
+    rescue Fontist::Errors::GeneralError => e
+      handle_error(e)
     end
 
     desc "create-formula URL", "Create a new formula with fonts from URL"
     option :name, desc: "Example: Times New Roman"
     option :mirror, repeatable: true
     option :subarchive, desc: "Subarchive to choose when there are several ones"
-    option :subdir, desc: "Subdirectory to take fonts from"
+    option :subdir, desc: "Subdirectory to take fonts from, starting with the " \
+      "root dir, e.g.: stixfonts-2.10/fonts/static_otf. May include `fnmatch` patterns."
     def create_formula(url)
       require "fontist/import/create_formula"
       name = Fontist::Import::CreateFormula.new(url, options).call
       Fontist.ui.say("#{name} formula has been successfully created")
+      success
+    end
+
+    desc "rebuild-index", "Rebuild formula index (used by formulas maintainers)"
+    long_desc <<-LONGDESC
+      This index is pre-built and served with formulas, so there is no need
+      update it unless something changes in the formulas repo.
+    LONGDESC
+    def rebuild_index
+      Fontist::Index.rebuild
+      Fontist.ui.say("Formula index has been rebuilt.")
       STATUS_SUCCESS
     end
 
+    desc "import-sil", "Import formulas from SIL"
+    def import_sil
+      require "fontist/import/sil_import"
+      Fontist::Import::SilImport.new.call
+    end
+
     private
 
     def success
       STATUS_SUCCESS
     end
 
-    def could_not_find_font(font)
-      error("Font '#{font}' not found locally nor available in the Fontist " \
-            "formula repository.\n" \
-            "Perhaps it is available at the latest Fontist formula " \
-            "repository.\n" \
-            "You can update the formula repository using the command " \
-            "`fontist update` and try again.")
+    def handle_error(exception)
+      status, message = ERROR_TO_STATUS[exception.class]
+      raise exception unless status
+
+      error(message || exception.message, status)
     end
 
-    def error(message)
+    def error(message, status)
       Fontist.ui.error(message)
-      STATUS_ERROR
+      status
     end
 
     def print_yaml(object)
@@ -133,7 +157,7 @@ module Fontist
 
     def print_formulas(formulas)
       formulas.each do |formula, fonts|
-        Fontist.ui.success(formula.installer)
+        Fontist.ui.success(formula.key)
 
         fonts.each do |font, styles|
           Fontist.ui.success(" #{font.name}")
@@ -148,7 +172,7 @@ module Fontist
     # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
     def print_list(formulas)
       formulas.each do |formula, fonts|
-        Fontist.ui.say(formula.installer)
+        Fontist.ui.say(formula.key)
 
         fonts.each do |font, styles|
           Fontist.ui.say(" #{font.name}")