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

Add support for more architectures and update install script #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DewaldDeJager
Copy link

Updates made to release pipeline:

  • Save version and repo name in variables so they can easily be changed
  • Lowercase the repo name so that it works for users with uppercase letters in their usernames
  • Add support for Linux/x86, Linux/ARMv7, macOS/ARM64, Windows/x86
  • Update the names of the executables to include the version and architecture and the .exe suffix for Windows
  • Use a glob pattern for selecting executable files to publish on the release
  • Simplify the Docker build and release steps

Updates made to install script:

  • Update the OS names so they match the executable names published on the release
  • Check the OS architecture so the correct binary can be downloaded
  • Download binaries to a temporary folder and clean up when the script exits
  • Use cURL to download the binary instead of wget so that only one tool is needed

Note: If you want to test the install script, be sure to update the URL with my username since the binaries in this repo won't match the expected naming convention yet.

@DewaldDeJager
Copy link
Author

You can see an example release with all the executables here: https://github.com/DewaldDeJager/netlify-dynamic-dns/releases/tag/v3.0.2
An example package with the Docker image here: https://github.com/DewaldDeJager/netlify-dynamic-dns/pkgs/container/netlify-dynamic-dns
An example run of the release workflow here: https://github.com/DewaldDeJager/netlify-dynamic-dns/actions/runs/5354011117

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

1 participant