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

nimsuggest doesn't run on a fresh choosenim Windows install #298

Closed
movpasd opened this issue Jun 13, 2022 · 3 comments
Closed

nimsuggest doesn't run on a fresh choosenim Windows install #298

movpasd opened this issue Jun 13, 2022 · 3 comments

Comments

@movpasd
Copy link

movpasd commented Jun 13, 2022

Came across this issue as it breaks the VSCode Nim extension on Windows (see issue)

Steps to reproduce:

  • Install choosenim using runme.bat on a Windows 10 machine
  • Once it's installed, try running nimsuggest on the command line
  • Observe the following error message

image

The DLL in question is not copied over from .choosenim/toolchains/mingw64/bin into .nimble/bin which might be the source of the issue. I think nimsuggest must assume there is a global MinGW installation in the PATH. This appears not to be an issue for the nim compiler nim, maybe because it doesn't directly access the DLLs but goes through choosenim's own MinGW proxies?

Current workaround: Adding .choosenim/toolchains/mingw64/bin manually to the global PATH works, but ofc this is not ideal since it kinda defeats the point of choosenim

Possible solution: Append ~/.choosenim/toolchains/mingw64/bin to the process PATH before each execution

@dom96
Copy link
Owner

dom96 commented Jun 13, 2022

Does this work when Nim is installed via finish.exe?

@movpasd
Copy link
Author

movpasd commented Jun 13, 2022

Doesn't seem to work either, so it might be a problem with nimsuggest -- but then I'm surprised the nim command works.

Then again it could be I didn't run my test properly, I did unset the relevant PATH variables for the test but perhaps I missed something (does Nim have any global config options or something that might have messed it up?)

@dom96
Copy link
Owner

dom96 commented Jun 13, 2022

nim doesn't need threads, nimsuggest could be different. Please report in Nim repo.

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

No branches or pull requests

2 participants