Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Commit

Permalink
Remove Multidex usages.
Browse files Browse the repository at this point in the history
Since the min SDK is at least 21, it is no longer necessary to use the Multidex library.

See the following for more info: https://developer.android.com/build/multidex#mdex-on-l

Resolves #13696
  • Loading branch information
MGaetan89 authored and greyson-signal committed Sep 20, 2024
1 parent a1bf4d6 commit b340097
Show file tree
Hide file tree
Showing 8 changed files with 2 additions and 22 deletions.
4 changes: 0 additions & 4 deletions app/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -178,8 +178,6 @@ android {
minSdk = signalMinSdkVersion
targetSdk = signalTargetSdkVersion

multiDexEnabled = true

vectorDrawables.useSupportLibrary = true
project.ext.set("archivesBaseName", "Signal")

Expand Down Expand Up @@ -504,7 +502,6 @@ dependencies {
implementation(libs.androidx.compose.runtime.livedata)
implementation(libs.androidx.activity.compose)
implementation(libs.androidx.constraintlayout)
implementation(libs.androidx.multidex)
implementation(libs.androidx.navigation.fragment.ktx)
implementation(libs.androidx.navigation.ui.ktx)
implementation(libs.androidx.navigation.compose)
Expand Down Expand Up @@ -598,7 +595,6 @@ dependencies {
testImplementation(testLibs.robolectric.robolectric) {
exclude(group = "com.google.protobuf", module = "protobuf-java")
}
testImplementation(testLibs.robolectric.shadows.multidex)
testImplementation(testLibs.bouncycastle.bcprov.jdk15on) {
version {
strictly("1.70")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,13 @@
*/
package org.thoughtcrime.securesms;

import android.app.Application;
import android.content.Context;

import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.annotation.VisibleForTesting;
import androidx.annotation.WorkerThread;
import androidx.multidex.MultiDexApplication;

import com.bumptech.glide.Glide;
import com.google.android.gms.security.ProviderInstaller;
Expand Down Expand Up @@ -129,7 +129,7 @@
*
* @author Moxie Marlinspike
*/
public class ApplicationContext extends MultiDexApplication implements AppForegroundObserver.Listener {
public class ApplicationContext extends Application implements AppForegroundObserver.Listener {

private static final String TAG = Log.tag(ApplicationContext.class);

Expand Down
1 change: 0 additions & 1 deletion app/src/main/res/raw/third_party_licenses
Original file line number Diff line number Diff line change
Expand Up @@ -356,7 +356,6 @@ The following dependencies are licensed under The MIT License:
* mockito-android (https://github.com/mockito/mockito)
* mockito-core (https://github.com/mockito/mockito)
* mockito-inline (https://github.com/mockito/mockito)
* multidex (http://robolectric.org)
* nativeruntime (http://robolectric.org)
* pluginapi (http://robolectric.org)
* reflector (http://robolectric.org)
Expand Down
2 changes: 0 additions & 2 deletions build-logic/plugins/src/main/java/signal-library.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ android {
defaultConfig {
minSdk = signalMinSdkVersion
targetSdk = signalTargetSdkVersion
multiDexEnabled = true
}

compileOptions {
Expand Down Expand Up @@ -58,7 +57,6 @@ dependencies {
implementation(libs.rxjava3.rxandroid)
implementation(libs.rxjava3.rxjava)
implementation(libs.rxjava3.rxkotlin)
implementation(libs.androidx.multidex)
implementation(libs.kotlin.stdlib.jdk8)

ktlintRuleset(libs.ktlint.twitter.compose)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ android {

minSdk = signalMinSdkVersion
targetSdk = signalTargetSdkVersion
multiDexEnabled = true
}

compileOptions {
Expand Down Expand Up @@ -69,7 +68,6 @@ dependencies {
implementation(libs.rxjava3.rxandroid)
implementation(libs.rxjava3.rxjava)
implementation(libs.rxjava3.rxkotlin)
implementation(libs.androidx.multidex)
implementation(libs.material.material)
implementation(libs.androidx.constraintlayout)
implementation(libs.kotlin.stdlib.jdk8)
Expand Down
2 changes: 0 additions & 2 deletions dependencies.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ dependencyResolutionManagement {
library("androidx-media3-ui", "androidx.media3", "media3-ui").versionRef("androidx-media3")
library("androidx-media3-decoder", "androidx.media3", "media3-decoder").versionRef("androidx-media3")
library("androidx-media3-common", "androidx.media3", "media3-common").versionRef("androidx-media3")
library("androidx-multidex", "androidx.multidex:multidex:2.0.1")
library("androidx-navigation-fragment-ktx", "androidx.navigation", "navigation-fragment-ktx").versionRef("androidx-navigation")
library("androidx-navigation-ui-ktx", "androidx.navigation", "navigation-ui-ktx").versionRef("androidx-navigation")
library("androidx-navigation-compose", "androidx.navigation", "navigation-compose").versionRef("androidx-navigation")
Expand Down Expand Up @@ -203,7 +202,6 @@ dependencyResolutionManagement {
library("mockito-kotlin", "org.mockito.kotlin:mockito-kotlin:4.0.0")
library("mockito-android", "org.mockito:mockito-android:4.6.1")
library("robolectric-robolectric", "org.robolectric", "robolectric").versionRef("robolectric")
library("robolectric-shadows-multidex", "org.robolectric", "shadows-multidex").versionRef("robolectric")
library("bouncycastle-bcprov-jdk15on", "org.bouncycastle:bcprov-jdk15on:1.70")
library("bouncycastle-bcpkix-jdk15on", "org.bouncycastle:bcpkix-jdk15on:1.70")
library("hamcrest-hamcrest", "org.hamcrest:hamcrest:2.2")
Expand Down
1 change: 0 additions & 1 deletion device-transfer/lib/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ dependencies {
testImplementation(testLibs.robolectric.robolectric) {
exclude(group = "com.google.protobuf", module = "protobuf-java")
}
testImplementation(testLibs.robolectric.shadows.multidex)
testImplementation(testLibs.hamcrest.hamcrest)

testImplementation(testFixtures(project(":libsignal-service")))
Expand Down
8 changes: 0 additions & 8 deletions gradle/verification-metadata.xml
Original file line number Diff line number Diff line change
Expand Up @@ -8979,14 +8979,6 @@ https://docs.gradle.org/current/userguide/dependency_verification.html
<sha256 value="126f485b5f1570021ab79a2b819bf44a2f64405bd248f7e819236f13728b9ce7" origin="Generated by Gradle"/>
</artifact>
</component>
<component group="org.robolectric" name="shadows-multidex" version="4.10.3">
<artifact name="shadows-multidex-4.10.3.jar">
<sha256 value="34d1da27044528c07ca43c6334af3890868f2570d48114459c25ec4331ea6966" origin="Generated by Gradle"/>
</artifact>
<artifact name="shadows-multidex-4.10.3.module">
<sha256 value="ad1276a3820bbe37672624ee412b14bf35660604fb4802ac9b2ce8de494b72ff" origin="Generated by Gradle"/>
</artifact>
</component>
<component group="org.robolectric" name="utils" version="4.10.3">
<artifact name="utils-4.10.3.jar">
<sha256 value="0081b1a65c2c6d7cf56a56f6b4ed85b35a91f5a9f40a4b81c6771b497265518e" origin="Generated by Gradle"/>
Expand Down

0 comments on commit b340097

Please sign in to comment.