-
Notifications
You must be signed in to change notification settings - Fork 21
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
Running Android has critical issues #3
Comments
I've had trouble running on Android too. I got the following error:
61 actionable tasks: 61 executed FAILURE: Build failed with an exception.
** Edit: manage to run on android using the rn-63 branch ** |
Can you please try the test repo?
…On Sat, 16 Jan 2021 at 5:20 AM, Júlia Miranda ***@***.***> wrote:
I've had trouble running on Android too. I got the following error:
Task :app:packageDebug FAILED
61 actionable tasks: 61 executed
Note:
<my_path>\react-native-vlc-media-player-test\node_modules\react-native-vlc-media-player\android\src\main\java\com\yuanzhou\vlc\vlcplayer\ReactVlcPlayerView.java
uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note:
<my_path>\react-native-vlc-media-player-test\android\app\src\debug\java\com\vlctest\ReactNativeFlipper.java
uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
FAILURE: Build failed with an exception.
- What went wrong:
Execution failed for task ':app:packageDebug'.
A failure occurred while executing
com.android.build.gradle.tasks.PackageAndroidArtifact$IncrementalSplitterRunnable
java.lang.OutOfMemoryError (no error message)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFB72EE2BLTEE2DY3H6RRD3S2BTL5ANCNFSM4NV5LWOA>
.
|
@razorRun i tried the test repo and the rn-63 branch was functioning perfectly. I tried to install the package in a brand new and unmodified react-native app and had many issues. I managed to install it by making some changes in the native modules. Here's what i did: i added this to my android/app/build.gradle ` packagingOptions {
also: `defaultConfig {
}` and finally upgraded my minSDKVersion to 17 ( android/build.gradle): `buildscript {
}` After these modifications I didn't have any more issues with building and using the package. |
Same issue for me on Android Dropping the minSdkVersion to 17 is not an option as other libraries rely on minSdkVersion 21 If I add the advice above but leave the minSdkVersion 21
|
Sorry guys. I am bit packed at the momeant and will have to leave these
issues to the community. See the note regarding big fixes on the readme ( I
am more than happy to merge or add you guys as contributers to the repo.)
Cheers
…On Sat, 18 Sep 2021 at 12:59 PM, slaymantis ***@***.***> wrote:
Same issue for me on Android
Works great on iOS though.
React 0.64.1
Dropping the minSdkVersion to 17 is not an option as other libraries rely
on minSdkVersion 21
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFB72EAY2E22MZGHGCD6PWDUCPP65ANCNFSM4NV5LWOA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Thanks, all in your own time when you ready :)
But I did finally manage to get it working by adding the tip above added this to android/gradle.properties added this to android/app/build.gradle defaultConfig { dependencies { Also not sure if it made a difference but I also rolled back to android studio 4.2.1 instead of 2021 |
@slaymantis Thank you so much you save my day |
a little change need:
|
Hi there
Feel free to send a pr please
Cheers
…On Thu, 26 May 2022 at 7:13 PM, Steven Gardner ***@***.***> wrote:
a little change need:
implementation 'androidx.multidex:multidex:2.0.1'
—
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFB72EFPGCKSY3E72YODWUDVL4QBBANCNFSM4NV5LWOA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***
com>
|
Hi there, Tested with react native 0.68, I did the following changes to make it work: In android/build.gradle In android/app/build.gradle defaultConfig{ } and
} in android/gradle.properties add this |
Under trying to run this test on Android I got these errors:
The text was updated successfully, but these errors were encountered: