From 09ea4e24fb08413ef2376556a72ede656e7dca7e Mon Sep 17 00:00:00 2001
From: Levon Gevorgyan <lgevorgyan03@gmail.com>
Date: Sat, 7 Feb 2026 13:57:40 -0600
Subject: [PATCH] fix freetype/harfbuzz circular dependency

---
 lib/0-utils.sh      | 13 +++++++++----
 lib/build.sh        | 27 ++++++++++++++++++---------
 lib/install_deps.sh |  2 +-
 3 files changed, 28 insertions(+), 14 deletions(-)

diff --git a/lib/0-utils.sh b/lib/0-utils.sh
index 692a435..6ca6818 100644
--- a/lib/0-utils.sh
+++ b/lib/0-utils.sh
@@ -116,7 +116,7 @@ is_root_owned() {
     local path=$1
     local uid
 
-    if stat --version >/dev/null 2>&1; then
+    if stat --version &>/dev/null; then
         # GNU coreutils (Linux)
         uid=$(stat -c '%u' "$path")
     else
@@ -153,7 +153,7 @@ has_cmd() {
     local cmds=("$@")
     local rv=0
     for cmd in "${cmds[@]}"; do
-        command -v "${cmd}" >/dev/null 2>&1 || rv=1
+        command -v "${cmd}" &>/dev/null || rv=1
     done
 
     return ${rv}
@@ -381,7 +381,8 @@ spinner() {
     local spinPidFile="${TMP_DIR}/.spinner-pid"
     case "${action}" in
     start)
-        test -f "${spinPidFile}" && rm "${spinPidFile}"
+        # if there is already a pidfile, we're spinning
+        test -f "${spinPidFile}" && return
 
         # don't want to clutter logs if running headless
         test "${HEADLESS}" == '1' && return
@@ -390,7 +391,11 @@ spinner() {
         echo $! >"${spinPidFile}"
         ;;
     stop)
-        test -f "${spinPidFile}" && kill "$(<"${spinPidFile}")"
+        if [[ -f ${spinPidFile} ]]; then
+            local pid="$(<"${spinPidFile}")"
+            kill -0 "${pid}" &>/dev/null && kill "${pid}"
+            rm "${spinPidFile}"
+        fi
         echo -ne ' \n'
         ;;
     esac
diff --git a/lib/build.sh b/lib/build.sh
index 3637f03..09f201a 100644
--- a/lib/build.sh
+++ b/lib/build.sh
@@ -286,14 +286,14 @@ fi' >"${compilerDir}/which"
     else
         arch_flags+=("-march=${ARCH}")
     fi
-	CFLAGS_ARR+=("${arch_flags[@]}")
-	RUSTFLAGS_ARR+=(-C "target-cpu=${ARCH}")
+    CFLAGS_ARR+=("${arch_flags[@]}")
+    RUSTFLAGS_ARR+=(-C "target-cpu=${ARCH}")
 
     # can fail static builds with -fpic
     # warning: too many GOT entries for -fpic, please recompile with -fPIC
     CFLAGS_ARR+=("-fPIC")
-	# add preprocessor flags
-	CFLAGS_ARR+=("${CPPFLAGS_ARR[@]}")
+    # add preprocessor flags
+    CFLAGS_ARR+=("${CPPFLAGS_ARR[@]}")
 
     # set exported env names to stringified arrays
     CPPFLAGS="${CPPFLAGS_ARR[*]}"
@@ -373,7 +373,8 @@ libnuma           2.0.19       tar.gz    https://github.com/numactl/numactl/arch
 
 libass            0.17.4       tar.xz    https://github.com/libass/libass/releases/download/${ver}/libass-${ver}.${ext} libfontconfig,libfreetype,libharfbuzz,libfribidi,libunibreak,libxml2,xz
 libfontconfig     2.17.1       tar.xz    https://gitlab.freedesktop.org/api/v4/projects/890/packages/generic/fontconfig/${ver}/fontconfig-${ver}.${ext} libharfbuzz,expat,brotli
-libfreetype       2.14.1       tar.xz    https://downloads.sourceforge.net/freetype/freetype-${ver}.${ext} bzip,libpng,zlib,brotli
+libfreetype       2.14.1       tar.xz    https://downloads.sourceforge.net/freetype/freetype-${ver}.${ext} bzip,libpng,zlib,brotli,libharfbuzzNFTP
+libharfbuzzNFTP   12.3.0       tar.xz    https://github.com/harfbuzz/harfbuzz/releases/download/${ver}/harfbuzz-${ver}.${ext}
 libharfbuzz       12.3.0       tar.xz    https://github.com/harfbuzz/harfbuzz/releases/download/${ver}/harfbuzz-${ver}.${ext} libfreetype
 libunibreak       6.1          tar.gz    https://github.com/adah1972/libunibreak/releases/download/libunibreak_${ver//./_}/libunibreak-${ver}.${ext}
 libxml2           2.15.1       tar.gz    https://github.com/GNOME/libxml2/archive/refs/tags/v${ver}.${ext}
@@ -1046,8 +1047,16 @@ build_libfontconfig() {
     sanitize_sysroot_libs libfontconfig || return 1
 }
 
+# harfbuzz No FreeType
+build_libharfbuzzNFTP() {
+    DISABLE_FREETYPE=1 build_libharfbuzz
+}
+
 build_libharfbuzz() {
-    meta_meson_build \
+    local addFlag
+    test "${DISABLE_FREETYPE}" -eq 1 && addFlag="-Dfreetype=disabled"
+
+    meta_meson_build ${addFlag} \
         -D tests=disabled \
         -D docs=disabled \
         -D doc_tests=false || return 1
@@ -1105,8 +1114,8 @@ meta_configure_build() {
 }
 
 build_libvpx() {
-	# remove preprocessor flags to not break the build
-	# when including old pre-built headers
+    # remove preprocessor flags to not break the build
+    # when including old pre-built headers
     CFLAGS="${CFLAGS//${CPPFLAGS}/}" meta_configure_build \
         --disable-examples \
         --disable-tools \
@@ -1118,7 +1127,7 @@ build_libvpx() {
         --enable-vp9 \
         --enable-vp9-highbitdepth \
         --enable-better-hw-compatibility \
-		--enable-runtime-cpu-detect \
+        --enable-runtime-cpu-detect \
         --enable-webm-io \
         --enable-libyuv || return 1
     sanitize_sysroot_libs libvpx || return 1
diff --git a/lib/install_deps.sh b/lib/install_deps.sh
index 84bbb07..a988968 100644
--- a/lib/install_deps.sh
+++ b/lib/install_deps.sh
@@ -151,7 +151,7 @@ check_for_req_pkgs() {
     for pkg in $(print_req_pkgs); do
         # pkg_check has ${pkg} unexpanded
         eval "pkg_check=\"${pkg_check}\""
-        ${pkg_check} "${pkg}" >/dev/null 2>&1 || missing_pkgs+=("${pkg}")
+        ${pkg_check} "${pkg}" &>/dev/null || missing_pkgs+=("${pkg}")
     done
 
     if [[ ${#missing_pkgs[@]} -gt 0 ]]; then