search

zulip / zulip-mobile

Zulip mobile apps for Android and iOS.
https://zulip.com/apps/
Apache License 2.0
1.3k stars 656 forks source link

android build: Set strict version 0.10.4 for RN's dependency SoLoader #5902

Closed chrisbobbe closed 1 month ago

chrisbobbe commented 1 month ago

Fixes: #5901

chrisbobbe commented 1 month ago

After this change:

$ tools/gradle :app:dependencies | grep com.facebook.soloader:soloader
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (c)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4
|    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
|    |    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (*)
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (c)
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4
|    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
|    |    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (*)
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:0.10.4 (n)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (c)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|         |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (*)
|    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (c)
|         |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4
+--- com.facebook.soloader:soloader:{strictly 0.10.4} -> 0.10.4 (*)
|    |    |    \--- com.facebook.soloader:soloader:0.10.1 -> 0.10.4 (*)
|    +--- com.facebook.soloader:soloader:0.10.3 -> 0.10.4 (*)

And a quick manual smoke test of the app on an emulator (API level 35) showed no issues.

gnprice commented 1 month ago

Thanks for taking care of this! Looks good; merging.