Skip to content

Add cross-platform build target#11

Merged
davebarnwell merged 1 commit intomainfrom
codex/add-cross-platform-build-target
Mar 7, 2026
Merged

Add cross-platform build target#11
davebarnwell merged 1 commit intomainfrom
codex/add-cross-platform-build-target

Conversation

@davebarnwell
Copy link
Copy Markdown
Owner

Summary

  • add a make build-cross target for common macOS, Linux, and Windows binaries
  • ignore the generated bin/ directory so cross-build artifacts do not show up in git status
  • document the new build command in the README

Verification

  • make help
  • make build-cross

Copilot AI review requested due to automatic review settings March 7, 2026 19:47
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a cross-compilation build target to produce common macOS/Linux/Windows binaries and keeps generated artifacts out of git status, with documentation updates for the new command.

Changes:

  • Add make build-cross to build multiple GOOS/GOARCH binaries into bin/
  • Ignore the generated bin/ build output directory in .gitignore
  • Document make build-cross in the README

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
README.md Documents the new make build-cross command alongside existing build/install commands.
Makefile Introduces CROSS_PLATFORMS and a build-cross target that loops over OS/arch pairs and builds stripped binaries into bin/.
.gitignore Adds an ignore rule for the bin/ directory so build artifacts don’t appear as untracked files.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@davebarnwell davebarnwell merged commit e6b6b54 into main Mar 7, 2026
5 checks passed
@davebarnwell davebarnwell deleted the codex/add-cross-platform-build-target branch March 7, 2026 19:49
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