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

deps: Use patch-package to fix Xcode build failure in react-native #5709

Merged
merged 1 commit into from
Apr 1, 2023

Conversation

chrisbobbe
Copy link
Contributor

This gets the effect of facebook/react-native@52d8a797e but without having to upgrade to RN v0.69.

I believe this is the first time we've used patch-package. It's a band-aid, but at least I think an effective one, while we focus our time on the Flutter project.

@chrisbobbe
Copy link
Contributor Author

chrisbobbe commented Apr 1, 2023

Oops, I forgot a key part of this: updating our postinstall script so patch-package actually runs 😄. Will update soon.

@chrisbobbe
Copy link
Contributor Author

OK, pushed with that fix.

This gets the effect of facebook/react-native@52d8a797e but without
having to upgrade to RN v0.69.

I believe this is the first time we've used patch-package. It's a
band-aid, but at least I think an effective one, while we focus our
time on the Flutter project.
@gnprice
Copy link
Member

gnprice commented Apr 1, 2023

Cool, seems best ­— thanks for working this out. Merging.

@gnprice gnprice force-pushed the pr-patch-package branch from b2432de to b546b07 Compare April 1, 2023 00:27
@gnprice gnprice merged commit b546b07 into zulip:main Apr 1, 2023
@chrisbobbe chrisbobbe deleted the pr-patch-package branch April 1, 2023 00:27
@chrisbobbe
Copy link
Contributor Author

Thanks!

@chrisbobbe
Copy link
Contributor Author

By the way I tested a yarn command and patch-package did its job, adding three lines to the output:

patch-package 6.5.1
Applying patches...
[email protected]

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

Successfully merging this pull request may close these issues.

2 participants