Commits · 7b5a4ba9fd1c59ee2c30d4d90ba5b20b4094271e · github.com / v8-riscv / node

Feb 24, 2017

doc: add changelogs for console · 7b5a4ba9

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

7b5a4ba9

doc: add changelogs for cluster · dc10c1aa

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

dc10c1aa

doc: add changelogs for cli · 7f24ec10

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

7f24ec10

doc: add changelogs for child_process · d6a3cfab

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

d6a3cfab

doc: add changelogs for buffer · 7b6e3021

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

7b6e3021

doc: add changelogs for assert · a0af3053

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

a0af3053

tools,doc: enable changelogs for items · 7cbb4b0e

Anna Henningsen authored Jan 10, 2017

PR-URL: https://github.com/nodejs/node/pull/11489


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>

7cbb4b0e

test: throw check in test-zlib-write-after-close · 817b28b4

Jason Wilson authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11482


Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

817b28b4

doc,test: args to `buffer.copy` can be Uint8Arrays · cba10205

Anna Henningsen authored Feb 22, 2017

This was semi-overlooked in https://github.com/nodejs/node/pull/10236
because it always worked and didn’t need additional changes.

Ref: https://github.com/nodejs/node/pull/10236
PR-URL: https://github.com/nodejs/node/pull/11486


Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

cba10205

test: increase coverage of vm · c7205ba9

DavidCai authored Feb 22, 2017

PR-URL: https://github.com/nodejs/node/pull/11377


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

c7205ba9

tools: enable unicode-bom ESLint rule · 5c1ec6ea

Anna Henningsen authored Feb 21, 2017

Disallow UTF-8 Byte Order Marks in the source tree.
That Node can read these files is still tested
by `test/sequential/test-module-loading.js`.

PR-URL: https://github.com/nodejs/node/pull/11479


Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

5c1ec6ea

benchmark: strip BOM in dgram/bind-params · 049709b0

Anna Henningsen authored Feb 21, 2017

PR-URL: https://github.com/nodejs/node/pull/11479


Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

049709b0

build: fail on CI if leftover processes · 189b49a4

Rich Trott authored Feb 09, 2017

If any tests leave processes running after testing results are complete,
fail the test run.

PR-URL: https://github.com/nodejs/node/pull/11269


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>

189b49a4

Feb 23, 2017

test: add support for --gtest_filter · 8e8fbd80

Daniel Bevenius authored Feb 21, 2017

It might be useful to sometimes run a single cctest while debugging/
developing. This commit adds support for Google Test's --gtest_filter
to enable this.

To list the tests available:

$ make list-gtests
UtilTest.
  ListHead
  StringEqualNoCase
  StringEqualNoCaseN
  ToLower
  Malloc
  Calloc
  UncheckedMalloc
  UncheckedCalloc
InspectorSocketTest.
  ReadsAndWritesInspectorMessage
  BufferEdgeCases
  AcceptsRequestInSeveralWrites
  ExtraTextBeforeRequest
  ExtraLettersBeforeRequest
  RequestWithoutKey
  KillsConnectionOnProtocolViolation
  CanStopReadingFromInspector
  CloseDoesNotNotifyReadCallback
  CloseWorksWithoutReadEnabled
  ReportsHttpGet
  HandshakeCanBeCanceled
  GetThenHandshake
  WriteBeforeHandshake
  CleanupSocketAfterEOF
  EOFBeforeHandshake
  Send1Mb
  ErrorCleansUpTheSocket
InspectorSocketServerTest.
  InspectorSessions
  ServerDoesNothing
  ServerWithoutTargets
  ServerCannotStart
  StoppingServerDoesNotKillConnections

Then run a single test:

$ env GTEST_FILTER=InspectorSocketTest.GetThenHandshake make cctest

Note: Google Test filter = InspectorSocketTest.GetThenHandshake
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from InspectorSocketTest
[ RUN      ] InspectorSocketTest.GetThenHandshake
[       OK ] InspectorSocketTest.GetThenHandshake (0 ms)
[----------] 1 test from InspectorSocketTest (0 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (0 ms total)
[  PASSED  ] 1 test.

PR-URL: https://github.com/nodejs/node/pull/11474


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>

8e8fbd80

src: update http-parser link · 1b651535

Daniel Bevenius authored Feb 21, 2017

I noticed that the link to http-parser is pointing to the joyent
organization. There is a redirect to the nodejs organization but
perhaps this should be updated anyway.

PR-URL: https://github.com/nodejs/node/pull/11477


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

1b651535

build: fix newlines in addon build output · 29ff16f0

Brian White authored Feb 20, 2017

Interpretation of escape sequences with echo is not as consistent
across platforms as printf, so use the latter instead.

PR-URL: https://github.com/nodejs/node/pull/11466


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

29ff16f0

fs: runtime deprecation for fs.SyncWriteStream · 7a55e34e

James M Snell authored Dec 26, 2016

This API was never intended to be made public and was docs-only
deprecated in Node.js 6.x. This upgrades to a runtime deprecation
for Node.js 8.0.0

PR-URL: https://github.com/nodejs/node/pull/10467


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

7a55e34e

domain,events: support non-object 'error' argument · 4f638f6f

Ben Noordhuis authored Feb 17, 2017

Fix a TypeError when emitting an 'error' argument with a non-object
argument (like a string) when domains are active.

Fixes: https://github.com/nodejs/help/issues/501
PR-URL: https://github.com/nodejs/node/pull/11438


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

4f638f6f

tty: avoid oob warning in TTYWrap::GetWindowSize() · 3f02b47b

Dmitry Tsvettsikh authored Feb 18, 2017

PR-URL: https://github.com/nodejs/node/pull/11454


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

3f02b47b

tls: do not crash on STARTTLS when OCSP requested · a1802e67

Fedor Indutny authored Jan 09, 2017

`TLSSocket` should not have a hard dependency on `tls.Server`, since it
may be running without it in cases like `STARTTLS`.

Fix: #10704
PR-URL: https://github.com/nodejs/node/pull/10706


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

a1802e67

lib: more consistent use of module.exports = {} model · 62e96096

James M Snell authored Feb 15, 2017

Switch to using the more efficient module.exports = {}
where possible.

PR-URL: https://github.com/nodejs/node/pull/11406


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

62e96096

lib: refactor internal/socket_list · 88c3f57c

James M Snell authored Feb 15, 2017

PR-URL: https://github.com/nodejs/node/pull/11406


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

88c3f57c

lib: refactor internal/freelist · f04387e9

James M Snell authored Feb 15, 2017

PR-URL: https://github.com/nodejs/node/pull/11406


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

f04387e9

lib: refactor internal/linkedlist · d61a5117

James M Snell authored Feb 15, 2017

PR-URL: https://github.com/nodejs/node/pull/11406


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

d61a5117

Feb 22, 2017

test: make test-intl-no-icu-data more robust · ca480719

Michaël Zasso authored Feb 16, 2017

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: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

ca480719

deps: fix gyp configuration for v8-inspector · 7c0c7baf

Michaël Zasso authored Feb 21, 2017

Cross-compiled builds need different toolsets.

PR-URL: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

7c0c7baf

deps: fix gyp build configuration for Windows · 00a2aa0a

Michaël Zasso authored Feb 14, 2017

PR-URL: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

00a2aa0a

deps: switch to v8_inspector in V8 · b30ec598

Ali Ijaz Sheikh authored Jan 17, 2017

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

PR-URL: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

b30ec598

deps: limit regress/regress-crbug-514081 v8 test · 3c53ad0d

Michael Dawson authored May 09, 2016

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: https://github.com/nodejs/node/issues/6340
PR-URL: https://github.com/nodejs/node/pull/6678


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Fedor Indutny <fedor@indutny.com>

3c53ad0d

src: update NODE_MODULE_VERSION to 53 · aae8f683

Michaël Zasso authored Jan 11, 2017

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: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

aae8f683

deps: update V8 to 5.6.326.55 · 7a77daf2

Michaël Zasso authored Feb 14, 2017

PR-URL: https://github.com/nodejs/node/pull/10992


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

7a77daf2

doc: change STYLE-GUIDE to STYLE_GUIDE · 5f08871e

Dean Coakley authored Feb 20, 2017

PR-URL: https://github.com/nodejs/node/pull/11460
Fixes: https://github.com/nodejs/node/issues/11456


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

5f08871e

lib: remove unnecessary assignments with _extend · dce37dc3

Sakthipriyan Vairamani (thefourtheye) authored Feb 14, 2017

The first parameter to `util._extend` is the target object. Assigning
the target object to the result of `util._extend` is not necessary.

PR-URL: https://github.com/nodejs/node/pull/11364



Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>

dce37dc3

2017-02-21, Version 7.6.0 (Current) · 2b5d83d4

Italo A. Casas authored Feb 14, 2017

Notable changes:

* deps:
    * update V8 to 5.5 (Michaël Zasso) [#11029](https://github.com/nodejs/node/pull/11029)
    * upgrade libuv to 1.11.0 (cjihrig) [#11094](https://github.com/nodejs/node/pull/11094)
    * add node-inspect 1.10.4 (Jan Krems) [#10187](https://github.com/nodejs/node/pull/10187)
    * upgrade zlib to 1.2.11 (Sam Roberts) [#10980](https://github.com/nodejs/node/pull/10980)
* lib: build `node inspect` into `node` (Anna Henningsen) [#10187](https://github.com/nodejs/node/pull/10187)
* crypto: Remove expired certs from CNNIC whitelist (Shigeki Ohtsu) [#9469](https://github.com/nodejs/node/pull/9469)
* inspector: add --inspect-brk (Josh Gavant) [#11149](https://github.com/nodejs/node/pull/11149)
* fs: allow WHATWG URL objects as paths (James M Snell) [#10739](https://github.com/nodejs/node/pull/10739)
* src: support UTF-8 in compiled-in JS source files (Ben Noordhuis) [#11129](https://github.com/nodejs/node/pull/11129)
* url: extend url.format to support WHATWG URL (James M Snell) [#10857](https://github.com/nodejs/node/pull/10857)

PR-URL: https://github.com/nodejs/node/pull/11185

2b5d83d4

2017-02-21, Version 6.10.0 'Boron' (LTS) · aecb0859

Myles Borins authored Jan 24, 2017

Notable Changes:

The SEMVER-MINOR changes include:

*  crypto: allow adding extra certs to well-known CAs (Sam Roberts)
   https://github.com/nodejs/node/pull/9139
*  deps: Upgrade INTL ICU to version 58 (Steven R. Loomis)
   https://github.com/nodejs/node/pull/9234
*  process: add `process.memoryUsage.external` (Fedor Indutny)
   https://github.com/nodejs/node/pull/9587
*  src: add wrapper for process.emitWarning() (Sam Roberts)
   https://github.com/nodejs/node/pull/9139

Notable SEMVER-PATCH changes include:

*  fs: cache non-symlinks in realpathSync. (Jeremy Yallop)
   https://github.com/nodejs/node/pull/10253
*  repl: allow autocompletion for scoped packages (Evan Lucas)
   https://github.com/nodejs/node/pull/10296

PR-URL: https://github.com/nodejs/node/pull/10974

aecb0859

2017-02-21, Version 4.8.0 'Argon' (LTS) · 49c5ff44

Myles Borins authored Jan 24, 2017

Notable Changes:

* child_process: add shell option to spawn() (cjihrig)
  https://github.com/nodejs/node/pull/4598
* crypto:
  * add ALPN Support (Shigeki Ohtsu)
    https://github.com/nodejs/node/pull/2564
  * allow adding extra certs to well-known CAs (Sam Roberts)
    https://github.com/nodejs/node/pull/9139
* deps:
  * v8: expose statistics about heap spaces (Ben Ripkens)
    https://github.com/nodejs/node/pull/4463
* fs: add the fs.mkdtemp() function. (Florian MARGAINE)
  https://github.com/nodejs/node/pull/5333
* process:
  * add `externalMemory` to `process` (Fedor Indutny)
    https://github.com/nodejs/node/pull/9587
  * add process.cpuUsage() (Patrick Mueller)
    https://github.com/nodejs/node/pull/10796

PR-URL: https://github.com/nodejs/node/pull/10973

49c5ff44

benchmark: add dgram bind(+/- params) benchmark · 1162e284

Vse Mozhet Byt authored Feb 11, 2017

Refs: https://github.com/nodejs/node/pull/11242
PR-URL: https://github.com/nodejs/node/pull/11313


Reviewed-By: Brian White <mscdex@mscdex.net>

1162e284

Feb 21, 2017

console: fixup `console.dir()` error handling · c969047d

Anna Henningsen authored Feb 17, 2017

Apply the `console: do not emit error events` changes properly
to `console.dir()`.

This was overlooked in f18e08d8
(https://github.com/nodejs/node/pull/9744).

Ref: https://github.com/nodejs/node/commit/f18e08d820dde161788d95a5603546ceca021e90#commitcomment-20934407
PR-URL: https://github.com/nodejs/node/pull/11443


Reviewed-By: James M Snell <jasnell@gmail.com>

c969047d

tools: enable one-var-declaration-per-line ESLint rule · 19346866

Michaël Zasso authored Feb 19, 2017

This rule enforces new lines around variable declarations. It is
configured to spot only variables that are initialized.

PR-URL: https://github.com/nodejs/node/pull/11462


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

19346866

test: add regex check to test-module-loading · a4c3e31a

Tarang Hirani authored Feb 16, 2017

Also removes extraneous argument.

PR-URL: https://github.com/nodejs/node/pull/11413


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

a4c3e31a