Page MenuHomeFreeBSD

New port: security/peda: Python-based GDB extension for exploit development and debugging
Needs ReviewPublic

Authored by 0mp on Wed, Dec 5, 11:23 AM.

Details

Reviewers
mat
krion
Summary

I am not sure if I should call this port py-peda instead. I think that I should not as PEDA itself does not care about the underlaying Python version.

Test Plan
  • portlint
  • poudriere testport
    • 11.2 i386, amd64
    • 12.0-RC3 amd64

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21514
Build 20827: arc lint + arc unit

Event Timeline

0mp created this revision.Wed, Dec 5, 11:23 AM
mat added inline comments.Thu, Dec 6, 3:54 PM
security/peda/Makefile
26

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

0mp updated this revision to Diff 51902.Wed, Dec 12, 12:29 PM

Thanks, @mat. I'm sorry for the trouble.

  • Put option-related variables into their section.
  • Set OPTIONS_SUB to use %%DOCS%% instead of %%PORTDOCS%% in pkg-plist.
0mp marked an inline comment as done.Wed, Dec 12, 12:29 PM
mat added a comment.Wed, Dec 12, 3:03 PM

I'd rather you kept PORTDOCS, and the whole tree gets converted at once.

security/peda/Makefile
32–33

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

0mp updated this revision to Diff 51908.Wed, Dec 12, 3:18 PM
  • Fix variable order once again.
  • Revert to using %%PORTDOCS%% in pkg-plist.
0mp marked 2 inline comments as done.Wed, Dec 12, 3:18 PM
0mp added inline comments.
security/peda/Makefile
32–33

I'm really sorry. It should be fixed now.