Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Goreleaser cross compile #74

Closed

Conversation

mightyshazam
Copy link
Contributor

What changed?
I noticed that the goreleaser did nothing, most likely because of the annoyance involved in cross compilation.

Why?
I did this so release binaries would be available for multiple platforms

How did you test it?
I created releases in my own repository. I tested some, but not all, of the binaries on various machines I have. I can confirm the darwin arm64, windows amd64, and linux amd64 binaries work.

Potential risks
There should be little risk in this because it doesn't change the code, but the binaries could be broken.

Is hotfix candidate?
No

@mightyshazam mightyshazam changed the title Goreleaser cross compile only Goreleaser cross compile May 18, 2022
@jlegrone
Copy link
Collaborator

The config for cross-compilation may be simpler after we update to the latest release of Temporal server, since the sqlite driver no longer requires CGO. We can keep this PR open until we've updated, then reasses what changes are needed to the goreleaser config.

@jlegrone
Copy link
Collaborator

#89 is merged, and we no longer require CGO in order to build! I think we should be able to stick with the goreleaser defaults now, so I'll close this.

@jlegrone jlegrone closed this Jul 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants