Skip to content

fix: bring back #715 to fix Android builds for Fabric components #769

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

Merged
merged 2 commits into from
Feb 25, 2025

Conversation

Serchinastico
Copy link
Contributor

Summary

I'm bringing back the fix implemented in #715. Unfortunately, it was overwritten in #721 so versions 0.48.x of create-react-native-library are failing Android builds for Fabric components.

Fixes #767

@Serchinastico Serchinastico changed the title Bring back #715 to fix Android builds for Fabric components fix: bring back #715 to fix Android builds for Fabric components Feb 18, 2025
@atlj
Copy link
Collaborator

atlj commented Feb 24, 2025

Thank you so much for the PR and sorry for the delay. I'm checking whether there is a fix without using type: all since the RN team asked us to not pass all for views since that's reserved for libraries that has views + modules

Copy link
Collaborator

@atlj atlj left a comment

Choose a reason for hiding this comment

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

Okay I double checked the RN discord channel and according to this discussion, we can get views back to all.
image

@atlj atlj added this pull request to the merge queue Feb 25, 2025
Merged via the queue into callstack:main with commit be8b107 Feb 25, 2025
21 checks passed
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.

Fabric view giving Cmake error with new arch enabled in React Native 0.77
2 participants