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

Adjust the default breakpoint name to match cortex-m-rt pull #144 #64

Merged
merged 1 commit into from Jan 22, 2019

Conversation

sstelfox
Copy link
Contributor

@sstelfox sstelfox commented Jan 22, 2019

I noticed as I was going through the rust-embedded book that I was receiving the following warning when running the compiled code through GDB (via cargo run):

Function "UserHardFault" not defined.

It looks like this was renamed and the template just didn't get updated. This PR just updates the associated default break point to match the change.

Copy link
Member

@adamgreig adamgreig left a comment

Choose a reason for hiding this comment

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

Thanks!

bors r+

@bors
Copy link
Contributor

bors bot commented Jan 22, 2019

Configuration problem

bors.toml: not found

@adamgreig adamgreig merged commit 3aa5ea9 into rust-embedded:master Jan 22, 2019
bors bot added a commit to rust-embedded/book that referenced this pull request Jan 22, 2019
117: Update the references of UserHardFault, which was renamed to HardFault r=korken89 a=sstelfox

This updates the book to match the [renamed symbol](rust-embedded/cortex-m-rt#144) and my [associated PR](rust-embedded/cortex-m-quickstart#64) updating the quickstart repository.

Co-authored-by: Sam Stelfox <sstelfox@bedroomprogrammers.net>
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.

None yet

2 participants