-
Notifications
You must be signed in to change notification settings - Fork 50
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
Merge build #2035
Merge build #2035
Conversation
27e606f
to
c857f42
Compare
c857f42
to
9869602
Compare
State at this point is passing locally with the following test.sh:
run from a WORKSPACE directory of the form:
|
707a8bc
to
11c2b98
Compare
3059caf
to
3e350e7
Compare
9497466
to
f8407fb
Compare
c08062b
to
441d96c
Compare
rm -rf $WORKSPACE/venv | ||
fi | ||
python3 -m venv --system-site-packages $WORKSPACE/venv || virtualenv --system-site-packages $WORKSPACE/venv | ||
$WORKSPACE/venv/bin/pip install -r $WORKSPACE/OMERO.server/share/web/requirements-py27.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$WORKSPACE/venv/bin/pip install -r $WORKSPACE/OMERO.server/share/web/requirements-py27.txt
seems strange with the decoupled work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. Happy to replace it, but requires something upstream.
see ome/omero-web#96
omero config set omero.web.application_server wsgi-tcp | ||
omero web config nginx | sed "s|$WORKSPACE/OMERO.server|/home/omero/OMERO.py|g" > omero/sysadmins/unix/install-web/nginx-omero.conf | ||
omero web config nginx-location | sed "s|$WORKSPACE/OMERO.server|/home/omero/OMERO.py|g" | grep -v '^#' > omero/sysadmins/unix/install-web/nginx-location.conf | ||
omero web config nginx-location | sed "s|/opt/omero/web|/home/omero|g" | grep '^##' | cut -c3- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to self. This will have to be reviewed when we rename omero ->omero-web/omero-server
@jburel : travis is green again. Any objections? |
No. merging |
Script for building a snapshot of the OMERO documentation in devspace rather than ci-master. This should merge upstream dependencies like omero-install and omeroweb-install, then run autogen, and finally build sphinx.
see: