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

Permit WSGI usage of anthracite-web.py #18

Merged
merged 1 commit into from
Nov 29, 2014
Merged

Conversation

rpetre
Copy link
Contributor

@rpetre rpetre commented Nov 26, 2014

In order to run anthracite under uwsgi, I made a couple of changes in the anthracite-web.py file. Please consider merging the changes or implementing them better :)

@Dieterbe
Copy link
Owner

what's up with the application variable? you don't seem to use it, or is that something wsgi specific?

@rpetre
Copy link
Contributor Author

rpetre commented Nov 29, 2014

It's a mod_wsgi thing (and it also applies to uwsgi): https://code.google.com/p/modwsgi/wiki/QuickConfigurationGuide#WSGI_Application_Script_File

Note that mod_wsgi requires that the WSGI application entry point be called 'application'. If you want to call it something else then you would need to configure mod_wsgi explicitly to use the other name. Thus, don't go arbitrarily changing the name of the function. If you do, even if you set up everything else correctly the application will not be found.

I admit I have very little experience hacking in Python, but my searches fo "running bottle under uwsgi" turned up a bunch of examples using application = app = bottle.Bottle() so I went for the same incantation and works for me, at least. The fix I made for graph-explorer is way worse, I was ashamed to submit it :)

@rpetre rpetre closed this Nov 29, 2014
@rpetre rpetre reopened this Nov 29, 2014
Dieterbe added a commit that referenced this pull request Nov 29, 2014
Permit WSGI usage of anthracite-web.py
@Dieterbe Dieterbe merged commit c50a314 into Dieterbe:master Nov 29, 2014
@Dieterbe
Copy link
Owner

ok then, all this stuff is a bit of a mistery to me too, but it looks allright.
thanks!

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