Skip to content
New issue

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

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

Already on GitHub? # to your account

riotbuild: update RISC-V Toolchain #248

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented Feb 12, 2024

Citing https://github.com/xpack-dev-tools/riscv-none-embed-gcc-xpack:

DEPRECATED! Replaced by xpack-dev-tools/riscv-none-elf-gcc-xpack

So let's do that.

Note: Depends on and includes:

Citing https://github.com/xpack-dev-tools/riscv-none-embed-gcc-xpack:

> DEPRECATED! Replaced by xpack-dev-tools/riscv-none-elf-gcc-xpack

So let's do that.
@kaspar030
Copy link
Contributor

So, if we merge this, it might fail some builds in RIOT master and possibly the release branch. @maribu if you're "on it" I'm ok with merging this now.

Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK. (merging when @maribu or others promise to watch RIOT master for fall-out)

@maribu
Copy link
Member Author

maribu commented Feb 12, 2024

I think I can do some local testing first, e.g. all apps for a single RISC-V board should be doable on my machine.

@maribu
Copy link
Member Author

maribu commented Feb 12, 2024

Yeah, there are some issue uncovered by the toolchain update. I think I can prepare a PR soonish that would fix that.

@maribu
Copy link
Member Author

maribu commented Feb 12, 2024

This should be updated in lock-step with RIOT-OS/RIOT#20380

@kaspar030
Copy link
Contributor

Let's go.

@kaspar030 kaspar030 added this pull request to the merge queue Feb 13, 2024
@kaspar030
Copy link
Contributor

So once this shows "merged", the master workflow needs to complete building & publishing the containers, then ~15min more to deploy, then we need to merge RIOT-OS/RIOT#20380.

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

Successfully merging this pull request may close these issues.

2 participants