Skip to content
This repository has been archived by the owner on Nov 12, 2024. It is now read-only.

plugin python is built into core uwsgi if installed via pip. #31

Merged
merged 1 commit into from
Dec 26, 2015

Conversation

mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Dec 26, 2015

Distros should package uWSGI in a modular way, with each feature as a plugin. But when you install using language specific ways (pip, gem...) the relevant language is embedded, so you do not need to load the plugin

I tested this with the stable branch of @bgruening docker-galaxy-stable, where this works fine.
(uwsgi is installed into the virtualenv environment using pip)
If we do have --plugin python in all of our current project, starting galaxy_web fails with

open("./python_plugin.so"): No such file or directory [core/utils.c line 3675]
!!! UNABLE to load uWSGI plugin: ./python_plugin.so: cannot open shared object file: No such file or directory !!!

This has also occured here
It also supersedes PR #21

jmchilton added a commit that referenced this pull request Dec 26, 2015
plugin python is built into core uwsgi if installed via pip.
@jmchilton jmchilton merged commit c3e6bce into galaxyproject:master Dec 26, 2015
@mvdbeek
Copy link
Member Author

mvdbeek commented Dec 26, 2015

@jmchilton thanks!

@jmchilton
Copy link
Member

@mvdbeek No, thank you this is awesome. I never even noticed #21 😰, too many projects I seem to miss PRs on things other than Galaxy core.

@jmchilton
Copy link
Member

@mvdbeek I have added you to the Galaxy "docker-devops" team that has write access to a few projects including this one. Feel free to commit directly or continue to open PRs of course, whichever you'd prefer.

@mvdbeek
Copy link
Member Author

mvdbeek commented Dec 26, 2015

@jmchilton no worries, i also miss notifications on github, and I'm involved in far less than what you must be seeing.
Thanks a lot for all the things you're doing!

@mvdbeek mvdbeek deleted the plugin_python branch March 1, 2016 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants