AGP Flamingo Alpha 8 Lint NewApi Desugar Regression [260755411]

Fixed

Bug

Status Update

No update yet.

Description

ey...@gmail.com

created issue #1

Nov 30, 2022 09:40AM

I just updated to Flamingo Alpha 8 from EE Beta 5 and running lint results in NewApi errors for java.time.* (272 occurrences in my app). My minApi is 24 and lint is saying that all of these calls require minApi 26. I have desugaring enabled.

Comments

rk...@google.com <rk...@google.com> Dec 2, 2022 09:37PM

Assigned to an...@google.com.

tn...@google.com <tn...@google.com> #2Dec 2, 2022 10:18PM

Reassigned to tn...@google.com.

Thank you for reporting this issue. For us to further investigate this issue, please provide the following additional information:

Android build
Which Android build are you using? (e.g. UQ1A.240205.002)

Device used
Which device did you use to reproduce this issue?

Are you reporting this as a Developer or User?

Android bug report (to be captured after reproducing the issue)
For steps to capture a bug report, please refer:

https://developer.android.com/studio/debug/bug-report#bugreportdevice

Alternate method
Navigate to “Developer options”, ensure “USB debugging” is enabled, then enable “Bug report shortcut”. Capture bug report by holding the power button and selecting the “Take bug report” option.

Note: Please upload the bug report to google drive and share the folder to android-bugreport@google.com, then share the link here.

Message last modified on Dec 2, 2022 10:22PM

DesugaringTest.zip

114 KB

Download

ey...@gmail.com <ey...@gmail.com> #3Dec 6, 2022 04:19AM

Please provide the requested information to proceed further. Unfortunately the issue will be closed within 7 days if there is no further update.

DesugaringTest.zip

113 KB

Download

tn...@google.com <tn...@google.com> #4Dec 7, 2022 03:10PM

Accepted by tn...@google.com.

We are closing this issue since we didn't receive a response. If you are still facing this problem, please open a new issue and add the relevant information along with reference to this issue.

ma...@gmail.com <ma...@gmail.com> #5Dec 12, 2022 03:38PM

I just bumped into this as well. To reproduce, you can use the reproducer-260755411 branch of the Confetti project, especially this commit and run ./gradlew :shared:lintDebug:

$ ./gradlew :shared:lintDebug

> Configure project :shared
> Task :shared:lintReportDebug
Wrote HTML report to file:///Users/mbonnin/git/Confetti/shared/build/reports/lint-results-debug.html

> Task :shared:lintDebug FAILED
Lint found 1 errors, 0 warnings. First failure:

/Users/mbonnin/git/Confetti/shared/src/androidMain/kotlin/dev/johnoreilly/confetti/utils/AndroidDateTimeFormatter.kt:15: Error: Cast from Instant to TemporalAccessor requires API level 26 (current min is 21) [NewApi]
        return formatter.withZone(ZoneId.of(timeZone.id)).format(instant.toJavaInstant())
                                                                 ~~~~~~~~~~~~~~~~~~~~~~~

There was no error on AGP 7.3.1

Message last modified on Dec 12, 2022 03:52PM

tn...@google.com <tn...@google.com> #6Dec 12, 2022 09:08PM

Marked as fixed.

This bug is fixed by https://cs.android.com/android-studio/platform/tools/base/+/7bc755425b0261bb6897e8975de1d8006c6c17df for both 8.1.0-dev and 8.0.0-beta; will also consider for 7.4.1.

There was also a related fix here for the scenario where you run lint on the library itself: https://cs.android.com/android-studio/platform/tools/base/+/a97632c01407e83dc3fa4fd9c393a5e54e4a0514

Thanks for the report and especially the clear repro!

tn...@google.com <tn...@google.com> #7Dec 12, 2022 09:09PM

Actually it sounds from the initial report like this wasn't broken in 7.4 (comment #5, did you jump straight from 7.3.1 to 8.0.0-alpha?) so I guess only 8.0 is relevant here.

ma...@gmail.com <ma...@gmail.com> #8Dec 29, 2022 04:46PM

comment #7 sorry I missed this notification but you're right, I went straight to 8.0.0-alpha09.

Also I just tried 8.0.0-alpha10 and it seems to be fixed there as well 👍

tn...@google.com <tn...@google.com> #9Dec 29, 2022 04:52PM

Thanks for confirming, glad to hear it!