Skip to content

Pyghidra - #6

Merged
edmcman merged 5 commits into
devfrom
pyghidra
Dec 18, 2025
Merged

Pyghidra#6
edmcman merged 5 commits into
devfrom
pyghidra

Conversation

@edmcman

@edmcman edmcman commented Dec 18, 2025

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 18, 2025 15:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the Ghidra headless analysis script from using analyzeHeadless to PyGhidra's pyghidraRun, enabling better Python integration with Ghidra. The changes adapt the code to work with PyGhidra's environment and JPype's Java interoperability layer.

  • Replaces analyzeHeadless with pyghidraRun for script execution
  • Updates Java interop to use JPype decorators instead of direct inheritance
  • Modifies PrintWriter usage from print() to write() method
  • Adds PyGhidra virtual environment setup in CI workflow

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
export.bash Switches from analyzeHeadless to pyghidraRun with -H flag for headless mode
cpp_exporter_headless.py Adds PyGhidra runtime annotation, JPype decorators, Python sort instead of Collections.sort, PrintWriter API corrections, and improved error handling
.github/workflows/test.yml Creates PyGhidra virtual environment during CI setup

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cpp_exporter_headless.py Outdated
Comment thread cpp_exporter_headless.py Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@edmcman
edmcman merged commit b844dae into dev Dec 18, 2025
4 checks passed
@edmcman
edmcman deleted the pyghidra branch December 18, 2025 16:19
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.

3 participants