From ef5d7febc826279e81aba84db8eb7ddf8289d2b0 Mon Sep 17 00:00:00 2001
From: Khaled Hosny <khaled@aliftype.com>
Date: Fri, 3 Jun 2022 08:46:30 +0200
Subject: [PATCH] [test/shape] Allow using hb-subset in record-test.sh

I think it is about time we use our own subsetter here. FontTools can
still be used.
---
 test/shape/record-test.sh | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/test/shape/record-test.sh b/test/shape/record-test.sh
index 41e13c3c6..3c1717eba 100755
--- a/test/shape/record-test.sh
+++ b/test/shape/record-test.sh
@@ -19,6 +19,8 @@ if test "x$1" == 'x-o'; then
 	out=$1
 	shift
 fi
+hb_subset=$1
+shift
 hb_shape=$1
 shift
 fontfile=$1
@@ -27,6 +29,10 @@ if test "x${fontfile:0:1}" == 'x-'; then
 	exit 1
 fi
 shift
+if ! echo "$hb_subset" | grep -q 'subset'; then
+	echo "Specify hb-subset (or \"fonttools subset\"): got "$hb_subset"." >&2
+	exit 1
+fi
 if ! echo "$hb_shape" | grep -q 'hb-shape'; then
 	echo "Specify hb-shape (not hb-view, etc): got "$hb_shape"." >&2
 	exit 1
@@ -61,7 +67,7 @@ fi
 glyph_ids=`echo "$text" | $hb_shape $options --no-glyph-names --no-clusters --no-positions "$fontfile" | sed 's/[][]//g; s/|/,/g'`
 
 cp "$fontfile" "$dir/font.ttf"
-echo fonttools subset \
+echo $hb_subset \
 	--glyph-names \
 	--no-hinting \
 	--layout-features='*' \
@@ -69,7 +75,7 @@ echo fonttools subset \
 	--text="$text" \
 	--output-file="$dir/font.subset.ttf" \
 	"$dir/font.ttf"
-fonttools subset \
+$hb_subset \
 	--glyph-names \
 	--no-hinting \
 	--layout-features='*' \