nim-lang / RFCs

A repository for your Nim proposals.
136 stars 26 forks source link

Unbundle DrNim #431

Closed juancarlospaco closed 1 year ago

juancarlospaco commented 2 years ago

"Do not bite off more than you can chew" kinda situation with DrNim, I think it should be unbundled from the compiler installers, because is not very useful in its current state.

https://github.com/nim-lang/Nim/tree/devel/drnim

Araq commented 2 years ago

I'm not against it but historically projects of mine that depend on the Nim compiler API bitrot quickly unless they are part of Nim's repository.

juancarlospaco commented 2 years ago

I'm not against it

Ok, then it can move forwards (?). 🤔

disruptek commented 2 years ago

Maybe you should find a new host for your rotten code.

PMunch commented 2 years ago

I'm not against it but historically projects of mine that depend on the Nim compiler API bitrot quickly unless they are part of Nim's repository.

Maybe this should be a sign that the Nim compiler API should be a bit better defined/documented. NimLSP seems to work fine (well, it doesn't crash more the further along the version of Nim you go), but I suspect that is only because it wraps Nimsuggest which does all the actual interfacing. Maybe moving some of your tools into their own separate repositories and maintain them there with the compiler changes would benefit other users of the same API.

juancarlospaco commented 2 years ago

Rotten code in your compiler distribution is not a good Marketing for new companies to migrate to Nim.

Araq commented 2 years ago

As if they would know, but fine, migrate it to nim-lang/drnim or araq/drnim then.

juancarlospaco commented 2 years ago

@Araq Companies do check issues, pull requests, documentations, licenses, github stars, github repo stats, etc when evaluating new programming languages to use.

I have seen it multiple times, thats the reason I try to remove the Telemetry from choosenim, some open source companies do not like bundled Telemetry, not even disabled Telemetry, simply because they can choose other languages that dont need that (Nim really dont need it neither), it is a random example, but just another thing of Nim Marketing that can be improved...

juancarlospaco commented 2 years ago

Roadmap for future is here https://github.com/nim-lang/RFCs/issues/437#issue-1058638395 No mention of DrNim developments, so I think is more reason to unbundle it, how can nim-lang/drnim be created?.

juancarlospaco commented 2 years ago

So this RFC has no concrete response since 2021, is a ton of code to maintain that nobody is using in prod everyday, maybe 2.0 is a great opportunity to Deprecate the old dead code?.

See also https://github.com/nim-lang/RFCs/issues/476

Araq commented 1 year ago

It's just a bad idea. The compiler API keeps changing and at the same time ensuring DrNim keeps working is close to zero work.