GC safety of Nimbus-related projects

[stefantalpalaru]

Projects we're able to use with --threads:on (guaranteed by enabling it in their test suite):

• cheatfate/nimcrypto - cheatfate/nimcrypto@eb151ac
• jangko/snappy - add --threads:on to tests nim-snappy#2
• status-im/nim-asyncdispatch2 - add "--threads:on" to tests nim-chronos#19
• status-im/nim-bncurve - enable "--threads:on" for tests nim-bncurve#4
• status-im/nim-byteutils - add "--threads:on" to tests nim-byteutils#5
• status-im/nim-chronicles - a test with threading enabled nim-chronicles#42
• status-im/nim-eth-bloom - add "--threads:on" to tests nim-eth-bloom#6
• status-im/nim-eth-common - add "--threads:on" to tests nim-eth-common#25
• status-im/nim-eth-keyfile - add "--threads:on" to tests nim-eth-keyfile#1
• status-im/nim-eth-keys - add "--threads:on" to tests nim-eth-keys#17
• status-im/nim-eth-p2p - Make the library gcsafe / ok to use with --threads:on nim-eth-p2p#60
• status-im/nim-eth-trie - add "--threads:on" to tests nim-eth-trie#32
• status-im/nim-faststreams - add "--threads:on" to tests nim-faststreams#1
• status-im/nim-http-utils - add "--threads:on" to tests nim-http-utils#3
• status-im/nim-json-rpc - add "--threads:on" to tests nim-json-rpc#47
• status-im/nim-json-serialization - add "--threads:on" to tests nim-json-serialization#1
• status-im/nim-ranges - add "--threads:on" to tests nim-ranges#35
• status-im/nim-rlp - add "--threads:on" to tests nim-rlp#23
• status-im/nim-rocksdb - tests refactoring nim-rocksdb#17
• status-im/nim-secp256k1 - add "--threads:on" to tests nim-secp256k1#14
• status-im/nim-serialization - no test suite
• status-im/nim-std-shims - no test suite
• status-im/nim-stint - Better bitsof implementation nim-stint#74
• zah/nim-package-visible-types - no test suite
• status-im/nimbus - add "--threads:on" to tests and main binary #213 GC-safety for Nimbus nim-rlp#24 GC-safety for Nimbus nim-eth-trie#34 GC-safety for Nimbus nim-eth-common#26 GC-safety for Nimbus nim-eth-p2p#63 disable Chronicles colors on Windows #215

[jangko]

status-im/nim-snappy#2

[stefantalpalaru]

Thanks. Added to the list.

[stefantalpalaru]

There's a stdlib bug on Windows that prevents threading there: nim-lang/Nim#8294 (comment)

[stefantalpalaru]

Goal achieved, but we might want to keep the issue open to track those 3 projects with no test suites.