-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Fix expo-av for usage with AndroidX #5575
Conversation
Co-Authored-By: Stanisław Chmiela <sjchmiela@users.noreply.github.com>
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! Thank you for taking care of this change. 💪 I'll wait for @mczernek's review before merging.
Please be aware of my comment here |
To make it clear: This is not a migration to AndroidX, but allows you to use jetifier to make it AndroidX compatible. |
At the moment projects with expo-av do not compile for Android with AndroidX. The package is missing the "support-annotations" dependency in order to compile. After the change it runs fine using jettifier. Manually tested on Android Device and Simulator. Fixes #5310, #5474 and unimodules/react-native-unimodules#61
This should be definitely mentioned here, took me 6 hours to find this pull request https://github.com/expo/expo/tree/master/packages/expo-av |
# Why At the moment projects with expo-av do not compile for Android with AndroidX. # How The package is missing the "support-annotations" dependency in order to compile. After the change it runs fine using jettifier. # Test Plan Manually tested on Android Device and Simulator. Fixes #5310, #5474 and unimodules/react-native-unimodules#61
# Why At the moment projects with expo-av do not compile for Android with AndroidX. # How The package is missing the "support-annotations" dependency in order to compile. After the change it runs fine using jettifier. # Test Plan Manually tested on Android Device and Simulator. Fixes expo#5310, expo#5474 and unimodules/react-native-unimodules#61
Why
At the moment projects with expo-av do not compile for Android with AndroidX.
How
The package is missing the "support-annotations" dependency in order to compile. After the change it runs fine using jettifier.
Test Plan
Manually tested on Android Device and Simulator.
Fixes #5310, #5474 and unimodules/react-native-unimodules#61