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

Improvement: Verify that tar is available #1267

Closed
gunnarmorling opened this issue Jan 9, 2024 · 2 comments · Fixed by sdkman/sdkman-hooks#62
Closed

Improvement: Verify that tar is available #1267

gunnarmorling opened this issue Jan 9, 2024 · 2 comments · Fixed by sdkman/sdkman-hooks#62

Comments

@gunnarmorling
Copy link

I just tried to use SDKMan on a fresh Linux install, and it would keep failing on me with this message:

Download has failed, aborting!

Enabling debug logging didn't provide any further insight about the cause. Only when inspecting the post-install hook, I realized that the issue was that the tar command wasn't available on this environment yet.

@marc0der
Copy link
Member

Ah, good catch! That's because tar is only used in the installation hooks, not in the CLI. I'd be happy to have it added it to the install script as a pre-requisite. If you feel up to it, you could even contribute that change yourself over here:
https://github.com/sdkman/sdkman-hooks/blob/master/app/views/includes/sanity.scala.txt

@gunnarmorling
Copy link
Author

Sure thing: sdkman/sdkman-hooks#62

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants