port VF metadata fixes from move to build script

tonsky · Apr 5, 2019 · b29e3f3 · b29e3f3
1 parent 11f505f
commit b29e3f3
Show file tree

Hide file tree

Showing 31 changed files with 97 additions and 401 deletions.
diff --git a/build.sh b/build.sh
@@ -1,19 +1,47 @@
 #!/bin/bash
-
+set -e
 source venv/bin/activate
 
+# ============================================================================
+# VARIABLE FONT ==============================================================
+
 # variable font
 rm -rf distr/variable_ttf
 fontmake -g FiraCode.glyphs -o variable --output-dir distr/variable_ttf
 
-# statics
+# -------------------------------------------------------------
+# fix variable font metadata as needed ------------------------
+
+firaCodeVF=distr/variable_ttf/FiraCode-VF.ttf
+
+# fix variable font metadata – very important
+gftools fix-vf-meta $firaCodeVF
+
+# other fixes for metadata and hinting
+gftools fix-nonhinting $firaCodeVF $firaCodeVF
+gftools fix-gasp --autofix $firaCodeVF
+gftools fix-dsig --autofix $firaCodeVF
+
+# cleanup of temp files
+tempFiles=$(ls distr/variable_ttf/*.fix && ls distr/variable_ttf/*-gasp*)
+for temp in $tempFiles
+do
+    rm -rf $temp
+done
+
+# TODO (late 2019?): use TTFautohint-VF for variable font (current support is minimal)
+
+# ============================================================================
+# STATIC FONTS ===============================================================
+
 rm -rf distr/ttf
 fontmake -g FiraCode.glyphs -o ttf --output-dir distr/ttf -i
+
 rm -rf distr/otf
 fontmake -g FiraCode.glyphs -o otf --output-dir distr/otf -i
 
-# ============================================================================
-# Autohinting ================================================================
+# -------------------------------------------------------------
+# Autohinting -------------------------------------------------
 
 statics=$(ls distr/ttf/*.ttf)
 for file in $statics; do 
@@ -32,7 +60,7 @@ done
 # ============================================================================
 # Build woff2 fonts ==========================================================
 
-# requires https://github.com/google/woff2
+# requires woff2_compress (get from https://github.com/bramstein/homebrew-webfonttools)
 
 rm -rf distr/woff2
 
@@ -50,7 +78,7 @@ done
 # ============================================================================
 # Build woff fonts ===========================================================
 
-# requires sfnt2woff-zopfli (https://github.com/bramstein/homebrew-webfonttools)
+# requires sfnt2woff-zopfli (get from https://github.com/bramstein/homebrew-webfonttools)
 
 rm -rf distr/woff
 

diff --git a/distr/otf/FiraCode-Bold.otf b/distr/otf/FiraCode-Bold.otf
diff --git a/distr/otf/FiraCode-Light.otf b/distr/otf/FiraCode-Light.otf
diff --git a/distr/otf/FiraCode-Medium.otf b/distr/otf/FiraCode-Medium.otf
diff --git a/distr/otf/FiraCode-Regular.otf b/distr/otf/FiraCode-Regular.otf
diff --git a/distr/otf/FiraCode-Retina.otf b/distr/otf/FiraCode-Retina.otf
diff --git a/distr/ttf/FiraCode-Bold.ttf b/distr/ttf/FiraCode-Bold.ttf
diff --git a/distr/ttf/FiraCode-Light.ttf b/distr/ttf/FiraCode-Light.ttf
diff --git a/distr/ttf/FiraCode-Medium.ttf b/distr/ttf/FiraCode-Medium.ttf
diff --git a/distr/ttf/FiraCode-Regular.ttf b/distr/ttf/FiraCode-Regular.ttf
diff --git a/distr/ttf/FiraCode-Retina.ttf b/distr/ttf/FiraCode-Retina.ttf
diff --git a/distr/variable_ttf/FiraCode-VF.ttf b/distr/variable_ttf/FiraCode-VF.ttf
diff --git a/distr/woff/FiraCode-Bold.woff b/distr/woff/FiraCode-Bold.woff
diff --git a/distr/woff/FiraCode-Light.woff b/distr/woff/FiraCode-Light.woff
diff --git a/distr/woff/FiraCode-Medium.woff b/distr/woff/FiraCode-Medium.woff
diff --git a/distr/woff/FiraCode-Regular.woff b/distr/woff/FiraCode-Regular.woff
diff --git a/distr/woff/FiraCode-Retina.woff b/distr/woff/FiraCode-Retina.woff
diff --git a/distr/woff/FiraCode-VF.woff b/distr/woff/FiraCode-VF.woff
diff --git a/distr/woff2/FiraCode-Bold.woff2 b/distr/woff2/FiraCode-Bold.woff2
diff --git a/distr/woff2/FiraCode-Light.woff2 b/distr/woff2/FiraCode-Light.woff2
diff --git a/distr/woff2/FiraCode-Medium.woff2 b/distr/woff2/FiraCode-Medium.woff2
diff --git a/distr/woff2/FiraCode-Regular.woff2 b/distr/woff2/FiraCode-Regular.woff2
diff --git a/distr/woff2/FiraCode-Retina.woff2 b/distr/woff2/FiraCode-Retina.woff2
diff --git a/distr/woff2/FiraCode-VF.woff2 b/distr/woff2/FiraCode-VF.woff2
diff --git a/googlefonts-qa/checks/FiraCode-Light.checks.md b/googlefonts-qa/checks/FiraCode-Light.checks.md
diff --git a/googlefonts-qa/checks/static/FiraCode-Bold.checks.md b/googlefonts-qa/checks/static/FiraCode-Bold.checks.md
diff --git a/googlefonts-qa/checks/static/FiraCode-Light.checks.md b/googlefonts-qa/checks/static/FiraCode-Light.checks.md
diff --git a/googlefonts-qa/checks/static/FiraCode-Medium.checks.md b/googlefonts-qa/checks/static/FiraCode-Medium.checks.md
diff --git a/googlefonts-qa/checks/static/FiraCode-Regular.checks.md b/googlefonts-qa/checks/static/FiraCode-Regular.checks.md
diff --git a/googlefonts-qa/checks/static/FiraCode-Retina.checks.md b/googlefonts-qa/checks/static/FiraCode-Retina.checks.md
diff --git a/googlefonts-qa/scripts/move-check.sh b/googlefonts-qa/scripts/move-check.sh
@@ -32,31 +32,6 @@ ttx -t head $firaCodeVF
 fontVersion=v$(xml sel -t --match "//*/fontRevision" -v "@value" ${firaCodeVF/".ttf"/".ttx"})
 rm ${firaCodeVF/".ttf"/".ttx"}
 
-# -------------------------------------------------------------------
-# fix variable font metadata as needed ------------------------------
-
-# TODO: Add gftools scripts as needed
-
-# TODO: test VFs with TTFautohint-VF vs no hinting
-
-gftools fix-nonhinting $firaCodeVF $firaCodeVF
-
-# TODO: decide if `--autofix` is really the best option, or if we should assert more control
-gftools fix-gasp --autofix $firaCodeVF
-
-gftools fix-dsig --autofix $firaCodeVF
-
-# fix variable font metadata
-gftools fix-vf-meta $firaCodeVF
-
-# cleanup
-tempFiles=$(ls distr/variable_ttf/*.fix && ls distr/variable_ttf/*-gasp*)
-for temp in $tempFiles
-do
-    rm -rf $temp
-done
-
-
 # -------------------------------------------------------------------
 # navigate to google/fonts repo, then fira code branch --------------