Skip to content

Commit

Permalink
deps: Upgrade react-native-unimodules to ^0.9.1, the latest.
Browse files Browse the repository at this point in the history
This is possible, now that the RN v0.60 -> v0.61 upgrade is
complete! [1]

Change the `@unimodules/core` version, following a valid instruction
in bc27f2d:

"""
So, declare @unimodules/core in our own "dependencies", with the
version specified by react-native-unimodules in its active version.
This matching should be done each time we change versions of
react-native-unimodules.
"""

Also, remove some now-unnecessary code in `android/build.gradle`, as
foreshadowed in cb87f90:

"""
Second, specify a new dependency for
`unimodules-react-native-adapter` in our own `android/build.gradle`,
in a fix that is necessary because we're locked on version 0.6.0 of
react-native-unimodules. Filed as
https://github.com/unimodules/react-native-unimodules/issues/130.
"""

There are some reasonable, automatic changes to the
version-controlled files that ensure that unimodules are
automatically linked [2]: `ios/Podfile.lock` and
`android/app/src/main/java/com/zulipmobile/generated/BasePackageList.java`.
These look like some modules that are part of
`react-native-unimodules`' core; these additions are caused by the
upgrade and don't stem from running something like `yarn add
expo-apple-authentication` to get a particular Unimodule.

Also, run `yarn yarn-deduplicate && yarn` as prompted by
`tools/test deps`.

[1]: See discussion of the various obstacles at
     https://github.com/unimodules/react-native-unimodules/issues/97#issuecomment-637714639
     and
     https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/Podfile.20dependency.20of.20a.20dependency/near/892373.
[2]: This is similar to but distinct from `react-native`'s
     "autolinking" feature that we activated in f460460 and
     a9a9ac7. See
     https://github.com/unimodules/react-native-unimodules/issues/75#issuecomment-536517253
     for a comment on that.

Fixes: #4091
  • Loading branch information
chrisbobbe committed Jun 11, 2020
1 parent 411d553 commit a5bb3f0
Show file tree
Hide file tree
Showing 5 changed files with 194 additions and 182 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ public List<Package> getPackageList() {
new expo.modules.application.ApplicationPackage(),
new expo.modules.constants.ConstantsPackage(),
new expo.modules.filesystem.FileSystemPackage(),
new expo.modules.imageloader.ImageLoaderPackage(),
new expo.modules.permissions.PermissionsPackage(),
new expo.modules.screenorientation.ScreenOrientationPackage()
);
Expand Down
6 changes: 0 additions & 6 deletions android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -57,12 +57,6 @@ project(":react-native-device-info").beforeEvaluate {
ext.googlePlayServicesVersion = "16.1.0"
}

project(":unimodules-react-native-adapter").afterEvaluate {
it.dependencies {
implementation 'com.facebook.react:react-native:+'
}
}

subprojects {
afterEvaluate {
if (project.hasProperty("android")) {
Expand Down
133 changes: 62 additions & 71 deletions ios/Podfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,17 @@ PODS:
- DoubleConversion (1.1.6)
- EXApplication (2.1.1):
- UMCore
- EXAppLoaderProvider (7.0.0)
- EXConstants (7.0.1):
- EXConstants (9.0.0):
- UMConstantsInterface
- UMCore
- EXFileSystem (7.0.0):
- EXFileSystem (8.1.0):
- UMCore
- UMFileSystemInterface
- EXPermissions (7.0.0):
- EXImageLoader (1.0.1):
- React-Core
- UMCore
- UMImageLoaderInterface
- EXPermissions (8.1.0):
- UMCore
- UMPermissionsInterface
- EXScreenOrientation (1.0.0):
Expand Down Expand Up @@ -281,29 +284,31 @@ PODS:
- Sentry/Core (= 4.4.3)
- Sentry/Core (4.4.3)
- Toast (4.0.0)
- UMBarCodeScannerInterface (4.0.0)
- UMCameraInterface (4.0.0)
- UMConstantsInterface (4.0.0)
- UMCore (4.0.0)
- UMFaceDetectorInterface (4.0.0)
- UMFileSystemInterface (4.0.0)
- UMFontInterface (4.0.0)
- UMImageLoaderInterface (4.0.0)
- UMPermissionsInterface (4.0.0)
- UMReactNativeAdapter (4.0.0):
- React
- UMAppLoader (1.0.2)
- UMBarCodeScannerInterface (5.1.0)
- UMCameraInterface (5.1.0)
- UMConstantsInterface (5.1.0)
- UMCore (5.1.2)
- UMFaceDetectorInterface (5.1.0)
- UMFileSystemInterface (5.1.0)
- UMFontInterface (5.1.0)
- UMImageLoaderInterface (5.1.0)
- UMPermissionsInterface (5.1.0):
- UMCore
- UMReactNativeAdapter (5.2.0):
- React-Core
- UMCore
- UMFontInterface
- UMSensorsInterface (4.0.0)
- UMTaskManagerInterface (4.0.0)
- UMSensorsInterface (5.1.0)
- UMTaskManagerInterface (5.1.0)
- Yoga (1.14.0)

DEPENDENCIES:
- DoubleConversion (from `../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`)
- EXApplication (from `../node_modules/expo-application/ios`)
- EXAppLoaderProvider (from `../node_modules/expo-app-loader-provider/ios`)
- EXConstants (from `../node_modules/expo-constants/ios`)
- EXFileSystem (from `../node_modules/expo-file-system/ios`)
- EXImageLoader (from `../node_modules/expo-image-loader/ios`)
- EXPermissions (from `../node_modules/expo-permissions/ios`)
- EXScreenOrientation (from `../node_modules/expo-screen-orientation/ios`)
- FBLazyVector (from `../node_modules/react-native/Libraries/FBLazyVector`)
Expand Down Expand Up @@ -348,6 +353,7 @@ DEPENDENCIES:
- "RNSentry (from `../node_modules/@sentry/react-native`)"
- RNSound (from `../node_modules/react-native-sound`)
- RNVectorIcons (from `../node_modules/react-native-vector-icons`)
- UMAppLoader (from `../node_modules/unimodules-app-loader/ios`)
- UMBarCodeScannerInterface (from `../node_modules/unimodules-barcode-scanner-interface/ios`)
- UMCameraInterface (from `../node_modules/unimodules-camera-interface/ios`)
- UMConstantsInterface (from `../node_modules/unimodules-constants-interface/ios`)
Expand All @@ -372,23 +378,17 @@ EXTERNAL SOURCES:
DoubleConversion:
:podspec: "../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec"
EXApplication:
:path: !ruby/object:Pathname
path: "../node_modules/expo-application/ios"
EXAppLoaderProvider:
:path: !ruby/object:Pathname
path: "../node_modules/expo-app-loader-provider/ios"
:path: "../node_modules/expo-application/ios"
EXConstants:
:path: !ruby/object:Pathname
path: "../node_modules/expo-constants/ios"
:path: "../node_modules/expo-constants/ios"
EXFileSystem:
:path: !ruby/object:Pathname
path: "../node_modules/expo-file-system/ios"
:path: "../node_modules/expo-file-system/ios"
EXImageLoader:
:path: "../node_modules/expo-image-loader/ios"
EXPermissions:
:path: !ruby/object:Pathname
path: "../node_modules/expo-permissions/ios"
:path: "../node_modules/expo-permissions/ios"
EXScreenOrientation:
:path: !ruby/object:Pathname
path: "../node_modules/expo-screen-orientation/ios"
:path: "../node_modules/expo-screen-orientation/ios"
FBLazyVector:
:path: "../node_modules/react-native/Libraries/FBLazyVector"
FBReactNativeSpec:
Expand Down Expand Up @@ -467,53 +467,43 @@ EXTERNAL SOURCES:
:path: "../node_modules/react-native-sound"
RNVectorIcons:
:path: "../node_modules/react-native-vector-icons"
UMAppLoader:
:path: "../node_modules/unimodules-app-loader/ios"
UMBarCodeScannerInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-barcode-scanner-interface/ios"
:path: "../node_modules/unimodules-barcode-scanner-interface/ios"
UMCameraInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-camera-interface/ios"
:path: "../node_modules/unimodules-camera-interface/ios"
UMConstantsInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-constants-interface/ios"
:path: "../node_modules/unimodules-constants-interface/ios"
UMCore:
:path: !ruby/object:Pathname
path: "../node_modules/@unimodules/core/ios"
:path: "../node_modules/@unimodules/core/ios"
UMFaceDetectorInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-face-detector-interface/ios"
:path: "../node_modules/unimodules-face-detector-interface/ios"
UMFileSystemInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-file-system-interface/ios"
:path: "../node_modules/unimodules-file-system-interface/ios"
UMFontInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-font-interface/ios"
:path: "../node_modules/unimodules-font-interface/ios"
UMImageLoaderInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-image-loader-interface/ios"
:path: "../node_modules/unimodules-image-loader-interface/ios"
UMPermissionsInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-permissions-interface/ios"
:path: "../node_modules/unimodules-permissions-interface/ios"
UMReactNativeAdapter:
:path: !ruby/object:Pathname
path: "../node_modules/@unimodules/react-native-adapter/ios"
:path: "../node_modules/@unimodules/react-native-adapter/ios"
UMSensorsInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-sensors-interface/ios"
:path: "../node_modules/unimodules-sensors-interface/ios"
UMTaskManagerInterface:
:path: !ruby/object:Pathname
path: "../node_modules/unimodules-task-manager-interface/ios"
:path: "../node_modules/unimodules-task-manager-interface/ios"
Yoga:
:path: "../node_modules/react-native/ReactCommon/yoga"

SPEC CHECKSUMS:
boost-for-react-native: 39c7adb57c4e60d6c5479dd8623128eb5b3f0f2c
DoubleConversion: 5805e889d232975c086db112ece9ed034df7a0b2
EXApplication: 7cf81de6fafccff42f5d1caa5c24a159db6b9437
EXAppLoaderProvider: 5d348813a9cf09b03bbe5b8b55437bc1bfbddbd1
EXConstants: 857aa7b1c84e2878f8402d712061860bca16a697
EXFileSystem: 7e53a2c30a2eb6987ba6d5158ab908f947523228
EXPermissions: df10ad83df2f6b647aec304619354f8ab48d5f63
EXConstants: 5304709b1bea70a4828f48ba4c7fc3ec3b2d9b17
EXFileSystem: cf4232ba7c62dc49b78c2d36005f97b6fddf0b01
EXImageLoader: 5ad6896fa1ef2ee814b551873cbf7a7baccc694a
EXPermissions: 24b97f734ce9172d245a5be38ad9ccfcb6135964
EXScreenOrientation: 44d3cd3a99a86b9cb681e742697bc2c057d7fbd2
FBLazyVector: aaeaf388755e4f29cd74acbc9e3b8da6d807c37f
FBReactNativeSpec: 118d0d177724c2d67f08a59136eb29ef5943ec75
Expand Down Expand Up @@ -556,18 +546,19 @@ SPEC CHECKSUMS:
RNVectorIcons: 0bb4def82230be1333ddaeee9fcba45f0b288ed4
Sentry: 14bdd673870e8cf64932b149fad5bbbf39a9b390
Toast: 91b396c56ee72a5790816f40d3a94dd357abc196
UMBarCodeScannerInterface: d5a6fdc98ed6241225b0a8432a7f4e2b397668bc
UMCameraInterface: 68870a3197fee85bd5afca5609ba4a5b7257d19d
UMConstantsInterface: d25b8e8887ca7aaf568c06caf08f4d40734ee4ef
UMCore: 402cee150324974974f5c32b5404d8af65e4cff5
UMFaceDetectorInterface: 7b4f1a92f0c726b58b086296048efe193b570678
UMFileSystemInterface: aadb9a67aa6470d7ebc06cf04dc54fee6781ac48
UMFontInterface: 2d3c128285086bbed3d2a650f1d698323ef3b25a
UMImageLoaderInterface: 2829a7571a12d2e754c73c55ffe7e327d8402c7d
UMPermissionsInterface: b6a6e96db0f4011a25aaca14e6022529dd3d6e4e
UMReactNativeAdapter: 93c2f520a1cdb6ef3058a7b6a95275a804e0923b
UMSensorsInterface: cf59dd7602764a2419e00167429be3e4be39c61d
UMTaskManagerInterface: 1e70fe58b872355f0ecb44fb81bb1a16484047f0
UMAppLoader: ee77a072f9e15128f777ccd6d2d00f52ab4387e6
UMBarCodeScannerInterface: 9dc692b87e5f20fe277fa57aa47f45d418c3cc6c
UMCameraInterface: 625878bbf2ba188a8548675e1d1d2e438a653e6d
UMConstantsInterface: 64060cf86587bcd90b1dbd804cceb6d377a308c1
UMCore: eb200e882eadafcd31ead290770835fd648c0945
UMFaceDetectorInterface: d6677d6ddc9ab95a0ca857aa7f8ba76656cc770f
UMFileSystemInterface: c70ea7147198b9807080f3597f26236be49b0165
UMFontInterface: d9d3b27af698c5389ae9e20b99ef56a083f491fb
UMImageLoaderInterface: 14dd2c46c67167491effc9e91250e9510f12709e
UMPermissionsInterface: 5e83a9167c177e4a0f0a3539345983cc749efb3e
UMReactNativeAdapter: 126da3486c1a1f11945b649d557d6c2ebb9407b2
UMSensorsInterface: 48941f70175e2975af1a9386c6d6cb16d8126805
UMTaskManagerInterface: cb890c79c63885504ddc0efd7a7d01481760aca2
Yoga: f2a7cd4280bfe2cca5a7aed98ba0eb3d1310f18b

PODFILE CHECKSUM: c5b1c4d273aa956722a3218f0da9a30ef8251c32
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"@react-native-community/cameraroll": "^1.7.2",
"@react-native-community/netinfo": "^3.2.1",
"@sentry/react-native": "^1.0.9",
"@unimodules/core": "~4.0.0",
"@unimodules/core": "~5.1.2",
"@zulip/shared": "^0.0.2",
"base-64": "^0.1.0",
"blueimp-md5": "^2.10.0",
Expand Down Expand Up @@ -68,7 +68,7 @@
"react-native-simple-toast": "^1.0.0",
"react-native-sound": "^0.11.0",
"react-native-text-input-reset": "^1.0.2",
"react-native-unimodules": "0.6.0",
"react-native-unimodules": "^0.9.1",
"react-native-url-polyfill": "1.2.0-rc.0",
"react-native-vector-icons": "^6.6.0",
"react-native-webview": "~10.0.0",
Expand Down
Loading

0 comments on commit a5bb3f0

Please sign in to comment.