PATH entry of de-installed SDKs persists

reitzig / sdkman-for-fish

Adds support for SDKMAN! to fish

MIT License

307 stars 15 forks source link

PATH entry of de-installed SDKs persists #10

Open reitzig opened 6 years ago

reitzig commented 6 years ago

After uninstalling all versions of a candidate, fish greets us with an error:

set: Warning: $PATH entry "/home/raphael/.sdkman/candidates/crash/current/bin" is not valid (No such file or directory)
Welcome to fish, the friendly interactive shell

reitzig commented 6 years ago

SDKMAN! doesn't seem to remove the candidate folder itself:

$ tree .sdkman/candidates/crash/
.sdkman/candidates/crash/

0 directories, 0 files

reitzig commented 6 years ago

We no longer try to add removed candidates to PATH of new shells.

sdk uninstall does not change the PATH of the current shell; this is a bug in SDKMAN! and won't be worked around here. A test exists on branch issue-10; waiting for a fix upstream. Tracked as sdkman/sdkman-cli#662.