Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support risc-v #1444

Open
come-maiz opened this issue Sep 13, 2024 · 10 comments
Open

Support risc-v #1444

come-maiz opened this issue Sep 13, 2024 · 10 comments

Comments

@come-maiz
Copy link
Contributor

RISC-V is a good start to get Ethereum nodes running in free hardware, which is a necessary step for a local-first decentralized economy. Lightclients are another necessary step, so these two worlds together are very powerful.

Please support the risc-v architecture in this project.

come-maiz added a commit to come-maiz/trin that referenced this issue Sep 13, 2024
This is a necessary step to update ring, which is necessary to build in risc-v.

Part of ethereum#1444.
come-maiz added a commit to come-maiz/trin that referenced this issue Sep 13, 2024
This is a necessary step to update ring, which is necessary to build in risc-v.

Part of ethereum#1444.
come-maiz added a commit to come-maiz/trin that referenced this issue Sep 13, 2024
This is a necessary step to update ring, which is necessary to build in risc-v.

Part of ethereum#1444.
@come-maiz
Copy link
Contributor Author

Next we have to update rustls, which is problematic because surf keeps it pinned to an old version and it is unmaintained:
http-rs/surf#352 (comment)

KolbyML pushed a commit that referenced this issue Sep 13, 2024
This is a necessary step to update ring, which is necessary to build in risc-v.

Part of #1444.
@KolbyML
Copy link
Member

KolbyML commented Sep 13, 2024

Next we have to update rustls, which is problematic because surf keeps it pinned to an old version and it is unmaintained: http-rs/surf#352 (comment)

Yeah I guess we should remove it as a dependency and switch to something else in that case

@KolbyML
Copy link
Member

KolbyML commented Sep 13, 2024

Next we have to update rustls, which is problematic because surf keeps it pinned to an old version and it is unmaintained: http-rs/surf#352 (comment)

#1446

I opened a PR to move us to reqwest instead of surf, so that should unblock that step

@come-maiz
Copy link
Contributor Author

Nice! Thank you @KolbyML. It works.

Next is ethers-providers:
gakonst/ethers-rs#2786

@KolbyML
Copy link
Member

KolbyML commented Sep 13, 2024

Nice! Thank you @KolbyML. It works.

Next is ethers-providers: gakonst/ethers-rs#2786

We want to remove etheres-rs in Trin and replace it with alloy

@come-maiz
Copy link
Contributor Author

That sounds perfect.

@KolbyML
Copy link
Member

KolbyML commented Sep 16, 2024

@come-maiz This PR #1456 removes our dependency on ethers-rs, are you able to compile on risc-v yet, or are there still some dependency issues?

@come-maiz
Copy link
Contributor Author

thanks @KolbyML!

Compilation almost finishes now. It's getting stuck in librocksdb-sys, I'll see what I can do on my machine to get it to compile.

@KolbyML
Copy link
Member

KolbyML commented Sep 20, 2024

thanks @KolbyML!

Compilation almost finishes now. It's getting stuck in librocksdb-sys, I'll see what I can do on my machine to get it to compile.

@come-maiz try installing clang and tell me if it works sudo apt install clang

@come-maiz
Copy link
Contributor Author

It was installed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants