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

[GDBStub] kconfig gdbstub runtime should be separate bool (IDFGH-10309) #11569

Conversation

chipweinberger
Copy link
Contributor

See: #11568 (comment)

This is a runtime feature, not a panic feature.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jun 1, 2023
@github-actions github-actions bot changed the title [GDBStub] kconfig gdbstub should be separate bool [GDBStub] kconfig gdbstub should be separate bool (IDFGH-10309) Jun 1, 2023
@chipweinberger chipweinberger force-pushed the user/chip/kconfig-gdbstub-separate-bool branch from e4962ba to adfe341 Compare June 1, 2023 09:07
@chipweinberger chipweinberger changed the title [GDBStub] kconfig gdbstub should be separate bool (IDFGH-10309) [GDBStub] kconfig gdbstub runtime should be separate bool (IDFGH-10309) Jun 1, 2023
@chipweinberger chipweinberger force-pushed the user/chip/kconfig-gdbstub-separate-bool branch from adfe341 to 3991e9d Compare June 1, 2023 09:09
@igrr igrr requested a review from Lapshin June 1, 2023 09:23
@chipweinberger chipweinberger force-pushed the user/chip/kconfig-gdbstub-separate-bool branch 3 times, most recently from fd8d786 to 2d64d12 Compare June 1, 2023 18:36
@chipweinberger
Copy link
Contributor Author

@ESP-Marius

@ESP-Marius
Copy link
Collaborator

@ESP-Marius

@Lapshin can you help take a look?

components/esp_system/Kconfig Outdated Show resolved Hide resolved
components/esp_system/Kconfig Outdated Show resolved Hide resolved
@chipweinberger
Copy link
Contributor Author

chipweinberger commented Jul 6, 2023

@Lapshin bump

@Lapshin
Copy link
Collaborator

Lapshin commented Jul 6, 2023

@chipweinberger , I still see open discussions for changes in this PR

@chipweinberger
Copy link
Contributor Author

oh I missed that. will update.

@chipweinberger chipweinberger force-pushed the user/chip/kconfig-gdbstub-separate-bool branch from 2d64d12 to ab75781 Compare July 8, 2023 21:23
@chipweinberger
Copy link
Contributor Author

@Lapshin updated.

@Lapshin
Copy link
Collaborator

Lapshin commented Jul 13, 2023

sha=ab757819aad60e0fb6a0e2d1b5c072cc71ab0520

@Lapshin Lapshin added the PR-Sync-Merge Pull request sync as merge commit label Jul 13, 2023
@chipweinberger
Copy link
Contributor Author

@Lapshin any updates? would be great if you could ping some people to review this.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new labels Aug 1, 2023
@chipweinberger
Copy link
Contributor Author

chipweinberger commented Aug 2, 2023

@Lapshin It looks like this did not get merged correctly 5f07ed8

It should be a separate bool, not part choice ESP_SYSTEM_PANIC

was this a mistake?

@Lapshin
Copy link
Collaborator

Lapshin commented Aug 3, 2023

@chipweinberger , it was separated into two commits, the second one will be synced with GitHub repo soon

@chipweinberger
Copy link
Contributor Author

merged a120ea8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants