Skip to content

RANGER-5666: update graalvm version to 25.1.3#1052

Open
mneethiraj wants to merge 1 commit into
apache:masterfrom
mneethiraj:RANGER-5666
Open

RANGER-5666: update graalvm version to 25.1.3#1052
mneethiraj wants to merge 1 commit into
apache:masterfrom
mneethiraj:RANGER-5666

Conversation

@mneethiraj

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Updated graalvm version from 23.0.11 to 25.1.3

How was this patch tested?

Verified that build with unit tests completed successfully

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the project’s GraalVM dependency version used across Ranger modules, aiming to align Graal.js/script-engine artifacts with a newer GraalVM release.

Changes:

  • Bumped the graalvm.version Maven property from 23.0.11 to 25.1.3.
  • Adjusted the org.graalvm.js:js dependency declaration in agents-common by adding an explicit artifact type.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pom.xml Updates the central graalvm.version property to 25.1.3.
agents-common/pom.xml Modifies the Graal.js dependency declaration (adds an explicit <type>).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread agents-common/pom.xml
Comment on lines 182 to 186
<groupId>org.graalvm.js</groupId>
<artifactId>js</artifactId>
<version>${graalvm.version}</version>
<type>pom</type>
</dependency>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

At GraalVM 25.1.3, org.graalvm.js:js is published as a POM aggregator, not a JAR:

js-25.1.3.jar → 404 (does not exist)
js-25.1.3.pom → exists, and pulls in js-language + truffle-runtime
Without pom, Maven defaults to type=jar and will fail to resolve the dependency (or behave incorrectly).

GraalVM’s own docs for 25.x require:

org.graalvm.js js 25.1.3 pom

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants