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.

I also have the same issue so it seems to be a real problem.

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

Dup here:

http://code.google.com/p/android/issues/detail?id=27959&can=1&q=fixed%20position&colspec=ID%20Type%20Status%20Owner%20Summary%20Stars

Though it's a bit more detailed with the minimum necessary to reproduce.

DesugaringTest.zip

113 KB

Download

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

Accepted by tn...@google.com.

why is this marked has Obsolete? this is still a bug that is happening and its difficult to fix.
for all who are looking for a solution use this:

http://stackoverflow.com/questions/16200972/android-css-positionfixed-after-a-device-rotate

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!