From d46476fba1f2baa8b8b260d191421ca804e06f89 Mon Sep 17 00:00:00 2001
From: Peter Baker <b.tarde@gmail.com>
Date: Wed, 5 Jul 2023 18:48:03 -0400
Subject: [PATCH] Change to use Ygt .yaml hints instead of .xgf.

---
 sources/build_font | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/sources/build_font b/sources/build_font
index 7798466..b2d7bcc 100755
--- a/sources/build_font
+++ b/sources/build_font
@@ -159,8 +159,8 @@ then
     exit 1
 fi
 
-# If this line fails, maybe you need sed instead of gsed. But in any case you need
-# Gnu sed.
+# Need Gnu sed. The default (BSD) sed on the Mac won't cut it. There use
+# Homebrew to get gsed.
 
 sedcmd="sed"
 if command -v gsed &> /dev/null
@@ -171,6 +171,8 @@ fi
 familyname=`$sedcmd -n '1,/<source.*familyname/s/.*familyname="\([^"]*\)"[^\n]*/\1/p' $des`
 echo "Building font(s) with family name \"${familyname}\""
 
+# PERFORM EDITS ON DESIGNSPACE AND UFO FILES.
+
 # Every glyph with a "us" anchor also needs a "_us" anchor or fontmake (glyphsLib,
 # I think) will mess up the anchors for that glyph. But there is no requirement
 # that anchors must be paired this way, and it's easy to imagine the extraneous
@@ -211,13 +213,11 @@ then
     xsltproc -o $des $instanceScript $des
 fi
 
-# build the fonts. First things to do for both static types
-
 if [ $outputType != "v" ];
 then
   # fix-spacing.xsl sets the default spacing via the Spacing axis. Change
-  # the number in l. 14 of the script (use anything between 0 and 1) to
-  # change the spacing.
+  # the number "sp" (use anything between 0 and 1) to change the spacing.
+  # Now use -p option for this (static fonts only).
 
   xsltproc --param sp $spacing -o $des fix-spacing.xsl $des
   if [ $? -ne 0 ]
@@ -226,8 +226,9 @@ then
     exit 1
   fi
 
-  # Set the slant of the static Italic fonts. Change the number in l. 14
-  # of the script (using anything between 0 and 15) to change the slant.
+  # Set the slant of the static Italic fonts. Change the number "sl"
+  # (using anything between 0 and 15) to change the slant.
+  # Now use -s option for this (static fonts only).
 
   if [ $fonttype == "italic" ]
   then
@@ -240,6 +241,8 @@ then
   fi
 fi
 
+# RUN FONTMAKE TO BUILD THE FONTS (s = static TrueType; o = static CFF; v = variable TrueType).
+
 if [ $outputType = "s" ];
 then
 
@@ -345,7 +348,7 @@ else
   elif [ $autohint -ne -1 ]
   then
     echo "Running Xgridfit"
-    xgridfit -q -i "${fontname}-VF-withSTAT.ttf" -o "${fontname}.ttf" "xgf/Elstob-${fonttype}-all.xgf"
+    xgridfit -q -i "${fontname}-VF-withSTAT.ttf" -o "${fontname}.ttf" "xgf/Elstob-${fonttype}-all.yaml"
     if [ $? -ne 0 ]
     then
       echo "Failed to hint with Xgridfit."