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

build: Add linux arm64 to build targets #61

Merged
merged 4 commits into from
May 17, 2022

Conversation

jimmidyson
Copy link
Contributor

See https://goreleaser.com/customization/build/ example config under key
builds[*].targets.- build: Add linux arm64 to build targets

Also use moved packer plugin check command.

Fixes #60

@jimmidyson
Copy link
Contributor Author

If possible, getting this in and cutting a release would be really appreciated 🙏

Previously errors would have been swallowed by goreleaser.
This was breaking in later versions with output of:

```
$ go run . describe | jq -r .api_version
panic: Please do not execute plugins directly. Packer will execute these for you.

goroutine 1 [running]:
main.main()
        /home/jimmi/go/src/github.com/YaleUniversity/packer-provisioner-goss/packer-provisioner-goss.go:106 +0x6a
exit status 2
```

The API version is unnecessary so this PR removes this to make builds
easier locally.
@jimmidyson
Copy link
Contributor Author

@fishnix Any chance of getting this merged and released?

@tenyo
Copy link

tenyo commented May 17, 2022

@jimmidyson Thanks for fixing up the goreleaser config! I'll merge this and create a new release.

@tenyo tenyo merged commit 0bdee3b into YaleUniversity:master May 17, 2022
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.

Please re-add arm64 binaries to release
2 participants