Remove the redundant install_requirement script when test iOS on CI (#1583)

huydhn · facebook-github-bot · commit 7a6abb4a1822 · 2024-01-11T10:41:38.000-08:00
Summary: All the dependency has already been setup by https://github.com/pytorch/executorch/blob/main/.github/workflows/app-build.yml#L40. Running the install script again here will overwrite the PyTorch version causing version mismatched failure, i.e. https://github.com/pytorch/executorch/actions/runs/7483610205/job/20369124052 Pull Request resolved: #1583 Reviewed By: shoumikhin Differential Revision: D52684492 Pulled By: huydhn fbshipit-source-id: b90a29d96a84e7e3b5c6f5b29b468786d5bcc332
diff --git a/build/test_ios_ci.sh b/build/test_ios_ci.sh
@@ -31,10 +31,6 @@ say() {
   echo -e "\033[1m\n\t** $1 **\n\033[0m"
 }
 
-say "Installing Requirements"
-
-./install_requirements.sh
-
 say "Installing CoreML Backend Requirements"
 
 ./backends/apple/coreml/scripts/install_requirements.sh

Original file line number	Diff line number	Diff line change
`@@ -31,10 +31,6 @@ say() {`
`31`	`31`	`echo -e "\033[1m\n\t $1 \n\033[0m"`
`32`	`32`	`}`
`33`	`33`
`34`		`-say "Installing Requirements"`
`35`		`-`
`36`		`-./install_requirements.sh`
`37`		`-`
`38`	`34`	`say "Installing CoreML Backend Requirements"`
`39`	`35`
`40`	`36`	`./backends/apple/coreml/scripts/install_requirements.sh`