Adds Gradle build that delegates to existing Ant build #214

NicoPiel · 2025-12-01T14:52:32Z

Adds an initial Gradle build system that delegates to the existing Ant build to enable gradual migration and tooling improvements.

Highlights:

Introduces a root Gradle configuration that imports and delegates to the project's Ant build targets, avoiding immediate rewrite of existing build logic.
Adds the Gradle wrapper and supporting files (wrapper properties, jar, and scripts) so the repository can be built consistently without requiring a local Gradle install.
Adds per-subproject Gradle stub files to make the repository a multi-project Gradle layout and allow incremental conversion of subprojects.
Adds Gradle metadata/configuration (gradle.properties, dependency platform TOML) and updates SDKMAN settings to include a Gradle version for reproducible developer environments.
Adds .gitattributes and .gitignore tweaks and normalizes an IDE classpath file to remove trailing encoding differences.
Configures Ant-to-Gradle interoperability (ant taskdefs and properties) and maps Ant targets to avoid conflicts with Gradle built-in tasks.

Why:

Allows us to start using Gradle tooling and the wrapper for builds, CI, and IDEs while preserving the working Ant build.
Supports incremental migration of subprojects to Gradle without disrupting existing build behavior.

Notes / Next steps:

Subprojects remain backed by the Ant build; plan incremental conversion of individual modules to native Gradle builds.
Verify CI pipelines and developer SDK setups pick up the wrapper and SDKMAN gradle setting.

mgaffigan

Other issues to address

CI build scripts need updated (.github/workflow/...)
Readme/Contributor documents need updated
ant mirth-build build target needs to be updated to block build outside of gradle with migration guidance in error message

client/build.gradle.kts

gradle/wrapper/gradle-wrapper.properties

gradle/libs.versions.toml

settings.gradle.kts

mgaffigan · 2025-12-01T23:57:03Z

build.gradle.kts

+// Pass Gradle properties to Ant
+project.findProperty("disableSigning")?.let {
+    ant.properties["disableSigning"] = it.toString()
+}
+
+project.findProperty("disableTests")?.let {
+    ant.properties["disableTests"] = it.toString()
+}


Are there other properties we need to be passing?

Good question. I'm not sure. These are the only two I use and the only two available in mirth-build.xml, afaia.

I know JUnit supports some properties for running specific tests instead of the whole suite, but I doubt anyone actually uses that. I have in the past, but I would need to look them up to even identify what they are called, because it's not something I do frequently. I wouldn't hold up this PR for that.

9adf3f3

This commit added disableSigning as well as cert and keystore_property_file

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Adds a top-level Gradle build that imports the existing Ant build to enable a gradual migration to Gradle while still using Ant for current tasks. Configures Ant to run JUnit (including ant-junit4) and exposes select Gradle properties to Ant; renames conflicting Ant targets to avoid collisions and makes the Gradle build task delegate to the Ant build target. Also adds placeholder Gradle files for subprojects and bumps the configured Gradle SDK version. Co-authored-by: Mitch Gaffigan <mitch.gaffigan@comcast.net> Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Changes the root project name to a lowercase value to avoid case-related build/tooling issues (improves compatibility with Gradle/Ant and CI environments). Signed-off-by: Nico Piel <nico.piel@hotmail.de>

.sdkmanrc

settings.gradle.kts

tonygermano · 2025-12-12T17:32:03Z

build.gradle.kts

+// Pass Gradle properties to Ant
+project.findProperty("disableSigning")?.let {
+    ant.properties["disableSigning"] = it.toString()
+}
+
+project.findProperty("disableTests")?.let {
+    ant.properties["disableTests"] = it.toString()
+}


I know JUnit supports some properties for running specific tests instead of the whole suite, but I doubt anyone actually uses that. I have in the past, but I would need to look them up to even identify what they are called, because it's not something I do frequently. I wouldn't hold up this PR for that.

tonygermano · 2025-12-15T23:44:09Z

build.gradle.kts

+// Pass Gradle properties to Ant
+project.findProperty("disableSigning")?.let {
+    ant.properties["disableSigning"] = it.toString()
+}
+
+project.findProperty("disableTests")?.let {
+    ant.properties["disableTests"] = it.toString()
+}


9adf3f3

This commit added disableSigning as well as cert and keystore_property_file

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Copilot

Pull request overview

This PR introduces Gradle build support to enable gradual migration from the existing Ant-based build system. It adds the Gradle wrapper (version 9.2.1), root and subproject build configurations that delegate to Ant, version catalog management, and updates to supporting files like .gitignore and .gitattributes.

Key Changes:

Adds Gradle wrapper scripts and configuration for reproducible builds without requiring local Gradle installation
Implements root build configuration that imports the Ant build and provides Ant-to-Gradle interoperability (JUnit taskdefs, property passing)
Creates stub subproject build files to establish multi-project structure while preserving Ant build delegation

Reviewed changes

