From feb91ad281791d3ffb114c25d5641250687f8955 Mon Sep 17 00:00:00 2001
From: "alexei.dolgolyov" <dolgolyov.alexei@gmail.com>
Date: Tue, 7 Apr 2026 23:41:42 +0300
Subject: [PATCH] fix(build): fix shell syntax error in smoke_test_imports
 heredoc
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The 'cmd <<EOF || { ... }' pattern confuses bash's parser — the closing
brace of the inline error block collides with the function's closing
brace. Rewrote to capture the python script into a local var via
$(cat <<EOF), then run it with -c and a plain 'if !' guard.
---
 build-common.sh | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/build-common.sh b/build-common.sh
index 5b8ef80..f35ca4c 100644
--- a/build-common.sh
+++ b/build-common.sh
@@ -217,10 +217,8 @@ smoke_test_imports() {
     # installation — Pillow for example is only a Windows dep. But if a
     # module's top-level package dir exists in site-packages and we
     # can't import it, that's a broken install and we abort.
-    PYTHONPATH="$pypath" "$py_cmd" - "$sp_dir" <<'PYEOF' || {
-        echo "  ERROR: smoke test failed — site-packages is broken, aborting build"
-        return 1
-    }
+    local smoke_script
+    smoke_script=$(cat <<'PYEOF'
 import importlib
 import os
 import sys
@@ -266,4 +264,10 @@ if failed:
 
 print(f'  Smoke test passed ({tested} imported, {skipped} not installed)')
 PYEOF
+)
+
+    if ! PYTHONPATH="$pypath" "$py_cmd" -c "$smoke_script" "$sp_dir"; then
+        echo "  ERROR: smoke test failed — site-packages is broken, aborting build"
+        return 1
+    fi
 }