Migrate from destination property to outputLocation property to address deprecation warning and prepare for Gradle 9.0 [242831042]

Fixed

Bug

Status Update

No update yet.

Description

tt...@gradle.com

created issue #1

Aug 17, 2022 02:13PM

Gradle is removing some @Deprecated methods from the org.gradle.api.reporting.ConfigurableReport class in Gradle 8.0. One method that is remaining is setDestination(File), which is still used by tasks created by AGP. All of Gradle's internal implementation classes for this interface (anything which ultimately extends org.gradle.api.reporting.internal.SimpleReport) will begin producing nagging deprecation warnings when this method is called.

Per the javadoc on that method, please switch to using the outputLocation property to avoid these warnings and prepare for it's removal in Gradle 9.0.

Comments

ag...@google.com <ag...@google.com> Aug 17, 2022 04:05PM

Assigned to an...@google.com.

ga...@google.com <ga...@google.com> #2Aug 18, 2022 11:53AM

Reassigned to bi...@google.com.

The reason this has not been seen before now is that JDK-8272564 was done for JDK-18 and that it also require min API level 28. The reason the Espresso library expose this is that (most likely) internal Google build tools have JDK-8272564 applied generating invoke-interface on Object methods.

bi...@google.com <bi...@google.com> Aug 18, 2022 10:24PM

Accepted by bi...@google.com.

bi...@google.com <bi...@google.com> #3Sep 28, 2022 12:10AM

Project: r8
Branch: main

commit 6041b16a28b5966e6975340c03261c4fef61b794
Author: Søren Gjesse <sgjesse@google.com>
Date: Thu Mar 02 14:39:27 2023

Also apply the rewriting for JDK-8272564 up to Andorid R

Fixes:

b/271408544

Change-Id: I240de7d301677668b9c13b3b49c7b7951e6434b6

M src/main/java/com/android/tools/r8/utils/InternalOptions.java

https://r8-review.googlesource.com/76861

tt...@gradle.com <tt...@gradle.com> #4Sep 28, 2022 06:53PM

Project: r8
Branch: main

commit 9e5692c8ce91b3ea3f4199740b41d57c097c9947
Author: Søren Gjesse <sgjesse@google.com>
Date: Fri Mar 03 08:40:44 2023

Fix test after changing API level for rewriting of JDK-8272564

Bug:

b/271408544

Change-Id: I67552a7fb13657ac9b4fb706592754d74d1b0653

M src/test/java/com/android/tools/r8/desugar/jdk8272564/Jdk8272564Test.java

https://r8-review.googlesource.com/76885

bi...@google.com <bi...@google.com> #5Sep 28, 2022 06:58PM

Project: r8
Branch: 8.0

commit 875ac43330d4404d3c481d7395368ec19defff9d
Author: Søren Gjesse <sgjesse@google.com>
Date: Fri Mar 03 08:43:48 2023

Version 8.0.33

Bug:

b/271408544

Change-Id: I675869c3c03cce5a0e7b367981e79e47ad1048ec

M src/main/java/com/android/tools/r8/Version.java

https://r8-review.googlesource.com/76888

bi...@google.com <bi...@google.com> #6Sep 28, 2022 08:56PM

Marked as fixed.

Project: r8
Branch: 8.0

commit 7b9dea5b4c29b5aa9dde2e4a7db4cb28b68c9caf
Author: Søren Gjesse <sgjesse@google.com>
Date: Fri Mar 03 08:43:32 2023

Fix test after changing API level for rewriting of JDK-8272564

Bug:

b/271408544

Change-Id: I67552a7fb13657ac9b4fb706592754d74d1b0653

M src/test/java/com/android/tools/r8/desugar/jdk8272564/Jdk8272564Test.java

https://r8-review.googlesource.com/76887

Issue 242831042

Description

Issue summary

Comments

ag...@google.com <ag...@google.com> Aug 17, 2022 04:05PM

ga...@google.com <ga...@google.com> #2Aug 18, 2022 11:53AM

bi...@google.com <bi...@google.com> Aug 18, 2022 10:24PM

bi...@google.com <bi...@google.com> #3Sep 28, 2022 12:10AM

tt...@gradle.com <tt...@gradle.com> #4Sep 28, 2022 06:53PM

bi...@google.com <bi...@google.com> #5Sep 28, 2022 06:58PM

bi...@google.com <bi...@google.com> #6Sep 28, 2022 08:56PM

Add comment

Issue metadata