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.

Sign up for GitHub

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

Jump to bottom

added instructions to utilize riscv64-unknown-elf-gdb #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added instructions to utilize riscv64-unknown-elf-gdb #10

wants to merge 1 commit into from

Conversation

U007D
Copy link

@U007D U007D commented Jul 31, 2019 •
edited
Loading

Following the directions did not run the leds_blink example using Rust v1.36.0/Pop!_OS; added instructions to specify the RISC-V toolchain's gdb to fix the issue.

@Disasm
Copy link
Member

Disasm commented Aug 2, 2019

Thanks! However, I think that it's better to describe how to add it to PATH. If you change .cargo/config this change will make your project not portable (difficult to run on another machine) or you will have to avoid committing .cargo/config every time if you want to share your project.

almindor
almindor approved these changes Jul 10, 2021
View reviewed changes
Copy link
Collaborator

@almindor almindor left a comment

Choose a reason for hiding this comment

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

LGTM

@almindor
Copy link
Collaborator

@Disasm I think this should get merged, I just got back and this step actually got me fishing in memory too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@almindor almindor almindor approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@U007D @Disasm @almindor