-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
Warning - Platform tools already installed #265
Comments
Maybe similar error
From https://github.com/yschimke/rememberwear/actions/runs/3078664216/jobs/4979543909#step:5:15 |
Can you post your .yml? @mattjohnsonpint Also, I don't know if we can assume the tools are installed on every flavor of runner. |
Sure. https://github.com/getsentry/sentry-dotnet/blob/main/.github/workflows/device-tests.yml
The runner images and their pre-installed software are listed here: https://github.com/actions/runner-images/blob/main/README.md |
@ychescale9 Any downsides to putting a check in before installing the tooling? Suppose we could do that. Runners already have the SDK tools needed and many people already run the setup-android action before their tests as well. |
I can't recall exactly why I did this but it could be something to do with the paths of the preinstalled SDK and the way we install all the tools via the Happy to accept a PR if anyone wants to have a go 😀 |
I'm also seeing this output on GH Actions using |
Seems like depending on the Java Version the app under test is using, they will need to use either So, for this PR we will want to have 2 workflows to test, one with Java 1.8 and one on a higher level, and make sure the appropriate tools are called in both cases. I'm fairly sure the Github Standard Runners come with sufficient android tooling out of the box, but we can confirm this in the PR. |
Note: I did find a discrepancy in the tooling as noted here: #286 (comment) |
In our setup, |
I'm seeing this in my runner logs:
Indeed it would appear everything that is trying to be installed is already on the runner image
https://github.com/actions/runner-images/blob/main/images/macos/macos-11-Readme.md#android
Is there a reason we're trying to install them again? Or is there a way to skip this?
Thanks.
The text was updated successfully, but these errors were encountered: