search

status-im / nim-blscurve

Nim implementation of BLS signature scheme (Boneh-Lynn-Shacham) over Barreto-Lynn-Scott (BLS) curve BLS12-381
Apache License 2.0
26 stars 11 forks source link

add `{.push: raises: [].}` annotation #158

Closed etan-status closed 12 months ago

etan-status commented 12 months ago

In Nim 2.0, bls_batch_verifier seems to need {.push: raises: [].}.

/home/runner/work/nimbus-eth2/nimbus-eth2/nimcache/release/resttest/@m..@svendor@snim-bearssl@sbearssl@scsources@ssrc@scodec@senc16be.c.o /home/runner/work/nimbus-eth2/nimbus-eth2/vendor/nim-bearssl/bearssl/abi/../csources/src/codec/enc16be.c
/home/runner/work/nimbus-eth2/nimbus-eth2/vendor/nim-blscurve/blscurve/bls_batch_verifier.nim(246, 22) template/generic instantiation of `spawn` from here
/home/runner/work/nimbus-eth2/nimbus-eth2/vendor/nim-taskpools/taskpools/taskpools.nim(503, 70) template/generic instantiation of `toTask` from here
/home/runner/work/nimbus-eth2/nimbus-eth2/vendor/nim-taskpools/taskpools/tasks.nim(216, 65) template/generic instantiation from here
/home/runner/work/nimbus-eth2/nimbus-eth2/vendor/nim-taskpools/taskpools/tasks.nim(119, 20) Error: taskpool_reducePartialPairings(tp_4664066530, contexts_4664066532, start_4664066534,
                               stopEx_4664066536, fut_4664066538) can raise an unlisted exception: Exception
etan-status commented 12 months ago

Nimble currently installs https://github.com/status-im/nim-taskpools v0.0.4, but at least 20677645212a544007763cb99c9faca10db657d6 is required.

etan-status commented 12 months ago

https://github.com/status-im/nim-taskpools/pull/35