Skip to content

Conversation

@ethauvin
Copy link

@ethauvin ethauvin commented Sep 13, 2024

Otherwise, the script will fail when rm is aliased to rm -i, etc.

@yusiwen
Copy link

yusiwen commented Oct 12, 2024

I've also mapped this alias for 'rm -iv' to 'rm'. And besides, I'm using sharkdp/bat to substitute vanilla 'cat' command via an alias. These changes make 'sdk list java' command failed at exit.

Copy link

@yusiwen yusiwen left a comment

Choose a reason for hiding this comment

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

LGTM

@flurdy
Copy link

flurdy commented May 12, 2025

LGTM. Took me a while to track down this annoying prompt. Not sure why, it is quite obvious now.

@reitzig
Copy link
Owner

reitzig commented Jun 10, 2025

FWIW: I'm happy to review and merge a PR that

Adding command seems a feasible fix, but we may also want to replace some external dependencies with a little fish scripting (looking at you, awk!).

This is not that, unfortunately, so I'll close it for now. You can track the overarching issue at #23.

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.

4 participants