Skip to content

Fix ARM64 container builds and local Docker support#8

Merged
varun-krishnamurthy merged 9 commits into
vyogotech:developfrom
mothomas:cursor/arm64-build-fixes
Jun 7, 2026
Merged

Fix ARM64 container builds and local Docker support#8
varun-krishnamurthy merged 9 commits into
vyogotech:developfrom
mothomas:cursor/arm64-build-fixes

Conversation

@mothomas
Copy link
Copy Markdown

Summary

  • Fix ARM64 image builds by compiling Redis from source, pinning frappe-bench, and pre-installing esbuild for aarch64
  • Resolve Containerfile BuildKit warnings (APP_ROOT, nginx env ordering, amd64-only base image)
  • Makefile auto-detects podman or docker and always builds from Containerfile

Test plan

  • make build-arm64 FRAPPE_VERSION=version-16 on Apple Silicon
  • Confirm no BuildKit warnings for InvalidBaseImagePlatform / UndefinedVar
  • Verify container starts and esbuild/watch runs on native ARM

Made with Cursor

Mobin Thomas and others added 9 commits May 31, 2026 15:08
Compile Redis from source, pin frappe-bench, pre-install esbuild for aarch64, and resolve Containerfile build warnings so Apple Silicon builds succeed with make build-arm64 via podman or docker.

Co-authored-by: Cursor <cursoragent@cursor.com>
- Add ARG TARGETARCH for arch-specific cache IDs in Containerfile
- Cache Redis compilation via --mount=type=cache
- Cache pip and npm downloads during bench init
- Update Makefile to pass --build-arg TARGETARCH to all build targets
- Implement S2I incremental caching in save-artifacts and assemble
- Fix GitHub workflow Podman build cache paths and key strategy
- Add .actrc for local act testing with catthehacker Ubuntu runner
- Expand .gitignore to exclude .secrets, .DS_Store, scratch files
@varun-krishnamurthy varun-krishnamurthy merged commit 03c1a2c into vyogotech:develop Jun 7, 2026
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants