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

v13.9.0 proposal #31837

Merged
merged 221 commits into from Feb 18, 2020
Merged

v13.9.0 proposal #31837

merged 221 commits into from Feb 18, 2020

Conversation

codebytere
Copy link
Member

@codebytere codebytere commented Feb 17, 2020

2020-02-18, Version 13.9.0 (Current), @codebytere

Notable changes

  • async_hooks
    • add executionAsyncResource (Matteo Collina) #30959
  • crypto
    • add crypto.diffieHellman (Tobias Nießen) #31178
    • add DH support to generateKeyPair (Tobias Nießen) #31178
    • simplify DH groups (Tobias Nießen) #31178
    • add key type 'dh' (Tobias Nießen) #31178
  • test
    • skip keygen tests on arm systems (Tobias Nießen) #31178
  • perf_hooks
    • add property flags to GCPerformanceEntry (Kirill Fomichev) #29547
  • process
    • report ArrayBuffer memory in memoryUsage() (Anna Henningsen) #31550
  • readline
    • make tab size configurable (Ruben Bridgewater) #31318
  • report
    • add support for Workers (Anna Henningsen) #31386
  • worker
    • add ability to take heap snapshot from parent thread (Anna Henningsen) #31569
  • added new collaborators
    • add ronag to collaborators (Robert Nagy) #31498

