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

Updated Req.txt #114

Merged
merged 6 commits into from
Aug 15, 2018
Merged

Updated Req.txt #114

merged 6 commits into from
Aug 15, 2018

Conversation

PSNAppz
Copy link
Member

@PSNAppz PSNAppz commented Aug 11, 2018

Fixes #115

Changes Proposed

  • Updated requirements.txt
  • Fixed a format error in savedb.py

requirements.txt Outdated
@@ -5,3 +5,5 @@ requests==2.18.4
requests_mock==1.4.0
tldextract==2.2.0
yattag==1.10.0
mysqlclient==1.3.13
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this is a pip install-able module yet. Didn't work for me personally.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I followed these steps and it didn't solve the issue.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try other solutions?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll try the rest in a little and let you know if they work.

Copy link
Contributor

@shivankar-madaan shivankar-madaan Aug 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failing for me as well
here's the error code

root@kali:~/github/TorBot# pip install mysqlclient
Collecting mysqlclient
  Using cached https://files.pythonhosted.org/packages/ec/fd/83329b9d3e14f7344d1cb31f128e6dbba70c5975c9e57896815dbb1988ad/mysqlclient-1.3.13.tar.gz
    Complete output from command python setup.py egg_info:
    /bin/sh: 1: mysql_config: not found
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-41snh5qi/mysqlclient/setup.py", line 18, in <module>
        metadata, options = get_config()
      File "/tmp/pip-install-41snh5qi/mysqlclient/setup_posix.py", line 53, in get_config
        libs = mysql_config("libs_r")
      File "/tmp/pip-install-41snh5qi/mysqlclient/setup_posix.py", line 28, in mysql_config
        raise EnvironmentError("%s not found" % (mysql_config.path,))
    OSError: mysql_config not found
    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-41snh5qi/mysqlclient/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shivankar-madaan Try this pip install --upgrade setuptools

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes tried all the steps in the stackoverflow link

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PSNAppz it worked after running this step

sudo apt-get install python-pip python-dev libmysqlclient-dev

If this becomes a dependency, can we try using Mongo instead of mysql, just a suggestion

@PSNAppz PSNAppz merged commit 2d863a6 into dev Aug 15, 2018
@PSNAppz PSNAppz deleted the bug_fixes branch September 18, 2018 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants