search

geekrelief / gdnim

godot-nim based bootstrapping framework supporting hot reloading
Other
55 stars 4 forks source link

Nim devel breaks godot-nim #3

Closed geekrelief closed 3 years ago

geekrelief commented 3 years ago

Merge core files so we don't have circular dependencies between modules.

geekrelief commented 3 years ago

https://github.com/pragmagic/godot-nim/issues/81 Need to produce a simpler test case to debug this.

geekrelief commented 3 years ago

Looks like the {.pure.} pragma is broken in nim devel.

geekrelief commented 3 years ago

https://github.com/nim-lang/Nim/issues/16462#issuecomment-751580422

geekrelief commented 3 years ago

3b963a81 is the latest working nim commit.

geekrelief commented 3 years ago

The workaround for this is to modify VariantType so that each enum entry is prefixed so {.pure.} can be removed. I tested this with nim devel branch recently, and it works except the latest commit is slower to compile than 3b963a81 for now.

geekrelief commented 3 years ago

The latest version of Nim devel compiles gdnim fine. Though it's twice as slow to compile a clean build compared wtih 3b963a81.