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

Debug: use proper hook for handle_exit in flipperapps #3842

Merged
merged 4 commits into from
Sep 5, 2024
Merged

Conversation

skotopes
Copy link
Member

@skotopes skotopes commented Aug 15, 2024

What's new

  • Debug: use proper hook for handle_exit in flipperapps

Verification

  • Use ./fbt blackmagic, disconnect shouldn't cause memory write error anymore, also starting app after disconnect shouldn't crash flipper

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

Copy link

github-actions bot commented Aug 15, 2024

Compiled f7 firmware for commit 9fd26830:

@hedger hedger added Build System & Scripts fbt, scripts and toolchain-related Bug labels Aug 15, 2024
Copy link

@fernandezmajr0 fernandezmajr0 left a comment

Choose a reason for hiding this comment

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

Great!

@skotopes skotopes merged commit 62bbf40 into dev Sep 5, 2024
11 checks passed
@skotopes skotopes deleted the aku/debugger_fix branch September 5, 2024 18:55
ofabel pushed a commit to ofabel/flipperzero-firmware that referenced this pull request Sep 26, 2024
…#3842)

* Debug: use proper hook for handle_exit in flipperapps
* fbt: flash firmware for `blackmagic` target

Co-authored-by: hedger <[email protected]>
Co-authored-by: Georgii Surkov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Build System & Scripts fbt, scripts and toolchain-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants