dom96 / choosenim

Tool for easily installing and managing multiple versions of the Nim programming language.
BSD 3-Clause "New" or "Revised" License
680 stars 64 forks source link

improve error message when tempDir does not exist #299

Open pietroppeter opened 2 years ago

pietroppeter commented 2 years ago

tested with 0.8.4 recent release installing on windows and had a very minor issue, thanks for the release!

When installing and temp directory is non existing there is an error reported when copying DLLs. Error message could probably be improved by checking if temp dir exists and is accessible and alerting the user to fix it (in source getTempDir is used which does not do these checks: https://nim-lang.org/docs/os.html#getTempDir).

I am assuming that this issue is not windows specific.

Note that the AdditionalInfo which is logged when there is an error copying the DLLs does indeed report the path used so from there it is not very difficult to find the issue that the given path is non existing.