Commits

  • [2db7593838] - assert: align character indicators properly (Ruben Bridgewater) #31429
  • [a840e9d639] - async_hooks: ensure event after been emitted on runInAsyncScope (legendecas) #31784
  • [6be51296e4] - (SEMVER-MINOR) async_hooks: add executionAsyncResource (Matteo Collina) #30959
  • [2de085fe93] - benchmark: use let instead of var (Daniele Belardi) #31592
  • [e37f5100e5] - benchmark: swap var for let in benchmarks (Alex Ramirez) #28958
  • [819fb76ba5] - Revert "benchmark: refactor helper into a class" (Anna Henningsen) #31722
  • [8974fa794c] - Revert "benchmark: add test and all options and improve errors" (Anna Henningsen) #31722
  • [30f55cebb6] - Revert "benchmark: remove special test entries" (Anna Henningsen) #31722
  • [1484f5ab6e] - benchmark: remove special test entries (Ruben Bridgewater) #31396
  • [ca343caee3] - benchmark: add test and all options and improve errors (Ruben Bridgewater) #31396
  • [9f2c742626] - benchmark: refactor helper into a class (Ruben Bridgewater) #31396
  • [161db608ae] - benchmark: check for and fix multiple end() (Brian White) #31624
  • [6fe8eda3ca] - benchmark: clean up config resolution in multiple benchmarks (Denys Otrishko) #31581
  • [ebdcafafeb] - benchmark: add MessagePort benchmark (Anna Henningsen) #31568
  • [eb3c6e9127] - benchmark: use let and const instead of var (Daniele Belardi) #31518
  • [b29badad81] - benchmark: fix getStringWidth() benchmark (Rich Trott) #31476
  • [519134ddb0] - buffer: improve from() performance (Brian White) #31615
  • [769154de07] - buffer: improve concat() performance (Brian White) #31522
  • [9d45393e95] - buffer: improve fill(number) performance (Brian White) #31489
  • [60a69770f5] - build: add configure option to debug only Node.js part of the binary (Anna Henningsen) #31644
  • [10f9abe81d] - build: ignore all the "Debug","Release" folders (ConorDavenport) #31565
  • [03eade01d7] - build: enable loading internal modules from disk (Gus Caplan) #31321
  • [a2b7006847] - build: build docs in GitHub Actions CI workflow (Richard Lau) #31504
  • [2e216aebcb] - build: do not use setup-node in build workflows (Richard Lau) #31349
  • [825d089763] - crypto: fix performance regression (Robert Nagy) #31742
  • [3c6545f0b4] - crypto: improve randomBytes() performance (Brian White) #31519
  • [f84b34d42c] - crypto: improve errors in DiffieHellmanGroup (Tobias Nießen) #31445
  • [4591202e66] - crypto: assign and use ERR_CRYPTO_UNKNOWN_CIPHER (Tobias Nießen) #31437
  • [bf46c304dd] - (SEMVER-MINOR) crypto: add crypto.diffieHellman (Tobias Nießen) #31178
  • [0d3e095941] - (SEMVER-MINOR) crypto: add DH support to generateKeyPair (Tobias Nießen) #31178
  • [15bd2c9f0c] - (SEMVER-MINOR) crypto: simplify DH groups (Tobias Nießen) #31178
  • [572322fddf] - (SEMVER-MINOR) crypto: add key type 'dh' (Tobias Nießen) #31178
  • [0ac124b6b9] - deps: upgrade npm to 6.13.7 (Michael Perrotte) #31558
  • [bf7097c77d] - deps: switch to chromium's zlib implementation (Brian White) #31201
  • [2eeaa5ce40] - deps: uvwasi: cherry-pick 7b5b6f9 (cjihrig) #31495
  • [464f4afa66] - deps: upgrade to libuv 1.34.2 (cjihrig) #31477
  • [9811ebe0c5] - deps: uvwasi: cherry-pick eea4508 (cjihrig) #31432
  • [2fe0ed3a2e] - deps: uvwasi: cherry-pick c3bef8e (cjihrig) #31432
  • [09566be899] - deps: uvwasi: cherry-pick ea73af5 (cjihrig) #31432
  • [04f2799ed2] - deps: update to uvwasi 0.0.5 (cjihrig) #31432
  • [7c4f1ed030] - deps: uvwasi: cherry-pick 941bedf (cjihrig) #31363
  • [00e38a749a] - deps: port uvwasi@676ba9a to gyp (cjihrig) #31363
  • [5bd3f6c258] - deps,test: update to uvwasi 0.0.4 (cjihrig) #31363
  • [2cd8461e56] - doc: add glossary.md (gengjiawen) #27517
  • [c4613c6b8b] - doc: add prerequisites information for Arch (Ujjwal Sharma) #31669
  • [b35f83e69b] - doc: fix typo on fs docs (Juan José Arboleda) #31620
  • [2ff812ca84] - doc: update contact email for @ryzokuken (Ujjwal Sharma) #31670
  • [2c83946757] - doc: fix default server timeout description for https (Andrey Pechkurov) #31692
  • [b56a21fdad] - doc: add directions to mark a release line as lts (Danielle Adams) #31724
  • [5ae40cd2b2] - doc: expand C++ README with information about exception handling (Anna Henningsen) #31720
  • [94a0ec1b99] - doc: update foundation name in onboarding (Tobias Nießen) #31719
  • [fda97fa772] - doc: reword possessive form of Node.js in zlib.md (Rich Trott) #31713
  • [eea58cd3d5] - doc: reword possessive form of Node.js in modules.md (Rich Trott) #31713
  • [d0238190a1] - doc: reword possessive form of Node.js in repl.md (Rich Trott) #31713
  • [55a25b3bbe] - doc: reword section title in addons.md (Rich Trott) #31713
  • [ba9fae058a] - doc: revise deepEqual() legacy assertion mode text (Rich Trott) #31704
  • [f6d78f959f] - doc: improve strict assertion mode color text (Rich Trott) #31703
  • [22cf3e3d4e] - doc: consolidate introductory text (Rich Trott) #31667
  • [1e2327d9e6] - doc: simplify async_hooks overview (Rich Trott) #31660
  • [77ec381ea2] - doc: clarify Worker exit/message event ordering (Anna Henningsen) #31642
  • [4b0085c7e3] - doc: update TSC name in "Release Process" (Tobias Nießen) #31652
  • [2e6c737281] - doc: remove .github/ISSUE_TEMPLATE.md in favor of the template folder (Joyee Cheung) #31656
  • [b61b85ccf9] - doc: add note in BUILDING.md about running make distclean (Swagat Konchada) #31542
  • [2991e7c0e3] - doc: correct getting an ArrayBuffer's length (tsabolov) #31632
  • [e27f24987e] - doc: ask more questions in the bug report template (Joyee Cheung) #31611
  • [b50a6cc54d] - doc: add example to fs.promises.readdir (Conor ONeill) #31552
  • [1dbe765b0b] - doc: add AsyncResource + Worker pool example (Anna Henningsen) #31601
  • [f40264980e] - doc: fix numbering (Steffen) #31575
  • [3ba0a22c57] - doc: clarify socket.setNoDelay() explanation (Rusty Conover) #31541
  • [faec87b7f1] - doc: list largepage values in --help (cjihrig) #31537
  • [2638110cce] - doc: clarify require() OS independence (Denys Otrishko) #31571
  • [7fe9d5ebd4] - doc: add protocol option in http2.connect() (Michael Lumish) #31560
  • [6626c4de3c] - doc: clarify that v8.serialize() is not deterministic (Anna Henningsen) #31548
  • [cde4b51a92] - doc: update job reference in COLLABORATOR_GUIDE.md (Richard Lau) #31557
  • [4cac2cccd6] - doc: simultaneous blog and email of sec announce (Sam Roberts) #31483
  • [e2b3e4e0e3] - doc: update collaborator guide citgm instructions (Robert Nagy) #31549
  • [43186e0046] - doc: change error message testing policy (Tobias Nießen) #31421
  • [a52df55b9a] - doc: remove redundant properties from headers (XhmikosR) #31492
  • [04d783ae71] - doc: update maintaining-V8.md (kenzo-spaulding) #31503
  • [f75fe9ab71] - doc: enable visual code indication in headers (Rich Trott) #31493
  • [8f25e51e4e] - doc: clean up and streamline vm.md examples (Denys Otrishko) #31474
  • [729b96137e] - doc: further fix async iterator example (Robert Nagy) #31367
  • [15b24b71ce] - doc: add ronag to collaborators (Robert Nagy) #31498
  • [e9462b4d44] - doc: fix code display in header glitch (Rich Trott) #31460
  • [b1c745877b] - doc: fix syntax in N-API documentation (Tobias Nießen) #31466
  • [67d8967f98] - doc: add explanatory to path.resolve description (Yakov Litvin) #31430
  • [1099524452] - doc: document process.std*.fd (Harshitha KP) #31395
  • [843c5c6f46] - doc: fix several child_process doc typos (cjihrig) #31393
  • [d77099856a] - doc: fix a broken link in fs.md (himself65) #31373
  • [1e08d3c2f1] - doc: correct added version for --abort-on-uncaught-exception (Anna Henningsen) #31360
  • [6055134db6] - doc: explain hex encoding in Buffer API (Harshitha KP) #31352
  • [bd54abe3f7] - doc: explain _writev() API (Harshitha KP) #31356
  • [91f5e9b0f7] - doc: document missing properties in child_process (Harshitha KP) #31342
  • [6874deef28] - doc,assert: rename "mode" to "assertion mode" (Rich Trott) #31635
  • [788ea36ce0] - doc,net: reword Unix domain path paragraph in net.md (Rich Trott) #31684
  • [e3e40a12b0] - doc,util: revise util.md introductory paragraph (Rich Trott) #31685
  • [e46cfaf146] - errors: make use of "cannot" consistent (Tobias Nießen) #31420
  • [f6392e9fde] - esm: import.meta.resolve with nodejs: builtins (Guy Bedford) #31032
  • [21fc81821f] - fs: set path when mkdir recursive called on file (bcoe) #31607
  • [8669ecc8a2] - fs: bail on permission error in recursive directory creation (bcoe) #31505
  • [2c2b3ba39c] - fs: do not emit 'close' twice if emitClose enabled (Robert Nagy) #31383
  • [32ac1be372] - fs: unset FileHandle fd after close (Anna Henningsen) #31389
  • [9ecae58643] - lib: delete dead code in SourceMap (Justin Ridgewell) #31512
  • [7ecf842429] - lib,src: switch Buffer::kMaxLength to size_t (Ben Noordhuis) #31406
  • [15c8d9ead1] - meta: move princejwesley to emeritus (Rich Trott) #31730
  • [f5ae510e03] - meta: move vkurchatkin to emeritus (Rich Trott) #31729
  • [cd520ddfef] - meta: move calvinmetcalf to emeritus (Rich Trott) #31736
  • [832255df89] - meta: fix collaborator list errors in README.md (James M Snell) #31655
  • [aa266628ba] - module: drop support for extensionless main entry points in esm (Geoffrey Booth) #31415
  • [ca81af7d73] - module: correct docs about when extensionless files are supported (Geoffrey Booth) #31415
  • [6797656d86] - module: revert esm: empty ext from pkg type/main doesnt affect format #31021 (Geoffrey Booth) #31415
  • [ae2141effc] - n-api: free instance data as reference (Gabriel Schulhof) #31638
  • [c8215699ab] - n-api: rename 'promise' parameter to 'value' (Tobias Nießen) #31544
  • [5982726ef9] - net: track state of setNoDelay() and prevent unnecessary system calls (Rusty Conover) #31543
  • [e7fea14c7b] - (SEMVER-MINOR) perf_hooks: add property flags to GCPerformanceEntry (Kirill Fomichev) #29547
  • [672315651d] - (SEMVER-MINOR) process: report ArrayBuffer memory in memoryUsage() (Anna Henningsen) #31550
  • [cd754337f8] - process: fix two overflow cases in SourceMap VLQ decoding (Justin Ridgewell) #31490
  • [98f3028c30] - readline: remove intermediate variable (cjihrig) #31676
  • [148dfde1d4] - (SEMVER-MINOR) readline: make tab size configurable (Ruben Bridgewater) #31318
  • [1bcf2f9423] - report: add support for Workers (Anna Henningsen) #31386
  • [7c2d33f38f] - src: use hex not decimal in IsArrayIndex (Shelley Vohr) #31758
  • [a095ef0d52] - src: keep main-thread Isolate attached to platform during Dispose (Anna Henningsen) #31795
  • [1dec9d196f] - src: wrap HostPort in ExclusiveAccess (Ben Noordhuis) #31717
  • [e23023d685] - src: add ExclusiveAccess class (Ben Noordhuis) #31717
  • [54caf76210] - src: allow to reuse env options handling (Denys Otrishko) #31711
  • [6ad8ca5ecf] - src: do not unnecessarily re-assign uv handle data (Anna Henningsen) #31696
  • [2837788849] - src: fix compile warnings in node_url.cc (Anna Henningsen) #31689
  • [1d34ab5e43] - src: modernized unique_ptr construction (Yuhanun Citgez) #31654
  • [0e44902b85] - src: remove dead code in InternalMakeCallback (Gerhard Stoebich) #31622
  • [348c7871b6] - src: remove fixed-size GetHumanReadableProcessName (Ben Noordhuis) #31633
  • [8964077935] - src: fix OOB reads in process.title getter (Ben Noordhuis) #31633
  • [af612bcc21] - src: various minor improvements to node_url (James M Snell) #31651
  • [f0ffa4cb80] - src: fix inspecting MessagePort from init async hook (Anna Henningsen) #31600
  • [425662e2d6] - src: remove unused Worker::child\_port\_ member (Anna Henningsen) #31599
  • [43e2c2e643] - src: change Fill() to use ParseArrayIndex() (ConorDavenport) #31591
  • [42b835412d] - src: remove duplicate field env in CryptoJob class (ConorDavenport) #31554
  • [9fd1e717e6] - src: fix console debug output on Windows (Denys Otrishko) #31580
  • [277980d288] - src: use __executable_start for linux hugepages (Ben Noordhuis) #31547
  • [6d5c3cd7ac] - src: remove preview for heap dump utilities (Anna Henningsen) #31570
  • [c167ae0a87] - src: fix minor typo in base_object.h (Daniel Bevenius) #31535
  • [f04576ede0] - src: fix debug crash handling null strings (Rusty Conover) #31523
  • [ef4d081660] - src: simplify native immediate queue running (Anna Henningsen) #31502
  • [bc0c1420f0] - src: define noreturn attribute for windows (Alexander Smarus) #31467
  • [9e9dbd44fe] - src: reduce code duplication in BootstrapNode (Denys Otrishko) #31465
  • [76aad0e5e1] - src: use custom fprintf alike to write errors to stderr (Anna Henningsen) #31446
  • [a685827a55] - src: add C++-style sprintf utility (Anna Henningsen) #31446
  • [049a1727d4] - src: harden running native SetImmediate()s slightly (Anna Henningsen) #31468
  • [f56de5a3b4] - src: move MemoryInfo() for worker code to .cc files (Anna Henningsen) #31386
  • [0cacc1facf] - src: add interrupts to Environments/Workers (Anna Henningsen) #31386
  • [f8c45b277f] - src: remove AsyncRequest (Anna Henningsen) #31386
  • [600e96ec04] - src: add a threadsafe variant of SetImmediate() (Anna Henningsen) #31386
  • [74a7cdbe05] - src: exclude C++ SetImmediate() from count (Anna Henningsen) #31386
  • [53e566bc50] - src: better encapsulate native immediate list (Anna Henningsen) #31386
  • [b8face28e7] - src: reduce large pages code duplication (Gabriel Schulhof) #31385
  • [83dd65a469] - src: fix ignore GCC -Wcast-function-type for older compilers (Denys Otrishko) #31524
  • [13c6965703] - src: ignore GCC -Wcast-function-type for v8.h (Daniel Bevenius) #31475
  • [3dd4089b9a] - (SEMVER-MINOR) src,lib: make ^C print a JS stack trace (legendecas) #29207
  • [6d0b2267ce] - stream: fix finished w/ 'close' before 'finish' (Robert Nagy) #31534
  • [80e75ab389] - stream: add regression test for async iteration completion (Matteo Collina) #31508
  • [538582b43d] - Revert "stream: fix async iterator destroyed error propagation" (Matteo Collina) #31508
  • [f255053033] - stream: fix finished writable/readable state (Robert Nagy) #31527
  • [3046648580] - stream: implement throw for async iterator (Robert Nagy) #31316
  • [5a95fa4aeb] - stream: normalize async iterator stream destroy (Robert Nagy) #31316
  • [20d0a0e9a7] - stream: add async iterator support for v1 streams (Robert Nagy) #31316
  • [0654e6790d] - test: mark test-fs-stat-bigint flaky on FreeBSD (Rich Trott) #31728
  • [6dbe6bde56] - test: fix flaky parallel/test-repl-history-navigation test (Ruben Bridgewater) #31708
  • [1dae7dc6bc] - test: improve test-fs-stat-bigint (Rich Trott) #31726
  • [fa9b59276d] - test: remove flaky designation for test-fs-stat-bigint (Rich Trott) #30437
  • [d36ba2b555] - test: fix flaky test-fs-stat-bigint (Duncan Healy) #30437
  • [5b3c4b3e7d] - Revert "test: refactor all benchmark tests to use the new test option" (Anna Henningsen) #31722
  • [2c0f3028c9] - test: add debugging output to test-net-listen-after-destroy-stdin (Rich Trott) #31698
  • [2224211609] - test: improve assertion message in test-dns-any (Rich Trott) #31697
  • [b0e37b7180] - test: fix flaky test-trace-sigint-on-idle (Anna Henningsen) #31645
  • [58f17c0e6b] - test: stricter assert color test (Ruben Bridgewater) #31429
  • [89dcf733c6] - test: improve logged errors (Ruben Bridgewater) #31425
  • [4878c7a197] - test: refactor all benchmark tests to use the new test option (Ruben Bridgewater) #31396
  • [3bcc2da887] - test: fix test-benchmark-http (Rich Trott) #31686
  • [6139d4ea3b] - test: fix flaky test-inspector-connect-main-thread (Anna Henningsen) #31637
  • [13c256d31d] - test: add test-dns-promises-lookupService (Rich Trott) #31640
  • [23fefba84c] - test: fix flaky test-http2-stream-destroy-event-order (Anna Henningsen) #31610
  • [435b9c977a] - test: abstract common assertions in readline-interface test (Ruben Bridgewater) #31423
  • [d2a12d3af8] - test: refactor test-readline-interface.js (Ruben Bridgewater) #31423
  • [7c3cc94b9f] - test: unset NODE_OPTIONS for cctest (Anna Henningsen) #31594
  • [62d0c6029d] - test: simplify test-https-simple.js (Sam Roberts) #31584
  • [49be50051c] - test: show child stderr output in largepages test (Ben Noordhuis) #31612
  • [c3247fedd9] - test: mark additional tests as flaky on Windows (Anna Henningsen) #31606
  • [3fdec1c790] - test: fix flaky test-memory-usage (Anna Henningsen) #31602
  • [23da559ab2] - test: verify threadId in reports (Dylan Coakley) #31556
  • [5a12cd636b] - test: remove --experimental-worker flag comment (Harshitha KP) #31563
  • [07525c317e] - test: make test-http2-buffersize more correct (Anna Henningsen) #31502
  • [c4a2f94a11] - test: cover property n-api null cases (Gabriel Schulhof) #31488
  • [f2dc694805] - test: fix test-heapdump-worker (Anna Henningsen) #31494
  • [b25ea9b1dc] - test: add tests for main() argument handling (cjihrig) #31426
  • [38ea53629b] - test: add wasi test for freopen() (cjihrig) #31432
  • [c2792aad44] - test: remove bluebird remnants from test fixture (Rich Trott) #31435
  • [583d1d9f55] - test: improve wasi stat test (cjihrig) #31413
  • [676b84a803] - (SEMVER-MINOR) test: skip keygen tests on arm systems (Tobias Nießen) #31178
  • [099c921f40] - test: add wasi test for symlink() and readlink() (cjihrig) #31403
  • [6256d0ae92] - test: update postmortem test with v12 constants (Matheus Marchini) #31391
  • [0bafb5c8c8] - test: export public symbols in addons tests (Ben Noordhuis) #28717
  • [6833f62e9d] - test: add promises metadata to postmortem test (Matheus Marchini) #31357
  • [41524282b5] - test,benchmark: fix test-benchmark-zlib (Rich Trott) #31538
  • [c34872e464] - test,dns: add coverage for dns exception (Rich Trott) #31678
  • [03aac4e65d] - tls: simplify errors using ThrowCryptoError (Tobias Nießen) #31436
  • [95d509e974] - tools: update Markdown linter to be cross-platform (Derek Lewis) #31239
  • [328b8a6444] - tools: unify make-v8.sh for ppc64le and s390x (Richard Lau) #31628
  • [39c86bbe4c] - tools: replace deprecated iteritems() for items() (Giovanny Andres Gongora Granada (Gioyik)) #31528
  • [be55f3ec4f] - tty: do not end in an infinite warning recursion (Ruben Bridgewater) #31429
  • [a0c1ceddbc] - util: throw if unreachable TypedArray checking code is reached (Rich Trott) #31737
  • [7b9d6d08f4] - util: add coverage for util.inspect.colors alias setter (Rich Trott) #31743
  • [9f9edc2c78] - util: throw if unreachable code is reached (Rich Trott) #31712
  • [5e1bee817c] - util: fix inspection of typed arrays with unusual length (Ruben Bridgewater) #31458
  • [3da4d5174c] - util: improve unicode support (Ruben Bridgewater) #31319
  • [822f2ac640] - worker: add support for .cjs extension (Antoine du HAMEL) #31662
  • [cd99dc7368] - worker: properly handle env and NODE_OPTIONS in workers (Denys Otrishko) #31711
  • [1592c474da] - worker: reset Isolate stack limit after entering Locker (Anna Henningsen) #31593
  • [3e5803f91b] - worker: improve MessagePort performance (Anna Henningsen) #31605
  • [8d3ffbeb55] - (SEMVER-MINOR) worker: add ability to take heap snapshot from parent thread (Anna Henningsen) #31569
  • [6fdef457c6] - worker: remove redundant closing of child port (aaccttrr) #31555
  • [5656ec9f71] - worker: move JoinThread() back into exit callback (Anna Henningsen) #31468

danbev and others added 30 commits February 17, 2020 10:18
This commit suggests that cast-function-type warnings be ignored
from v8.h. Currently, GCC reports a number of warnings like this:

In file included from ../src/util.h:27,
                 from ../src/aliased_buffer.h:7,
                 from ../src/memory_tracker.h:5,
                 from ../src/base_object.h:27,
                 from ../src/async_wrap.h:27,
                 from ../src/req_wrap.h:6,
                 from ../src/req_wrap-inl.h:6,
                 from ../src/connect_wrap.h:6,
                 from ../src/connect_wrap.cc:1:
../deps/v8/include/v8.h: In instantiation of
‘void v8::PersistentBase<T>::SetWeak(
    P*,
    typename v8::WeakCallbackInfo<P>::Callback, v8::WeakCallbackType)
[with
P = node::BaseObject;
T = v8::Object;
typename v8::WeakCallbackInfo<P>::Callback =
    void (*)(const v8::WeakCallbackInfo<node::BaseObject>&)]’:

../src/base_object-inl.h:123:42:   required from here
../deps/v8/include/v8.h:10374:16: warning:
cast between incompatible function types from
‘v8::WeakCallbackInfo<node::BaseObject>::Callback’
{aka ‘void (*)(const v8::WeakCallbackInfo<node::BaseObject>&)’} to
‘Callback’
{aka ‘void (*)(const v8::WeakCallbackInfo<void>&)’}
[-Wcast-function-type]
                reinterpret_cast<Callback>(callback), type);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The motivation for doing this that it makes it difficult to spot
