From 0ef392e0158501f275b61b1713a0fe7bc91390bc Mon Sep 17 00:00:00 2001
From: subframe7536 <1667077010@qq.com>
Date: Mon, 29 Jul 2024 13:21:36 +0800
Subject: [PATCH] add fontforge bin config #216

---
 build.py           | 57 ++++++++++++++++++++++++++--------------------
 source/schema.json |  4 ++++
 2 files changed, 36 insertions(+), 25 deletions(-)

diff --git a/build.py b/build.py
index 57f063a..1a1f402 100644
--- a/build.py
+++ b/build.py
@@ -14,12 +14,41 @@ from fontTools.ttLib import TTFont, newTable
 from fontTools.merge import Merger
 from fontTools.ttLib.tables import otTables
 
+# =========================================================================================
+
+package_name = "foundryToolsCLI"
+package_installed = importlib.util.find_spec(package_name) is not None
+
+if not package_installed:
+    print(f"{package_name} is not found. Please run `pip install foundrytools-cli`")
+    exit(1)
+
+# =========================================================================================
+
 # whether to archieve fonts
 release_mode = True
 # whether to clean built fonts
 clean_cache = True
 # build process pool size
 
+# =========================================================================================
+
+WIN_FONTFORGE_PATH = "C:/Program Files (x86)/FontForgeBuilds/bin/fontforge.exe"
+MAC_FONTFORGE_PATH = (
+    "/Applications/FontForge.app/Contents/Resources/opt/local/bin/fontforge"
+)
+LINUX_FONTFORGE_PATH = "/usr/bin/fontforge"
+
+system_name = platform.uname()[0]
+
+font_forge_bin = LINUX_FONTFORGE_PATH
+if "Darwin" in system_name:
+    font_forge_bin = MAC_FONTFORGE_PATH
+elif "Windows" in system_name:
+    font_forge_bin = WIN_FONTFORGE_PATH
+
+# =========================================================================================
+
 build_config = {
     # the number of parallel tasks
     # when run in codespace, this will be 1
@@ -49,6 +78,8 @@ build_config = {
         "version": "3.2.1",
         # whether to make icon width fixed
         "mono": False,
+        # font forgee bin path
+        "font_forge_bin": font_forge_bin,
         # prefer to use Font Patcher instead of using prebuild NerdFont base font
         # if you want to custom build nerd font using font-patcher, you need to set this to True
         "use_font_patcher": False,
@@ -86,12 +117,6 @@ try:
 except:
     print("config.json is not found. Use default config.")
 
-package_name = "foundryToolsCLI"
-package_installed = importlib.util.find_spec(package_name) is not None
-
-if not package_installed:
-    print(f"{package_name} is not found. Please run `pip install foundrytools-cli`")
-    exit(1)
 
 family_name = build_config["family_name"]
 family_name_compact = family_name.replace(" ", "")
@@ -109,24 +134,6 @@ output_cn = path.join(output_dir, "CN")
 
 ttf_dir_path = output_ttf_autohint if build_config["use_hinted"] else output_ttf
 
-# =========================================================================================
-
-WIN_FONTFORGE_PATH = "C:/Program Files (x86)/FontForgeBuilds/bin/fontforge.exe"
-MAC_FONTFORGE_PATH = (
-    "/Applications/FontForge.app/Contents/Resources/opt/local/bin/fontforge"
-)
-LINUX_FONTFORGE_PATH = "/usr/local/bin/fontforge"
-
-system_name = platform.uname()[0]
-
-font_forge_bin = LINUX_FONTFORGE_PATH
-if "Darwin" in system_name:
-    font_forge_bin = MAC_FONTFORGE_PATH
-elif "Windows" in system_name:
-    font_forge_bin = WIN_FONTFORGE_PATH
-
-# =========================================================================================
-
 if build_config["cn"]["with_nerd_font"]:
     cn_base_font_dir = output_nf
     suffix = "NF CN"
@@ -232,7 +239,7 @@ def check_font_patcher() -> bool:
 def get_nerd_font_patcher_args():
     # full args: https://github.com/ryanoasis/nerd-fonts?tab=readme-ov-file#font-patcher
     _nf_args = [
-        font_forge_bin,
+        build_config["nerd_font"]["font_forge_bin"],
         "FontPatcher/font-patcher",
         "-l",
         "--careful",
diff --git a/source/schema.json b/source/schema.json
index 678ac1e..dbb561e 100644
--- a/source/schema.json
+++ b/source/schema.json
@@ -142,6 +142,10 @@
           "description": "Whether to make icon width fixed",
           "default": false
         },
+        "font_forge_bin":{
+          "type": "string",
+          "description": "Font Forge bin path. Default: \n Windows: C:/Program Files (x86)/FontForgeBuilds/bin/fontforge.exe \n MacOS: /Applications/FontForge.app/Contents/Resources/opt/local/bin/fontforge\" \n Linux: /usr/bin/fontforge"
+        },
         "use_font_patcher": {
           "type": "boolean",
           "description": "Prefer to use Font Patcher instead of using prebuilt NerdFont base font. If you want to custom build nerd font using font-patcher, you need to set this to True",