Skip to content

Conversation

@SimonMarquis
Copy link
Contributor

@SimonMarquis SimonMarquis commented Sep 14, 2025

This neat website can be used to track the current issues with AGP 9: https://agp-status.frybits.com/agp-9.0.0/

⚠️ TODO before merging PR

  • Hilt Gradle Plugin does not work with AGP 9.0.0-alpha04 google/dagger#4944
    Temporary workaround with android.newDsl=false
  • androidx.baselineprofile issue, flagged as error starting with AGP beta05:
    A problem occurred configuring project ':app'.
    > Using kotlin.sourceSets DSL to add Kotlin sources is not allowed with built-in Kotlin.
      Kotlin source set 'demoBenchmarkRelease' contains: [/home/runner/work/nowinandroid/nowinandroid/app/build/generated/ksp/demoBenchmarkRelease/kotlin, /home/runner/work/nowinandroid/nowinandroid/app/build/generated/ksp/demoBenchmarkRelease/java]
      Solution: Use android.sourceSets DSL instead.
      For more information, see https://developer.android.com/r/tools/built-in-kotlin
      To suppress this error, set android.disallowKotlinSourceSets=false in gradle.properties.
    

✅ Fixed issues

@SimonMarquis SimonMarquis marked this pull request as draft September 14, 2025 12:35
@SimonMarquis SimonMarquis force-pushed the agp-9 branch 2 times, most recently from d2767d1 to a19fcd9 Compare September 21, 2025 08:45
`error: cannot find symbol` on:

- `DataStoreModule.INSTANCE.providesUserPreferencesDataStore$datastore_demoDebug`
- `SyncModule.Companion.provideFirebaseMessaging$work_prodDebug()`
They must have been removed as they are no longer resolved by default.
Updated Android Gradle Plugin and Tools versions.
@liutikas
Copy link
Collaborator

liutikas commented Nov 6, 2025

release build type has isDebuggable forced to true, even when explicitly turned off.

Can you point me to how to repro this?

@SimonMarquis
Copy link
Contributor Author

I've not been able to reproduce it on my other projects, but on this one, simply build a release app with gradlew :app:assembleDemoRelease.

not sure if they were dropped on purpose...
@SimonMarquis SimonMarquis force-pushed the agp-9 branch 2 times, most recently from 101662a to cf6e7dd Compare November 7, 2025 20:54
…buildTypes

leading to "debuggable" release builds.
@SimonMarquis
Copy link
Contributor Author

I found the issue, fixed in 52b118a.
enableAndroidTestCoverage forces the app to be debuggable, probably because of a new logic internal to AGP.

I've refactored the code to configure only the debug buildTypes:
https://github.com/SimonMarquis/nowinandroid/blob/52b118a1e073c07c4b7843e3ca93ea70333e41a0/build-logic/convention/src/main/kotlin/com/google/samples/apps/nowinandroid/Jacoco.kt#L68-L71

Add comment to clarify Jacoco configuration for debug builds.
@liutikas liutikas changed the title 🚧 AGP 9 AGP 9 Dec 19, 2025
@liutikas
Copy link
Collaborator

Can you please rebase the merge conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants