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

Update dependencies #14

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

Update dependencies #14

wants to merge 1 commit into from

Conversation

vallsv
Copy link
Contributor

@vallsv vallsv commented Nov 30, 2019

Hi,

Here is few fixes on the readme.

  • I try to use python 3.8, 3.7, 3.6. Nothing was working cause the tornado web.asynchonous do not exists anymore. I guess because of the new python async keyword. It is working with python 3.5.
  • python3-inotify looks to be expected now

On side note, i created on my side, a conda requirement file, in case you are interested in a small how to with conda.

@falkTX
Copy link
Member

falkTX commented Dec 2, 2019

I have python 3.6 and mod-sdk still runs fine here.
The removal of web.asynchonous is something in python-tornado, not python itself (but yes, it is due to python natively supporting async since 3.6)

@vallsv
Copy link
Contributor Author

vallsv commented Dec 3, 2019

Oh i see, then there is a constrain to put on the tornado version. Ill check.

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.

2 participants