Must use Nim 0.18.0, an official "release", not "devel"

jangko / msgpack4nim

MessagePack serializer/deserializer implementation for Nim / msgpack.org[Nim]

http://msgpack.org/

119 stars 21 forks source link

Must use Nim 0.18.0, an official "release", not "devel" #23

Closed pb-cdunn closed 5 years ago

pb-cdunn commented 5 years ago

I have a problem. There is a bug in Nim's memory allocation on devel, so I need to use 0.18.0 until this is solved. I think I had to switch to devel because of something in your new code. We need to revert msgpack4nim to a version that works with a released version of Nim, namely 0.18.0. What SHA1 do you think would work?

https://forum.nim-lang.org/t/4088

jangko commented 5 years ago

Hmm, git bisect? You can try any commit before nim 0.18.0 release date

pb-cdunn commented 5 years ago

I think made various bug-fixes for me. I'll try things before March 1 and cross my fingers.

pb-cdunn commented 5 years ago

Nvm. They fixed devel.