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.

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

Added setup.py #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added setup.py #29

wants to merge 1 commit into from

Conversation

ALSchwalm
Copy link

This PR adds the conventional setup.py file to make installation a bit easier. I believe this is entirely backwards compatible, so you can still run subbrute directly from the project folder, if desired. However, users can now install subbrute globally with sudo python setup.py install or (without root) python setup.py install --user.

The wrapper being executed is automatically generated via the entry_points feature of setuptools (this also generates an exe on windows). I added a short section to the README about installation with this script.

Currently, the script is configured to pull the latest dnslib, unless you are running from the project folder (in which case the one in the repo is used). I'm not sure I love this, but it does maintain exactly the existing behavior.

Ultimately, this is a step towards what I think would be the best installation setup: uploading to pypi. Then people can just pip install subbrute and never touch github. This will only get better in the future, as pip is shipped with current python (3.4+).

@regner
Copy link

regner commented Jul 27, 2016

:( WIsh this repo was still active.

@q2dg
Copy link

q2dg commented Aug 3, 2020

Well, it is clear it is not

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