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

[RN][CI] Try to restore caches for Hermes #45908

Closed
wants to merge 3 commits into from

Conversation

cipolleschi
Copy link
Contributor

@cipolleschi cipolleschi commented Aug 6, 2024

Summary:

We had CI on main failing consistently the past couple of days.
The problem is that the hermes pipeline is failing to create the iOS XCFramework with the error:

unable to create a Mach-O from the binary at '/Users/runner/work/react-native/react-native/packages/react-native/sdks/hermes/destroot/Library/Frameworks/catalyst/hermes.framework/hermes'

The main cause is this upgrade of upload-artifacts which breaks symlinks.

The solution is to bump the caches and downgrade the upload-artifact actions.

Changelog:

[Internal] - Try to fix CI for Hermes

Test Plan:

GHA must be green

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 6, 2024
@cipolleschi cipolleschi marked this pull request as draft August 6, 2024 10:03
@cipolleschi cipolleschi force-pushed the cipolleschi/fix-hermes-build branch from eae2874 to 6b1376d Compare August 6, 2024 10:27
@cipolleschi cipolleschi force-pushed the cipolleschi/fix-hermes-build branch from 743d13b to 16e7484 Compare August 6, 2024 12:58
@cipolleschi cipolleschi marked this pull request as ready for review August 6, 2024 13:16
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Aug 6, 2024
@facebook-github-bot
Copy link
Contributor

@cipolleschi merged this pull request in 24e7f7d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants