Skip to content

Make Hermes source archive extraction repository-agnostic - #59589

Merged
javache merged 1 commit into
react:mainfrom
javache:export-D128177063
Oct 28, 2026
Merged

javache merged 1 commit into
react:mainfrom
javache:export-D128177063

Conversation

@javache

@javache javache commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Summary:
Download Hermes sources from the canonical meta/hermes repository and strip the archive’s top-level directory without matching its owner or repository name. This keeps CMakeLists.txt at the expected source root after repository moves; exclude empty directories so the archive wrapper is not recreated.

Changelog:
[Android][Fixed] - Fix Hermes source builds after the Hermes repository move

Differential Revision: D128177063

Summary:
Download Hermes sources from the canonical `meta/hermes` repository and strip the archive’s top-level directory without matching its owner or repository name. This keeps `CMakeLists.txt` at the expected source root after repository moves; exclude empty directories so the archive wrapper is not recreated.

Changelog:
[Android][Fixed] - Fix Hermes source builds after the Hermes repository move

Differential Revision: D128177063
@meta-cla meta-cla 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 Oct 28, 2026
@meta-codesync

meta-codesync Bot commented Oct 28, 2026

Copy link
Copy Markdown

@javache has exported this pull request. If you are a Meta employee, you can view the originating Diff in D128177063.

@javache
javache merged commit f6a5042 into react:main Oct 28, 2026
62 of 63 checks passed
alanjhughes pushed a commit to expo/react-native that referenced this pull request Oct 29, 2026
Summary:
Download Hermes sources from the canonical `meta/hermes` repository and strip the archive’s top-level directory without matching its owner or repository name. This keeps `CMakeLists.txt` at the expected source root after repository moves; exclude empty directories so the archive wrapper is not recreated.

Changelog:
[Android][Fixed] - Fix Hermes source builds after the Hermes repository move

Differential Revision: D128177063
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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants