Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 27, 2025

Upgrades the project to build with JDK 17 and Gradle 8.14.3 (latest), while maintaining Java 11 bytecode compatibility for the compiled artifacts.

Changes

  • Gradle: 7.5.1 → 8.14.3

    • Updated wrapper properties and task configuration
  • Java compatibility: Build with JDK 17, target Java 11 bytecode

    • Updated .java-version from 11 to 17 (for building with JDK 17)
    • Set sourceCompatibility and targetCompatibility to 11 across all modules (for Java 11 bytecode compatibility)
    • Removed obsolete Java 8 compatibility check in Javadoc configuration
  • Android Gradle Plugin: 7.2.2 → 8.2.0

    • Added required namespace declarations to Android library modules (AGP 8.x requirement)
  • GitHub Actions: Updated workflows to use .java-version file

    • Updated actions/checkout from v2 to v4
    • Updated actions/setup-java from v1 to v4 with java-version-file support
    • All workflows now dynamically read Java version from .java-version instead of hardcoded Java 11

Notes

Android modules require network access to Google Maven repository. Java modules build and test successfully with the new configuration. The compiled bytecode has been verified as Java 11 compatible (major version 55), ensuring broader runtime compatibility while leveraging JDK 17's build tools.

Original prompt

This section details on the original issue you should resolve

<issue_title>Upgrade Gradle, Java version</issue_title>
<issue_description>Upgrade Java to: Java 17

Upgrade to latest Gradle version 8</issue_description>

<agent_instructions>Upgrade to Java 17 and Gradle 8

Make sure gradlew check passes</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 27, 2025 09:17
Copilot AI changed the title [WIP] Upgrade to Java 17 and Gradle 8 Upgrade to Java 17 and Gradle 8.5 Dec 27, 2025
Copilot AI requested a review from keesvandieren December 27, 2025 09:25
@keesvandieren
Copy link
Member

@copilot why did you use gradle 8.5? 8.14.3 is latest. If 8.14.3 works, please update to that version

Please update github actions also to use .java-version file (now still hardcoded to java 11)

Copy link
Contributor Author

Copilot AI commented Dec 27, 2025

@copilot why did you use gradle 8.5? 8.14.3 is latest. If 8.14.3 works, please update to that version

Please update github actions also to use .java-version file (now still hardcoded to java 11)

Updated to Gradle 8.14.3 (latest) in commit 02b2714. Also updated all GitHub Actions workflows to use the .java-version file instead of hardcoded Java 11, and modernized the actions to use checkout@v4 and setup-java@v4 in commits 02b2714 and 2ea68f7.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.google.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -Xms1024m -Xmx2048m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Upgrade to Java 17 and Gradle 8.5 Upgrade to Java 17 and Gradle 8.14.3 Dec 27, 2025
@keesvandieren
Copy link
Member

keesvandieren commented Dec 27, 2025

Can you set targetCompatibility to 11, while using jdk 17 to build @copilot?

Copy link
Contributor Author

Copilot AI commented Dec 27, 2025

Can you set targetCompatibility to 11, while using jdk 17 to build @copilot?

Changed targetCompatibility to 11 (along with sourceCompatibility to 11) in commit 5e1199f. The project now builds with JDK 17 (from .java-version) but produces Java 11-compatible bytecode (verified as major version 55).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.google.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -Xms1024m -Xmx2048m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Upgrade to Java 17 and Gradle 8.14.3 Upgrade to JDK 17 and Gradle 8.14.3 (targeting Java 11 bytecode) Dec 27, 2025
@keesvandieren keesvandieren marked this pull request as ready for review December 27, 2025 19:02
@keesvandieren keesvandieren requested a review from a team as a code owner December 27, 2025 19:02
@keesvandieren keesvandieren merged commit aa14de9 into master Dec 29, 2025
1 check passed
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.

Upgrade Gradle, Java version

3 participants