celo-org / celo-bls-go

Go module for https://github.com/celo-org/bls-zexe/
Apache License 2.0
14 stars 8 forks source link

Remove per-arch go files #2

Closed gakonst closed 4 years ago

gakonst commented 4 years ago

Copied from: https://github.com/celo-org/bls-zexe/issues/123

Currently we have 1 .go file per architecture, due to the celo-blockchain toolchain being rigid for compiling with various CGO flags.

It'd be nice if we could get rid of these files, fix celo-blockchain and specify explicit instructions on how to cross-compile / link this library in various architectures.

An approach for this can be seen in https://github.com/celo-org/bls-zexe/pull/122 (which we partially reverted due to geth's inflexibility)

kobigurk commented 4 years ago

Still not sure it can be done. We have to tell the Go module where to look for the libraries using build tags.