Skip to content
This repository has been archived by the owner on Feb 4, 2024. It is now read-only.

Nim devel breaks godot-nim #3

Closed
geekrelief opened this issue Dec 26, 2020 · 6 comments
Closed

Nim devel breaks godot-nim #3

geekrelief opened this issue Dec 26, 2020 · 6 comments
Labels
bug Something isn't working

Comments

@geekrelief
Copy link
Owner

Merge core files so we don't have circular dependencies between modules.
@geekrelief
Copy link
Owner Author

pragmagic/godot-nim#81 Need to produce a simpler test case to debug this.

@geekrelief
Copy link
Owner Author

geekrelief commented Dec 28, 2020 •
edited
Loading

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

@geekrelief
Copy link
Owner Author

nim-lang/Nim#16462 (comment)

@geekrelief geekrelief added the bug Something isn't working label Dec 30, 2020
@geekrelief
Copy link
Owner Author

3b963a81 is the latest working nim commit.

@geekrelief
Copy link
Owner Author

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
Copy link
Owner Author

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@geekrelief