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

Fix risc-v by updating dependencies #94

Closed
wants to merge 1 commit into from
Closed

Conversation

fgaz
Copy link
Contributor

@fgaz fgaz commented Nov 5, 2023

In particular we want to update all instances of ring to 0.17

@fgaz
Copy link
Contributor Author

fgaz commented Nov 6, 2023

Looks like I ran into SeaQL/sea-orm#1832

@fgaz fgaz force-pushed the fix-riscv branch 2 times, most recently from 3a1a693 to 0c3d3fb Compare November 6, 2023 10:30
@fgaz
Copy link
Contributor Author

fgaz commented Nov 6, 2023

I applied the workaround in a separate commit so it's easy to revert.

It doesn't seem to do anything though

In particular we want to update all instances of ring to 0.17
@zhaofengli
Copy link
Owner

Thank you! I have applied the updates with changes.

@zhaofengli zhaofengli closed this Dec 18, 2023
@fgaz
Copy link
Contributor Author

fgaz commented Dec 18, 2023

Excellent, thanks! Could you also add riscv64-linux to the flake platforms?

@zhaofengli
Copy link
Owner

Done in 75c0482.

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

Successfully merging this pull request may close these issues.

2 participants