-
-
Notifications
You must be signed in to change notification settings - Fork 847
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
Can't build on Android [RN 0.72.14] #3598
Comments
Yes same here ++ |
Force the rnmapbox version to be My workiing setup although I am using expo:
|
Hey folks, I'm having a similar issue: EnvironmentDev OS: Windows Result running yarn android
|
Here is my working setup for expo right now
|
Thank you, you just saved me a LOT of headache. Granted it's my own fault, I should've checked the issues earlier. I Thought It was something I did in my code. For now this is the best solution and can confirm works. |
I also had the issue on RN 0.74, reverting to rnmapbox/maps 10.1.27 solved the issue. |
@snowdamiz Thank you, reverting to |
@TobiDevloft you can open the issue. I'll need to take a look, to see if we can make rnmapbox compatible with both RN<0.75 and >=0.75. In 10.1.30 I've made it compile with RN 0.75, but it sounds like this breaks with <=RN 0.74. At worst we'll need to provide patches so ppl can use the latest version with older react using |
Alright, reopening! Thanks for your work :) |
I have the same problem, did you find a solution? "react-native": "0.72.1", |
same: "@rnmapbox/maps": "^10.1.28", |
Remove the |
I fixed by changing to: |
released v10.1.31-rc.0 this should work with RN 0.75, and 0.72 pls test |
I still have this issue with v10.1.31-rc.0:
ext.kotlinVersion = '1.9.21' Just did another test with 0.72 but it still fails with rc.0: npx react-native init MapboxTest --version 0.72.1
cd MapboxTest
yarn add @rnmapbox/[email protected]
# update android/build.grale with maven repo details
yarn android
|
I think it's better for now to either not override class ResourceDrawableIdHelperCompat {
companion object {
fun getInstance(): ResourceDrawableIdHelper {
return ResourceDrawableIdHelper.getInstance() // deprecated, .instance is preferred but can't be used on rn <0.75
}
}
} and reference |
@mattijsf thanks, do you have new arch enabled? Not sure why I haven't seen this issue, while testing... |
I'm not using new arch. Just the instructions I wrote in #3598 (comment) are enough. Maybe you hit some caching / node_modules situation where the build succeeded with the change 🤷 |
I am using the same version, but still failed |
same here |
It's unbelievable the amount of people that don't bother reading comments. Pure laziness 😂 |
Can you try with v10.1.31-rc.1 ? released v10.1.31-rc.1 this should work with RN 0.75, and 0.72 pls test |
I can confirm this is working for me
|
Continuation of #3593
Environment
Steps to reproduce
Result
The text was updated successfully, but these errors were encountered: