-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Command-Line Tool Refactor #660
Comments
sounds awesome... just a tiny side note aka feature request: It would be gorgeous if the |
zachriggle
added a commit
to zachriggle/pwntools
that referenced
this issue
Aug 28, 2016
Adds an install-time option to disable legacy commands, and only use 'pwn': $ pip install --install-option='--only-use-pwn-command' Documentation is updated to show 'pwn' entry point and sub-commands. Fixes Gallopsled#404 Fixes Gallopsled#660
Kyle-Kyle
pushed a commit
to Kyle-Kyle/pwntools
that referenced
this issue
Apr 25, 2021
This was referenced Mar 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have a lot of command-line tools that nobody knows about unless they go and read the documentation. Additionally, when using the command-line tools, it's not apparent where they come from (e.g. "What is that
asm
command you used?")I suggeset refactoring all of the tools to have a single entry point, a la
git
. Instead ofasm foo
we will have e.g.pwn asm foo
.This also resolves some other issues with naming conflicts with e.g. Arch linux in #573.
In order to maintain backward compatibility, we will also provide a command-line option to
setup.py
to install the fully-named tools, which can be passed topip install
via--install-option
.I also recommend that in addition to the short
pwn
, that we also have an entry point with the full namepwntools
.Finally, to go with #659, I recommend a new entry point so that
pwn update
andpwntools update
just magically works, and grabs the latest released-or-tagged version directly from Pip or Github, and does The Right Thing.The text was updated successfully, but these errors were encountered: