ome / ome/openmicroscopy

CI new setup workarounds

Open
#6,463 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Java
Stars
222
Forks
105
Avg merge
2h 21m
Merged PRs (30d)
1

Description

In order to restart all the jobs using new CI infrastructure. A few adjustments have been made.

guava, dependency introduced in https://github.com/ome/ome-common-java/blob/master/pom.xml#L148 offers android or jre option.
The option is considered when building Bio-Formats itself but fails to be resolved when building the OMERO Java components. This is due to the Gradle version
The problem only occurs with Bio-Formats 8.6.0-SNAPSHOT so the version in the build is currently pinned to 8.5.0. See https://merge-ci.openmicroscopy.org/jenkins/job/BIOFORMATS-push/

For the guava option to be considered by Gradle, test using Gradle 7 sorted out the issue but failed later on due to Gradle 7 not being supported by other tasks

cc @sbesson

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with ome-common-java/pom.xml around line 148 and the build configuration that pins Bio-Formats to 8.5.0. Review the BIOFORMATS-push Jenkins job and the Gradle version constraints, including the Gradle 7 experiment. Done means the CI jobs can use the new infrastructure and resolve the Bio-Formats snapshot without breaking other tasks.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, ci-cd
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.