Skip to content

fix: autoinstall script curl failure#49

Merged
rgwood merged 1 commit intorgwood:masterfrom
mattrighetti:fix/auto-install-script
Jul 5, 2025
Merged

fix: autoinstall script curl failure#49
rgwood merged 1 commit intorgwood:masterfrom
mattrighetti:fix/auto-install-script

Conversation

@mattrighetti
Copy link
Copy Markdown
Contributor

@mattrighetti mattrighetti commented Jul 5, 2025

This will notify user when curl fails downloading the correct binary

Closes #48

This will notify user when curl fails downloading the correct binary

Closes rgwood#48
@mattrighetti mattrighetti force-pushed the fix/auto-install-script branch from bc37008 to da981d1 Compare July 5, 2025 14:25
@mattrighetti
Copy link
Copy Markdown
Contributor Author

mattrighetti commented Jul 5, 2025

Running this will return the following

$ ./install.sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     9    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
The requested file 'systemctl-tui-arm64-unknown-linux-musl.tar.gz' for version 'v0.4.0' and architecture 'arm64' may not exist.

@rgwood
Copy link
Copy Markdown
Owner

rgwood commented Jul 5, 2025

Thanks for the PR!

This improves the error reporting but I think we should fix the underlying issue too.

It looks like the install script is looking for systemctl-tui-arm64-unknown-linux-musl.tar.gz when it should be looking for systemctl-tui-aarch64-unknown-linux-musl.tar.gz. I'll fix that separately.

@rgwood rgwood merged commit a80bc06 into rgwood:master Jul 5, 2025
@mattrighetti
Copy link
Copy Markdown
Contributor Author

I usually have a target for that arch, if you're okay with it I can try and open another PR later with that target in the release CI?

@rgwood
Copy link
Copy Markdown
Owner

rgwood commented Jul 5, 2025

Already fixed in 88ad73b

CI was already publishing releases for aarch64, the install script was using the wrong name.

@mattrighetti
Copy link
Copy Markdown
Contributor Author

Aah I see - That's nice, thanks!

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.

auto install script does not work

2 participants