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.

Sign up for GitHub

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

Jump to bottom

Unified argument parsing support for commands #659

Closed
hugsy opened this issue Jun 11, 2021 · 1 comment · Fixed by #661
Closed

Unified argument parsing support for commands #659

hugsy opened this issue Jun 11, 2021 · 1 comment · Fixed by #661
Assignees
@hugsy
Labels
enhancement new feature
Milestone
Release 2021.07

Comments

@hugsy
Copy link
Owner

hugsy commented Jun 11, 2021

Branch dev-argparse, started by bd8e338
@hugsy hugsy added this to the 2021-07 Release milestone Jun 11, 2021
@hugsy hugsy self-assigned this Jun 11, 2021
hugsy pushed a commit that referenced this issue Jun 11, 2021
New addition for #659 :
d6fd038 
- added `pattern` command
- changed the default period to 4 for better compat with `pwntools` (fixed #658)
- updated unit tests
hugsy pushed a commit that referenced this issue Jun 12, 2021
Added argparsing support and unit test for processlisting (#659)
bbe84db
hugsy pushed a commit that referenced this issue Jun 12, 2021
- added RemoteCommand for argparsing (#659)
782dd88 
- fixed ArgumentParser prog name (#659)
@hugsy
Copy link
Owner Author

hugsy commented Jun 12, 2021

Pretty much done, all tests pass. getopt was removed from import list. Each gef command can be decorated with @parse_arguments() (see docs/api.md)

image

@hugsy hugsy mentioned this issue Jun 12, 2021
Merged
5 tasks
hugsy pushed a commit that referenced this issue Jul 3, 2021
adding parse_arguments for pie * commands (#659)
5fb2e90
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hugsy hugsy

Labels
enhancement new feature
Projects
None yet
Milestone
Release 2021.07
Development

Successfully merging a pull request may close this issue.

Proper command argument parsing hugsy/gef
1 participant
@hugsy