Repository navigation
Add omero-zarr-pixel-buffer - #6478
Conversation
|
This now breaks the OMERO-build on merge-ci with: I have absolutely no clue why and how to fix it. |
sbesson
left a comment
There was a problem hiding this comment.
A few additional cleanup/organization suggestions but overall the proposed changes are consistent with the way Glencoe has been building OMERO.server with omero-zarr-pixel-buffer and all its dependencies included.
| @@ -85,6 +85,10 @@ | |||
| usepoms="true" useMavenMetadata="true" | |||
| m2compatible="true" | |||
| root="https://nexus.senbox.net/nexus/content/groups/public/"/> | |||
There was a problem hiding this comment.
We should be able to clean up the bc.releases resolver and the associated chain resolver below
Moved the 'versions.omero-zarr-pixel-buffer' property to the appended values section.
sbesson
left a comment
There was a problem hiding this comment.
Can you clean the references to bc.releases from etc/ivysettings.xml as well. As the latest state of this PR has been successfully tested in the nightly integration tests on OME CI, I think we should be in a place to get it merged soon.
Was only needed for jzarr.
sbesson
left a comment
There was a problem hiding this comment.
The nightly CI tests have been passing with the last commit. Before merging, could you update the body of the PR to briefly summarize its content?
This PR adds the omero-zarr-pixel-buffer to the OMERO.server build.
In order for that to work it also:
Note: Replaces #6450: Had to close the other one, because it seemed to have some kind of hidden "failed" flag which prevented scc from merging it, even after several all successful rebuilds.