Skip to content

Conversation

@jeisinger
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

deps/v8

Description of change

This is an non-ABI breaking version port of
https://chromium.googlesource.com/v8/v8/+/46428e45e9982a7490685ff1af6ffe680096c1a4
which adds an API that will be used by the inspector code.

This is an non-ABI breaking version port of https://chromium.googlesource.com/v8/v8/+/46428e45e9982a7490685ff1af6ffe680096c1a4 which adds an API that will be used by the inspector code.
@nodejs-github-bot nodejs-github-bot added the v8 engine Issues and PRs related to the V8 dependency. label Jul 7, 2016
@jeisinger
Copy link
Contributor Author

@nodejs/v8

@bnoordhuis
Copy link
Member

LGTM but can't it land in 5.1? That way we can pick it up through a regular upgrade.

@jeisinger
Copy link
Contributor Author

@natorion wdyt

Ben Noordhuis notifications@github.com schrieb am Do., 7. Juli 2016,
17:28:

LGTM but can't it land in 5.1? That way we can pick it up through a
regular upgrade.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#7586 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAL88kQ2rzwwr8-AFR2q2pVIhiBysUP0ks5qTRsegaJpZM4JHMdP
.

@natorion
Copy link

natorion commented Jul 8, 2016

Unfortunately V8 switched its branch builders to 5.2 and 5.3 already (V8 branched last week). There is no test coverage from the V8 side anymore and 5.1 is abandoned.

@MylesBorins
Copy link
Contributor

@natorion this could be included in the LTS branch no?

@natorion
Copy link

natorion commented Jul 8, 2016

Change of plans. @jeisinger can merge this one to 5.1 on the V8 repo and I will manually trigger the builders and have a look at the test results.

@jeisinger jeisinger closed this Jul 8, 2016
bnoordhuis added a commit to bnoordhuis/io.js that referenced this pull request Jul 9, 2016
Pick up the latest branch-head for V8 5.1. Introduces a semver-minor overload of v8::Function::New() for use by v8_inspector. Refs: nodejs#7586 PR-URL: nodejs#7615 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com> Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com>
ofrobots added a commit to ofrobots/node that referenced this pull request Aug 25, 2016
Pick up the latest branch-head for V8 5.1. This branch brings in improved language support and performance improvements. For full details: http://v8project.blogspot.com/2016/04/v8-release-51.html * Picks up the latest branch head for 5.1 [1] * Edit v8 gitignore to allow trace_event copy * Update V8 DEP trace_event as per deps/v8/DEPS [2] [1] https://chromium.googlesource.com/v8/v8.git/+/5.1.281.75 [2] https://chromium.googlesource.com/chromium/src/base/trace_event/common/+/c8c8665 Introduces a semver-minor overload of v8::Function::New() for use by v8_inspector. PR-URL: nodejs#8054 Refs: nodejs#7016 Refs: nodejs#7586 Refs: nodejs#7615 Reviewed-By: addaleax - Anna Henningsen <anna@addaleax.net> Reviewed-By: thealphanerd - Myles Borins <myles.borins@gmail.com> Reviewed-By: mhdawson - Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: evanlucas - Evan Lucas <evanlucas@me.com> Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
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.

5 participants