Skip to content

Commit

Permalink
💚 👷 Remove the check for patcher
Browse files Browse the repository at this point in the history
We removed patcher in 41ff1a7
so we should not expect to see it in the file structure any more

This should fix the failing build

```
+ ls -al /tmp/new-install/Android/sdk
+ '[' '!' -d /tmp/new-install/Android/sdk/emulator ']'
+ '[' '!' -d /tmp/new-install/Android/sdk/build-tools ']'
+ '[' '!' -d /tmp/new-install/Android/sdk/patcher ']'
+ exit 1
```

Also, run CI on changes to the SDK package list (i.e.
`setup/android_sdk_packages`) so that such errors are caught in the PR stage in
the future
  • Loading branch information
shankari committed May 5, 2024
1 parent 0a43a4d commit f504f68
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/android-automated-sdk-install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,12 @@ on:
push:
paths:
- 'setup/prereq_android_sdk_install.sh'
- 'setup/android_sdk_packages'
- '.github/workflows/android-automated-sdk-install.yml'
pull_request:
paths:
- 'setup/prereq_android_sdk_install.sh'
- 'setup/android_sdk_packages'
- '.github/workflows/android-automated-sdk-install.yml'
schedule:
# * is a special character in YAML so you have to quote this string
Expand Down Expand Up @@ -64,7 +66,6 @@ jobs:
ls -al $ANDROID_SDK_ROOT
if [ ! -d $ANDROID_SDK_ROOT/emulator ]; then exit 1; fi
if [ ! -d $ANDROID_SDK_ROOT/build-tools ]; then exit 1; fi
if [ ! -d $ANDROID_SDK_ROOT/patcher ]; then exit 1; fi
if [ ! -d $ANDROID_SDK_ROOT/extras ]; then exit 1; fi
if [ ! -d $ANDROID_SDK_ROOT/platforms ]; then exit 1; fi
if [ ! -d $ANDROID_SDK_ROOT/platform-tools ]; then exit 1; fi
Expand Down

0 comments on commit f504f68

Please sign in to comment.