Skip to content

Remove GDB/MI debugger host support - #39

Merged
davidly merged 1 commit into
davidly:mainfrom
gloveboxes:fix/remove-gdb-mi-debugger-host
Aug 25, 2026
Merged

davidly merged 1 commit into
davidly:mainfrom
gloveboxes:fix/remove-gdb-mi-debugger-host

Conversation

@gloveboxes

Copy link
Copy Markdown
Contributor

Summary

ntvcm is no longer used as a debugger host. Remove the embedded GDB Machine Interface implementation and all app-facing integration for that role.

Changes

  • remove the GDB/MI command parser, protocol responses, metadata loader, expression evaluator, stack-frame handling, and variable-object support
  • remove --interpreter=mi startup and help-text support
  • remove CPU breakpoint, pause, continue, and single-step state and APIs
  • simplify console input/output and the emulation loop to the normal execution path
  • delete the obsolete tests/debugger_mi.py regression suite
  • retain unrelated native compiler symbols, debug/release build flavors, execution tracing, and profiling

Validation

  • compiled and linked ntvcm.cxx and x80.cxx successfully with G++
  • ran 8080PRE.COM successfully under a hard timeout
  • verified workspace diagnostics are clean
  • searched source, tests, scripts, README files, workflows, hidden configuration, linked symbols, binary strings, and public help output for residual GDB/MI debugger-host references
  • verified patch whitespace with CRLF-aware Git settings

ntvcm is no longer used as a debugger host, so remove the embedded GDB Machine Interface server and its command-line mode.

Remove the CPU breakpoint, pause, continue, and single-step hooks; simplify console I/O and the emulation loop to the normal execution path; and delete the obsolete MI regression suite.
@gloveboxes

Copy link
Copy Markdown
Contributor Author

Closing so the branch owner can open the upstream pull request directly.

@gloveboxes gloveboxes closed this Aug 25, 2026
@gloveboxes gloveboxes reopened this Aug 25, 2026
@davidly
davidly merged commit 5c85a57 into davidly:main Aug 25, 2026
2 checks passed
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.

2 participants