From c6adccc5de4dcad865fbe2840b8ea897b3d02cda Mon Sep 17 00:00:00 2001 From: Jeroen Schweitzer Date: Sat, 27 Jun 2026 16:32:44 +0200 Subject: [PATCH] build(make): route `build` through `build-` instead of duplicating the flutter build MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `build` ran `flutter build $(FLUTTER_OS)` inline while build-linux/-macos/ -windows ran the same command separately — two places to keep in sync. Make `build` depend on `build-$(FLUTTER_OS)` so the per-OS target is the single source of the flutter-build command; `build` just adds the C-client bundling on top. `make install` → build → build-, and `make build-linux` still works standalone (release.sh + CI reference it). Co-Authored-By: Claude Opus 4.8 (1M context) --- Makefile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 3c9182d0..0c16c3bc 100644 --- a/Makefile +++ b/Makefile @@ -169,8 +169,7 @@ ui-smoke: ## Build + serve + run Playwright smoke + stop. @sh -c 'trap "tools/ui/stop.sh >/dev/null 2>&1" EXIT; cd tools/ui && npx playwright test smoke.spec.ts' .PHONY: build -build: gen-build-info clide-cli ## flutter build for the current OS (incl. the C CLI client). - flutter build $(FLUTTER_OS) +build: clide-cli build-$(FLUTTER_OS) ## flutter build for the current OS (via build-) + bundle the C CLI client. @install -m 755 $(CLIDE_CLI_BIN) $(CLI_BUNDLE_DEST) @echo "==> bundled C client at $(CLI_BUNDLE_DEST)"