Revert "Gradle version change from AGP 7.4.2 to 8.1.1, Fixes AB#2779502" (#2651)

somalaya · web-flow · commit 654872fef4db · 2025-05-27T17:38:21.000-07:00
Reverts #2587 Reverting since the builds are failing after this change. Fixes [AB#2779502](https://identitydivision.visualstudio.com/fac9d424-53d2-45c0-91b5-ef6ba7a6bf26/_workitems/edit/2779502)
diff --git a/LabApiUtilities/build.gradle b/LabApiUtilities/build.gradle
@@ -1,7 +1,7 @@
 plugins {
     id 'java-library'
     id 'maven-publish'
-    id 'com.microsoft.identity.buildsystem' version '0.2.5'
+    id 'com.microsoft.identity.buildsystem' version '0.2.3'
 
     // the below build plugin allows to generate BuildConfig files in a JAVA gradle project.
     // BuildConfig files are generated by default in an Android project, however, a non-Android java
diff --git a/azure-pipelines/pull-request-validation/build-consumers.yml b/azure-pipelines/pull-request-validation/build-consumers.yml
@@ -125,15 +125,15 @@ stages:
       displayName: Assemble msal
       inputs:
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
         tasks: clean msal:assembleLocal
     - template: ../templates/steps/automation-cert.yml
     - task: Gradle@3
       displayName: Run msal Unit tests
       inputs:
         tasks: msal:testLocalDebugUnitTest -Plabtest -PlabSecret=$(LabVaultAppCert) -ProbolectricSdkVersion=${{variables.robolectricSdkVersion}} -PmockApiUrl=$(MOCK_API_URL)
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
   # broker
   - job: brokerValidation
     displayName: Broker
@@ -171,13 +171,13 @@ stages:
         tasks: AADAuthenticator:clean AADAuthenticator:assembleLocal --build-cache --info
         publishJUnitResults: false
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
     - task: Gradle@3
       displayName: Run broker Unit tests
       inputs:
         tasks: AADAuthenticator:localDebugAADAuthenticatorUnitTestCoverageReport --build-cache --info -Plabtest -PlabSecret=$(LabVaultAppCert) -ProbolectricSdkVersion=${{variables.robolectricSdkVersion}} -PpowerLiftApiKey=$(powerliftApiKey) -PcodeCoverageEnabled=true ${{variables.shouldSkipLongRunningTest}}
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
   # Linux broker
   - job: linuxBrokerValidation
     displayName: Linux Broker
@@ -226,7 +226,6 @@ stages:
           tasks: LinuxBroker:clean LinuxBroker:assemble --build-cache --info
           publishJUnitResults: false
           jdkArchitecture: x86
-          jdkVersionOption: "1.17"
       - task: Bash@3
         displayName: Execute tests
         inputs:
@@ -278,10 +277,10 @@ stages:
       inputs:
         tasks: clean adal:assembleLocal
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
     - task: Gradle@3
       displayName: Run adal Unit tests
       inputs:
         tasks: adal:testLocalDebugUnitTest -Plabtest -ProbolectricSdkVersion=${{variables.robolectricSdkVersion}} -PlabSecret=$(LabVaultAppCert)
         jdkArchitecture: x64
-        jdkVersionOption: "1.17"
+        jdkVersionOption: "1.11"
diff --git a/azure-pipelines/pull-request-validation/common.yml b/azure-pipelines/pull-request-validation/common.yml
@@ -38,7 +38,7 @@ jobs:
     displayName: Assemble Local
     inputs:
       javaHomeSelection: JDKVersion
-      jdkVersionOption: "1.17"
+      jdkVersionOption: "1.11"
       tasks: clean common:assembleLocal
       publishJUnitResults: false
       jdkArchitecture: x64
@@ -48,14 +48,14 @@ jobs:
     displayName: Run Unit tests
     inputs:
       javaHomeSelection: JDKVersion
-      jdkVersionOption: "1.17"
+      jdkVersionOption: "1.11"
       tasks: common:testLocalDebugUnitTest -PlabSecret=$(LabVaultAppCert) -PmockApiUrl=$(MOCK_API_URL) 
   - task: Gradle@3
     displayName: Check Dependencies size
     condition: eq(variables['system.pullRequest.targetBranch'], 'dev')
     inputs:
       tasks: common:dependenciesSizeCheck -PdependenciesSizeCheckConfig=distReleaseRuntimeClasspath
-      jdkVersionOption: "1.17"
+      jdkVersionOption: "1.11"
 
 - job: spotbugs
   displayName: SpotBugs
@@ -68,7 +68,6 @@ jobs:
   - template: ../templates/steps/spotbugs.yml
     parameters:
       project: common
-    
 - job: lint
   displayName: Lint
   cancelTimeoutInMinutes: 1
@@ -88,5 +87,5 @@ jobs:
       tasks: clean common:lintLocalDebug
       publishJUnitResults: false
       javaHomeSelection: JDKVersion
-      jdkVersionOption: "1.17"
+      jdkVersionOption: "1.11"
 ...
diff --git a/azure-pipelines/pull-request-validation/common4j.yml b/azure-pipelines/pull-request-validation/common4j.yml
@@ -31,8 +31,6 @@ jobs:
     inputs:
       cwd: $(Build.SourcesDirectory)/common-java-root
       tasks: clean common4j:assemble
-      javaHomeOption: JDKVersion
-      jdkVersionOption: 1.17
   - template: ../templates/steps/spotbugs.yml
     parameters:
       project: common4j
@@ -43,6 +41,4 @@ jobs:
     inputs:
       cwd: $(Build.SourcesDirectory)/common-java-root
       tasks: common4j:test -Psugar=true -PmockApiUrl=$(MOCK_API_URL)
-      javaHomeOption: JDKVersion
-      jdkVersionOption: 1.17
 ...
diff --git a/azure-pipelines/pull-request-validation/lab-utilities.yml b/azure-pipelines/pull-request-validation/lab-utilities.yml
@@ -31,8 +31,6 @@ jobs:
         inputs:
           cwd: $(Build.SourcesDirectory)/common-java-root
           tasks: LabApiUtilities:clean LabApiUtilities:assemble
-          javaHomeSelection: JDKVersion
-          jdkVersionOption: "1.17"
       - template: ../templates/steps/spotbugs.yml
         parameters:
           project: LabApiUtilities
@@ -43,6 +41,4 @@ jobs:
         inputs:
           cwd: $(Build.SourcesDirectory)/common-java-root
           tasks: LabApiUtilities:test -Psugar=true -PlabSecret=$(LabVaultAppCert)
-          javaHomeSelection: JDKVersion
-          jdkVersionOption: "1.17"
 ...
diff --git a/azure-pipelines/templates/steps/continuous-delivery/assemble-publish-projversion.yml b/azure-pipelines/templates/steps/continuous-delivery/assemble-publish-projversion.yml
@@ -37,19 +37,19 @@ steps:
   displayName: Assemble ${{ parameters.project }} ${{ parameters.variant }}
   inputs:
     javaHomeSelection: JDKVersion
-    jdkVersionOption: "1.17"
+    jdkVersionOption: "1.11"
     tasks: '${{ parameters.project }}:clean ${{ parameters.project }}:assemble${{ parameters.variant }} ${{ parameters.buildArguments }}'
 - task: Gradle@2
   displayName: Publish
   inputs:
     javaHomeSelection: JDKVersion
-    jdkVersionOption: "1.17"
+    jdkVersionOption: "1.11"
     tasks: '${{ parameters.project }}:publish${{ parameters.variant }} ${{ parameters.buildArguments }}'
 - task: Gradle@2
   displayName: Run unit tests
   inputs:
     javaHomeSelection: JDKVersion
-    jdkVersionOption: "1.17"
+    jdkVersionOption: "1.11"
     tasks: '${{ parameters.project }}:test${{ parameters.variant }} -PlabSecret=$(LabVaultAppCert)'
 - ${{ if ne(parameters.artifactFolder, '') }}:
   - task: CopyFiles@2
diff --git a/azure-pipelines/templates/steps/spotbugs.yml b/azure-pipelines/templates/steps/spotbugs.yml
@@ -10,14 +10,13 @@ parameters:
   default: CodeAnalysisLogs
 - name: cwd
   default: $(Build.SourcesDirectory)
-- name: javaVersion
-  default: "1.17"
+
 steps:
 - task: Gradle@3
   displayName: 'Run Spotbugs'
   inputs:
     javaHomeSelection: JDKVersion
-    jdkVersionOption: ${{parameters.javaVersion}}
+    jdkVersionOption: "1.11"
     jdkArchitecture: x64
     cwd: ${{ parameters.cwd }}
     tasks: ${{ parameters.project }}:${{ parameters.spotbugsCommand }}
diff --git a/changelog.txt b/changelog.txt
@@ -23,7 +23,6 @@ Version 21.0.0
 - [PATCH] Fix multiple prompts issue in cross cloud request (#2599)
 - [PATCH] Corrected error handling in cross cloud scenario (#2602)
 - [MINOR] Native auth: Add claimsRequest also to getAccessToken and signIn after signUp/SSPR flows (#2622)
-- [MINOR] Upgrading AGP Version to 8.1.1 (#2587)
 
 Version 20.1.1
 ----------
diff --git a/common-java-root/gradle.properties b/common-java-root/gradle.properties
@@ -10,8 +10,7 @@ org.gradle.daemon=true
 
 # See https://stackoverflow.com/questions/56075455/expiring-daemon-because-jvm-heap-space-is-exhausted
 # we must make sure that the total size is <7G, as that's the RAM size of VM on the build pipeline.
-# MaxMetaspaceSize replaced MaxPermSize as of java 8+.https://www.oracle.com/java/technologies/javase/8-whats-new.html
-org.gradle.jvmargs=-Xmx2048m -XX:MaxMetaspaceSize=512m -XX:+HeapDumpOnOutOfMemoryError
+org.gradle.jvmargs=-Xmx2048m -XX:MaxPermSize=512m -XX:+HeapDumpOnOutOfMemoryError
 
 # This the TSL versionName...
 versionName=1.5.9
diff --git a/common-java-root/gradle/wrapper/gradle-wrapper.properties b/common-java-root/gradle/wrapper/gradle-wrapper.properties
@@ -1,6 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.1-all.zip
-
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
diff --git a/common/build.gradle b/common/build.gradle
@@ -1,5 +1,5 @@
 plugins {
-    id 'com.microsoft.identity.buildsystem' version '0.2.5'
+    id 'com.microsoft.identity.buildsystem' version '0.2.3'
     id 'com.android.library'
     id 'pmd'
     id 'checkstyle'
@@ -61,11 +61,7 @@ android {
         buildConfigField("boolean", "trustDebugBrokerFlag", "$trustDebugBrokerFlag")
         buildConfigField("boolean", "bypassRedirectUriCheck", "$bypassRedirectUriCheck")
     }
-    //aidl is being set to true since aidl only generated with this set
-    buildFeatures {
-        buildConfig = true
-        aidl = true
-    }
+
     buildTypes {
         debug {
             testCoverageEnabled enableCodeCoverage
@@ -122,7 +118,6 @@ android {
     sourceSets {
         main {
             manifest.srcFile 'src/main/AndroidManifest.xml'
-            aidl.srcDirs = ['src/main/aidl']
 
             // Also load source from common-java
             java.srcDirs = ['src/main/java', 'src/main/kotlin']
@@ -245,15 +240,15 @@ android.libraryVariants.all { variant ->
 // Create tasks to generate javadoc jar
 android.libraryVariants.all { variant ->
     task("${variant.name}JavadocJar", type: Jar, dependsOn: "${variant.name}Javadoc") {
-        archiveClassifier.set('javadoc')
+        classifier 'javadoc'
         from tasks["${variant.name}Javadoc"].destinationDir
     }
 }
 
 
 tasks.register('sourcesJar', Jar) {
     from android.sourceSets.main.java.srcDirs
-    archiveClassifier.set('sources')
+    classifier 'sources'
     destinationDirectory = reporting.file("$project.buildDir/outputs/jar/")
 }
 
@@ -289,10 +284,11 @@ tasks.register('pmd', Pmd) {
     source = fileTree('src/main/java')
 
     reports {
-        xml.required = false
-        html.required = true
+        xml.enabled = false
+        html.enabled = true
     }
 }
+
 // For publishing to the remote maven repo.
 afterEvaluate {
 
@@ -339,7 +335,6 @@ afterEvaluate {
                 groupId 'com.microsoft.identity'
                 artifactId 'common-debug'
                 //Edit the 'version' here for VSTS RC build
-
                 version = project.version
             }
         }
@@ -369,34 +364,11 @@ tasks.configureEach { task ->
     if (task.name == 'assembleDistRelease') {
         task.dependsOn 'sourcesJar'
     }
-    //Due to gradle version increase, we need to be more explicit in the dependency chain
-    // More information https://discuss.gradle.org/t/g./gradle-8-project-dependency-requirement-clarification/46336
-    if (task.name == "publishDistDebugPublicationToVsts-maven-adal-androidRepository") {
-        task.dependsOn 'generatePomFileForDistReleasePublication'
-    }
-    if (task.name == "publishDistReleasePublicationToVsts-maven-adal-androidRepository") {
-        task.dependsOn 'bundleDistReleaseAar'
-    }
-    if (task.name == "publishDistDebugPublicationToVsts-maven-adal-androidRepository") {
-        task.dependsOn 'bundleDistReleaseAar'
-    }
-    if (task.name == "generateMetadataFileForDistDebugPublication") {
-        task.dependsOn 'bundleDistReleaseAar'
-    }
-    if (task.name == "generateMetadataFileForDistReleasePublication") {
-        task.dependsOn 'bundleDistDebugAar'
-    }
 }
 
-// This is used to generate the pom file for publishing to external maven in maven-release-jobs.yml.
-// Need to distinguish between distDebug and distRelease for the distination
+// This is used to generate the pom file for publishing to external maven in maven-release-jobs.yml
 tasks.withType(GenerateMavenPom).all {
-    def functionName = name.toLowerCase()
-    if (functionName.contains("distdebug")) {
-        destination = layout.buildDirectory.file("poms/common-debug-${project.version}.pom").get().asFile
-    } else if (functionName.contains("distrelease")) {
-        destination = layout.buildDirectory.file("poms/common-${project.version}.pom").get().asFile
-    }
+    destination = layout.buildDirectory.file("poms/${project.name}-${project.version}.pom").get().asFile
 }
 
 def dependenciesSizeInMb = project.hasProperty("dependenciesSizeMb") ? project.dependenciesSizeMb : "15"
diff --git a/common4j/build.gradle b/common4j/build.gradle
@@ -8,7 +8,7 @@
 
 plugins {
     id 'java-library'
-    id 'com.microsoft.identity.buildsystem' version '0.2.5'
+    id 'com.microsoft.identity.buildsystem' version '0.2.3'
     id 'maven-publish'
 
     // the below build plugin allows to generate BuildConfig files in a JAVA gradle project.
@@ -61,15 +61,13 @@ project.ext.vstsPassword = System.getenv("ENV_VSTS_MVN_ANDROIDCOMMON_ACCESSTOKEN
 version = getAppVersionName()
 
 task sourcesJar(type: Jar) {
-    dependsOn generateBuildConfig
     from sourceSets.main.java.srcDirs
-    archiveClassifier.set('sources')
+    classifier 'sources'
     destinationDirectory = reporting.file("$project.buildDir/outputs/jars")
 }
 
 // Task to generate javadoc
 task generateJavadoc(type: Javadoc) {
-    dependsOn generateBuildConfig
     failOnError false
     title = "Microsoft Identity Common Java"
     source = sourceSets.main.java
@@ -86,7 +84,7 @@ task generateJavadoc(type: Javadoc) {
 // Task to generate javadoc.jar
 task javadocJar(type: Jar, dependsOn: generateJavadoc) {
     from javadoc.destinationDir
-    archiveClassifier.set('javadoc')
+    classifier 'javadoc'
     destinationDirectory = reporting.file("$project.buildDir/outputs/jars")
 }
 
diff --git a/common4j/gradle/wrapper/gradle-wrapper.properties b/common4j/gradle/wrapper/gradle-wrapper.properties
@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.1-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
diff --git a/gradle.properties b/gradle.properties
@@ -9,8 +9,7 @@ org.gradle.daemon=true
 
 # See https://stackoverflow.com/questions/56075455/expiring-daemon-because-jvm-heap-space-is-exhausted
 # we must make sure that the total size is <7G, as that's the RAM size of VM on the build pipeline.
-# MaxMetaspaceSize replaced MaxPermSize as of java 8+.https://www.oracle.com/java/technologies/javase/8-whats-new.html
-org.gradle.jvmargs=-Xmx3072m -XX:MaxMetaspaceSize=1024m -XX:+HeapDumpOnOutOfMemoryError
+org.gradle.jvmargs=-Xmx3072m -XX:MaxPermSize=1024m -XX:+HeapDumpOnOutOfMemoryError
 
 # This the TSL versionName...
 versionName=1.5.9
diff --git a/gradle/versions.gradle b/gradle/versions.gradle
@@ -10,7 +10,7 @@ ext {
     buildToolsVersion = "28.0.3"
 
     // Plugins
-    gradleVersion = '8.1.1'
+    gradleVersion = '7.4.2'
     kotlinVersion = '1.7.21'
     spotBugsGradlePluginVersion = '4.7.1'
     jupiterApiVersion = '5.6.0'
@@ -47,7 +47,7 @@ ext {
     uiAutomatorVersion = "2.2.0"
     daggerVersion = "2.31.2"
     daggerCompilerVersion = "2.31.2"
-    msal4jVersion = "1.20.1"
+    msal4jVersion = "1.10.0"
     moshiVersion = "1.14.0"
     moshiAdaptersVersion = "1.14.0"
     openpojoVersion = "0.9.1"
diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.1-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-bin.zip
diff --git a/testutils/build.gradle b/testutils/build.gradle
diff --git a/testutils/src/main/AndroidManifest.xml b/testutils/src/main/AndroidManifest.xml
diff --git a/uiautomationutilities/build.gradle b/uiautomationutilities/build.gradle
diff --git a/uiautomationutilities/src/main/AndroidManifest.xml b/uiautomationutilities/src/main/AndroidManifest.xml
