delta/node-new.git/tools, branch try_Python3_on_macOS

tools: apply more stringent blank-line linting for markdown files

2019-09-07T14:07:26+00:00

Update remark-preset-lint-node to 1.9.0, which adds linting for
multiple consecutive blank lines.

PR-URL: https://github.com/nodejs/node/pull/29447
Reviewed-By: Michaël Zasso 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Luigi Pinca

doc: prepare markdown files for more stringent blank-line linting

2019-09-07T14:07:25+00:00

PR-URL: https://github.com/nodejs/node/pull/29447
Reviewed-By: Michaël Zasso 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Luigi Pinca

build,win: fix Python detection on localized OS

2019-09-06T02:14:29+00:00

This code previously assumed the default value was always printed to
the console by reg.exe as "(default)", but this is not true on
localized versions of Windows and can contain spaces.

Fixes: https://github.com/nodejs/node/issues/29417

PR-URL: https://github.com/nodejs/node/pull/29423
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Minwoo Jung 
Reviewed-By: David Carlier

tools: delete v8_external_snapshot.gypi

2019-09-05T20:53:49+00:00

Delete the v8_external_snapshot target from gyp and disable the
v8_use_external_startup_data option since it is never used anyway.

Refs: https://github.com/nodejs/node/pull/29363#issuecomment-526103293
Fixes: https://github.com/nodejs/node/issues/28964

PR-URL: https://github.com/nodejs/node/pull/29369
Reviewed-By: Ben Noordhuis 
Reviewed-By: Jiawen Geng 
Reviewed-By: David Carlier

tools: fix GYP ninja generator for Python 3

2019-09-05T11:53:24+00:00

PR-URL: https://github.com/nodejs/node/pull/29416
Reviewed-By: Christian Clauss 
Reviewed-By: Rich Trott 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Ben Noordhuis

gyp: make StringIO work in ninja.py

2019-09-03T16:31:14+00:00

Allow both Python 2 and 3 to access StringIO.

This fixes `./configure --ninja`, which was broken by
https://github.com/nodejs/node/pull/29371.

See: https://github.com/nodejs/node/pull/29371#issuecomment-527331969

PR-URL: https://github.com/nodejs/node/pull/29414
Reviewed-By: Michaël Zasso 
Reviewed-By: Sam Roberts

tools: update ESLint to 6.3.0

2019-09-03T04:51:40+00:00

Update ESLint to 6.3.0

PR-URL: https://github.com/nodejs/node/pull/29382
Reviewed-By: Rich Trott 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Yongsheng Zhang

build,win: accept Python 3 if 2 is not available

2019-09-03T04:07:08+00:00

If there is no Python 2 available, use Python 3. This allows to test
running configure with Python 3.

PR-URL: https://github.com/nodejs/node/pull/29236
Reviewed-By: Christian Clauss 
Reviewed-By: Rich Trott

build,win: find Python in paths with spaces

2019-09-03T04:07:06+00:00

When looking for Python in the registry, as specified in PEP514,
this was not able to handle installations in a path with spaces,
like Program Files. This ensures the whole path is used, fixing the
issue.

PR-URL: https://github.com/nodejs/node/pull/29236
Reviewed-By: Christian Clauss 
Reviewed-By: Rich Trott

tools: use 'from io import StringIO' in ninja.py

2019-09-03T04:01:09+00:00

PR-URL: https://github.com/nodejs/node/pull/29371
Reviewed-By: Ben Noordhuis 
Reviewed-By: Jiawen Geng 
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca