Skip to content

Build perf: publish broker libraries directly to NewAndroid + decouple common4j sources/javadoc from assemble #3113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
fadidurah wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Build perf: publish broker libraries directly to NewAndroid + decouple common4j sources/javadoc from assemble #3113

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d98d3e1
Build: publish libraries directly to NewAndroid feed
fadidurah May 14, 2026
baae279
common4j: don't tie sourcesJar/javadocJar to assemble
fadidurah May 14, 2026
46c3b31
Add gradle configure-on-demand for build perf
fadidurah May 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion LabApiUtilities/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ publishing {
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down
2 changes: 1 addition & 1 deletion common/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -420,7 +420,7 @@ afterEvaluate {
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
Comment thread
fadidurah marked this conversation as resolved.
credentials {
username project.vstsUsername
password project.vstsMavenAccessToken
Expand Down
9 changes: 6 additions & 3 deletions common4j/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ publishing {
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down Expand Up @@ -339,8 +339,11 @@ buildConfig {
}

afterEvaluate {
assemble.dependsOn sourcesJar, javadocJar

// sourcesJar and javadocJar are wired into the publish graph via components.java + maven-publish,
// so they will run automatically as part of `publish`. There is no need to also tie them to
// `assemble`, which made every CI build (including test-only / spotbugs-only flows) generate
// javadoc unnecessarily. Generating javadoc for common4j is one of the slower tasks in the build.
// Local devs who explicitly want the artifacts can still run `./gradlew sourcesJar javadocJar`.
// these tasks are generated by the buildconfig plugin..for more details, read comment about it
// above in the plugins block.
compileJava.dependsOn generateBuildConfig, generateTestBuildConfig
Expand Down
1 change: 1 addition & 0 deletions gradle.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ android.useAndroidX=true
# https://office.visualstudio.com/Outlook%20Mobile/_wiki/wikis/Outlook-Mobile.wiki/3780/Android-Studio-Gradle-Performance-tips-and-tricks
org.gradle.parallel=true
org.gradle.daemon=true
org.gradle.configureondemand=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.
Expand Down
2 changes: 1 addition & 1 deletion keyvault/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ publishing {
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down
2 changes: 1 addition & 1 deletion labapi/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ publishing {
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down
2 changes: 1 addition & 1 deletion testutils/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ project.afterEvaluate{
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down
2 changes: 1 addition & 1 deletion uiautomationutilities/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ project.afterEvaluate{
repositories {
maven {
name "vsts-maven-adal-android"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/AndroidADAL/maven/v1"
url "https://identitydivision.pkgs.visualstudio.com/_packaging/NewAndroid/maven/v1"
credentials {
username project.ext.vstsUsername
password project.ext.vstsMavenAccessToken
Expand Down
Loading