search

reitzig / sdkman-for-fish

Adds support for SDKMAN! to fish
MIT License
280 stars 13 forks source link

Respect globally set SDKMAN_DIR #52

Closed reitzig closed 2 months ago

reitzig commented 2 months ago

Re #46:

I set SDKMAN_DIR globally, through environment.d with system. It should not be over-written by sdk.fish.

In my opinion, if both are defined, __sdkman_custom_dir should be preferred.

Labelling this as bug since, as @xtexChooser mentions, the original sdkman-init.sh does blindly trust and use SDKMAN_DIR. I think that's probably a bad idea, but well.