-
Notifications
You must be signed in to change notification settings - Fork 481
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
Add some trouble shootings for android installation #145
Conversation
I had to run |
@pistonsky is that something related to my PR? |
README.md
Outdated
For error **_CMake 'X.X.X' was not found in SDK, PATH, or by cmake.dir property._** | ||
|
||
open _Tools > SDK Manager_, switch to the _SDK Tools_ tab. | ||
Find `CMake`and click _Show Package Details_ and download compatiable version **'X.X.X'**, and apply to install. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You forgot a space after CMake
README.md
Outdated
@@ -44,6 +44,16 @@ If the NDK is not installed, you can install it via Android Studio by going to t | |||
|
|||
And then the _SDK Location_ section. It will show you the NDK path, or the option to Download it if you don't have it installed. | |||
|
|||
And them the _Modules_ section. click on `shopify_react-native-skia` and select _NDK version_ with dropdown, and click on apply |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a dot after apply
This is probably not related to this PR, I saw and commented on the issue you filed about this. I don't think you should need to run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks!
I am not sure i am the only one who got in this error, but found it will be helpful for others to configure there settings.