other warnings that might be important. Since it is v8 that
performs this cast I was not able to find a way around it.

PR-URL: #31475
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Fixes: #31517

PR-URL: #31524
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
The setup-node GitHub Action installs problem matchers that happen
to match the warning message format of Visual Studio's C/C++ compiler.
This is resulting in all of our pull requests being annotated with
`Unchanged files with check annotations` which are confusing to new
contributors as they are not due to the changes in the pull request.

The action is used to run `npx envinfo` to dump some information into
the logs before the actual build. All GitHub hosted runners already
have a version of Node.js installed (12.x at the time of this commit)
which we can use to run `envinfo`. Remove the action to avoid using
the problematic problem matcher.

PR-URL: #31349
Fixes: #31347
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Add documentation for subprocess.exitCode, subprocess.signalCode,
subprocess.spawnargs, and subprocess.spawnfile.

PR-URL: #31342
Fixes: #27206
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
The exact context of invocation of _writev API is not well known.
Also, the choice between _write and _writev is not well known.
Add a description to make it explicit.

Fixes: #28408
Refs: #28690

Co-authored-by: Parker Bjur <bjur.parker45@gmail.com>

PR-URL: #31356
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
type_JSPromise__JS_PROMISE_TYPE and
type_JSMessageObject__JS_MESSAGE_OBJECT_TYPE will be used on llnode to
identify Promises in memory and core dumps:
nodejs/llnode#272. Add these to our postmortem
test so we're aware of any changes to this metadata.

PR-URL: #31357
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
fixes: #29786
refs: #29792
refs: #24491

PR-URL: #31352
Fixes: #29786
Refs: #29792
Refs: #24491
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit updates the uvwasi dependency to version 0.0.4. The
most notable change is a refactor of the way paths are resolved.
All paths, including symlinks, are now resolved in terms of
sandboxed paths instead of leaking host system paths.

PR-URL: #31363
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
This commit ports nodejs/uvwasi@676ba9a
from CMake to gyp.

PR-URL: #31363
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Original commit message:

    nodejs/uvwasi@20fd9e2 was
    accidentally reverted in
    nodejs/uvwasi#73. This commit
    relands it.

PR-URL: #31363
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
PR-URL: #31360
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #31373
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Upcoming changes to node-gyp will turn on `-fvisibility=hidden` on
macOS. Ensure that public symbols that are dlsym'd have default
visibility.

Refs: #28647
Refs: nodejs/node-gyp#1828

PR-URL: #28717
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
- Do not set the fd as a property on the native object.
- Use the already-existent `GetFD()` method to pass the
  fd from C++ to JS.
- Cache the fd in JS to avoid repeated accesses to the
  C++ getter.
- Set the fd to `-1` after close, thus reliably making
  subsequent calls using the `FileHandle` return `EBADF`.

Fixes: #31361

PR-URL: #31389
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Ref: nodejs/llnode#330

PR-URL: #31391
Refs: nodejs/llnode#330
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Declaring and initializing a `struct text_region` is common to all
three implementations of the large pages mapping code. Let's make it
unconditional.

PR-URL: #31385
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: JungMinu - Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Gus Caplan <me@gus.host>
PR-URL: #31393
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
fs streams have some backwards compat behavior that does not
behave well if emitClose: true is passed in options. This
fixes this edge case until the backwards compat is removed.

PR-URL: #31383
Fixes: #31366
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This test provides missing coverage for __wasi_path_symlink()
and __wasi_path_readlink().

PR-URL: #31403
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Fixes: #28386
Refs: #31292
Refs: nodejs/help#2136

PR-URL: #31395
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
The new key type 'dh' corresponds to EVP_PKEY_DH.

PR-URL: #31178
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
PR-URL: #31178
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
This allows using the generateKeyPair API for DH instead of the old
stateful DH APIs.

PR-URL: #31178
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Currently, Node.js has separate (stateful) APIs for DH/ECDH, and no
support for ECDH-ES. This commit adds a single stateless function to
compute the DH/ECDH/ECDH-ES secret based on two KeyObjects.

PR-URL: #31178
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
test-crypto-keygen and test-crypto-dh-stateless are currently flaky
on ARM CI systems due to their slow CPUs.

PR-URL: #31178
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
This commit improved the stat test a bit by verifying that
S_ISDIR() works properly. It also adds missing coverage for
__wasi_path_remove_directory().

PR-URL: #31413
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Change the type of `Buffer::kMaxLength` to size_t because upcoming
changes in V8 will allow typed arrays > 2 GB on 64 bits platforms.

Not all platforms handle file reads and writes > 2 GB though so keep
enforcing the 2 GB typed array limit for I/O operations.

Fixes: #31399
Refs: libuv/libuv#1501

PR-URL: #31406
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Refactor for clarity and reusability. Make it more obvious that the
list is a FIFO queue.

PR-URL: #31386
Refs: openjs-foundation/summit#240
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
There is no real reason to manage a count manually, given that
checking whether there are C++ callbacks is a single pointer
comparison.

This makes it easier to add other kinds of native C++ callbacks
that are managed in a similar way.

PR-URL: #31386
Refs: openjs-foundation/summit#240
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Add a variant of `SetImmediate()` that can be called from any thread.
This allows removing the `AsyncRequest` abstraction and replaces it
with a more generic mechanism.

PR-URL: #31386
Refs: openjs-foundation/summit#240
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@addaleax
Copy link
Member

All the src: commits should be dropped from the notable changes list, I think. Unfortunately, we currently don’t have a way to mark individual commits with a specific semvar label, but only the commit that actually adds Worker support to the report feature is semver-minor, the rest just happened in the same PR.

I think all other semver-minor changes are worth listing, though.

@MylesBorins
Copy link
Member

MylesBorins commented Feb 18, 2020

If possible could we please include #31801 and #30965 #31784 included in this release

edit: whoops wrong PR

codebytere added a commit that referenced this pull request Feb 18, 2020
Notable changes:

* async_hooks:
  * add executionAsyncResource (Matteo Collina) #30959
* doc:
  * add ronag to collaborators (Robert Nagy) #31498
* crypto:
  * add crypto.diffieHellman (Tobias Nießen) #31178
  * add DH support to generateKeyPair (Tobias Nießen) #31178
  * simplify DH groups (Tobias Nießen) #31178
  * add key type 'dh' (Tobias Nießen) #31178
* report:
  * add support for Workers (Anna Henningsen) #31386
* test:
  * skip keygen tests on arm systems (Tobias Nießen) #31178

PR-URL: #31837
@nodejs nodejs deleted a comment from nodejs-github-bot Feb 18, 2020
@nodejs-github-bot
Copy link
Collaborator

@codebytere
Copy link
Member Author

codebytere commented Feb 18, 2020

https://ci.nodejs.org/job/node-test-binary-arm-12+/ seems to be failing - anyone have ideas? failed on a few runs now - latest is here.

Looks like:

22:12:13 ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?
22:12:13 Collecting metadata...
22:12:13 Metadata collection done.
22:12:13 Notifying upstream projects of job completion
22:12:13 Finished: FAILURE

cc @nodejs/build? I don't think this is a blocker but i'd like to confirm.

@Trott
Copy link
Member

Trott commented Feb 18, 2020

cc @nodejs/build? I don't think this is a blocker but i'd like to confirm.

Looks like the failure is always on the same host (test-requireio_notthetup_sayanee-debian10-arm64_pi3-1) so let's see if it passes now that I've temporarily marked it offline.

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 18, 2020

doc/changelogs/CHANGELOG_V13.md Outdated Show resolved Hide resolved
codebytere added a commit that referenced this pull request Feb 18, 2020
Notable changes:

* async_hooks:
  * add executionAsyncResource (Matteo Collina) #30959
* doc:
  * add ronag to collaborators (Robert Nagy) #31498
* crypto:
  * add crypto.diffieHellman (Tobias Nießen) #31178
  * add DH support to generateKeyPair (Tobias Nießen) #31178
  * simplify DH groups (Tobias Nießen) #31178
  * add key type 'dh' (Tobias Nießen) #31178
* report:
  * add support for Workers (Anna Henningsen) #31386
* test:
  * skip keygen tests on arm systems (Tobias Nießen) #31178

PR-URL: #31837
@codebytere
Copy link
Member Author

Given that the only change between this run and the last was a changelog typo fix, I don't think I need to run another full CI, but can if others would prefer.

@addaleax
Copy link
Member

@codebytere Fwiw, I think it’s still worth listing the other semver-minor commits in the notable changes list (not the src ones). Also, usually releasers match the notable changes list content in three places: The release commit message, the release PR description, and the changelog itself.

I don’t think you’d need a full CI after those either, but these changes might be nice to have before this goes out as a release :)

Notable changes:

* async_hooks
  * add executionAsyncResource (Matteo Collina) #30959
* crypto
  * add crypto.diffieHellman (Tobias Nießen) #31178
  * add DH support to generateKeyPair (Tobias Nießen) #31178
  * simplify DH groups (Tobias Nießen) #31178
  * add key type 'dh' (Tobias Nießen) #31178
* test
  * skip keygen tests on arm systems (Tobias Nießen) #31178
* perf_hooks
  * add property flags to GCPerformanceEntry (Kirill Fomichev) #29547
* process
  * report ArrayBuffer memory in `memoryUsage()` (Anna Henningsen) #31550
* readline
  * make tab size configurable (Ruben Bridgewater) #31318
* report
  * add support for Workers (Anna Henningsen) #31386
* worker
  * add ability to take heap snapshot from parent thread (Anna Henningsen) #31569
* added new collaborators
  * add ronag to collaborators (Robert Nagy) #31498

PR-URL: #31837
Copy link
Member

@BethGriggs BethGriggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codebytere
Copy link
Member Author

codebytere commented Feb 18, 2020

Release builds are FTBFS on what looks like a V8 issue:

02:48:22 ../deps/v8/src/utils/vector.h:213:12: error: no viable conversion from returned value of type 'OwnedVector<unsigned char>' to function return type 'OwnedVector<const unsigned char>'
02:48:22     return vec;
02:48:22            ^~~
02:48:22 ../deps/v8/src/wasm/wasm-module.cc:116:52: note: in instantiation of function template specialization 'v8::internal::OwnedVector<const unsigned char>::Of<v8::internal::Vector<const unsigned char>, const unsigned char *>' requested here
02:48:22     : encoded_offsets_(OwnedVector<const uint8_t>::Of(encoded_offsets)) {}
02:48:22                                                    ^
02:48:22 ../deps/v8/src/utils/vector.h:147:39: note: candidate constructor not viable: no known conversion from 'v8::internal::OwnedVector<unsigned char>' to 'v8::internal::OwnedVector<const unsigned char> &&' for 1st argument
02:48:22   MOVE_ONLY_WITH_DEFAULT_CONSTRUCTORS(OwnedVector);
02:48:22                                       ^
02:48:22 ../deps/v8/src/base/macros.h:132:36: note: expanded from macro 'MOVE_ONLY_WITH_DEFAULT_CONSTRUCTORS'
02:48:22   MOVE_ONLY_NO_DEFAULT_CONSTRUCTOR(TypeName)
02:48:22                                    ^
02:48:22 ../deps/v8/src/base/macros.h:137:3: note: expanded from macro 'MOVE_ONLY_NO_DEFAULT_CONSTRUCTOR'
02:48:22   TypeName(TypeName&&) V8_NOEXCEPT = default;            \
02:48:22   ^

Thoughts on how best to proceed?

@codebytere codebytere merged commit 1f0f58c into v13.x Feb 18, 2020
codebytere added a commit that referenced this pull request Feb 18, 2020
codebytere added a commit that referenced this pull request Feb 18, 2020
Notable changes:

* async_hooks
  * add executionAsyncResource (Matteo Collina) #30959
* crypto
  * add crypto.diffieHellman (Tobias Nießen) #31178
  * add DH support to generateKeyPair (Tobias Nießen) #31178
  * simplify DH groups (Tobias Nießen) #31178
  * add key type 'dh' (Tobias Nießen) #31178
* test
  * skip keygen tests on arm systems (Tobias Nießen) #31178
* perf_hooks
  * add property flags to GCPerformanceEntry (Kirill Fomichev) #29547
* process
  * report ArrayBuffer memory in `memoryUsage()` (Anna Henningsen) #31550
* readline
  * make tab size configurable (Ruben Bridgewater) #31318
* report
  * add support for Workers (Anna Henningsen) #31386
* worker
  * add ability to take heap snapshot from parent thread (Anna Henningsen) #31569
* added new collaborators
  * add ronag to collaborators (Robert Nagy) #31498

PR-URL: #31837
codebytere added a commit to nodejs/nodejs.org that referenced this pull request Feb 18, 2020
codebytere added a commit to nodejs/nodejs.org that referenced this pull request Feb 18, 2020
@devsnek devsnek deleted the v13.9.0-proposal branch February 19, 2020 07:55
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 3, 2020

CI: https://ci.nodejs.org/job/node-test-pull-request/29527/

(running for v13.10.0 sanity check)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet