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: upgrade to V8 5.0.71.34 #6320

Closed
wants to merge 1 commit into from
Closed

Conversation

ofrobots
Copy link
Contributor

@ofrobots ofrobots commented Apr 21, 2016

Checklist
  • tests and code linting passes
  • a test and/or benchmark is included
  • the commit message follows commit guidelines
Affected core subsystem(s)
Description of change

Pick up the latest bug fix from the V8 5.0 branch.

Original commit message:
V8-Commit: v8/v8@c36773f

Version 5.0.71.34 (cherry-pick)
Merged 9acbca1

[es6] Fix bug in pattern re-writing

BUG=v8:4891
LOG=N
R=[email protected]

Review URL: https://codereview.chromium.org/1906633002 .

R=@nodejs/v8
/cc @jasnell
CI: https://ci.nodejs.org/job/node-test-pull-request/2350/

Pick up the latest bug fix from the V8 5.0 branch.

Original commit message:
V8-Commit: v8/v8@c36773f
  Version 5.0.71.34 (cherry-pick)
  Merged 9acbca1

  [es6] Fix bug in pattern re-writing

  BUG=v8:4891
  LOG=N
  [email protected]

  Review URL: https://codereview.chromium.org/1906633002 .
@ofrobots ofrobots mentioned this pull request Apr 21, 2016
@mscdex mscdex added the v8 engine Issues and PRs related to the V8 dependency. label Apr 21, 2016
@mscdex mscdex added this to the 6.0.0 milestone Apr 21, 2016
@cjihrig
Copy link
Contributor

cjihrig commented Apr 21, 2016

LGTM pending CI

@jasnell
Copy link
Member

jasnell commented Apr 21, 2016

LGTM

1 similar comment
@targos
Copy link
Member

targos commented Apr 21, 2016

LGTM

@jasnell
Copy link
Member

jasnell commented Apr 21, 2016

@ofrobots ... want to go ahead and get this landed today? I might take some time this afternoon to put together a v6 RC.4

@ofrobots
Copy link
Contributor Author

test-regress-GH-5727 on ARM seems to be a machine/network related issue. I will go ahead and land this.

@cjihrig
Copy link
Contributor

cjihrig commented Apr 21, 2016

@ofrobots that test has been fixed since the time you started your CI run, so it's safe to ignore.

@ofrobots
Copy link
Contributor Author

Thanks. Landed as 49e42c5.

@ofrobots ofrobots closed this Apr 21, 2016
ofrobots added a commit that referenced this pull request Apr 21, 2016
Pick up the latest bug fix from the V8 5.0 branch.

Original commit message:
V8-Commit: v8/v8@c36773f
  Version 5.0.71.34 (cherry-pick)
  Merged 9acbca1

  [es6] Fix bug in pattern re-writing

  BUG=v8:4891
  LOG=N
  [email protected]

  Review URL: https://codereview.chromium.org/1906633002 .

PR-URL: #6320
Reviewed-By: cjihrig - Colin Ihrig <[email protected]>
Reviewed-By: jasnell - James M Snell <[email protected]>
Reviewed-By: targos - Michaël Zasso <[email protected]>
@ofrobots ofrobots deleted the 5.0.71.34 branch April 21, 2016 17:07
joelostrowski pushed a commit to joelostrowski/node that referenced this pull request Apr 25, 2016
Pick up the latest bug fix from the V8 5.0 branch.

Original commit message:
V8-Commit: v8/v8@c36773f
  Version 5.0.71.34 (cherry-pick)
  Merged 9acbca1

  [es6] Fix bug in pattern re-writing

  BUG=v8:4891
  LOG=N
  [email protected]

  Review URL: https://codereview.chromium.org/1906633002 .

PR-URL: nodejs#6320
Reviewed-By: cjihrig - Colin Ihrig <[email protected]>
Reviewed-By: jasnell - James M Snell <[email protected]>
Reviewed-By: targos - Michaël Zasso <[email protected]>
jasnell pushed a commit that referenced this pull request Apr 26, 2016
Pick up the latest bug fix from the V8 5.0 branch.

Original commit message:
V8-Commit: v8/v8@c36773f
  Version 5.0.71.34 (cherry-pick)
  Merged 9acbca1

  [es6] Fix bug in pattern re-writing

  BUG=v8:4891
  LOG=N
  [email protected]

  Review URL: https://codereview.chromium.org/1906633002 .

PR-URL: #6320
Reviewed-By: cjihrig - Colin Ihrig <[email protected]>
Reviewed-By: jasnell - James M Snell <[email protected]>
Reviewed-By: targos - Michaël Zasso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants