Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Downgrade R8 to fix regression (DEV) #2003

Merged
merged 4 commits into from
Dec 28, 2020
Merged

Conversation

d4rken
Copy link
Member

@d4rken d4rken commented Dec 28, 2020

Our gradle update (#1992) implicitly upgraded the R8 code shrinker to 2.1.75 which now throws a compile error:

/home/darken/git/cwa-app-android/Corona-Warn-App/build/tmp/kotlin-classes/deviceForTestersRelease/de/rki/coronawarnapp/appconfig/internal/AppConfigSource.class: R8: com.android.tools.r8.errors.b: Undefined value encountered during compilation. This is typically caused by invalid dex input that uses a register that is not defined on all control-flow paths leading to the use.
Task :Corona-Warn-App:minifyDeviceForTestersReleaseWithR8 in Corona-Warn-App Finished

Downgrading to the previous one R8 version used by the gradle plugin at 4.0.0 which is 2.0.74, until this is resolved.

Corresponding ticket on the R8 issue tracker:
https://issuetracker.google.com/issues/176381203

@d4rken d4rken added bug Something isn't working maintainers Tag pull requests created by maintainers labels Dec 28, 2020
@d4rken d4rken added this to the 1.11.0 milestone Dec 28, 2020
@d4rken d4rken requested a review from a team December 28, 2020 13:42
@ralfgehrer ralfgehrer self-assigned this Dec 28, 2020
@sonarcloud
Copy link

sonarcloud bot commented Dec 28, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@d4rken d4rken merged commit 7e61aec into release/1.11.x Dec 28, 2020
@d4rken d4rken deleted the fix/R8-issue-2.1.X branch December 28, 2020 16:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants