-
Notifications
You must be signed in to change notification settings - Fork 15
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
OMERO versions and describing dependencies in a release package file #103
Comments
This file can be placed at a well known location e.g. |
Another use-case: OMERO.web links to an appropriate Insight version based on its version. This is now broken due to the decoupling of OMERO.server, OMERO.web and OMERO.insight versions. |
The 5.6.0 version is broken for non-dev version strings. This should handle 5.6.0, 5.6.dev1, and 5.6.0.dev1 style version strings. Ultimately, this method should either be refactored out to a util method so we can write unit tests or removed in favor of a more complete solution a la ome/design#103
@manics additionally realized that the omero-web configuration properties are only exposed as part of the final docs, meaning a doc re-release may be required after an omero-web release. |
see discussion under ome/omero-documentation#2123 (comment) on possible using an |
This issue has been mentioned on Image.sc Forum. There might be relevant details there: https://forum.image.sc/t/omero-release-versions-and-strategy/38336/5 |
See:
From @joshmoore ome/ansible-role-omero-server#41 (comment)
From @joshmoore ome/ansible-role-omero-web#27 (comment)
We could have an
omero-release.json
artifact which describes all recommended dependencies (perhaps even includingOMERO-server.zip
?). The general idea is to have a file that describes all suggested production dependencies separate from any single release artifact.Note this is in YAML format because it's easier to type and read but the real file would be JSON.
The advantage of a dict instead of the
name==version
fromrequirements.txt
is it should be easier to parse and override in Ansible and other tools, e.g. by merging theweb.python-requirements
dictionary with a variableomero_web_python_requirements_override
(needs to verify this works).The text was updated successfully, but these errors were encountered: