Merge pull request #333 from android/wk/agp8-changes

Migrate packages to namespaces in build files

app-nia-catalog/build.gradle.kts

@@ -32,6 +32,7 @@ android {
             excludes.add("/META-INF/{AL2.0,LGPL2.1}")
         }
     }
+    namespace = "com.google.samples.apps.niacatalog"
 }
 
 dependencies {

app-nia-catalog/src/main/AndroidManifest.xml

@@ -14,8 +14,7 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.niacatalog">
 
     <application
         android:allowBackup="true"

app/build.gradle.kts

@@ -13,8 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import com.google.samples.apps.nowinandroid.Flavor
-import com.google.samples.apps.nowinandroid.FlavorDimension
 
 plugins {
     id("nowinandroid.android.application")
@@ -75,6 +73,7 @@ android {
             isIncludeAndroidResources = true
         }
     }
+    namespace = "com.google.samples.apps.nowinandroid"
 }
 
 dependencies {

app/src/main/AndroidManifest.xml

@@ -15,8 +15,7 @@
      limitations under the License.
 -->
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:tools="http://schemas.android.com/tools"
+    xmlns:tools="http://schemas.android.com/tools">
-    package="com.google.samples.apps.nowinandroid">
 
     <uses-permission android:name="android.permission.INTERNET" />
 

core/common/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.common"
+}
+
 dependencies {
     implementation(libs.kotlinx.coroutines.android)
     testImplementation(project(":core:testing"))

core/common/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.common">
 
 </manifest>

core/data-test/build.gradle.kts

@@ -18,6 +18,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.data.test"
+}
+
 dependencies {
     api(project(":core:data"))
     implementation(project(":core:testing"))

core/data-test/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.data.test">
 
 </manifest>

core/data/build.gradle.kts

@@ -20,6 +20,10 @@ plugins {
     id("kotlinx-serialization")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.data"
+}
+
 dependencies {
     implementation(project(":core:common"))
     implementation(project(":core:model"))
@@ -35,4 +39,4 @@ dependencies {
     implementation(libs.kotlinx.datetime)
     implementation(libs.kotlinx.coroutines.android)
     implementation(libs.kotlinx.serialization.json)
-}
+}

core/data/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.data">
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
 </manifest>

core/database/build.gradle.kts

@@ -33,6 +33,7 @@ android {
 
         testInstrumentationRunner = "com.google.samples.apps.nowinandroid.core.testing.NiaTestRunner"
     }
+    namespace = "com.google.samples.apps.nowinandroid.core.database"
 }
 
 dependencies {

core/database/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.database">
 
 </manifest>

core/datastore-test/build.gradle.kts

@@ -18,6 +18,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.datastore.test"
+}
+
 dependencies {
     api(project(":core:datastore"))
     implementation(project(":core:testing"))

core/datastore-test/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.datastore.test">
 
 </manifest>

core/datastore/build.gradle.kts

@@ -32,6 +32,7 @@ android {
     defaultConfig {
         consumerProguardFiles("consumer-proguard-rules.pro")
     }
+    namespace = "com.google.samples.apps.nowinandroid.core.datastore"
 }
 
 // Setup protobuf configuration, generating lite Java and Kotlin classes

core/datastore/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.datastore">
 
 </manifest>

core/designsystem/build.gradle.kts

@@ -26,6 +26,7 @@ android {
     lint {
         checkDependencies = true
     }
+    namespace = "com.google.samples.apps.nowinandroid.core.designsystem"
 }
 
 dependencies {

core/designsystem/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.designsystem">
 
 </manifest>

core/navigation/build.gradle.kts

@@ -21,6 +21,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.navigation"
+}
+
 dependencies {
     api(libs.androidx.hilt.navigation.compose)
     api(libs.androidx.navigation.compose)

core/navigation/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.navigation">
 
 </manifest>

core/network/build.gradle.kts

@@ -26,6 +26,7 @@ android {
     buildFeatures {
         buildConfig = true
     }
+    namespace = "com.google.samples.apps.nowinandroid.core.network"
 }
 
 secrets {

core/network/src/main/AndroidManifest.xml

@@ -14,8 +14,7 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.network">
     <uses-permission android:name="android.permission.INTERNET" />
 
 </manifest>

core/testing/build.gradle.kts

@@ -18,6 +18,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.testing"
+}
+
 dependencies {
     implementation(project(":core:common"))
     implementation(project(":core:data"))

core/testing/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.testing">
 
 </manifest>

core/ui/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.library.jacoco")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.ui"
+}
+
 dependencies {
     implementation(project(":core:designsystem"))
     implementation(project(":core:model"))

core/ui/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.ui">
 
 </manifest>

feature/author/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.library.jacoco")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.feature.author"
+}
+
 dependencies {
     implementation(libs.kotlinx.datetime)
 }

feature/author/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.feature.author">
 
 </manifest>

feature/bookmarks/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.library.jacoco")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.feature.bookmarks"
+}
+
 dependencies {
     implementation(libs.androidx.compose.material3.windowSizeClass)
 }

feature/bookmarks/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.feature.bookmarks">
 
 </manifest>

feature/foryou/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.library.jacoco")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.feature.foryou"
+}
+
 dependencies {
     implementation(libs.kotlinx.datetime)
 

feature/foryou/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.feature.foryou">
 
 </manifest>

feature/interests/build.gradle.kts

@@ -18,3 +18,6 @@ plugins {
     id("nowinandroid.android.library.compose")
     id("nowinandroid.android.library.jacoco")
 }
+android {
+    namespace = "com.google.samples.apps.nowinandroid.feature.interests"
+}

feature/interests/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.feature.interests">
 
 </manifest>

feature/topic/build.gradle.kts

@@ -19,6 +19,10 @@ plugins {
     id("nowinandroid.android.library.jacoco")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.feature.topic"
+}
+
 dependencies {
     implementation(libs.kotlinx.datetime)
 }

feature/topic/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.feature.topic">
 
 </manifest>

sync/sync-test/build.gradle.kts

@@ -18,6 +18,10 @@ plugins {
     id("nowinandroid.android.hilt")
 }
 
+android {
+    namespace = "com.google.samples.apps.nowinandroid.core.sync.test"
+}
+
 dependencies {
     api(project(":sync:work"))
     implementation(project(":core:data"))

sync/sync-test/src/main/AndroidManifest.xml

@@ -14,7 +14,6 @@
      See the License for the specific language governing permissions and
      limitations under the License.
 -->
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    package="com.google.samples.apps.nowinandroid.core.sync.test">
 
 </manifest>

sync/work/build.gradle.kts

@@ -23,6 +23,7 @@ android {
     defaultConfig {
         testInstrumentationRunner = "com.google.samples.apps.nowinandroid.core.testing.NiaTestRunner"
     }
+    namespace = "com.google.samples.apps.nowinandroid.sync"
 }
 
 dependencies {

sync/work/src/main/AndroidManifest.xml

@@ -15,8 +15,7 @@
      limitations under the License.
 -->
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:tools="http://schemas.android.com/tools"
+    xmlns:tools="http://schemas.android.com/tools">
-    package="com.google.samples.apps.nowinandroid.sync">
 
     <application>
         <provider