Skip to content

Modernize Gradle build infrastructure#6

Merged
gsidhwani-nr merged 1 commit into
mainfrom
modernize-gradle-build
Jun 11, 2026
Merged

Modernize Gradle build infrastructure#6
gsidhwani-nr merged 1 commit into
mainfrom
modernize-gradle-build

Conversation

@gsidhwani-nr

Copy link
Copy Markdown
Contributor

Summary

  • Upgraded Gradle wrapper from 7.3 to 8.5 for Java 21 compatibility
  • Replaced root build.gradle with standardized template using verify plugin 4.0 and Maven-based agent resolution
  • Created gradle.properties with java.agent.version=8.6.0
  • Renamed rootProject.name from java-instrumentation-template to apache-axis2-instrumentation
  • Updated all module build.gradle files to use project.findProperty('java.agent.version') and added agent-bridge dependency
  • Replaced sourceCompatibility/targetCompatibility with Java toolchain (Java 8 default, Java 17 for axis2-2.0 module)
  • Changed passes to passesOnly, compile to implementation in verify blocks
  • Widened axis2-kernel range to [1.4,1.5) since 1.4 and 1.4.1 have identical classes
  • Removed newrelic-dependencies.sh (no longer needed with Maven resolution)
  • Updated release.yml: removed newrelic-dependencies.sh invocation, upgraded JDK to 21

Test plan

  • ./gradlew clean build verifyInstrumentation passes locally
  • CI release workflow runs successfully

- Replace root build.gradle with standardized template (verify plugin 4.0, Maven-based agent resolution)
- Create gradle.properties with java.agent.version=8.6.0
- Upgrade Gradle wrapper from 7.3 to 8.5
- Rename rootProject.name from java-instrumentation-template to apache-axis2-instrumentation
- Update all module build.gradle files to use project.findProperty('java.agent.version')
- Add agent-bridge dependency to all modules
- Replace sourceCompatibility/targetCompatibility with Java toolchain
- Change passes to passesOnly, compile to implementation in verify blocks
- Widen axis2-kernel range to [1.4,1.5) since 1.4 and 1.4.1 have identical classes
- apache-axis2-2.0 module uses Java 17 toolchain
- Remove newrelic-dependencies.sh (no longer needed)
- Update release.yml: remove newrelic-dependencies.sh call, upgrade JDK to 21

@Kav91 Kav91 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@gsidhwani-nr gsidhwani-nr merged commit 7d629ec into main Jun 11, 2026
5 checks passed
@gsidhwani-nr gsidhwani-nr deleted the modernize-gradle-build branch June 11, 2026 11:47
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.

2 participants