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

[sw] remove unused variable RISCV_TOOLCHAIN #832

Merged
merged 2 commits into from
Feb 25, 2024

Conversation

umarcor
Copy link
Collaborator

@umarcor umarcor commented Feb 25, 2024

While trying to build and install the bootloader on MSYS2 following my notes from #55, I found the following typo: s/RISCV_TOLLCHAIN/RISCV_TOOLCHAIN/. So I fixed it.

However, then I found out that it seems not to be used anymore. Now RISCV_PREFIX is used: RISCV_PREFIX=riscv64-unknown-elf- MARCH=rv32i make -C sw/bootloader/ install. Therefore, I removed it. Yet, I don't know if that's correct.

@stnolting wdyt?

@stnolting
Copy link
Owner

Looks like a leftover from the early days...

@stnolting stnolting added the SW Software-related label Feb 25, 2024
@stnolting stnolting merged commit ec62004 into stnolting:main Feb 25, 2024
7 checks passed
@umarcor umarcor deleted the umarcor/make-unused-variable branch February 25, 2024 17:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
SW Software-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants