Skip to content

Bump ethnum-rs and dtc dependencies - #333

Merged
jwnrt merged 2 commits into
lowRISC:ot-10.2.0from
jwnrt:bump-ethnum-rs
Jul 30, 2026
Merged

Bump ethnum-rs and dtc dependencies#333
jwnrt merged 2 commits into
lowRISC:ot-10.2.0from
jwnrt:bump-ethnum-rs

Conversation

@jwnrt

@jwnrt jwnrt commented Jul 30, 2026

Copy link
Copy Markdown

Both dependencies have errors which fail to compile using recent compilers. We build QEMU with host tools in OpenTitan (due to how rules_foreign_cc works) and the build recently started failing due to the ethnum-rs Rust error.

When I fixed the Rust error locally, my C host compiler rejected libdtc due to another problem (Ubuntu 26.04). See the commit messages for details.

jwnrt added 2 commits July 30, 2026 12:11
Required to include nlordell/ethnum-rs#58 which
is needed for more recent Rust compilers which reject this failure.

Signed-off-by: James Wainwright <james.wainwright@lowrisc.org>
Includes commit 9a1c801a1a3c102bf95c5339c9e985b26b823a21 [0] which is
needed for more recent compilers which reject the `const` qualifier
being dropped by default.

[0]: https://gitlab.com/qemu-project/dtc/-/commit/9a1c801a1a3c102bf95c5339c9e985b26b823a21

Signed-off-by: James Wainwright <james.wainwright@lowrisc.org>
@jwnrt
jwnrt requested a review from AlexJones0 July 30, 2026 11:13

@AlexJones0 AlexJones0 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me, lets wait for CI to finish and then merge. There may be some breakage from upstream OpenTitan changes but as long as the build remains sensibly working this seems good.

@jwnrt

jwnrt commented Jul 30, 2026

Copy link
Copy Markdown
Author

Thanks Alex, the CI failures do look like upstream changes and not build failures, so I'll merge and cut a release

@jwnrt
jwnrt merged commit d765528 into lowRISC:ot-10.2.0 Jul 30, 2026
7 of 10 checks passed
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