AndroidLintAnalysisTask has cache misses due to relative path normalization [236524525]

Fixed

Bug

Status Update

No update yet.

Description

no...@gradle.com

created issue #1

Jun 20, 2022 03:49PM

DESCRIBE THE ISSUE IN DETAIL:

STEPS TO REPRODUCE:

Run AndroidLintAnalysisTask like lintAnalyzeDebug on CI.
Run AndroidLintAnalysisTask like lintAnalyzeDebug on another CI machine.
See if task inputs are the same in both builds.

Expected result: Cache input is reused across invocations across machines.

The AndroidLintAnalysisTask should use RuntimeClasspath normalization for jar and aar inputs instead of Relative Path normalization. See screenshot for details of the cache miss. @Classpath.

Version of Gradle Plugin: 7.2.1 Version of Gradle: 7.4.2 OS: Linux (Circle CI)

Screen Shot 2022-06-17 at 11.41.15 AM.png

1.1 MB

View

Download

Comments

cm...@google.com <cm...@google.com> Jun 20, 2022 03:51PM

Assigned to sp...@google.com.

sp...@google.com <sp...@google.com> #2Aug 1, 2022 05:55PM

Marked as fixed.

Actually I believe it would throw a runtime error, not just be redundant:

if (mRecycled) {
            throw new RuntimeException(toString() + " recycled twice!");
        }

yc...@gmail.com <yc...@gmail.com> #3Mar 2, 2023 07:56AM

Internal note: looks related to Change I39727bc67

sp...@google.com <sp...@google.com> #4Mar 2, 2023 05:56PM

This will be fixed by Change I9baebec66991dd308edda2ce66f1225d25037644 in an upcoming release.

Thank you for the report!

Issue 236524525

Description

Issue summary

Comments

cm...@google.com <cm...@google.com> Jun 20, 2022 03:51PM

sp...@google.com <sp...@google.com> #2Aug 1, 2022 05:55PM

yc...@gmail.com <yc...@gmail.com> #3Mar 2, 2023 07:56AM

sp...@google.com <sp...@google.com> #4Mar 2, 2023 05:56PM

Add comment

Issue metadata