-
-
Notifications
You must be signed in to change notification settings - Fork 818
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
Feature Request: System install (as a module) support and setup.py for easier installation #433
Comments
Hello @blshkv Kindly confirm that the attached setup.py.zip is ok :) |
Thanks for working on it. I have downloaded the latest git, dropped setup.py into the folder, the output was the following:
That's probably a different issue and a separate patch is required, so could you push your changes into git? |
patch done @ 3a3188c :) |
I installed the latest using the following:
and got this:
|
Please take a look at this: setup.py.zip |
Can you please test it before asking?
|
That setup.py works fine for me. |
Ill test with python 3.7 in a bit. Have seen such problem with 3.6 recently |
Cool! Please download and try the latest setup.py |
ok, it works with python 3.7 The root cause seems mutual imports: Not sure if you want to fix the code or lift the requirement. I would suggest fixing the code. |
Added df5544d |
great job, thanks. Would be good to have a fresh release too, since the previous version does not support python3 (if I'm not wrong). |
Please have a look at laramies/theHarvester#52 for reasons why it is important for us.
You can also have a look at the following example on the implementation:
laramies/theHarvester#229
The text was updated successfully, but these errors were encountered: