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

JS: Upgrade to v8-compile-cache@^2.4.0 & eslint@^8.23.1 #39012

Closed
wants to merge 1 commit into from

Conversation

yungsters
Copy link
Contributor

Summary:
Upgrades many of our projects to v8-compile-cache@^2.4.0, which includes zertosh/v8-compile-cache#45. This fixes a very opaque segmentation fault when v8-compile-cache is used on Apple Silicon with mixed architecture modes.

This also upgrades our projects to eslint@^2.3.1, which no longer has the dependency on an outdated version of v8-compile-cache.

Changelog:
[Internal]

Reviewed By: NickGerleman

Differential Revision: D48336030

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Aug 14, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48336030

@github-actions
Copy link

github-actions bot commented Aug 14, 2023

Warnings
⚠️ 🔒 package.json - Changes were made to package.json. This will require a manual import by a Facebook employee.

Generated by 🚫 dangerJS against 14e1c55

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48336030

yungsters added a commit to yungsters/react-native that referenced this pull request Aug 14, 2023
Summary:
Pull Request resolved: facebook#39012

Upgrades many of our projects to `v8-compile-cache@^2.4.0`, which includes [zertosh/v8-compile-cache#45](zertosh/v8-compile-cache#45). This fixes a very opaque segmentation fault when `v8-compile-cache` is used on Apple Silicon with mixed architecture modes.

This also upgrades our projects to `eslint@^2.3.1`, which no longer has the dependency on an outdated version of `v8-compile-cache`.

Changelog:
[Internal]

Reviewed By: NickGerleman

Differential Revision: D48336030

fbshipit-source-id: 5cf68aea0abed1a4081d0182f5fb9ef5b16511c5
@analysis-bot
Copy link

analysis-bot commented Aug 15, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,946,475 +0
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 9,540,248 +12
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 540c41b
Branch: main

Summary:
Pull Request resolved: facebook#39012

Upgrades many of our projects to `v8-compile-cache@^2.4.0`, which includes [zertosh/v8-compile-cache#45](zertosh/v8-compile-cache#45). This fixes a very opaque segmentation fault when `v8-compile-cache` is used on Apple Silicon with mixed architecture modes.

This also upgrades our projects to `eslint@^2.3.1`, which no longer has the dependency on an outdated version of `v8-compile-cache`.

Changelog:
[Internal]

Reviewed By: zertosh, NickGerleman

Differential Revision: D48336030

fbshipit-source-id: 59e772d777e0775a9bd7c31218025cc2f34a1979
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48336030

facebook-github-bot pushed a commit to facebook/metro that referenced this pull request Aug 15, 2023
Summary:
X-link: facebook/react-native#39012

Upgrades many of our projects to `v8-compile-cache@^2.4.0`, which includes [zertosh/v8-compile-cache#45](zertosh/v8-compile-cache#45). This fixes a very opaque segmentation fault when `v8-compile-cache` is used on Apple Silicon with mixed architecture modes.

This also upgrades our projects to `eslint@^2.3.1`, which no longer has the dependency on an outdated version of `v8-compile-cache`.

Changelog:
[Internal]

Reviewed By: zertosh, NickGerleman

Differential Revision: D48336030

fbshipit-source-id: afa73d1f00bff826a8b76fa09949b9d35a97905b
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e637a33.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Aug 15, 2023
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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants