v11.14.0 proposal by BethGriggs · Pull Request #27163 · nodejs/node (original) (raw)

BethGriggs

Notable changes

Commits

@refack @targos

Backport-PR-URL: #26649 PR-URL: #26280 Fixes: #25593 Reviewed-By: Joyee Cheung joyeec9h3@gmail.com

@refack @targos

Backport-PR-URL: #26649 PR-URL: #26280 Fixes: #25593 Reviewed-By: Joyee Cheung joyeec9h3@gmail.com

@joyeecheung @targos

PR-URL: #26476 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Richard Lau riclau@uk.ibm.com

@joyeecheung @targos

PR-URL: #26513 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ben Coe bencoe@gmail.com

@joyeecheung @targos

PR-URL: #26513 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ben Coe bencoe@gmail.com

@joyeecheung @targos

Since these depends on process runtime states, delay them until pre-execution.

PR-URL: #26517 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@joyeecheung @targos

Shim legacy process object properties of CLI options during pre-execution instead of serializing them during bootstrap.

PR-URL: #26517 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@ZYSzys @targos

Backport-PR-URL: #26650 PR-URL: #26548 Refs: #26546 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Colin Ihrig cjihrig@gmail.com

@gengjiawen @targos

PR-URL: #26493 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Refael Ackermann refack@gmail.com Reviewed-By: Ben Noordhuis info@bnoordhuis.nl

@addaleax @targos

Use a RAII approach by default, and make it possible for embedders to use the ArrayBufferAllocator directly as a V8 ArrayBuffer::Allocator, e.g. when passing to Isolate::CreateParams manually.

PR-URL: #26525 Reviewed-By: Gireesh Punathil gpunathi@in.ibm.com Reviewed-By: Joyee Cheung joyeec9h3@gmail.com

@addaleax @targos

Split the API up into its essential parts, namely setting up the creation parameters for the Isolate, creating it, and performing Node.js-specific customization afterwards.

PR-URL: #26525 Reviewed-By: Gireesh Punathil gpunathi@in.ibm.com Reviewed-By: Joyee Cheung joyeec9h3@gmail.com

@joyeecheung @targos

Instead of patching process._breakFirstLine to inform the JS land to wait for the debugger, check that the JS land has not yet serialized the options and then patch the debug options from C++. The changes will be carried into JS later during option serialization.

PR-URL: #26602 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Eugene Ostroukhov eostroukhov@google.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@ZYSzys @targos

PR-URL: #23081 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Sakthipriyan Vairamani thechargingvolcano@gmail.com Reviewed-By: Weijia Wang starkwang@126.com Reviewed-By: Joyee Cheung joyeec9h3@gmail.com Reviewed-By: Refael Ackermann refack@gmail.com Reviewed-By: Denys Otrishko shishugi@gmail.com Reviewed-By: John-David Dalton john.david.dalton@gmail.com Reviewed-By: Anna Henningsen anna@addaleax.net

@ZYSzys @targos

PR-URL: #23081 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Sakthipriyan Vairamani thechargingvolcano@gmail.com Reviewed-By: Weijia Wang starkwang@126.com Reviewed-By: Joyee Cheung joyeec9h3@gmail.com Reviewed-By: Refael Ackermann refack@gmail.com Reviewed-By: Denys Otrishko shishugi@gmail.com Reviewed-By: John-David Dalton john.david.dalton@gmail.com Reviewed-By: Anna Henningsen anna@addaleax.net

@toshi1127 @targos

PR-URL: #26769 Reviewed-By: Masashi Hirano shisama07@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Yongsheng Zhang zyszys98@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@toshi1127 @targos

PR-URL: #26810 Refs: #26546 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Yongsheng Zhang zyszys98@gmail.com Reviewed-By: Michaël Zasso targos@protonmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@Fishrock123 @targos

Another nail in the coffin here, farewell ye ol C-style apis.

These apis caused numerous other issues that required far too many safeguards. This gets us one step closer to not having to worry about those issues anymore.

Refs: #18066 Refs: #20298

PR-URL: #26760

Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Anatoli Papirovski apapirovski@mac.com Reviewed-By: Matteo Collina matteo.collina@gmail.com Reviewed-By: Joyee Cheung joyeec9h3@gmail.com

@toshi1127 @targos

PR-URL: #26814 Refs: #26546 Reviewed-By: Michaël Zasso targos@protonmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Masashi Hirano shisama07@gmail.com Reviewed-By: Yongsheng Zhang zyszys98@gmail.com

@toshi1127 @targos

PR-URL: #26783 Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Yongsheng Zhang zyszys98@gmail.com

@joyeecheung @targos

Instead of relying on the value of the CLI option when executing bootstrap/loaders.js.

PR-URL: #26759 Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

@Trott @targos

In the current code, line 497 checks if item is null or undefined. However, item is guaranteed to be a non-null object or function at that point.

Here are the current lines 484 through 497:

  const primitive = item === null ||
      (typeof item !== 'function' && typeof item !== 'object');
  if (properties === undefined && primitive) {
    hasPrimitives = true;
    valuesKeyArray[i] = _inspect(item);
  } else {
    const keys = properties || ObjectKeys(item);
    for (const key of keys) {
      if (map[key] === undefined)
        map[key] = [];
      if ((primitive && properties) || !hasOwnProperty(item, key))
        map[key][i] = '';
      else
        map[key][i] = item == null ? item : _inspect(item[key]);

This change removes the unnecessary ternary in that final line, simplifying it to:

        map[key][i] = _inspect(item[key]);

PR-URL: #26906 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Michaël Zasso targos@protonmail.com Reviewed-By: Tobias Nießen tniessen@tnie.de Reviewed-By: Yuta Hiroto hello@hiroppy.me Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Yongsheng Zhang zyszys98@gmail.com

@sasurau4 @targos

PR-URL: #26898 Refs: #24928 Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Tobias Nießen tniessen@tnie.de

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@gengjiawen @targos

PR-URL: #26815 Reviewed-By: Refael Ackermann refack@gmail.com

@BridgeAR @targos

Use destructuring and arrow functions and make one test stricter. Also inline the error object as there's only a sinlge error that can currently be thrown in the path module.

PR-URL: #26912 Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Michaël Zasso targos@protonmail.com

@BridgeAR @targos

This fixes the parse function for single character input that are not a path separator.

PR-URL: #26912 Fixes: #26911 Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Michaël Zasso targos@protonmail.com

@ZYSzys @BethGriggs

PR-URL: #26591 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@Hakerh400 @BethGriggs

When console.log is called for the first time it initializes TTYWrap object. However, if there is not enough space on the V8 stack, creating function template fails and triggers empty maybe local exception.

PR-URL: #26832 Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: Joyee Cheung joyeec9h3@gmail.com Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@panva @BethGriggs

PR-URL: #27041 Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@apapirovski @BethGriggs

Cleanup some code and make the emit hooks very slightly faster.

PR-URL: #27034 Reviewed-By: Yongsheng Zhang zyszys98@gmail.com Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Reviewed-By: James M Snell jasnell@gmail.com Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@bnoordhuis @BethGriggs

It was reported that parallel builds on Windows sometimes error because of missing intermediate files.

On closer inspection I noticed that some files are copied from src/ to the intermediate build directory in a way where they don't participate in dependency resolution. Put another way, the build system doesn't know to wait for the copy to complete because we don't tell it to.

Fix that by not copying around files but instead making the script that processes them a little smarter about where to find them and where to store the results.

PR-URL: #27026 Fixes: #27025 Reviewed-By: Eugene Ostroukhov eostroukhov@google.com Reviewed-By: Richard Lau riclau@uk.ibm.com Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@panva @BethGriggs

PR-URL: #27041 Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Ruben Bridgewater ruben@bridgewater.de Signed-off-by: Beth Griggs Bethany.Griggs@uk.ibm.com

@richardlau @BethGriggs

Refs: #26982

PR-URL: #27164 Reviewed-By: Vse Mozhet Byt vsemozhetbyt@gmail.com Reviewed-By: Yongsheng Zhang zyszys98@gmail.com

BethGriggs added a commit that referenced this pull request

Apr 10, 2019

@BethGriggs

Notable changes:

PR-URL: #27163

richardlau

@nodejs-github-bot

targos

cjihrig

@BethGriggs

@BethGriggs

@richardlau

@BridgeAR

spawn-wrap has come up before. I am not sure anymore if it always comes up for 11 or if it's flaky.

@refack

citgm.leveldown-v5.0.1

npm ERR! C:\Program Files\Git\mingw64/libexec/git-core\git-submodule: line 19: .: git-sh-setup: file not found

Git setup issue? @nodejs/build

Looks very very strange... I keep getting git timeouts, so this might be a network issue...

@Trott

I imagine (SEMVER-MINOR) fs: remove experimental warning for fs.promises (Anna Henningsen) #26581 might be something to add to the Notable changes section.

@BethGriggs

Notable changes:

PR-URL: #27163

@nodejs-github-bot

@nodejs-github-bot

BethGriggs added a commit that referenced this pull request

Apr 11, 2019

@BethGriggs

BethGriggs added a commit that referenced this pull request

Apr 11, 2019

@BethGriggs

Notable changes:

PR-URL: #27163

BethGriggs added a commit to BethGriggs/nodejs.org that referenced this pull request

Apr 11, 2019

@BethGriggs

BethGriggs added a commit to nodejs/nodejs.org that referenced this pull request

Apr 11, 2019

@BethGriggs

@targos targos deleted the v11.14.0-proposal branch

June 4, 2019 14:09