Copilot reviewed 12 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
`settings.gradle.kts`	Configures multi-project structure with 5 subprojects and foojay-resolver plugin
`build.gradle.kts`	Root build that imports Ant, configures JUnit tasks, passes properties, and renames conflicting targets
`gradlew` / `gradlew.bat`	Gradle wrapper shell and batch scripts for cross-platform execution
`gradle/wrapper/gradle-wrapper.properties`	Wrapper configuration specifying Gradle distribution URL and settings
`gradle/wrapper/gradle-wrapper.jar`	Binary wrapper JAR for bootstrapping Gradle
`gradle.properties`	Gradle build optimization settings (configuration cache, parallel, caching)
`gradle/libs.versions.toml`	Version catalog with Guava dependency declaration
`client/build.gradle.kts`	Stub build file with commented Ant import for client subproject
`command/build.gradle.kts`	Stub build file with commented Ant import for command subproject
`donkey/build.gradle.kts`	Stub build file with commented Ant import for donkey subproject
`generator/build.gradle.kts`	Stub build file with commented Ant import for generator subproject
`server/build.gradle.kts`	Stub build file with commented Ant import for server subproject
`.gitattributes`	Adds line ending normalization for gradlew and binary handling for JARs
`.gitignore`	Adds exclusion for Gradle `build` output directories
`.sdkmanrc`	Formatting adjustment (no functional change)

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

Copilot · 2025-12-30T19:09:56Z

.sdkmanrc

 # Add key=value pairs of SDKs to use below
 java=17.0.17.fx-zulu
-ant=1.10.14
+ant=1.10.14


The PR description states that the SDKMAN settings should include a Gradle version for reproducible developer environments, but no Gradle version has been added to this file. Consider adding a line like "gradle=8.11.1" (or the appropriate version) to ensure developers have a consistent Gradle version installed via SDKMAN.

Copilot · 2025-12-30T19:09:57Z

settings.gradle.kts

+ * This file was generated by the Gradle 'init' task.
+ *
+ * The settings file is used to specify which projects to include in your build.
+ * For more detailed information on multi-project builds, please refer to https://docs.gradle.org/8.14/userguide/multi_project_builds.html in the Gradle documentation.


The comment references documentation for Gradle 8.14, but the wrapper is configured to use Gradle 9.2.1 (which doesn't exist). These version references should be consistent and both should reference a valid, existing Gradle version.

Suggested change

* For more detailed information on multi-project builds, please refer to https://docs.gradle.org/8.14/userguide/multi_project_builds.html in the Gradle documentation.

* For more detailed information on multi-project builds, please refer to https://docs.gradle.org/current/userguide/multi_project_builds.html in the Gradle documentation.

NicoPiel force-pushed the feature/gradle-build-ant branch 2 times, most recently from 6a6e3c7 to ddf0ada Compare December 1, 2025 15:00

mgaffigan requested changes Dec 1, 2025

View reviewed changes

NicoPiel and others added 5 commits December 29, 2025 01:29

Add root settings and gradle to SDKMAN

1c79151

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Gradle init and build files for all subprojects

1130609

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Update docs link

26514ca

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

Upgrade Gradle Wrapper to 9.2.1

6284f3b

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

NicoPiel force-pushed the feature/gradle-build-ant branch 3 times, most recently from 4018468 to 6284f3b Compare December 29, 2025 00:31

Normalize Gradle project name and classpath formatting

bf79ea8

Changes the root project name to a lowercase value to avoid case-related build/tooling issues (improves compatibility with Gradle/Ant and CI environments). Signed-off-by: Nico Piel <nico.piel@hotmail.de>

tonygermano reviewed Dec 29, 2025

View reviewed changes

Remove gradle from SDKMAN env

8fbaf99

Signed-off-by: Nico Piel <nico.piel@hotmail.de>

pacmano1 requested a review from Copilot December 30, 2025 19:06

Copilot started reviewing on behalf of pacmano1 December 30, 2025 19:06 View session

pacmano1 requested review from Copilot and removed request for Copilot December 30, 2025 19:06

Copilot started reviewing on behalf of pacmano1 December 30, 2025 19:08 View session

Copilot AI reviewed Dec 30, 2025

View reviewed changes

	* For more detailed information on multi-project builds, please refer to https://docs.gradle.org/8.14/userguide/multi_project_builds.html in the Gradle documentation.
	* For more detailed information on multi-project builds, please refer to https://docs.gradle.org/current/userguide/multi_project_builds.html in the Gradle documentation.

Adds Gradle build that delegates to existing Ant build #214

Are you sure you want to change the base?

Adds Gradle build that delegates to existing Ant build #214

Uh oh!

Conversation

NicoPiel commented Dec 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mgaffigan left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mgaffigan Dec 1, 2025

Choose a reason for hiding this comment

Uh oh!

NicoPiel Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

tonygermano Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

tonygermano Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

tonygermano Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

tonygermano Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Dec 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 30, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

NicoPiel commented Dec 1, 2025 •

edited

Loading