Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automate cross-compilation process using goreleaser #18

Merged

Conversation

jossemarGT
Copy link
Contributor

@jossemarGT jossemarGT commented Oct 20, 2021

Hey @yusukebe.

As we discussed on #11, I have tackled the following enhancement:

  • Statically compile revealgo for Windows, MacOS and Linux operative systems (at least for x64 architectures)

We can continue doing the usual go build if we need to, but when the time comes to cross-compile multiple OS's and architectures gorelease is a better solution.

Please take a look, and let's discuss any rough edge.

PS: I do not think having submodules is as bad as we initially thought. At the end, we would need to move that logic into another tool, for example: a shell script that reads a file which does several curl commands.

@jossemarGT
Copy link
Contributor Author

Oh yeah, I forgot to mention that gorelease already have built-in functionality for homebrew formula creation. But I rather prefer to explore those options in a separate PR, for order and peace of mind.

@yusukebe
Copy link
Owner

Hey @jossemarGT!

Using goreleaser is a very good solution to make multiple binaries I think so too. I succeeded to build the binaries with goreleaser on my local machine. It works fine.

If we ship revealgo as binaries do not use go get or go install commands, we don't need to include reveal.js and socket.io in this repository. It is no problem to use submobules because git submodule update command will be called only when we release revealgo.

gorelease already have built-in functionality for homebrew formula creation

Year! That's good.

I'll merge this PR. Thank you!

@yusukebe yusukebe merged commit b0b7cbd into yusukebe:master Oct 21, 2021
@jossemarGT jossemarGT deleted the feature/build-multiple-binaries branch October 22, 2021 05:34
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.

None yet

2 participants