[fontc] Set --drop-implied-oncurves

googlefonts · Oct 31, 2024 · 31d7c78 · 31d7c78
1 parent 4d011a3
commit 31d7c78
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/Lib/gftools/builder/fontc.py b/Lib/gftools/builder/fontc.py
@@ -54,7 +54,7 @@ def modify_config(self, config: dict):
             config["splitItalic"] = False
             # set --no-production-names, because it's easier to debug
             extra_args = config.get("extraFontmakeArgs") or ""
-            extra_args += " --no-production-names"
+            extra_args += " --no-production-names --drop-implied-oncurves"
             config["extraFontmakeArgs"] = extra_args
             # override config to turn not build instances if we're variable
             if self.will_build_variable_font(config):

diff --git a/Lib/gftools/builder/operations/fontc/__init__.py b/Lib/gftools/builder/operations/fontc/__init__.py
@@ -53,6 +53,9 @@ def rewrite_one_arg(args: List[str]) -> str:
             next_ = f"{next_} {filter_}"
     elif next_ == "--no-production-names":
         return next_
+    elif next_ == "--drop-implied-oncurves":
+        # this is our default behaviour so no worries
+        return ""
     else:
         raise ValueError(f"unknown fontmake arg '{next_}'")
     return ""