langserver icon indicating copy to clipboard operation
langserver copied to clipboard

Update Nimble file checks to work with Atlas and Nimbus build system

Open elcritch opened this issue 1 year ago • 4 comments

Some projects don't / can't use Nimble. It'd be great to have support for projects using Atlas (or the older Nimbus build system). My previous tests with nimlangserver resulted in a bunch of nimble errors occuring.

Idealling this could be done by modifying to check for a nim.cfg which specifies a --noNimblePath switch. I'm not sure if Nim's dump config will specifies that or not.

Alternatively, a nimlangserver config to manually disable using Nimble would work as well.

elcritch avatar Jan 30 '24 15:01 elcritch

I also wonder if it's possible to replace the getProjectFileAutoGuess with running nim to dump the path configs?

Would running nim dump --verbosity:0 --dump.format:json dump.json | jq give all the path info necessary? It seems like it's just trying to find paths, but I haven't dug into getProjectFileAutoGuess to see exactly what it's getting.

elcritch avatar Jan 30 '24 15:01 elcritch

nimlangserver binaries don't need to be compiled anymore for common platforms . after each commit , automatically compiled fresh new binaries are available at https://github.com/nim-lang/langserver/releases

dreamflow avatar Jan 30 '24 18:01 dreamflow

nimlangserver binaries don't need to be compiled anymore for common platforms . after each commit , automatically compiled fresh new binaries are available at https://github.com/nim-lang/langserver/releases

Did you mean to post this elsewhere @dreamflow?

elcritch avatar Jan 31 '24 13:01 elcritch

Did you mean to post this elsewhere @dreamflow?

no , but seems like i misunderstood your first question( nimble , atlas , older build system , ... ) .

dreamflow avatar Jan 31 '24 14:01 dreamflow