Skip to content
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

Upgrade Ivy to 2.5.2 #6388

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Upgrade Ivy to 2.5.2 #6388

wants to merge 1 commit into from

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented May 9, 2024

What this PR does

The current version (2.4.0) was released in December 2015. Since then three releases have been made including CVE fixes and some features. Ivy 2.5+ requires Java 8 as the minimum version and includes new features like the ability to depend on timestamped SNAPSHOTs.

Testing this PR

The CI builds should still be passing and build the OMERO components. The artifacts and the resolved dependencies should be unmodified and can be compared directly to a build of OMERO without this change. All integration tests should be unaffected

Related reading

See https://ant.apache.org/ivy/history/2.5.0/release-notes.html, https://ant.apache.org/ivy/history/2.5.1/release-notes.html and and https://ant.apache.org/ivy/history/2.5.2/release-notes.html for the list of associated changes

The current version (2.4.0) was released in December 2015. Since
then three releases have been made including CVE fixes and some
features. Ivy 2.5+ requires Java 8 as the minimum version and
includes new features like the ability to depend on timestamped
SNAPSHOTs.
See https://ant.apache.org/ivy/history/2.5.0/release-notes.html,
https://ant.apache.org/ivy/history/2.5.1/release-notes.html and
and https://ant.apache.org/ivy/history/2.5.2/release-notes.html
for the list of associated changes
@sbesson sbesson requested review from jburel and dominikl May 10, 2024 05:17
Copy link
Member

@dominikl dominikl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me. Builds are fine and the new ivy def gets used:

...
01:37:58 check-ivy:
01:37:58 Creating /home/omero/workspace/OMERO-build/src/etc/local.properties
01:37:58 Created dir: /home/omero/workspace/OMERO-build/src/target
01:37:58 :: Apache Ivy 2.5.2 - 20230817170011 :: https://ant.apache.org/ivy/ ::
01:37:58 :: loading settings :: file = /home/omero/workspace/OMERO-build/src/etc/ivysettings.xml
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants