★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/21727Nouvelle recherche | Portail wanayoo
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: patch V8 to 6.7.288.49 #21727

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
6 participants
@MylesBorins
Copy link
Member

MylesBorins commented Jul 9, 2018

Refs: v8/v8@6.7.288.46...6.7.288.49

@addaleax

This comment has been minimized.

Copy link
Member

addaleax commented Jul 9, 2018

It doesn’t look like these changes affect us, right?

I guess it’s not an issue if you’re a Googler, but if your dev machine is near the lower end, recompiling all of V8 can be a real pain…

@MylesBorins

This comment has been minimized.

Copy link
Member Author

MylesBorins commented Jul 9, 2018

@addaleax we can hold off on landing this one and wait for changes that are more meaningful, generally trying to just keep us up to date with LKGR. Do these changes force a recompile?

@addaleax

This comment has been minimized.

Copy link
Member

addaleax commented Jul 9, 2018

Do these changes force a recompile?

I guess not, no … seems okay to me then?

@jasnell
Copy link
Member

jasnell left a comment

rubber-stamp lgtm

@MylesBorins

This comment has been minimized.

Copy link
Member Author

MylesBorins commented Jul 13, 2018

landed in 9374a83

MylesBorins added a commit that referenced this pull request Jul 13, 2018

deps: patch V8 to 6.7.288.49
PR-URL: #21727
Refs: v8/v8@6.7.288.46...6.7.288.49
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

targos added a commit that referenced this pull request Jul 14, 2018

deps: patch V8 to 6.7.288.49
PR-URL: #21727
Refs: v8/v8@6.7.288.46...6.7.288.49
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

@targos targos referenced this pull request Jul 17, 2018

Merged

v10.7.0 proposal #21851

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.