pcbjam/docker/build.sh

426 lines
18 KiB
Shell
Raw Normal View History

#!/bin/bash
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
# Build a KiCad app (pcbnew, eeschema, calculator) inside Docker, then run
# asyncify and friends on the host.
#
# Usage:
# ./docker/build.sh <app>[,<app>...] [args...]
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
#
# Apps:
feat(wasm): kicad_editor — merge the pcbnew+eeschema kifaces into ONE bundle (Part 2) All four editors (PCB / Footprint / Schematic / Symbol) are now runtime --frame choices of a single kicad_editor.wasm (178 MB at -O1 vs 147+82 separate; shared wx/common/boost linked once). One editor per page load, as before; frames pcb / fpedit / sch / symedit. - wasm/editor/: the merged executable target (single_top + both kiface library sets, whole-archive pcbcommon) + the safety-net focus-walk Kiface() dispatch TU. Gated by KICAD_WASM_MERGED_EDITOR (kicad submodule bump carries the fork side: per-engine Kiface/getter binding + ODR renames + dual-kiface launcher). - wasm/bindings/: per-editor collab entries renamed pcbCollab*/schCollab* (JS names unchanged); duplicate kicadOpenFile/kicadCollabOnSave + shared-name registrations guarded behind KICAD_MERGED_EMBIND; new kicad_editor_embind.cpp registers each shared JS name once, dispatching on the live frame. - Build: kicad_editor app (build wrapper, target case arms, 3-object embind compile with the ABI-critical flags, STUB_APP=pcbnew); docker/build.sh "all" = kicad_editor calculator pl_editor gerbview (pcbnew/eeschema stay as explicit debug apps); scripts/kicad/audit-merged-symbols.sh = repeatable ODR-collision audit (run on kicad bumps). - Frontend: Bundle type (bundle ≠ tool); TOOL_BUNDLE maps all four editors to kicad_editor; explicit --frame tokens for pcbnew (pcb) and eeschema (sch); publish list = the 4 real bundles. - Tests/CI: five harnesses load kicad_editor.js with explicit frame tokens; PCBNEW_FAMILY_SPECS renamed BIG_MODULE_SPECS + the 8 eeschema-family specs (they now boot the merged module — SpiderMonkey x86 CI OOM routing); frame-runtime spec covers all four frames from the one bundle. Validated so far: frame-runtime 4/4 (each frame boots with the right title, no aborts, no duplicate embind registration); 24-spec merged-module regression green; 3D raytracer renders. Known pre-existing failure: 3d-viewer title-bar drag deadlock, fixed on main by 7630c7e (2N+8 pthread pre-warm) — picked up by the follow-up rebase. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-02 14:48:11 +02:00
# kicad_editor merged PCB + schematic editor image — serves all four editors
# (PCB / Footprint / Schematic / Symbol) via runtime --frame
2026-06-02 13:35:37 +02:00
# calculator PCB calculator
# pl_editor drawing-sheet editor
2026-06-03 14:23:57 +02:00
# gerbview Gerber viewer
# all build all of the above
feat(wasm): kicad_editor — merge the pcbnew+eeschema kifaces into ONE bundle (Part 2) All four editors (PCB / Footprint / Schematic / Symbol) are now runtime --frame choices of a single kicad_editor.wasm (178 MB at -O1 vs 147+82 separate; shared wx/common/boost linked once). One editor per page load, as before; frames pcb / fpedit / sch / symedit. - wasm/editor/: the merged executable target (single_top + both kiface library sets, whole-archive pcbcommon) + the safety-net focus-walk Kiface() dispatch TU. Gated by KICAD_WASM_MERGED_EDITOR (kicad submodule bump carries the fork side: per-engine Kiface/getter binding + ODR renames + dual-kiface launcher). - wasm/bindings/: per-editor collab entries renamed pcbCollab*/schCollab* (JS names unchanged); duplicate kicadOpenFile/kicadCollabOnSave + shared-name registrations guarded behind KICAD_MERGED_EMBIND; new kicad_editor_embind.cpp registers each shared JS name once, dispatching on the live frame. - Build: kicad_editor app (build wrapper, target case arms, 3-object embind compile with the ABI-critical flags, STUB_APP=pcbnew); docker/build.sh "all" = kicad_editor calculator pl_editor gerbview (pcbnew/eeschema stay as explicit debug apps); scripts/kicad/audit-merged-symbols.sh = repeatable ODR-collision audit (run on kicad bumps). - Frontend: Bundle type (bundle ≠ tool); TOOL_BUNDLE maps all four editors to kicad_editor; explicit --frame tokens for pcbnew (pcb) and eeschema (sch); publish list = the 4 real bundles. - Tests/CI: five harnesses load kicad_editor.js with explicit frame tokens; PCBNEW_FAMILY_SPECS renamed BIG_MODULE_SPECS + the 8 eeschema-family specs (they now boot the merged module — SpiderMonkey x86 CI OOM routing); frame-runtime spec covers all four frames from the one bundle. Validated so far: frame-runtime 4/4 (each frame boots with the right title, no aborts, no duplicate embind registration); 24-spec merged-module regression green; 3D raytracer renders. Known pre-existing failure: 3d-viewer title-bar drag deadlock, fixed on main by 7630c7e (2N+8 pthread pre-warm) — picked up by the follow-up rebase. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-02 14:48:11 +02:00
# pcbnew standalone PCB engine (debug aid; not deployed — kicad_editor is)
# eeschema standalone schematic engine (debug aid; not deployed)
#
# A comma-separated list builds just those apps in order (e.g.
# "calculator,pl_editor" — used to exercise the multi-app pipeline cheaply).
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
#
# Any extra args are forwarded to scripts/kicad/build-<app>.sh (e.g. -j 8,
# --full, --release, --diag=gal).
#
# The build is split into two phases:
# 1. Docker: Compile KiCad to WASM (without asyncify)
# 2. Host: dyncall shims + finalize + asyncify + wasm-opt -O1 (Binaryen submodule via build-wasm-opt.sh)
#
# KICAD_PIPELINE=1 (multi-app builds only): run phase 2 of each app in the
# background while the next app compiles in the container. wasm-opt is
# Amdahl-capped at ~4 effective cores, so on a many-core CI box the container
# would otherwise sit idle for the 1-2h of host-side wasm-opt (run 27226030304:
# 103 min of the 4h was tools serialized behind each other's wasm-opt). At most
# KICAD_PIPELINE_JOBS (default 2) postprocesses run concurrently — pcbnew's wasm-opt
# pass peaks ~34 GB RSS, so 2 fits the 128 GB CI box but NOT a dev Mac: leave
# KICAD_PIPELINE unset locally.
#
# Binaryen is downloaded automatically - no prerequisites needed.
2026-06-01 17:31:03 +02:00
# Auto-launch the live progress dashboard in this terminal (handled by logging.sh,
# which owns the TTY before it re-execs us with output redirected). Set KICAD_NO_MONITOR=1
# to disable. MUST be set before sourcing logging.sh — that's where the dashboard is
# launched, in the pre-re-exec process.
export KICAD_MONITOR=1
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
# Redirect all output to a log file (re-execs script with redirection).
# MUST be sourced before arg parsing — the re-exec relies on the original
# "$@", so any shifts before this point would strip args from the re-exec.
source "$(dirname "$0")/../scripts/common/logging.sh"
2026-06-01 17:31:03 +02:00
# Build-progress markers (parsed by scripts/build-monitor.sh).
source "$(dirname "$0")/../scripts/common/stages.sh"
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
# Pinned toolchain version (single source of truth). Exported so the compose build.args can pass it
# into the Docker image's emsdk install — bumping the toolchain is then a one-line edit in versions.sh.
source "$(dirname "$0")/../scripts/common/versions.sh"
export EMSCRIPTEN_VERSION
set -e
# Stop the builder container when the build ends (any exit path). A lingering
# idle builder keeps the Docker Desktop VM ballooned — each project's builder
# is capped at ${KICAD_DOCKER_MEM:-32G}, and with per-worktree compose projects
# two forgotten containers once read as ~56 GB of host RAM (page cache +
# high-water ballooning). The container is pure scaffolding: all caches live in
# the named volumes and `up -d` restarts it in seconds. Set
# KICAD_KEEP_CONTAINER=1 to keep it running for interactive exec/debugging.
_COMPOSE_FILE="$(cd "$(dirname "$0")" && pwd)/docker-compose.yml"
stop_builder() {
if [[ "${KICAD_KEEP_CONTAINER:-0}" != "1" ]]; then
docker compose -f "${_COMPOSE_FILE}" stop >/dev/null 2>&1 || true
fi
}
2026-06-01 17:31:03 +02:00
# Emit a completion/failure marker no matter how the build ends, so the monitor
# can stop on a clean "done" or show an aborted state instead of hanging.
trap '_rc=$?; if [ $_rc -eq 0 ]; then kw_done; else kw_fail $_rc; fi; stop_builder' EXIT
2026-06-01 17:31:03 +02:00
# On Ctrl-C, mark the build aborted so the final dashboard frame shows failed
# (not a stale "running" state). The EXIT trap above also fires; the monitor reads
# the last marker, so the duplicate is harmless.
trap 'kw_fail 130; exit 130' INT TERM
cd "$(dirname "$0")/.."
VALID_APPS="kicad_editor | pcbnew | eeschema | calculator | pl_editor | gerbview | kicad_tools | occ_service | all"
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
usage() {
echo "Usage: ./docker/build.sh <app>[,<app>...] [args...]" >&2
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
echo " <app>: ${VALID_APPS}" >&2
echo " args: forwarded to scripts/kicad/build-<app>.sh (e.g. -j 8, --release)" >&2
}
# First positional arg must be the app name. No default — picking one would
# silently build the wrong thing for someone who forgot the argument.
if [[ "${1:-}" == "-h" ]] || [[ "${1:-}" == "--help" ]]; then
usage
exit 0
fi
if [[ $# -lt 1 ]] || [[ "$1" == -* ]]; then
echo "Error: missing <app> argument" >&2
usage
exit 1
fi
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
APP_NAME="$1"
shift
# Expand the app argument into APPS[]: "all", a single app, or a comma list.
feat(wasm): kicad_editor — merge the pcbnew+eeschema kifaces into ONE bundle (Part 2) All four editors (PCB / Footprint / Schematic / Symbol) are now runtime --frame choices of a single kicad_editor.wasm (178 MB at -O1 vs 147+82 separate; shared wx/common/boost linked once). One editor per page load, as before; frames pcb / fpedit / sch / symedit. - wasm/editor/: the merged executable target (single_top + both kiface library sets, whole-archive pcbcommon) + the safety-net focus-walk Kiface() dispatch TU. Gated by KICAD_WASM_MERGED_EDITOR (kicad submodule bump carries the fork side: per-engine Kiface/getter binding + ODR renames + dual-kiface launcher). - wasm/bindings/: per-editor collab entries renamed pcbCollab*/schCollab* (JS names unchanged); duplicate kicadOpenFile/kicadCollabOnSave + shared-name registrations guarded behind KICAD_MERGED_EMBIND; new kicad_editor_embind.cpp registers each shared JS name once, dispatching on the live frame. - Build: kicad_editor app (build wrapper, target case arms, 3-object embind compile with the ABI-critical flags, STUB_APP=pcbnew); docker/build.sh "all" = kicad_editor calculator pl_editor gerbview (pcbnew/eeschema stay as explicit debug apps); scripts/kicad/audit-merged-symbols.sh = repeatable ODR-collision audit (run on kicad bumps). - Frontend: Bundle type (bundle ≠ tool); TOOL_BUNDLE maps all four editors to kicad_editor; explicit --frame tokens for pcbnew (pcb) and eeschema (sch); publish list = the 4 real bundles. - Tests/CI: five harnesses load kicad_editor.js with explicit frame tokens; PCBNEW_FAMILY_SPECS renamed BIG_MODULE_SPECS + the 8 eeschema-family specs (they now boot the merged module — SpiderMonkey x86 CI OOM routing); frame-runtime spec covers all four frames from the one bundle. Validated so far: frame-runtime 4/4 (each frame boots with the right title, no aborts, no duplicate embind registration); 24-spec merged-module regression green; 3D raytracer renders. Known pre-existing failure: 3d-viewer title-bar drag deadlock, fixed on main by 7630c7e (2N+8 pthread pre-warm) — picked up by the follow-up rebase. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-02 14:48:11 +02:00
# kicad_editor first in "all" — the merged image is the largest bundle, so its
# host-side wasm-opt chain is the critical path and must start as early as
# possible (especially with KICAD_PIPELINE=1). pcbnew/eeschema stay buildable as
# standalone debug aids but are not part of "all" (not deployed).
# kicad_tools joined "all" for the runner-image CI (tasks-runner 0001 R2) —
# it finalizes in-container (no host wasm-opt tail), so it never contends
# with the editor's critical path.
if [[ "$APP_NAME" == "all" ]]; then
APPS=(kicad_editor occ_service calculator pl_editor gerbview kicad_tools)
else
IFS=',' read -r -a APPS <<< "$APP_NAME"
for app in "${APPS[@]}"; do
case "$app" in
kicad_editor|pcbnew|eeschema|calculator|pl_editor|gerbview|kicad_tools|occ_service) ;;
*)
echo "Error: unknown app '$app' (expected: ${VALID_APPS})" >&2
usage
exit 1
;;
esac
done
fi
# Use branch name as Docker Compose project name for isolated containers/volumes.
# Honor a pre-set COMPOSE_PROJECT_NAME so a build can target an existing volume
# (e.g. reuse another branch's already-provisioned deps).
BRANCH_NAME=$(git rev-parse --abbrev-ref HEAD | tr '/' '-' | tr '[:upper:]' '[:lower:]')
export COMPOSE_PROJECT_NAME="${COMPOSE_PROJECT_NAME:-kicad-wasm-${BRANCH_NAME}}"
echo "Using Docker project: ${COMPOSE_PROJECT_NAME}"
echo "Building app: ${APP_NAME}"
# Build phase (cache split). The container compile produces an OPT-INDEPENDENT
# base wasm; the only opt-DEPENDENT work is the final `wasm-opt -O$LEVEL` shrink
# inside the host post-process (apply-asyncify.sh). Splitting them lets CI cache
# the expensive compile once (shared regardless of the asyncify tail) and re-run just the
# asyncify/-O tail per opt level — see .github/workflows/.
# (default) both — compile in-container, then host post-process.
# --compile-only — only the in-container compile → base wasm in output/.
# --postprocess-only — only the host post-process (dyncall + finalize +
# asyncify + wasm-opt -O$BINARYEN_OPT_LEVEL) on the
# existing output/ base wasm; NO container needed
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
# (build-wasm-opt.sh self-provisions the Binaryen submodule).
# Extracted here so they are NOT forwarded to the inner build-<app>.sh scripts.
PHASE="both"
_FILTERED=()
for _arg in "$@"; do
case "$_arg" in
--compile-only) PHASE="compile" ;;
--postprocess-only) PHASE="postprocess" ;;
*) _FILTERED+=("$_arg") ;;
esac
done
set -- "${_FILTERED[@]+"${_FILTERED[@]}"}"
# Add -j 10 by default if no -j flag is given
ARGS=("$@")
if [[ ! " ${ARGS[*]} " =~ " -j " ]]; then
ARGS+=("-j" "10")
fi
# Container compile + source sync — skipped entirely for --postprocess-only,
# which is pure host work on the already-built base wasm in output/.
if [[ "$PHASE" != "postprocess" ]]; then
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
# Start container if not running. --build so the image is rebuilt when the pinned EMSCRIPTEN_VERSION
# (build-arg from versions.sh) changes; Docker layer-caches it to a near no-op when unchanged.
docker compose -f docker/docker-compose.yml up -d --build
# Sync source code to container volume (fixes macOS Docker VirtioFS issues)
# Use --checksum to only transfer files with different CONTENT, not timestamps.
# This avoids the timestamp mismatch cycle that caused full rebuilds every time.
# Transferred files get current container time, so make detects them correctly.
2026-06-01 17:31:03 +02:00
kw_stage container-sync
echo "Syncing source code to container..."
2026-05-28 17:21:26 +02:00
# rsync into the macOS-backed volume intermittently hits transient VirtioFS glitches:
# temp-file rename failures (exit 23) or vanished-source files (exit 24, harmless).
# --inplace avoids the temp-file+rename pattern that triggers exit 23; retry up to 3x
# for any residual flakiness (--checksum makes each retry skip already-synced files).
sync_rc=0
for sync_attempt in 1 2 3; do
if docker compose -f docker/docker-compose.yml exec kicad-wasm-builder \
rsync -r --delete --checksum --inplace \
--exclude="build-wasm" \
--exclude="output" \
--exclude=".git" \
--exclude="logs" \
--exclude=".idea" \
--exclude="node_modules" \
--exclude="tools/emsdk" \
/workspace-host/ /workspace/
then
sync_rc=0
else
sync_rc=$?
fi
{ [ $sync_rc -eq 0 ] || [ $sync_rc -eq 24 ]; } && break
echo "rsync attempt ${sync_attempt} failed (exit ${sync_rc}); retrying in 2s..."
sleep 2
done
if [ $sync_rc -ne 0 ] && [ $sync_rc -ne 24 ]; then
echo "ERROR: source sync failed after retries (exit ${sync_rc})"; exit 1
fi
fi # end: container compile + sync guard ("$PHASE" != postprocess)
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
# Map an app name to its inner CMake build subdirectory. Most apps share their
# subdir name with the app name; pcb_calculator emits OUTPUT_NAME=calculator
# but lives under the pcb_calculator/ subtree.
kicad_subdir_for() {
case "$1" in
calculator) echo "pcb_calculator" ;;
pl_editor) echo "pagelayout_editor" ;;
*) echo "$1" ;;
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
esac
}
# Phase 1 of one app: compile in the container and copy the output to ./output.
2026-06-01 17:31:03 +02:00
# Args: <app> [index] [total] — index/total drive the monitor's app counter.
compile_app() {
local app="$1"
2026-06-01 17:31:03 +02:00
local index="${2:-1}"
local total="${3:-1}"
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
local subdir
subdir=$(kicad_subdir_for "$app")
2026-06-01 17:31:03 +02:00
kw_app "$app" "$index" "$total"
echo ""
2026-06-01 17:31:03 +02:00
echo "=== Building ${app} (${index}/${total}) ==="
refactor: collapse dual-mode plumbing — the DOM port is the only WASM build The canvas (wxUniversal) mode is gone (wxwidgets submodule); remove every piece of side-by-side plumbing so there is exactly one build and one test flow: - scripts/build-wxuniversal-wasm.sh -> scripts/build-wx-wasm.sh; no --dom/--enable-universal; builds into build-wasm/wxwidgets - build-wasm-test.sh: no DOM_BUILD / apps-dom rsync mirror / PORT=dom; apps build straight into tests/apps (Makefile.wasm PORT conditionals collapsed; wx.js + wx-dom.js always pre-js) - docker/build.sh, build-kicad-target.sh, env.sh: WX_PORT / -dom / -universal suffixes removed; kicad builds to kicad-<app>, outputs to output/; wx.js/wx-dom.js copied from the real source path (/workspace/wxwidgets/build/wasm — the old build-wasm path never existed and silently failed) - setup-kicad-wasm.sh: single target dir; the perl wx-dom.js injection is gone — the 7 checked-in kicad pages now reference wx-dom.js directly - playwright configs serve apps/; fixtures drop the test-results/dom and logs/wxwidgets/dom namespacing; boot.spec asserts wxDomPort unconditionally; pcbnew.spec uses one reference image; appearance.spec assertions unconditional - compare/update-baseline-screenshots.sh: --port removed - tests/gal-regression/wasm/Makefile: links build-wasm/wxwidgets and carries wx-dom.js as a second pre-js — the gal-webgl suite (30 specs) now actually builds and runs here (it needed host-side boost+glm via scripts/deps; the bundle had been missing, timing the whole spec out) - tests: clickCanvas() dispatches via page.mouse (DOM widgets legitimately cover the canvas; locator actionability refused the click); the comprehensive spec drives wxChoice through its native <select> (browser-owned popup cannot be coordinate-clicked) - docs: README/CLAUDE.md/build.md script names and dirs; features/wx-dom-port README reframed (DOM is THE port), visual-notes bugs 26-28; FindwxWidgets.cmake config label drops 'wasmuniv' - wxwidgets submodule -> 9dbacc9448 (DOM-only port, fork diff shrunk) Gate: full wx e2e suite 292 passed / 1 skipped / 0 failed — first run ever with the gal-webgl specs green (28 scenarios + load + sequential). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-06-12 10:18:16 +02:00
local out_dir="output"
local kicad_build="kicad-${app}"
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
# Run build inside the container.
# -e EMSDK=/emsdk: `docker compose exec` bypasses the entrypoint that sources
refactor(build): fold calculator into unified docker/build.sh dispatch Convert the calculator port (commit 0464470) from the parallel/copy-paste pattern to the dispatch pattern used by pcbnew + eeschema. - docker/build.sh: <app> is now required (no pcbnew default); missing, unknown, and --help paths all print "pcbnew | eeschema | calculator | all". Adds calculator to build_app() via kicad_subdir_for() (pcb_calculator inner subdir, calculator.* output basename). Also fixes the EMSDK=/emsdkkicad-wasm-builder typo that snuck in during the calc merge. - scripts/kicad/build-kicad-target.sh: accepts calculator; introduces KICAD_TARGET (pcb_calculator for calc, == APP_NAME otherwise) used for the make target, embind include path, and final-log line. - scripts/kicad/build-calculator.sh: 305-line copy of build-pcbnew.sh collapsed to a 7-line wrapper around build-kicad-target.sh. - tests/scripts/setup-kicad-wasm.sh: copy_app calculator added with the same pcb_calculator subdir mapping for the docker-volume fallback path. - tests/package.json: test:calculator* routed through the shared playwright-kicad.config.ts kicad/calculator.spec.ts (mirrors eeschema). - Delete: docker/build-calculator.sh, tests/scripts/setup-calculator-wasm.sh, tests/playwright-calculator.config.ts. Bumps wxwidgets d1d1627 -> 6fb2eac (origin/wasm-port). The new sha includes "unic/combobox: add GetCurrentSelection() inline default" which calc's kicad/pcb_calculator/widgets/unit_selector.cpp needs to compile. Without this bump the unified dispatch would expose calc as a buildable target but the build itself would fail. Verified: build.sh all completes clean across all three apps; pcbnew/eeschema e2e pass; calc compiles and launches (test stability separate from this refactor). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-29 12:01:44 +02:00
# emsdk_env.sh, so the build shell would lack emcc/embuilder on PATH. Setting
# EMSDK lets scripts/common/env.sh source /emsdk/emsdk_env.sh and activate the toolchain.
# BUILD_3D_VIEWER passes through EMPTY when the host didn't set it, so
# build-kicad-target.sh can apply per-app defaults (ON for editors, OFF
# for headless CLIs like kicad_tools — the gl1 shim needs glm).
refactor: collapse dual-mode plumbing — the DOM port is the only WASM build The canvas (wxUniversal) mode is gone (wxwidgets submodule); remove every piece of side-by-side plumbing so there is exactly one build and one test flow: - scripts/build-wxuniversal-wasm.sh -> scripts/build-wx-wasm.sh; no --dom/--enable-universal; builds into build-wasm/wxwidgets - build-wasm-test.sh: no DOM_BUILD / apps-dom rsync mirror / PORT=dom; apps build straight into tests/apps (Makefile.wasm PORT conditionals collapsed; wx.js + wx-dom.js always pre-js) - docker/build.sh, build-kicad-target.sh, env.sh: WX_PORT / -dom / -universal suffixes removed; kicad builds to kicad-<app>, outputs to output/; wx.js/wx-dom.js copied from the real source path (/workspace/wxwidgets/build/wasm — the old build-wasm path never existed and silently failed) - setup-kicad-wasm.sh: single target dir; the perl wx-dom.js injection is gone — the 7 checked-in kicad pages now reference wx-dom.js directly - playwright configs serve apps/; fixtures drop the test-results/dom and logs/wxwidgets/dom namespacing; boot.spec asserts wxDomPort unconditionally; pcbnew.spec uses one reference image; appearance.spec assertions unconditional - compare/update-baseline-screenshots.sh: --port removed - tests/gal-regression/wasm/Makefile: links build-wasm/wxwidgets and carries wx-dom.js as a second pre-js — the gal-webgl suite (30 specs) now actually builds and runs here (it needed host-side boost+glm via scripts/deps; the bundle had been missing, timing the whole spec out) - tests: clickCanvas() dispatches via page.mouse (DOM widgets legitimately cover the canvas; locator actionability refused the click); the comprehensive spec drives wxChoice through its native <select> (browser-owned popup cannot be coordinate-clicked) - docs: README/CLAUDE.md/build.md script names and dirs; features/wx-dom-port README reframed (DOM is THE port), visual-notes bugs 26-28; FindwxWidgets.cmake config label drops 'wasmuniv' - wxwidgets submodule -> 9dbacc9448 (DOM-only port, fork diff shrunk) Gate: full wx e2e suite 292 passed / 1 skipped / 0 failed — first run ever with the gal-webgl specs green (28 scenarios + load + sequential). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-06-12 10:18:16 +02:00
docker compose -f docker/docker-compose.yml exec -e EMSDK=/emsdk \
-e BUILD_3D_VIEWER="${BUILD_3D_VIEWER:-}" \
kicad-wasm-builder \
"/workspace/scripts/kicad/build-${app}.sh" "${ARGS[@]}"
# Copy output to host-accessible directory.
# ${app}.wasm.debug.wasm contains DWARF debug info (when built with -gseparate-dwarf).
2026-06-01 17:31:03 +02:00
kw_stage copy-output
echo "Copying ${app} build output to ./${out_dir}/..."
docker compose -f docker/docker-compose.yml exec kicad-wasm-builder \
bash -c "mkdir -p /workspace/${out_dir} && \
cp /workspace/build-wasm/${kicad_build}/${subdir}/${app}.{js,wasm,wasm.debug.wasm,wasm.map,worker.js} /workspace/${out_dir}/ 2>/dev/null || \
cp /workspace/build-wasm/${kicad_build}/${subdir}/${app}.{js,wasm} /workspace/${out_dir}/; \
cp /workspace/build-wasm/${kicad_build}/resources/images.tar.gz /workspace/${out_dir}/ 2>/dev/null || true; \
refactor: collapse dual-mode plumbing — the DOM port is the only WASM build The canvas (wxUniversal) mode is gone (wxwidgets submodule); remove every piece of side-by-side plumbing so there is exactly one build and one test flow: - scripts/build-wxuniversal-wasm.sh -> scripts/build-wx-wasm.sh; no --dom/--enable-universal; builds into build-wasm/wxwidgets - build-wasm-test.sh: no DOM_BUILD / apps-dom rsync mirror / PORT=dom; apps build straight into tests/apps (Makefile.wasm PORT conditionals collapsed; wx.js + wx-dom.js always pre-js) - docker/build.sh, build-kicad-target.sh, env.sh: WX_PORT / -dom / -universal suffixes removed; kicad builds to kicad-<app>, outputs to output/; wx.js/wx-dom.js copied from the real source path (/workspace/wxwidgets/build/wasm — the old build-wasm path never existed and silently failed) - setup-kicad-wasm.sh: single target dir; the perl wx-dom.js injection is gone — the 7 checked-in kicad pages now reference wx-dom.js directly - playwright configs serve apps/; fixtures drop the test-results/dom and logs/wxwidgets/dom namespacing; boot.spec asserts wxDomPort unconditionally; pcbnew.spec uses one reference image; appearance.spec assertions unconditional - compare/update-baseline-screenshots.sh: --port removed - tests/gal-regression/wasm/Makefile: links build-wasm/wxwidgets and carries wx-dom.js as a second pre-js — the gal-webgl suite (30 specs) now actually builds and runs here (it needed host-side boost+glm via scripts/deps; the bundle had been missing, timing the whole spec out) - tests: clickCanvas() dispatches via page.mouse (DOM widgets legitimately cover the canvas; locator actionability refused the click); the comprehensive spec drives wxChoice through its native <select> (browser-owned popup cannot be coordinate-clicked) - docs: README/CLAUDE.md/build.md script names and dirs; features/wx-dom-port README reframed (DOM is THE port), visual-notes bugs 26-28; FindwxWidgets.cmake config label drops 'wasmuniv' - wxwidgets submodule -> 9dbacc9448 (DOM-only port, fork diff shrunk) Gate: full wx e2e suite 292 passed / 1 skipped / 0 failed — first run ever with the gal-webgl specs green (28 scenarios + load + sequential). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-06-12 10:18:16 +02:00
cp /workspace/wxwidgets/build/wasm/wx.js /workspace/${out_dir}/ 2>/dev/null || true; \
cp /workspace/wxwidgets/build/wasm/wx-dom.js /workspace/${out_dir}/ 2>/dev/null || true"
# The container runs as root, so files in the bind-mounted ./output land
# root-owned on the host. macOS Docker Desktop remaps ownership to the host
# user, but on a Linux CI runner the following host-side steps (dyncall,
# finalize, asyncify) can't write into ./output. Hand ownership back.
docker compose -f docker/docker-compose.yml exec kicad-wasm-builder \
chown -R "$(id -u):$(id -g)" /workspace/output || true
}
# Phase 2 of one app: host-side post-processing (dyncall shims, finalize,
# asyncify + wasm-opt -O1). Pure host work on output/${app}.* — independent of the
# container, which is what makes it safe to run in the background while the
# next app compiles.
postprocess_app() {
local app="$1"
refactor: collapse dual-mode plumbing — the DOM port is the only WASM build The canvas (wxUniversal) mode is gone (wxwidgets submodule); remove every piece of side-by-side plumbing so there is exactly one build and one test flow: - scripts/build-wxuniversal-wasm.sh -> scripts/build-wx-wasm.sh; no --dom/--enable-universal; builds into build-wasm/wxwidgets - build-wasm-test.sh: no DOM_BUILD / apps-dom rsync mirror / PORT=dom; apps build straight into tests/apps (Makefile.wasm PORT conditionals collapsed; wx.js + wx-dom.js always pre-js) - docker/build.sh, build-kicad-target.sh, env.sh: WX_PORT / -dom / -universal suffixes removed; kicad builds to kicad-<app>, outputs to output/; wx.js/wx-dom.js copied from the real source path (/workspace/wxwidgets/build/wasm — the old build-wasm path never existed and silently failed) - setup-kicad-wasm.sh: single target dir; the perl wx-dom.js injection is gone — the 7 checked-in kicad pages now reference wx-dom.js directly - playwright configs serve apps/; fixtures drop the test-results/dom and logs/wxwidgets/dom namespacing; boot.spec asserts wxDomPort unconditionally; pcbnew.spec uses one reference image; appearance.spec assertions unconditional - compare/update-baseline-screenshots.sh: --port removed - tests/gal-regression/wasm/Makefile: links build-wasm/wxwidgets and carries wx-dom.js as a second pre-js — the gal-webgl suite (30 specs) now actually builds and runs here (it needed host-side boost+glm via scripts/deps; the bundle had been missing, timing the whole spec out) - tests: clickCanvas() dispatches via page.mouse (DOM widgets legitimately cover the canvas; locator actionability refused the click); the comprehensive spec drives wxChoice through its native <select> (browser-owned popup cannot be coordinate-clicked) - docs: README/CLAUDE.md/build.md script names and dirs; features/wx-dom-port README reframed (DOM is THE port), visual-notes bugs 26-28; FindwxWidgets.cmake config label drops 'wasmuniv' - wxwidgets submodule -> 9dbacc9448 (DOM-only port, fork diff shrunk) Gate: full wx e2e suite 292 passed / 1 skipped / 0 failed — first run ever with the gal-webgl specs green (28 scenarios + load + sequential). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-06-12 10:18:16 +02:00
local out_dir="output"
# The headless CLI and the OCC service are finalized in-container (real
# tools, small -g0 wasm) and build with ASYNCIFY=0, so they need no host
feat(wasm): occ-split — lazy occ_service worker; kicad_editor drops OCC (−31%) Move OpenCASCADE out of the merged editor image into occ_service: a separate emscripten module (-sASYNCIFY=0, MODULARIZE, in-container -Oz finalize, 2N+8 pre-warmed pthread pool) booted lazily in a dedicated Web Worker on the first STEP export or STEP/IGES model parse. kicad_editor.wasm ~190 MB -> 130 MB; sessions that never touch OCC never fetch its 57 MB. STEP export works in the browser for the first time: the unchanged desktop dialog runs EXPORTER_STEP, whose wasm shadow suspends into globalThis.occService and the export bytes go straight to a browser download (never entering the editor heap). STEP/IGES 3D models parse in the worker via the oce shadow (S3D WriteCache/ReadCache wire). - wasm/occ-service/: service CMake target (hooked from the kicad fork's top-level CMakeLists, wasm/editor pattern), embind entry (occExport/occLoadModel), wxConfig pre-js. - wasm/stubs/{exporter_step,oce_plugin}_stub.cpp: EM_ASYNC_JS worker bridges (callee-shadowing; no caller #ifdefs). - web/standalone: provider installed whenever the kicad_editor bundle boots (cross-face safe); ONE shared worker-boot source occ-worker.js (vite ?raw; the e2e stub reads the same file) — blob worker with locateFile absolutized against the glue URL; export download-name guard. - deps: OCC builds with RapidJSON so its glTF/GLB writer exists — pinned to the vcpkg master snapshot 2025-02-26 (24b5e7a8b27f), the same code official KiCad consumes via vcpkg.json's opencascade[rapidjson]; rapidjson's latest tag (v1.1.0, 2016) is ill-formed under modern clang. - tests: occ-export dialog e2e (lazy-fetch boundary + STEP download bytes), occ-probe incl. a 9-format matrix (step/stpz/brep/xao/ply/stl/glb/u3d/pdf), 3d-viewer-models hard-asserts the worker parse; occ provider stub installed ambiently by the kicad fixtures. Validated against desktop kicad-cli 10.0.4: geometric exact equality (bbox delta 0 um, volume delta 0.0000%) for STEP/GLB/STL/BREP/STPZ across three boards and option sweeps — with desktop OCC 7.9 vs wasm OCC 7.8; PLY/XAO/PDF structurally equal; U3D same-size (quantizer float LSBs differ). Full kicad e2e green on Firefox and Chromium; standalone verified end to end (lazy fetch only on the Export click; export.step 60,628 B ISO-10303-21; loadModel 700 KB STEP -> 569 KB scenegraph cache). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-03 12:39:58 +02:00
# post-processing (no dyncall shims, no finalize, no asyncify).
if [ "$app" = "kicad_tools" ] || [ "$app" = "occ_service" ]; then
echo "Skipping host post-processing for ${app} (finalized in-container)"
return 0
fi
# Inject dynCall shims (fixes "dynCall_* is not defined" errors in Emscripten 4.x)
2026-06-01 17:31:03 +02:00
kw_stage dyncall-shims
./scripts/common/inject-dyncall-shims.sh "${out_dir}/${app}.js"
# Merge Module.ENV into the runtime ENV: the emscripten glue never merges it,
# so ?trace= (boot.ts sets Module.ENV.KICAD_TRACE) was a silent no-op —
# environ_get on the app pthread proxies to the main thread, whose ENV stayed
# empty. Replaces a manual per-build glue edit. Idempotent; runtime no-op when
# Module.ENV is unset. See docs/features/libs/0013.
node ./scripts/common/patch-env-shim.mjs "${out_dir}/${app}.js"
# Apply wasm-emscripten-finalize on host (skipped in Docker due to memory limits)
2026-06-01 17:31:03 +02:00
kw_stage finalize
./scripts/common/apply-finalize.sh "${out_dir}/${app}.wasm" "${out_dir}/${app}.wasm"
# Apply asyncify transformation on host (the ASYNCIFY=0 CLIs returned
# early above and never reach this).
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
# apply-asyncify always runs the --hoist-cpp-catches pass FIRST (native wasm-EH is the only build
# mode) so Asyncify can suspend from inside C++ catch arms, then asyncify + removelist + wasm-opt -O1.
kw_stage asyncify
./scripts/common/apply-asyncify.sh "${out_dir}/${app}.wasm" "${out_dir}/${app}.wasm"
}
# --- Pipelined driver state (KICAD_PIPELINE=1) ---
# One background postprocess per app; logs + rc files land in logs/build/ so the
# interleaved output stays readable and failures survive until the final wait.
PIPELINE_PIDS=()
PIPELINE_APPS_BG=()
PIPELINE_LOG_DIR="logs/build"
PIPELINE_TS="$(date +%Y%m%d-%H%M%S)"
pipeline_running_count() {
local n=0 pid
for pid in "${PIPELINE_PIDS[@]}"; do
kill -0 "$pid" 2>/dev/null && n=$((n + 1))
done
echo "$n"
}
# Launch postprocess_app in the background, capped at KICAD_PIPELINE_JOBS
# concurrent jobs (default 2: pcbnew's wasm-opt pass peaks ~34 GB RSS; two postprocesses
# plus the container compile fit the 128 GB CI box). Portable poll loop instead
# of `wait -n` (absent in macOS bash 3.2).
pipeline_postprocess() {
local app="$1"
local max_jobs="${KICAD_PIPELINE_JOBS:-2}"
while [ "$(pipeline_running_count)" -ge "$max_jobs" ]; do
sleep 10
done
local log_file="${PIPELINE_LOG_DIR}/postprocess-${app}-${PIPELINE_TS}.log"
echo "Pipelining host-side postprocess of ${app} (log: ${log_file})"
(
postprocess_app "$app" >"$log_file" 2>&1
echo $? >"${log_file}.rc"
) &
PIPELINE_PIDS+=($!)
PIPELINE_APPS_BG+=("$app")
}
# Wait for all background postprocesses, replay their logs into the main log,
# and fail if any of them failed.
pipeline_wait_all() {
local failed=0 i pid app log_file rc
for i in "${!PIPELINE_PIDS[@]}"; do
pid="${PIPELINE_PIDS[$i]}"
app="${PIPELINE_APPS_BG[$i]}"
log_file="${PIPELINE_LOG_DIR}/postprocess-${app}-${PIPELINE_TS}.log"
wait "$pid" || true
rc="$(cat "${log_file}.rc" 2>/dev/null || echo 1)"
echo ""
echo "=== Postprocess ${app} (rc=${rc}) — ${log_file} ==="
cat "$log_file" 2>/dev/null || true
if [ "$rc" != "0" ]; then
echo "ERROR: postprocess of ${app} failed (rc=${rc})"
failed=1
fi
done
return "$failed"
}
TOTAL_APPS="${#APPS[@]}"
# Shared pipeline trap: on a failure, kill orphaned background wasm-opt jobs
# (each ~30 GB) and keep the monitor's done/fail marker from the EXIT trap.
_install_pipeline_trap() {
trap '_rc=$?; for p in "${PIPELINE_PIDS[@]}"; do kill "$p" 2>/dev/null || true; done; if [ $_rc -eq 0 ]; then kw_done; else kw_fail $_rc; fi; stop_builder' EXIT
}
if [[ "$PHASE" == "compile" ]]; then
# --compile-only: produce the opt-independent base wasm; no host post-process.
idx=1
for app in "${APPS[@]}"; do
compile_app "$app" "$idx" "$TOTAL_APPS"
idx=$((idx + 1))
done
elif [[ "$PHASE" == "postprocess" ]]; then
# --postprocess-only: pure host post-process on the existing output/ base
# wasm (no container). Parallelize across apps when pipelining.
if [[ "${KICAD_PIPELINE:-0}" == "1" ]] && [ "$TOTAL_APPS" -gt 1 ]; then
mkdir -p "$PIPELINE_LOG_DIR"
fix(wasm-eh): build wasm-emscripten-finalize from the Binaryen submodule (unbreak CI) The native-EH migration pointed apply-finalize.sh at emsdk's bundled wasm-emscripten-finalize (${EMSDK:-tools/emsdk}/upstream/bin), which exists on a dev machine (tools/emsdk persists from a prior build) but never on the ephemeral CI host — emsdk lives only inside the Docker image, and nothing on the host post-process path provisions it. So the host post-process died at finalize for every app once the binaryen-SHA cache re-key forced it to actually run. (Not the missing-ninja theory — ninja was installed; the run never reached the wasm-opt build, which is downstream of finalize.) Finish the "Binaryen submodule everywhere" migration: build-wasm-opt.sh now also builds wasm-emscripten-finalize, and apply-finalize.sh takes it from that build (next to wasm-opt). The host post-process is now emsdk-free (dyncall=node, finalize+asyncify=submodule v130) and finalize/wasm-opt share one Binaryen version (previously finalize was emsdk's v121, wasm-opt the submodule's v130). get-wasm-opt.sh stays for bench only; dropped it from the wasm cache key and added build-wasm-opt.sh. Also surface the from-source Binaryen build as a "Build Binaryen" stage in build-monitor.sh (it had no marker), emitted at the pipelined pre-warm sites. Validated: a clean from-scratch `docker/build.sh calculator --build-deps` (deps + compile + finalize via the submodule binary + asyncify + -O2) succeeded end-to-end; finalize resolved to build-wasm/tools/binaryen-hoist-build/bin. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-30 11:55:52 +02:00
kw_stage binaryen
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
./scripts/binaryen-hoist-pass/build-wasm-opt.sh >/dev/null # pre-warm Binaryen (submodule) once
_install_pipeline_trap
for app in "${APPS[@]}"; do
pipeline_postprocess "$app"
done
pipeline_wait_all
else
for app in "${APPS[@]}"; do
postprocess_app "$app"
done
fi
elif [[ "${KICAD_PIPELINE:-0}" == "1" ]] && [ "$TOTAL_APPS" -gt 1 ]; then
# both, pipelined: overlap app[i+1]'s container compile with app[i]'s host
# post-process (KICAD_PIPELINE=1).
mkdir -p "$PIPELINE_LOG_DIR"
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
# Pre-build the Binaryen submodule once — two concurrent postprocesses racing
# the first from-source build would collide.
fix(wasm-eh): build wasm-emscripten-finalize from the Binaryen submodule (unbreak CI) The native-EH migration pointed apply-finalize.sh at emsdk's bundled wasm-emscripten-finalize (${EMSDK:-tools/emsdk}/upstream/bin), which exists on a dev machine (tools/emsdk persists from a prior build) but never on the ephemeral CI host — emsdk lives only inside the Docker image, and nothing on the host post-process path provisions it. So the host post-process died at finalize for every app once the binaryen-SHA cache re-key forced it to actually run. (Not the missing-ninja theory — ninja was installed; the run never reached the wasm-opt build, which is downstream of finalize.) Finish the "Binaryen submodule everywhere" migration: build-wasm-opt.sh now also builds wasm-emscripten-finalize, and apply-finalize.sh takes it from that build (next to wasm-opt). The host post-process is now emsdk-free (dyncall=node, finalize+asyncify=submodule v130) and finalize/wasm-opt share one Binaryen version (previously finalize was emsdk's v121, wasm-opt the submodule's v130). get-wasm-opt.sh stays for bench only; dropped it from the wasm cache key and added build-wasm-opt.sh. Also surface the from-source Binaryen build as a "Build Binaryen" stage in build-monitor.sh (it had no marker), emitted at the pipelined pre-warm sites. Validated: a clean from-scratch `docker/build.sh calculator --build-deps` (deps + compile + finalize via the submodule binary + asyncify + -O2) succeeded end-to-end; finalize resolved to build-wasm/tools/binaryen-hoist-build/bin. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-30 11:55:52 +02:00
kw_stage binaryen
feat(wasm-eh): migrate the WASM build to native wasm exceptions (+ 3D viewer default-on) Replace the legacy Emscripten JS-exceptions model with native wasm-EH (legacy encoding) across the whole build, keeping Asyncify coroutines working via a from-source Binaryen --hoist-cpp-catches pre-pass. Net result: native-EH is the only build mode, the 3D viewer is on by default, and pcbnew shrinks substantially. Highlights: - Binaryen submodule everywhere + --hoist-cpp-catches integration in apply-asyncify; post-link Asyncify covers every app wasm (not just standalone test wasm). - Build deps (incl. OpenCASCADE without OCC_CONVERT_SIGNALS) and all KiCad apps with -fwasm-exceptions; emscripten_sleep added to the post-link asyncify-imports. - libcontext fiber entry wired under native exceptions; while-loop main loop + currData shim injected into all wx apps. - Native-EH collab apply fixed: DEBUG-define the embind TU + match all out-of-CMake C++ TUs' ABI flags to the core, fixing the vtable-layout skew / mis-dispatch. - 3D viewer enabled by default (real raytracer linked, not the stub). - Retire the EH-spike scaffolding; flip the asyncify-races ablation pins to shim-redundancy pins (native-EH stays clean with the legacy shims ablated). - Fix the asyncify-races quiescence check to not require Asyncify.currData==0: under the native-EH per-frame-yield top loop the main stack is asyncify-suspended every frame, so currData legitimately churns (a freed-but-not-yet-nulled buffer, not a leak). Refresh the pcbnew toolbar screenshot baseline for the new kicad. - CI: drop the obsolete binaryen_version input/env (the build uses the binaryen submodule fork's wasm-opt, not a version download); key the wasm-output cache on the binaryen submodule SHA instead. Bumps the wxwidgets + binaryen submodules to their squashed feature commits. Validated green: all 7 apps native-EH (real 3D in pcbnew); KiCad e2e 63/63 Firefox + Chromium (3D viewer renders); wx 336; coroutine 34/34 both engines; asyncify 7/7 both engines. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-29 19:50:18 +02:00
./scripts/binaryen-hoist-pass/build-wasm-opt.sh >/dev/null
_install_pipeline_trap
idx=1
for app in "${APPS[@]}"; do
compile_app "$app" "$idx" "$TOTAL_APPS"
pipeline_postprocess "$app"
idx=$((idx + 1))
done
pipeline_wait_all
else
# both, sequential.
idx=1
for app in "${APPS[@]}"; do
compile_app "$app" "$idx" "$TOTAL_APPS"
postprocess_app "$app"
idx=$((idx + 1))
done
fi
echo ""
echo "Build complete. Output files in ./output/"
ls -lh output/