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

Support RN64 for android #1667

Merged
merged 6 commits into from
Feb 3, 2021
Merged

Conversation

piaskowyk
Copy link
Member

@piaskowyk piaskowyk commented Jan 27, 2021

Description

Added support for React Native v0.64.0.-rc.2 for android devices. This PR is based on: https://github.com/osdnk/react-native-reanimated/tree/64 - thanks @osdnk 🙌

I compiled with android-ndk-r19c

Fixes: #1680

@piaskowyk piaskowyk changed the title Test android build for RN64 Support RN64 for android Jan 27, 2021
@piaskowyk piaskowyk marked this pull request as ready for review January 29, 2021 11:50
Copy link

@kaszperro kaszperro left a comment

Choose a reason for hiding this comment

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

LGTM

@tomekzaw
Copy link
Member

Good job!

android/build.gradle Outdated Show resolved Hide resolved
createNPMPackage.sh Outdated Show resolved Hide resolved
@Szymon20000
Copy link
Contributor

Have you checked if RN 63 and 62 are still working with this pr?

@piaskowyk
Copy link
Member Author

Yes, I tested this PR with RN 62, 63, 64, and works without any problems.

@piaskowyk piaskowyk requested a review from Szymon20000 February 1, 2021 09:42

# PART II (clean)
for index in {0..2}
Copy link
Member

Choose a reason for hiding this comment

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

length="${#versions[@]}"

for ((i=0; i < $length; i++ )); do echo $i; done

I think explicitly setting length and using for loop is more readable.

android/src/main/JNI/Android.mk Show resolved Hide resolved
android/src/main/JNI/Application.mk Show resolved Hide resolved
android/src/main/JNI/react/Android.mk Show resolved Hide resolved
@piaskowyk piaskowyk merged commit bb5184e into master Feb 3, 2021
@piaskowyk piaskowyk deleted the @piaskowyk/support-rn64-for-android branch February 3, 2021 10:39
@ajpaulingalls
Copy link

Any ETA on a new release with support for RN64?

@karol-bisztyga
Copy link
Contributor

next few days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reanimatedv2 Task :app:checkDebugAarMetadata FAILED
10 participants