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

Compatiblity with Nim v1.4 #88

Closed
jfilby opened this issue Oct 17, 2020 · 6 comments
Closed

Compatiblity with Nim v1.4 #88

jfilby opened this issue Oct 17, 2020 · 6 comments

Comments

@jfilby
Copy link

jfilby commented Oct 17, 2020

Compilation fails after upgrading to Nim v1.4:

.nimble/pkgs/yaml-0.14.0/yaml/serialization.nim(407, 18) Error: attempting to call undeclared routine: 'getGMTime'

It looks like this function is no longer available in the times module.

@flyx
Copy link
Owner

flyx commented Oct 19, 2020

This has already been fixed in #84, which has not yet made it into a release. In the meantime, you can install yaml#head.

@telatin
Copy link

telatin commented Nov 16, 2020

I tried using yaml#head in a .nimble file, but got this during compilation with nim 1.2.4:

[...]yaml-#head/yaml/data.nim(131, 28) Error: undeclared identifier: 'FieldDefect'

It's not urgent and I can work with a pathed 0.14 meanwhile, just reporting in case others have the same problem

@flyx
Copy link
Owner

flyx commented Nov 17, 2020

@telatin FieldDefect is part of the system module since 1.4.0 (introduced here). HEAD doesn't support older compiler versions.

@telatin
Copy link

telatin commented Nov 17, 2020

Thanks for clarifying!
I installed nim 1.4.0, but now nimble install yaml#head does not work, did also nimble syntax change?

@flyx
Copy link
Owner

flyx commented Nov 17, 2020

Syntax seems to be nimble install yaml@#head now.

@jfilby
Copy link
Author

jfilby commented Feb 26, 2021

Works fine now (last tested with Nim 1.4.4).

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

3 participants