-
-
Notifications
You must be signed in to change notification settings - Fork 12
Properly setup DOTNET_ROOT env #26
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
Conversation
Closing as it currenly only sets |
@hensou can you please provide more context? I use |
Sure @uvlad7. First, I've been using nix so I'm might be lagging behind in My test case was done by:
However, let's say you want to install a tool like This also happens with I think it would be nice to have your PR, but at the same time I think it could lead to unexpected problems, as it won't be clear why all the other commands that have the Do you have any idea if it is possible to achieve something like I've tried with |
There is this issue for more context #54 |
I'm unable to look at this problem right now, it'll take a week or more. I'll take a look, but can't promise anything, because I'm not an expert, I just copied this from elixir plugin. But it indeed worked for globally installed tools in my environment. I use |
@hensou So I finally took a look and tried to test your example, and yes, I was able to reproduce the same behavior, but only when outside of a directory with dotnet activated, and it makes sense as I don't have it enabled globally (but it's possible in mise, don't see that option in asdf). So you need to run your tool from a directory with dotnet activated. I'd be better to customize installation directory to make it present only if dotnet is active, though, or (I guess) it can cause cryptic errors if another dotnet version is activated. Now it is installed in AFAIK there is no common approach to (global) install path configuration, same happens with Still a mess, will compare all used plugins later. |
I think I'll accept this MR and add a note on the I'll just add that to this PR and will merge it. Thanks a lot! |
No description provided.