Make repository usable on JDK11. (#1370)

* Make repository usable on JDK11.

* Fix build errors.

Author: vkryachko

buildSrc/src/main/groovy/com/google/firebase/gradle/plugins/ci/CheckCoveragePlugin.groovy

@@ -30,6 +30,12 @@ class CheckCoveragePlugin implements Plugin<Project> {
                 toolVersion = '0.8.5'
             }
 
+            tasks.withType(Test) {
+                jacoco {
+                    excludes = ['jdk.internal.*']
+                }
+            }
+
             task('checkCoverage', type: JacocoReport) {
                 dependsOn 'check'
                 description 'Generates check coverage report and uploads to Codecov.io.'

encoders/firebase-encoders-processor/firebase-encoders-processor.gradle

@@ -16,6 +16,9 @@ plugins {
     id 'java-library'
 }
 
+def jvm = org.gradle.internal.jvm.Jvm.current()
+
+
 dependencies {
     implementation project(':encoders:firebase-encoders-processor:test-support')
     implementation 'com.google.auto.service:auto-service-annotations:1.0-rc6'
@@ -30,7 +33,7 @@ dependencies {
     annotationProcessor 'com.google.auto.service:auto-service:1.0-rc6'
 
     testImplementation 'com.google.testing.compile:compile-testing:0.18'
-    testImplementation files(org.gradle.internal.jvm.Jvm.current().getToolsJar())
+    if (jvm.getToolsJar() != null) testImplementation files(jvm.getToolsJar())
     testImplementation 'com.google.truth:truth:1.0'
 
 }

encoders/firebase-encoders-processor/src/main/java/com/google/firebase/encoders/processor/EncodableProcessor.java

@@ -20,7 +20,6 @@
 import com.google.firebase.encoders.annotations.Encodable;
 import com.google.firebase.encoders.processor.getters.Getter;
 import com.google.firebase.encoders.processor.getters.GetterFactory;
-import com.squareup.javapoet.AnnotationSpec;
 import com.squareup.javapoet.ClassName;
 import com.squareup.javapoet.FieldSpec;
 import com.squareup.javapoet.JavaFile;
@@ -63,6 +62,11 @@ public class EncodableProcessor extends AbstractProcessor {
   private Types types;
   private GetterFactory getterFactory;
 
+  @Override
+  public SourceVersion getSupportedSourceVersion() {
+    return SourceVersion.latestSupported();
+  }
+
   @Override
   public synchronized void init(ProcessingEnvironment processingEnvironment) {
     super.init(processingEnvironment);
@@ -88,15 +92,14 @@ private void processClass(Element element) {
     ClassName className =
         ClassName.bestGuess("Auto" + Names.generatedClassName(element) + "Encoder");
     ClassName configurator = ClassName.get("com.google.firebase.encoders.config", "Configurator");
+
+    // TODO(vkryachko): add @Generated annotation in a way that is compatible with Java versions
+    // before and after 9. See https://github.com/google/dagger/pull/882
     TypeSpec.Builder encoderBuilder =
         TypeSpec.classBuilder(className)
             .addJavadoc("@hide")
             .addModifiers(Modifier.PUBLIC, Modifier.FINAL)
             .addSuperinterface(configurator)
-            .addAnnotation(
-                AnnotationSpec.builder(ClassName.get("javax.annotation", "Generated"))
-                    .addMember("value", "$S", getClass().getName())
-                    .build())
             .addField(
                 FieldSpec.builder(
                         TypeName.INT,

encoders/firebase-encoders-processor/src/test/resources/ExpectedGenericsEncoder.java

@@ -18,9 +18,7 @@
 import com.google.firebase.encoders.config.EncoderConfig;
 import java.io.IOException;
 import java.lang.Override;
-import javax.annotation.Generated;
 
-@Generated("com.google.firebase.encoders.processor.EncodableProcessor")
 public final class AutoGenericsEncoder implements Configurator {
     public static final int CODEGEN_VERSION = 1;
 

encoders/firebase-encoders-processor/src/test/resources/ExpectedGenericsEncoderWithUnknownType.java

@@ -18,11 +18,9 @@
 import com.google.firebase.encoders.config.EncoderConfig;
 import java.io.IOException;
 import java.lang.Override;
-import javax.annotation.Generated;
 
 /**
  * @hide */
-@Generated("com.google.firebase.encoders.processor.EncodableProcessor")
 public final class AutoGenericClassEncoder implements Configurator {
     public static final int CODEGEN_VERSION = 1;
 

encoders/firebase-encoders-processor/src/test/resources/ExpectedRecursiveGenericEncoder.java

@@ -20,9 +20,7 @@
 import com.google.firebase.encoders.config.EncoderConfig;
 import java.io.IOException;
 import java.lang.Override;
-import javax.annotation.Generated;
 
-@Generated("com.google.firebase.encoders.processor.EncodableProcessor")
 public final class AutoMainClassEncoder implements Configurator {
     public static final int CODEGEN_VERSION = 1;
 

encoders/firebase-encoders-processor/src/test/resources/ExpectedSimpleClassEncoder.java

@@ -18,9 +18,7 @@
 import com.google.firebase.encoders.config.EncoderConfig;
 import java.io.IOException;
 import java.lang.Override;
-import javax.annotation.Generated;
 
-@Generated("com.google.firebase.encoders.processor.EncodableProcessor")
 public final class AutoSimpleClassEncoder implements Configurator {
   public static final int CODEGEN_VERSION = 1;
 

encoders/firebase-encoders-processor/src/test/resources/ExpectedTypeWithListEncoder.java

@@ -18,11 +18,9 @@
 import com.google.firebase.encoders.config.EncoderConfig;
 import java.io.IOException;
 import java.lang.Override;
-import javax.annotation.Generated;
 
 /**
  * @hide */
-@Generated("com.google.firebase.encoders.processor.EncodableProcessor")
 public final class AutoTypeWithListEncoder implements Configurator {
     public static final int CODEGEN_VERSION = 1;
 

fiamui-app/fiamui-app.gradle

@@ -56,11 +56,19 @@ android {
 dependencies {
   implementation project(path: ":firebase-inappmessaging-display")
   implementation project(path: ":firebase-inappmessaging")
-  implementation "com.google.firebase:firebase-measurement-connector:18.0.0"
   implementation 'com.google.guava:guava:24.1-jre-android'
 
-  implementation('com.google.firebase:firebase-analytics:17.0.0') {
+  implementation('com.google.firebase:firebase-analytics:17.2.3') {
     exclude group: 'com.google.firebase', module: 'firebase-common'
+    exclude group: 'com.google.firebase', module: 'firebase-components'
+    exclude group: 'com.google.firebase', module: 'firebase-installations'
+    exclude group: 'com.google.firebase', module: 'firebase-installations-interop'
+  }
+  implementation('com.google.firebase:firebase-iid:20.1.1') {
+      exclude group: 'com.google.firebase', module: 'firebase-common'
+      exclude group: 'com.google.firebase', module: 'firebase-components'
+      exclude group: 'com.google.firebase', module: 'firebase-installations'
+      exclude group: 'com.google.firebase', module: 'firebase-installations-interop'
   }
 
   implementation "com.android.installreferrer:installreferrer:1.0"

firebase-database-collection/firebase-database-collection.gradle

@@ -26,6 +26,10 @@ android {
         targetSdkVersion project.targetSdkVersion
         minSdkVersion project.minSdkVersion
     }
+    compileOptions {
+        sourceCompatibility JavaVersion.VERSION_1_8
+        targetCompatibility JavaVersion.VERSION_1_8
+    }
 }
 
 dependencies {

firebase-inappmessaging/firebase-inappmessaging.gradle

@@ -119,6 +119,9 @@ dependencies {
     implementation project(':firebase-installations-interop')
     runtimeOnly project(':firebase-installations')
 
+    //To provide @Generated annotations
+    compileOnly 'javax.annotation:jsr250-api:1.0'
+
     implementation 'com.google.android.gms:play-services-tasks:17.0.0'
     implementation 'io.grpc:grpc-stub:1.21.0'
     implementation 'io.grpc:grpc-protobuf-lite:1.21.0'

gradle.properties

@@ -17,9 +17,12 @@ org.gradle.jvmargs=-Xmx8g -XX:MaxPermSize=8g
 
 android.enableJetifier=true
 android.useAndroidX=true
+android.jetifier.blacklist=errorprone,firebase-encoders-processor
 
 org.gradle.parallel=true
 org.gradle.caching=true
 
 firebase.checks.errorproneProjects=:tools:errorprone
 firebase.checks.lintProjects=:tools:lint
+
+systemProp.illegal-access=warn