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 examples #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update examples #7

wants to merge 4 commits into from

Conversation

mt7479
Copy link
Contributor

@mt7479 mt7479 commented Sep 26, 2018

I fixed up a bunch of things and added uwsgi examples like I promised last February 😄

Add rewrite statement
Add uwsgi ini and nginx config
Switch to /usr/local/bin
swap package installation for pip install instructions
@@ -69,7 +69,8 @@ Clone and install::
sudo mv xmpp-http-upload /opt/xmpp-http-upload
cd /opt/xmpp-http-upload
copy config.example.py config.py
sudo apt-get install python3-flask
sudo apt-get install python3-pip
sudo python3 -m pip install --upgrade flask-cors
Copy link
Owner

Choose a reason for hiding this comment

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

I would prefer to not recommend installing flask from pip. flask-cors is still optional for that reason.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So just keep the uwsgi examples and my updated nginx conf ? Is there a better way than to create a new branch, cherry pick, submit a new pr and abandon this one ?

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