Skip to content

Add macOS (darwin) binaries to release distribution - #1255

Merged
asvoboda merged 1 commit into
palantir:developfrom
bells17:add-darwin-release-binaries
Jun 8, 2026
Merged

asvoboda merged 1 commit into
palantir:developfrom
bells17:add-darwin-release-binaries

Conversation

@bells17

@bells17 bells17 commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Before this PR

The release distribution only ships Linux binaries (linux-amd64, linux-arm64).
Each release asset (policy-bot-<version>.tgz) contains:

policy-bot-<version>/
  bin/
    linux-amd64/policy-bot
    linux-arm64/policy-bot

Because of this, macOS users who want to run helper commands such as
policy-bot validate locally have no prebuilt binary to download. It can still
be installed via go install, but that requires a Go toolchain and a local
build. Having a prebuilt binary available for download is much more convenient.

After this PR

==COMMIT_MSG==
Add darwin (macOS) binaries to the release distribution.

darwin/amd64 and darwin/arm64 are added to the os-archs list in
godel/config/dist-plugin.yml, so ./godelw dist now also produces macOS
binaries and they are bundled into the same release .tgz:

policy-bot-<version>/
  bin/
    linux-amd64/policy-bot
    linux-arm64/policy-bot
    darwin-amd64/policy-bot   # new (Intel Mac)
    darwin-arm64/policy-bot   # new (Apple Silicon)

macOS users can now download the release archive and use the prebuilt binary
(e.g. for policy-bot validate) without needing a Go toolchain.
==COMMIT_MSG==

Possible downsides?

  • None for existing users. The release archive layout is unchanged; only two
    additional bin/<os>-<arch>/ directories are added.
  • The release archive becomes slightly larger due to the two extra binaries.
  • The binaries are built with CGO_ENABLED=0 (matching the existing Linux
    builds), so they cross-compile cleanly and require no new build dependencies.
    The Docker image is unaffected since its Dockerfile explicitly references
    bin/linux-amd64/policy-bot.

@palantirtech

Copy link
Copy Markdown
Member

Thanks for your interest in palantir/policy-bot, @bells17! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@changelog-app

changelog-app Bot commented Jun 6, 2026 •

Copy link
Copy Markdown

Generate changelog in changelog/@unreleased

Type (Select exactly one)

  • Feature (Adding new functionality)
  • Improvement (Improving existing functionality)
  • Fix (Fixing an issue with existing functionality)
  • Break (Creating a new major version by breaking public APIs)
  • Deprecation (Removing functionality in a non-breaking way)
  • Migration (Automatically moving data/functionality to a new system)

Description

Add darwin (macOS) binaries to the release distribution.

Check the box to generate changelog(s)

  • Generate changelog entry

@changelog-app

changelog-app Bot commented Jun 8, 2026

Copy link
Copy Markdown

✅ Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.


📋Changelog Preview

💡 Improvements

  • Add darwin (macOS) binaries to the release distribution. (#1255)

@asvoboda asvoboda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@asvoboda
asvoboda merged commit bdd4c7d into palantir:develop Jun 8, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants