Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update react-native to 0.7x #21122

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 24 commits
Commits
Show all changes
61 commits
Select commit Hold shift + click to select a range
acf9f65
Update pool to MacOS-13
jchen351 Aug 30, 2023
1af95ce
Merge branch 'main' into Cjian/macos13
jchen351 Nov 17, 2023
32d19a8
Merge branch 'refs/heads/main' into Cjian/macos13
jchen351 May 15, 2024
a6a85d7
update react-native and npm audit fix
jchen351 Jun 20, 2024
55a21b0
Merge remote-tracking branch 'origin/main' into Cjian/macos13
jchen351 Jun 21, 2024
912c9a5
update jave sdk from 11 to 17
jchen351 Jun 24, 2024
f820cf5
Use Xcode version 14.3 instead of
jchen351 Jun 24, 2024
3d3389e
14.3.1
jchen351 Jun 24, 2024
93ec7e2
macOS-13
jchen351 Jun 24, 2024
ee9dfc4
macOS-13
jchen351 Jun 24, 2024
53a03b1
Merge branch 'refs/heads/main' into Cjian/macos13
jchen351 Jun 26, 2024
f0ca208
update ios target to 15
jchen351 Jun 28, 2024
ffb6f5c
Merge remote-tracking branch 'origin/main' into Cjian/macos13
snnn Jun 29, 2024
c85fdd7
Merge branch 'refs/heads/main' into Cjian/macos13
jchen351 Jul 17, 2024
da23b72
Installing gcovr from apt-get
jchen351 Jul 17, 2024
462782e
Merge branch 'refs/heads/Cjian/macos13' into Cjian/cg
jchen351 Jul 17, 2024
b43bc45
js/react_native/ios/OnnxruntimeModule.xcodeproj/project.pbxproj
jchen351 Jul 19, 2024
6184cd5
remove post_install
jchen351 Jul 19, 2024
fa5cf75
Podfile
jchen351 Jul 19, 2024
37bbfd0
../node_modules/react-native/
jchen351 Jul 19, 2024
68e0075
use_react_native
jchen351 Jul 19, 2024
b74cf35
js/react_native/ios/Podfile
jchen351 Jul 22, 2024
cb92bd5
js/react_native/ios/Podfile
jchen351 Jul 22, 2024
ecfa1bf
js/react_native/ios/Podfile
jchen351 Jul 22, 2024
58b7b5f
--enable_lto \
jchen351 Jul 22, 2024
90818b0
platform :ios, '15.0'
jchen351 Jul 23, 2024
776d071
Merge branch 'refs/heads/main' into Cjian/cg
jchen351 Jul 24, 2024
e7e574f
react-native": "
jchen351 Jul 24, 2024
aff66bb
react-native-community/cli
jchen351 Jul 24, 2024
c22daf3
Podfile
jchen351 Jul 26, 2024
c8266af
use_react_native!(:path => '../node_modules/react-native')
jchen351 Jul 27, 2024
a153abd
platform :ios, '15.0'
jchen351 Jul 29, 2024
ab4c79f
"@react-native-community/cli": "^13.6.5",
jchen351 Jul 29, 2024
e75be4e
yarn
jchen351 Jul 29, 2024
b5990e1
downgade the react to 0.72
jchen351 Jul 30, 2024
0246310
remove react.gradle
jchen351 Jul 30, 2024
c345fcb
220
jchen351 Jul 30, 2024
0debec3
Check if emulator is running
jchen351 Jul 31, 2024
dcca550
Check if emulator is running
jchen351 Jul 31, 2024
2459208
react-native
jchen351 Jul 31, 2024
aba2284
Build React Native Detox Android e2e Tests
jchen351 Jul 31, 2024
383ed7f
Do not allow clearing Android logs if the emulator is not running
jchen351 Jul 31, 2024
ab78e66
implementation 'com.facebook.react:react-native:+'
jchen351 Jul 31, 2024
cce7cca
if is_emulator_running(sdk_tool_paths.adb):
jchen351 Jul 31, 2024
22da19c
result = run(f'{adb_path} devices', capture_stdout=True)
jchen351 Jul 31, 2024
283ca73
sdk_tool_paths
jchen351 Jul 31, 2024
1f58393
sdk_tool_paths
jchen351 Jul 31, 2024
bfe7869
lines = output.strip().split("\n")
jchen351 Jul 31, 2024
033442c
lines = output.strip().split("\n")
jchen351 Jul 31, 2024
10ae66c
lines = output.strip().split("\n")
jchen351 Jul 31, 2024
5f08d97
lines = output.strip().split("\n")
jchen351 Jul 31, 2024
49f3be9
lines = output.strip().split("\n")
jchen351 Jul 31, 2024
b9d7e34
splitlines
jchen351 Aug 1, 2024
eef0213
splitlines
jchen351 Aug 1, 2024
c9a006e
Merge branch 'refs/heads/Cjian/emu' into Cjian/cg
jchen351 Aug 1, 2024
ab20327
Merge branch 'refs/heads/main' into Cjian/cg
jchen351 Aug 19, 2024
e57c638
Reverting pipeline changes
jchen351 Aug 26, 2024
e1aae89
Merge branch 'refs/heads/main' into Cjian/cg
jchen351 Aug 26, 2024
2ff5454
Checkout java/build-android.gradle
jchen351 Aug 27, 2024
0ee4354
using fb 0.72.15
jchen351 Aug 28, 2024
d948315
Merge branch 'refs/heads/main' into Cjian/cg
jchen351 Sep 17, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion js/react_native/e2e/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
},
"dependencies": {
"react": "^18.1.0",
"react-native": "^0.69.1",
"react-native": "^0.74.2",
"react-native-fs": "^2.20.0"
},
"devDependencies": {
Expand Down
5,565 changes: 2,486 additions & 3,079 deletions js/react_native/e2e/yarn.lock

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion js/react_native/ios/Podfile
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '13.0'
platform :ios, '15.0'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this change needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specs satisfying the RCTRequired (from ../node_modules/react-native//Libraries/Required) dependency were found, but they required a higher minimum deployment target.
https://dev.azure.com/onnxruntime/onnxruntime/_build/results?buildId=1439006&view=logs&j=de302ec2-2305-57e0-e8c6-cd89c569f2a3&t=65d5dc62-c4eb-539e-a5bd-5c7416a33c7a&l=63


def shared
config = use_native_modules!

if config["reactNativePath"] != nil
use_react_native!(:path => config["reactNativePath"])
else
use_react_native!(:path => '../node_modules/react-native/')
end

# Comment the next line if you don't want to use dynamic frameworks
use_frameworks!
Expand Down
2 changes: 1 addition & 1 deletion js/react_native/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"pod-install": "^0.1.36",
"prettier": "^2.6.2",
"react": "^18.1.0",
"react-native": "^0.69.7",
"react-native": "^0.74.2",
"react-native-builder-bob": "^0.18.2"
},
"peerDependencies": {
Expand Down
4,907 changes: 2,197 additions & 2,710 deletions js/react_native/yarn.lock

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,9 @@ stages:
versionSpec: $(pythonVersion)

- task: JavaToolInstaller@0
displayName: Use jdk 11
displayName: Use jdk 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'

Expand Down Expand Up @@ -123,16 +123,17 @@ stages:
timeoutInMinutes: ${{ variables.JobsTimeout }}
workspace:
clean: all
condition: notIn(variables['Build.Reason'], 'IndividualCI', 'BatchedCI')
steps:
- task: UsePythonVersion@0
displayName: Use Python $(pythonVersion)
inputs:
versionSpec: $(pythonVersion)

- task: JavaToolInstaller@0
displayName: Use jdk 11
displayName: Use jdk 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'

Expand All @@ -149,7 +150,6 @@ stages:

- script: |
python3 tools/ci_build/build.py \
--enable_lto \
--android \
--build_dir build_nnapi \
--android_sdk_path $ANDROID_HOME \
Expand All @@ -159,7 +159,6 @@ stages:
--skip_submodule_sync \
--parallel \
--use_nnapi \
--build_shared_lib \
snnn marked this conversation as resolved.
Show resolved Hide resolved
--cmake_generator=Ninja \
--build_java
displayName: NNAPI EP, Build, Test on Android Emulator
Expand Down Expand Up @@ -194,9 +193,9 @@ stages:
versionSpec: $(pythonVersion)

- task: JavaToolInstaller@0
displayName: Use jdk 11
displayName: Use jdk 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'

Expand All @@ -209,7 +208,6 @@ stages:

- script: |
python3 tools/ci_build/build.py \
--enable_lto \
snnn marked this conversation as resolved.
Show resolved Hide resolved
--android \
--build_dir build_nnapi \
--android_sdk_path $ANDROID_HOME \
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:
workspace:
clean: all
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
variables:
MACOSX_DEPLOYMENT_TARGET: '11.0'
TODAY: $[format('{0:dd}{0:MM}{0:yyyy}', pipeline.startTime)]
Expand All @@ -44,7 +44,7 @@ jobs:

- template: templates/use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- template: templates/mac-build-step-with-cache.yml
parameters:
Expand Down
4 changes: 2 additions & 2 deletions tools/ci_build/github/azure-pipelines/mac-ios-ci-pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ pr:
jobs:
- job: iOS_CI_on_Mac
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
variables:
PROTO_CACHE_DIR: $(Pipeline.Workspace)/proto_ccache
ORT_CACHE_DIR: $(Pipeline.Workspace)/ort_ccache
Expand All @@ -39,7 +39,7 @@ jobs:
steps:
- template: templates/use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1
- template: templates/mac-build-step-with-cache.yml
parameters:
WithCache: true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ stages:
displayName: "Set common variables"

pool:
vmImage: "macOS-latest"
vmImage: "macOS-13"

timeoutInMinutes: 5

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ stages:
clean: all
timeoutInMinutes: 120
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'

variables:
- name: OnnxRuntimeBuildDirectory
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ stages:
workspace:
clean: all
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'

variables:
- name: OnnxRuntimeBuildDirectory
Expand Down
8 changes: 4 additions & 4 deletions tools/ci_build/github/azure-pipelines/post-merge-jobs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -425,7 +425,7 @@ stages:
- job: IosDynamicFramework
timeoutInMinutes: 120
pool:
vmImage: "macOS-latest"
vmImage: "macOS-13"

steps:
- task: UsePythonVersion@0
Expand All @@ -436,7 +436,7 @@ stages:

- template: templates/use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- script: |
pip install -r tools/ci_build/github/apple/ios_packaging.requirements.txt
Expand All @@ -463,7 +463,7 @@ stages:
- job: IosMinimalTrainingBuild
timeoutInMinutes: 120
pool:
vmImage: "macOS-latest"
vmImage: "macOS-13"

steps:
- task: UsePythonVersion@0
Expand All @@ -474,7 +474,7 @@ stages:

- template: templates/use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- script: |
pip install -r tools/ci_build/github/apple/ios_packaging.requirements.txt
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ stages:
parameters:
job_name: Test_MAC_Wheels
machine_pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
itemPattern: '*/*mac*x86_64.whl'
- template: templates/py-package-smoking-test.yml
parameters:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,7 @@ jobs:
workspace:
clean: all
pool:
# We need macOS-12 to run the Android emulator for now.
# https://github.com/actions/runner-images/issues/7671
vmImage: 'macOS-12'
vmImage: 'macOS-13'
variables:
- name: runCodesignValidationInjection
value: false
Expand All @@ -39,9 +37,9 @@ jobs:
targetPath: '$(Build.BinariesDirectory)/final-android-aar'

- task: JavaToolInstaller@0
displayName: Use jdk 11
displayName: Use jdk 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'

Expand Down
8 changes: 4 additions & 4 deletions tools/ci_build/github/azure-pipelines/templates/c-api-cpu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,14 +83,14 @@ stages:
workspace:
clean: all
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
timeoutInMinutes: 300
steps:
- template: set-version-number-variables-step.yml

- template: use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- template: download-deps.yml

Expand Down Expand Up @@ -780,7 +780,7 @@ stages:

- template: ../nuget/templates/test_macos.yml
parameters:
AgentPool : macOS-latest
AgentPool : macOS-13
ArtifactSuffix: 'CPU'

- template: ../nodejs/templates/test_win.yml
Expand Down Expand Up @@ -816,4 +816,4 @@ stages:
OS: MacOS
BuildId: ${{ parameters.BuildId }}
SpecificArtifact: ${{ parameters.SpecificArtifact }}
PoolName: 'macOS-latest'
PoolName: 'macOS-13'
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,9 @@ stages:
- job: MacOS_C_API_Package_Publish
pool:
${{ if eq(parameters.DoESRP, true)}}:
vmImage: 'macOS-12'
vmImage: 'macOS-13'
${{ else }}:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
steps:
- checkout: none
- template: flex-downloadPipelineArtifact.yml
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
PROTO_CACHE_DIR: $(Pipeline.Workspace)/ccache_proto
ORT_CACHE_DIR: $(Pipeline.Workspace)/ccache_ort
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
timeoutInMinutes: 300
steps:
- checkout: self
Expand All @@ -62,7 +62,7 @@ jobs:

- template: use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- template: download-deps.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -393,7 +393,7 @@ stages:
workspace:
clean: all
pool:
vmImage: 'macOS-latest'
vmImage: 'macOS-13'
variables:
MACOSX_DEPLOYMENT_TARGET: '11.0'
strategy:
Expand All @@ -420,7 +420,7 @@ stages:

- template: use-xcode-version.yml
parameters:
xcodeVersion: 14.2
xcodeVersion: 14.3.1

- template: download-deps.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ stages:

- job: Build_Ios_Pod_For_React_Native
pool:
vmImage: 'macOS-12'
vmImage: 'macOS-13'

timeoutInMinutes: 90

Expand Down Expand Up @@ -99,9 +99,7 @@ stages:
jobs:
- job: ReactNative_CI
pool:
# We need macOS-12 to run the Android emulator for now.
# https://github.com/actions/runner-images/issues/7671
vmImage: 'macOS-12'
vmImage: 'macOS-13'
variables:
runCodesignValidationInjection: false
timeoutInMinutes: 90
Expand All @@ -118,9 +116,9 @@ stages:
architecture: "x64"

- task: JavaToolInstaller@0
displayName: Use jdk 11
displayName: Use jdk 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: 'x64'
jdkSourceOption: 'PreInstalled'

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@ stages:
displayName: "Build iOS package for variant: ${{ parameters.packageVariant}}"

pool:
vmImage: "macOS-latest"
vmImage: "macOS-13"

variables:
xcodeVersion: "14.2"
xcodeVersion: "14.3.1"
ortPodVersion: $[stageDependencies.IosPackaging_SetCommonVariables.j.outputs['SetCommonVariables.ORT_POD_VERSION']]

${{ if eq(parameters.packageVariant, 'Full') }}:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
parameters:
- name: xcodeVersion
type: string
default: "14.2"
default: "14.3.1"

steps:
- bash: |
Expand Down
Loading