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

feat: Extend release targets and add install.bash script #288

Merged
merged 16 commits into from
Feb 21, 2025

Conversation

nieomylnieja
Copy link
Contributor

@nieomylnieja nieomylnieja commented Feb 13, 2025

Motivation

Currently we're only building binaries for amd64 architecture, it would be good to support arm64 as well.
In addition, many tools often provide an install script which automates the process of installing/upgrading these tools.

Testing

Run install script:

./install.bash -v v0.11.0-rc1 -d <YOUR_PATH>

You can also test the commands for installment from README, for instance:

curl -fsSL -o install.bash https://raw.githubusercontent.com/nobl9/sloctl/extend-release-targets-and-add-install-script/install.bash

Release Notes

Sloctl pre-built binaries are now also built for arm64 architecture.
In addition there's now a now method of installing sloctl, via an install script.
For more details refer to the README.

Breaking Changes

Sloctl released binaries are no longer of a form sloctl-${OS}-${VERSION} (e.g. sloctl-linux-0.10.1), instead they now follow this pattern: _sloctl-${VERSION}-${OS}-${ARCH} (e.g. sloctl-0.11.0-linux-adm64).
In addition, the checksum for each release is changing it's name from _sloctl-${VERSION}-SHA256SUMS to sloctl-${VERSION}.sha256.

Copy link
Member

@skrolikiewicz skrolikiewicz left a comment

Choose a reason for hiding this comment

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

Tested on the M1 and couldn't find any issues.

@nieomylnieja nieomylnieja force-pushed the extend-release-targets-and-add-install-script branch from 13f661f to e4e44ac Compare February 21, 2025 10:48
@nieomylnieja nieomylnieja force-pushed the extend-release-targets-and-add-install-script branch from e4e44ac to 47280e1 Compare February 21, 2025 10:49
@nieomylnieja
Copy link
Contributor Author

Tested on Windows 11 23h2 with Cygwin:
image

@nieomylnieja nieomylnieja merged commit af51534 into main Feb 21, 2025
5 checks passed
@nieomylnieja nieomylnieja deleted the extend-release-targets-and-add-install-script branch February 21, 2025 13:44
nieomylnieja added a commit that referenced this pull request Feb 24, 2025
Changes introduced in #288 broke
this glob pattern:

```ruby
bin.install Dir['sloctl-*-0.11.0'].first()  => "sloctl"
```

Since this is not a regular expression but a glob pattern, the newly
added arch suffix (and the moved OS name) failed to match.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants