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

fix: . in list-bin-paths was taken as is to form PATH #2077

Merged
merged 1 commit into from
May 12, 2024

Conversation

FranklinYinanDing
Copy link
Contributor

PATH generated from this was like /Users/user/.local/share/mise/installs/dotnet/8.0.204/.:/Users/user/.local/share/mise/installs/dotnet/8.0.204/shared:other-stuff... which lead to executable not found.

Now with the fix, the PATH becomes /Users/user/.local/share/mise/installs/dotnet/8.0.204/:/Users/user/.local/share/mise/installs/dotnet/8.0.204/shared:other-stuff..., and the executable can be found.

This should fix #2076.

@FranklinYinanDing FranklinYinanDing changed the title Fixed an issue where . in list-bin-paths was taken as is fix: . in list-bin-paths was taken as is to form PATH May 12, 2024
`PATH` generated from this was like `/Users/user/.local/share/mise/installs/dotnet/8.0.204/.:/Users/user/.local/share/mise/installs/dotnet/8.0.204/shared:other-stuff...` which lead to executable not found.
Now with the fix, the `PATH` becomes `/Users/user/.local/share/mise/installs/dotnet/8.0.204/:/Users/user/.local/share/mise/installs/dotnet/8.0.204/shared:other-stuff...`, and the executable can be found.
@jdx jdx enabled auto-merge (squash) May 12, 2024 22:25
@jdx jdx merged commit a88fee8 into jdx:main May 12, 2024
6 of 7 checks passed
@FranklinYinanDing FranklinYinanDing deleted the list-bin-paths branch May 12, 2024 22:28
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.

PATH is not assembled correctly when list-bin-paths has .
2 participants