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.

Sign up for GitHub

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

Jump to bottom

deps: update V8 to 5.6 #10992

Merged
merged 7 commits into from
Feb 22, 2017
Merged

deps: update V8 to 5.6 #10992

merged 7 commits into from
Feb 22, 2017

Commits on Feb 22, 2017

  1. deps: update V8 to 5.6.326.55 

    PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @targos
    targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    7a77daf View commit details
    Browse the repository at this point in the history

  2. src: update NODE_MODULE_VERSION to 53 

    V8 5.6 is not API/ABI compatible with 5.5.
This commit increments NODE_MODULE_VERSION by one.

Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @targos
    targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    aae8f68 View commit details
    Browse the repository at this point in the history

  3. deps: limit regress/regress-crbug-514081 v8 test 

    regress/regress-crbug-514081 allocates a 2G block of memory
and if there  are multiple variants running at the
same time this can lead to crashes, OOM kills or
the OS failing to allocate memory.  This patch
limits us to running a single variant of the test

Fixes: nodejs#6340
PR-URL: nodejs#6678
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor@indutny.com>
    @mhdawson @targos
    mhdawson authored and targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    3c53ad0 View commit details
    Browse the repository at this point in the history

  4. deps: switch to v8_inspector in V8 

    * Delete deps/v8_inspector
* Start building v8_inspector from V8
* Remove licenses related to the deleted code from LICENSE file

PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @ofrobots @targos
    ofrobots authored and targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    b30ec59 View commit details
    Browse the repository at this point in the history

  5. deps: fix gyp build configuration for Windows 

    PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @targos
    targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    00a2aa0 View commit details
    Browse the repository at this point in the history

  6. deps: fix gyp configuration for v8-inspector 

    Cross-compiled builds need different toolsets.

PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @targos
    targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    7c0c7ba View commit details
    Browse the repository at this point in the history

  7. test: make test-intl-no-icu-data more robust 

    In V8 5.6, String#toLocaleUpperCase can work even when no ICU data is
loaded. Use another method to check the --icu-data-dir option pointing
to an empty directory.

PR-URL: nodejs#10992
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    @targos
    targos committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    ca48071 View commit details
    Browse the repository at this point in the history