IssueTracker

Thank you for creating an issue. Looks like there may be a problem with the input.

If you are using R8 from source, which it looks like since you have a non-obfuscated stack trace, can you then place a break-point at:
com.android.tools.r8.naming.ProguardMapMinifier:489

and report what the reservedName, reference and definition is. Then we will be able to back-track from there.

You can attach the debugger to the gradle compilation by invoking the following command from the terminal:
./gradlew clean transformClassesAndResourcesWithR8ForReleaseAndroidTest --info --stacktrace -Dorg.gradle.debug=true --no-daemon --stacktrace

(you may have to change the name of the task)

Also, can I ask you to enable assertions in you build to see if there are any other problems we are not aware of. You can do it by modifying gradle.properties and extend the org.gradle.jvmargs with -ea:com.android.tools.r8...

You can also share you project if that is possible - that would allow me to find out why the definition is null.

I am not using R8 from source, could you advise how can I do that? Which debugger / dev tools should I use as I am not familiar with it.

And btw, it seems I cannot solely execute transformClassesAndResourcesWithR8ForAppReleaseAndroidTest, it said task not found

Another update:

I've tried to update my release buildType to this:

        release {
            signingConfig signingConfigs.gagConfig
            minifyEnabled true
            testProguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-project.txt', 'proguard-fresco.pro'
            proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-project.txt', 'proguard-fresco.pro'
        }

Changing from proguard-android-optimize.txt to proguard-android.txt in getDefaultProguardFile(), NullPointerException is gone when building test APK

Thank you for responding. The reason why I asked how you was running R8 is because this is the stacktrace:

Caused by: java.lang.NullPointerException
        at com.android.tools.r8.naming.ProguardMapMinifier$ApplyMappingMemberNamingStrategy.reportReservationError(ProguardMapMinifier.java:542)
        at com.android.tools.r8.naming.ProguardMapMinifier$ApplyMappingMemberNamingStrategy.nextName(ProguardMapMinifier.java:489)
        at com.android.tools.r8.naming.ProguardMapMinifier$ApplyMappingMemberNamingStrategy.next(ProguardMapMinifier.java:445)
        at com.android.tools.r8.naming.MethodNamingState.newOrReservedNameFor(MethodNamingState.java:64)
        at com.android.tools.r8.naming.MethodNamingState.newOrReservedNameFor(MethodNamingState.java:46)
        at com.android.tools.r8.naming.MethodNameMinifier.assignNameToMethod(MethodNameMinifier.java:236)
        at com.android.tools.r8.naming.MethodNameMinifier.assignNamesToClassesMethods(MethodNameMinifier.java:215)
        at com.android.tools.r8.naming.MethodNameMinifier.assignNamesToClassesMethods(MethodNameMinifier.java:222)
        at com.android.tools.r8.naming.MethodNameMinifier.assignNamesToClassesMethods(MethodNameMinifier.java:222)
        at com.android.tools.r8.naming.MethodNameMinifier.computeRenaming(MethodNameMinifier.java:188)
        at com.android.tools.r8.naming.ProguardMapMinifier.run(ProguardMapMinifier.java:146)
        at com.android.tools.r8.R8.run(R8.java:740)
        at com.android.tools.r8.R8.run(R8.java:243)

We shrink the R8 compiler (using R8) so all of the classes and line-positions are not available - I guess when adding the dependency in gradle, the dependency will not use the R8 compiled library but take the full one. Sorry for the confusion.

Is it possible for you to share the project? If not, I have to add some logging to R8 to figure out what the exact problem is.

You may get the project here. (Please let me know when you got the project, so that I will revert the access, thanks!)
https://drive.google.com/file/d/1pJNMTrKfgCmixfqow_UoNScL0WCbx03l/view?usp=sharing (9gag-android-e36681ef273f346f1d054bc42ace0c3568e30d79.zip)

To setup, you will need this steps:

1. Create a keystore, 9gag-android-release-key.keystore, put it to folder root. You will need to update the alias, store and key password in repo/app/gradle.properties
2. Execute ./run.sh

I have downloaded the project now. You can revert access! Will let you know when I have found the issue.

Project: r8
Branch: master

commit acebc712d7ad92a3885f36ca7991319ac607eed5
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date:   Thu Jan 16 12:12:51 2020

    Use toReference instead of asDexReference
   
    Bug: 147480426
    Change-Id: Ie123e7529ed8b4f53de6499cb3c04e28427e81b6

M       src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java

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

Project: r8
Branch: 1.6

commit d57c4db9259a1b119fca4c7d12016d0bf3e318bb
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date:   Thu Jan 16 16:16:09 2020

    Version 1.6.60
   
    Cherry-pick: Use toReference instead of asDexReference
    CL: https://r8-review.googlesource.com/47616
   
    Bug: 147480426
    Change-Id: I51aae47809ea12786302fddc8acee4584f4570c5

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

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

Raymond, can I ask you to try out version 1.6.60 that should resolve the issue? You can try it out by adding the following to your build.gradle file (your setup is a bit different but I am certain you can find the place :):

buildscript {

    repositories {
        maven {
            url 'http://storage.googleapis.com/r8-releases/raw'
        }
    }

    dependencies {
        classpath 'com.android.tools:r8:1.6.60'          // Must be before the Gradle Plugin for Android.
        classpath 'com.android.tools.build:gradle:X.Y.Z' // Your current AGP version.
     }
}

Project: r8
Branch: 2.0

commit d848786cc22672789d3f0e341860f9eb5dfaf7aa
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date:   Fri Jan 17 11:22:36 2020

    Version 2.0.17
   
    Cherry-pick: Use toReference instead of asDexReference
    CL: https://r8-review.googlesource.com/47685
   
    Cherry-pick: Ensure handling of optional type args and inner if class
    was merged
    CL: https://r8-review.googlesource.com/47682
   
    Cherry-pick: Use JDK9 in keep attributes dots test to fix windows
    invocation
    CL: https://r8-review.googlesource.com/47514
   
    Bug: 147480426
    Bug: 147800709
    Change-Id: Ibe37414efed8604559e8785ccc4fccdda69e8f8c

M       src/main/java/com/android/tools/r8/Version.java
M       src/main/java/com/android/tools/r8/naming/ProguardMapMinifier.java
M       src/main/java/com/android/tools/r8/naming/signature/GenericSignatureParser.java
M       src/main/java/com/android/tools/r8/naming/signature/GenericSignatureRewriter.java
M       src/test/java/com/android/tools/r8/naming/signature/SignatureOfMergedClassesTest.java
A       src/test/java/com/android/tools/r8/naming/signature/merging/ImplL.java
A       src/test/java/com/android/tools/r8/naming/signature/merging/L.java
M       src/test/java/com/android/tools/r8/shaking/attributes/KeepAttributesDotsTest.java

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

Hi, yes, version 1.6.60 works! Thanks!