Bug:"Unknown target CPU" & "Must define either OPENSSL_32_BIT or OPENSSL_64_BIT"

[gcp-development]

Hello,

My target is "riscv32imac-esp-espidf" and I am using the riscv64-unknown-elf-gcc to compile the ring library.

But I am getting the error below :

warning: [email protected]: In file included from include/ring-core/base.h:74,
warning: [email protected]:                  from include/ring-core/mem.h:60,
warning: [email protected]:                  from crypto/curve25519/curve25519.c:22:
warning: [email protected]: include/ring-core/target.h:63:2: error: #error "Unknown target CPU"
warning: [email protected]:    63 | #error "Unknown target CPU"
warning: [email protected]:       |  ^~~~~
warning: [email protected]: In file included from crypto/curve25519/internal.h:20,
warning: [email protected]:                  from crypto/curve25519/curve25519.c:24:
warning: [email protected]: crypto/curve25519/../internal.h:210:2: error: #error "Must define either OPENSSL_32_BIT or OPENSSL_64_BIT"
warning: [email protected]:   210 | #error "Must define either OPENSSL_32_BIT or OPENSSL_64_BIT"

I am assuming that the current ring crate does not have the target "riscv32imac-esp-espidf".

There is any plan in the future to add this target or any riscv32 target ?

Thanks

[briansmith]

I am happy to accept a PR that adds riscv32 support. I suggest we start with riscv32gc-unknown-linux-gnu since we can just copy-paste-edit the changes from PR #1627, at least for for target.h, mk/cargo.sh, mk/install-build-tools.sh.

As for espidf, that probably needs a couple of additional tweaks for getrandom. If there's a way to use qemu-user to target espidf then that would help.

[briansmith]

Closing as a duplicate of #1765.

[gcp-development]

Fair enough.

Nevertheless, the "riscv32imac-esp-espidf" is more similar the way the "armv7-unknown-linux-musleabihf" (With the emu setup for riscv32) was setup in your code than the current "riscv64gc-unknown-linux-gnu".

Thanks.