diff --git a/pyproject.toml b/pyproject.toml
index 7d523ff..5326128 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -12,7 +12,7 @@ authors = [
]
keywords = ["font", "fonts"]
license = {text = "GNU General Public License v3"}
-dependencies = ["fontra", "fontmake"]
+dependencies = ["fontra", "fontmake", "fontc"]
dynamic = ["version"]
requires-python = ">=3.10"
classifiers = [
@@ -39,6 +39,7 @@ fontra-compile = "fontra_compile.__main__:main"
[project.entry-points."fontra.workflow.actions"]
compile_varc = "fontra_compile.compile_varc_action"
compile_fontmake = "fontra_compile.compile_fontmake_action"
+compile_fontc = "fontra_compile.compile_fontc_action"
[tool.hatch.build.targets.wheel]
diff --git a/requirements-dev.txt b/requirements-dev.txt
index 7399175..b1a9155 100644
--- a/requirements-dev.txt
+++ b/requirements-dev.txt
@@ -1,4 +1,5 @@
mypy==1.10.0
-pre-commit==3.7.0
-pytest==8.1.1
+pre-commit==3.7.1
+pytest==8.2.1
+pytest-asyncio==0.23.7
types-PyYAML==6.0.12.20240311
diff --git a/requirements.txt b/requirements.txt
index 483b268..7740297 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -2,5 +2,6 @@
git+https://github.com/fonttools/fonttools.git@varc-table
# fonttools==4.51.0
fontmake==3.9.0
+fontc==0.0.1.post5
git+https://github.com/googlefonts/fontra.git
git+https://github.com/googlefonts/fontra-rcjk.git
diff --git a/src/fontra_compile/compile_fontc_action.py b/src/fontra_compile/compile_fontc_action.py
new file mode 100644
index 0000000..2d4a65d
--- /dev/null
+++ b/src/fontra_compile/compile_fontc_action.py
@@ -0,0 +1,22 @@
+import os
+import subprocess
+from dataclasses import dataclass
+
+from fontra.workflow.actions import registerOutputAction
+
+from .compile_fontmake_action import CompileFontMakeAction
+
+
+@registerOutputAction("compile-fontc")
+@dataclass(kw_only=True)
+class CompileFontCAction(CompileFontMakeAction):
+
+ def compileFromDesignspace(self, designspacePath, outputFontPath, extraArguments):
+ arguments = [
+ "fontc",
+ "--output-file",
+ os.fspath(outputFontPath),
+ os.fspath(designspacePath),
+ ]
+
+ subprocess.run(arguments + extraArguments, check=True)
diff --git a/src/fontra_compile/compile_fontmake_action.py b/src/fontra_compile/compile_fontmake_action.py
index d9fe1b1..6ff31aa 100644
--- a/src/fontra_compile/compile_fontmake_action.py
+++ b/src/fontra_compile/compile_fontmake_action.py
@@ -12,6 +12,7 @@
from fontra.core.protocols import ReadableFontBackend
from fontra.workflow.actions import OutputActionProtocol, registerOutputAction
from fontTools.designspaceLib import DesignSpaceDocument
+from fontTools.ufoLib import UFOReaderWriter
@registerOutputAction("compile-fontmake")
@@ -49,22 +50,27 @@ async def process(
await copyFont(self.input, dsBackend, continueOnError=continueOnError)
addInstances(designspacePath)
+ addGlyphOrder(designspacePath)
- arguments = [
- "-m",
- os.fspath(designspacePath),
- "-o",
- "variable",
- "--output-path",
- os.fspath(outputFontPath),
- ]
-
+ extraArguments = []
for option, value in self.options.items():
- arguments.append(f"--{option}")
+ extraArguments.append(f"--{option}")
if value:
- arguments.append(value)
+ extraArguments.append(value)
+
+ self.compileFromDesignspace(designspacePath, outputFontPath, extraArguments)
+
+ def compileFromDesignspace(self, designspacePath, outputFontPath, extraArguments):
+ arguments = [
+ "-m",
+ os.fspath(designspacePath),
+ "-o",
+ "variable",
+ "--output-path",
+ os.fspath(outputFontPath),
+ ]
- fontmake_main(arguments)
+ fontmake_main(arguments + extraArguments)
def addInstances(designspacePath):
@@ -94,6 +100,8 @@ def addInstances(designspacePath):
for axis in axes
]
+ axesByName = {axis.name: axis for axis in dsDoc.axes}
+
for items in itertools.product(*axisLabels):
location = {name: value for (name, valueLabel, value) in items}
nameParts = [valueLabel for (name, valueLabel, value) in items if valueLabel]
@@ -103,6 +111,24 @@ def addInstances(designspacePath):
# TODO: styleName seems to be ignored, and the instance names are derived
# from axis labels elsewhere. Figure out where this happens.
- dsDoc.addInstanceDescriptor(styleName=styleName, userLocation=location)
+ location = mapLocationForward(location, axesByName)
+ dsDoc.addInstanceDescriptor(
+ familyName="Testing", styleName=styleName, location=location
+ )
dsDoc.write(designspacePath)
+
+
+def mapLocationForward(location, axes):
+ return {name: axes[name].map_forward(value) for name, value in location.items()}
+
+
+def addGlyphOrder(designspacePath):
+ dsDoc = DesignSpaceDocument.fromfile(designspacePath)
+ defaultSource = dsDoc.findDefault()
+ ufo = UFOReaderWriter(defaultSource.path)
+ lib = ufo.readLib()
+ if "public.glyphOrder" not in lib:
+ glyphSet = ufo.getGlyphSet()
+ lib["public.glyphOrder"] = sorted(glyphSet.keys())
+ ufo.writeLib(lib)
diff --git a/tests/data/MutatorSans-fontc.ttx b/tests/data/MutatorSans-fontc.ttx
new file mode 100644
index 0000000..84cc37c
--- /dev/null
+++ b/tests/data/MutatorSans-fontc.ttx
@@ -0,0 +1,5551 @@
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ License same as MutatorMath. BSD 3-clause. [test-token: C]
+
+
+ MutatorMathTest
+
+
+ Regular
+
+
+ 1.002;NONE;MutatorMathTest-Regular
+
+
+ MutatorMathTest Regular
+
+
+ Version 1.002
+
+
+ MutatorMathTest-Regular
+
+
+ License same as MutatorMath. BSD 3-clause. [test-token: C]
+
+
+ weight
+
+
+ width
+
+
+ Light Condensed
+
+
+ Light
+
+
+ Light Wide
+
+
+ Condensed
+
+
+ Regular
+
+
+ Wide
+
+
+ Medium Condensed
+
+
+ Medium
+
+
+ Medium Wide
+
+
+ Black Condensed
+
+
+ Black
+
+
+ Black Wide
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+