-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
wasm: update V8 to 9.6.152 #18375
wasm: update V8 to 9.6.152 #18375
Conversation
The current (v9.2.230.13) contain bug which doesn't let build envoy on centos 7 Signed-off-by: Bart Smykla <[email protected]>
Hi @bartsmykla, welcome and thank you for your contribution. We will try to review your Pull Request as quickly as possible. In the meantime, please take a look at the contribution guidelines if you have not done so already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opened a PR with update to v9.5 (#18621), since it was a bit more involved than a version bump due to addition of a semi-complete Bazel support in v9.3.
@@ -808,14 +808,14 @@ REPOSITORY_LOCATIONS_SPEC = dict( | |||
project_name = "V8", | |||
project_desc = "Google’s open source high-performance JavaScript and WebAssembly engine, written in C++", | |||
project_url = "https://v8.dev", | |||
version = "9.2.230.13", | |||
version = "9.6.152", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v9.6 didn't reach chromium beta channel yet, so it might be a bit too early for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it, thank you for the PR - I'm closing this one then :-)
I'm closing this PR. Thank you for your help! |
Before merging this PR, someone with proper access (@PiotrSikora maybe?) should deploy artifacts built by https://storage.googleapis.com/envoyproxy-wee8/wee8-archive.sh to https://storage.googleapis.com/envoyproxy-wee8/wee8-9.6.152.tar.gz
Commit Message: wasm: The current (v9.2.230.13) contain bug which doesn't let build
envoy on centos 7
Additional Description: none
Risk Level: it's hard to asses by me as I don't have a lot of experience with v8 and couldn't find a good amount of information of changes between these versions
Testing: CI build checks
Docs Changes: none
Release Notes: none
Platform Specific Features: none
Fixes #18365