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

v12.13.1 release proposal #30352

Merged
merged 102 commits into from
Nov 19, 2019
Merged

v12.13.1 release proposal #30352

merged 102 commits into from
Nov 19, 2019

Conversation

targos
Copy link
Member

@targos targos commented Nov 10, 2019

2019-11-19, Version 12.13.1 'Erbium' (LTS), @targos

Notable changes

  • Experimental support for building Node.js with Python 3 is improved.
  • ICU time zone data is updated to version 2019c. This fixes the date
    offset in Brazil.

Commits

  • [56be32d22d] - async_hooks: only emit after for AsyncResource if stack not empty (Anna Henningsen) #30087
  • [e16e3d5b90] - benchmark: remove double word "then" in comments (Nick Schonning) #29823
  • [dcdb96c7bb] - benchmark: add benchmark for vm.createContext (Joyee Cheung) #29845
  • [680e9cc7e1] - buffer: improve performance caused by primordials (Jizu Sun) #30235
  • [bcd2238b3e] - build: add workaround for WSL (gengjiawen) #30221
  • [c5d312f821] - build: find Python syntax errors in dependencies (Christian Clauss) #30143
  • [468f203809] - build: fix pkg-config search for libnghttp2 (Ben Noordhuis) #30145
  • [0415dd7cb3] - build: python3 support for configure (Rod Vagg) #30047
  • [032c23d360] - build: make linter failures fail test-doc target (Richard Lau) #30012
  • [a86648c8d2] - build: log the found compiler version if too old (Richard Lau) #30028
  • [02f6e5cc40] - build: fix version checks in configure.py (Michaël Zasso) #29965
  • [a1adce1b4f] - build: build benchmark addons like test addons (Richard Lau) #29995
  • [735ec1bf96] - build: fix version checks in gyp files (Ben Noordhuis) #29931
  • [8da83e8c24] - build: always use strings for compiler version in gyp files (Michaël Zasso) #29897
  • [b7bdfd346c] - crypto: guard with OPENSSL_NO_GOST (Shelley Vohr) #30050
  • [e175d0beb6] - crypto: reject public keys properly (Tobias Nießen) #29913
  • [b1529c6bc2] - deps: V8: cherry-pick a7dffcd767be (Christian Clauss) #30218
  • [6bc7a6db0e] - deps: V8: cherry-pick e5dbc95 (Gabriel Schulhof) #30130
  • [b88314f735] - deps: update npm to 6.12.1 (Michael Perrotte) #30164
  • [ce49a412ef] - deps: V8: cherry-pick c721203 (Michaël Zasso) #30065
  • [d2756fd14d] - deps: V8: cherry-pick ed40ab1 (Michaël Zasso) #30064
  • [58c585e3ed] - deps: npm: patch support for 13.x (Jordan Harband) #30079
  • [2764567f90] - deps: upgrade to libuv 1.33.1 (Colin Ihrig) #29996
  • [33bd1281fc] - doc: add missing hash for header link (Nick Schonning) #30188
  • [b159b91798] - doc: linkify .setupMaster() in cluster doc (Trivikram Kamat) #30204
  • [9c4a9e7337] - doc: explain http2 aborted event callback (dev-313) #30179
  • [d7bfc6c987] - doc: linkify .fork() in cluster documentation (Anna Henningsen) #30163
  • [a71f210206] - doc: update AUTHORS list (Michaël Zasso) #30142
  • [7b5047454b] - doc: improve doc Http2Session:Timeout (dev-313) #30161
  • [0efe9a0c97] - doc: move inactive Collaborators to emeriti (Rich Trott) #30177
  • [98d31da342] - doc: add options description for send APIs (dev-313) #29868
  • [d0f5bc1aa7] - doc: fix an error in resolution algorithm steps (Alex Zherdev) #29940
  • [28db99932a] - doc: remove incorrect and outdated example (Tobias Nießen) #30138
  • [c2108d4919] - doc: adjust code sample for stream.finished (Cotton Hou) #29983
  • [2ac76e3055] - doc: remove "it is important to" phrasing (Rich Trott) #30108
  • [ec992878e8] - doc: revise os.md (Rich Trott) #30102
  • [a56e78c8c8] - doc: delete "a number of" things in the docs (Rich Trott) #30103
  • [ee954d5570] - doc: remove dashes (Rich Trott) #30101
  • [c4c8e01af1] - doc: add legendecas to collaborators (legendecas) #30115
  • [22e10fd15a] - doc: --enable-source-maps and prepareStackTrace are incompatible (Benjamin Coe) #30046
  • [870c320f31] - doc: join parts of disrupt section in cli.md (vsemozhetbyt) #30038
  • [8df5bdbd66] - doc: update collaborator email address (Minwoo Jung) #30007
  • [d9b5508fc8] - doc: fix tls version typo (akitsu-sanae) #29984
  • [5616f22839] - doc: clarify readable.unshift null/EOF (Robert Nagy) #29950
  • [b57fe3b370] - doc: remove unused Markdown reference links (Nick Schonning) #29961
  • [12f24542b8] - doc: re-enable passing remark-lint rule (Nick Schonning) #29961
  • [c0cbfae0e3] - doc: add server header into the discarded list of http message.headers (Huachao Mao) #29962
  • [a23b5cbf61] - doc: prepare miscellaneous docs for new markdown lint rules (Rich Trott) #29963
  • [c66bc20bbf] - doc: fix some recent nits in fs.md (vsemozhetbyt) #29906
  • [1fefd7fddc] - doc: fs dir modifications may not be reflected by dir.read (Anna Henningsen) #29893
  • [66c6818473] - doc,meta: prefer aliases and stubs over Runtime Deprecations (Rich Trott) #30153
  • [5ade490505] - doc,meta: reduce npm PR wait period to one week (Rich Trott) #29922
  • [0ec63ee27a] - doc,n-api: sort bottom-of-the-page references (Gabriel Schulhof) #30124
  • [8a333a4519] - domain: do not import util for a simple type check (Ruben Bridgewater) #29825
  • [94ac44f3fc] - esm: modify resolution order for specifier flag (Myles Borins) #29974
  • [216e200fa9] - fs: buffer dir entries in opendir() (Anna Henningsen) #29893
  • [5959023b76] - http2: fix file close error condition at respondWithFd (Anna Henningsen) #29884
  • [4277066afd] - inspector: turn platform tasks that outlive Agent into no-ops (Anna Henningsen) #30031
  • [b0837fead3] - meta: use contact_links instead of issue templates (Michaël Zasso) #30172
  • [2695f822bc] - module: warn on require of .js inside type: module (Guy Bedford) #29909
  • [ee3c3ad0f5] - n-api,doc: add info about building n-api addons (Jim Schlight) #30032
  • [da58301054] - net: treat ENOTCONN at shutdown as success (Anna Henningsen) #29912
  • [62bc80c906] - process: add lineLength to source-map-cache (Benjamin Coe) #29863
  • [ab03c29587] - src: isolate->Dispose() order consistency (Shelley Vohr) #30181
  • [c52b292adf] - src: change env.h includes for forward declarations (Alexandre Ferrando) #30133
  • [b215b1665a] - src: split up InitializeContext (Shelley Vohr) #30067
  • [d586070388] - src: allow inspector without v8 platform (Shelley Vohr) #30049
  • [f6655b41fa] - src: remove unnecessary std::endl usage (Daniel Bevenius) #30003
  • [abfac9640e] - src: make implementing CancelPendingDelayedTasks for platform optional (Anna Henningsen) #30034
  • [693bf73b06] - src: expose ListNode<T>::prev_ on postmortem metadata (legendecas) #30027
  • [4b57088c25] - src: fewer uses of NODE_USE_V8_PLATFORM (Shelley Vohr) #30029
  • [6269a3c92a] - src: remove unused iomanip include (Daniel Bevenius) #30004
  • [aa0aacbba9] - src: initialize openssl only once (Sam Roberts) #29999
  • [45c5ad7922] - src: refine maps parsing for large pages (Gabriel Schulhof) #29973
  • [aac2476346] - src: render N-API weak callbacks as cleanup hooks (Gabriel Schulhof) #28428
  • [f3115c4d62] - src: fix largepages regression (Gabriel Schulhof) #29914
  • [ddbf150edb] - src: remove unused using declarations in worker.cc (Daniel Bevenius) #29883
  • [8a31136a95] - stream: extract Readable.from in its own file (Matteo Collina) #30140
  • [21a43bd2fd] - stream: simplify uint8ArrayToBuffer helper (Luigi Pinca) #30041
  • [ae390393b6] - stream: remove dead code (Luigi Pinca) #30041
  • [56e986aa23] - test: do not run release-npm test without crypto (Michaël Zasso) #30265
  • [d96e8b662e] - test: use arrow functions for callbacks (Minuk Park) #30069
  • [00dab3495d] - test: verify npm compatibility with releases (Michaël Zasso) #30082
  • [ecf6ae89f4] - test: expand Worker test for non-shared ArrayBuffer (Anna Henningsen) #30044
  • [2ebd1a0d3f] - test: fix test runner for Python 3 on Windows (Michaël Zasso) #30023
  • [9fed62f7cb] - test: remove common.skipIfInspectorEnabled() (Rich Trott) #29993
  • [3e39909022] - test: add cb error test for fs.close() (Matteo Rossi) #29970
  • [b93c8a77a3] - test: fix flaky doctool and test (Rich Trott) #29979
  • [aec8e77ae1] - test: fix fs benchmark test (Rich Trott) #29967
  • [b9fd18f9fb] - tools: pull xcode_emulation.py from node-gyp (Christian Clauss) #30272
  • [2810f1aec3] - tools: update tzdata to 2019c (Myles Borins) #30478
  • [41d1f166bc] - tools: fix Python 3 deprecation warning in test.py (Loris Zinsou) #30208
  • [b6546736a0] - tools: fix Python 3 syntax error in mac_tool.py (Christian Clauss) #30146
  • [87cb6b2418] - tools: use print() function in buildbot_run.py (Christian Clauss) #30148
  • [309c395aba] - tools: undefined name opts -> args in gyptest.py (Christian Clauss) #30144
  • [df0fbf2e46] - tools: git rm -r tools/v8_gypfiles/broken (Christian Clauss) #30149
  • [375f349760] - tools: update ESLint to 6.6.0 (Colin Ihrig) #30123
  • [0b6fb3d1db] - tools: doc: improve async workflow of generate.js (Theotime Poisseau) #30106
  • [8d030131a4] - tools: fix test runner in presence of NODE_REPL_EXTERNAL_MODULE (Gus Caplan) #29956
  • [59033f618a] - tools: fix GYP MSVS solution generator for Python 3 (Michaël Zasso) #29897
  • [41430bea3c] - tools: port Python 3 compat patches from node-gyp to gyp (Michaël Zasso) #29897

addaleax and others added 30 commits October 25, 2019 09:16
While it is not entirely clear why this condition is being
triggered, it does resolve a reported bug.

Fixes: #26315

PR-URL: #29912
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit removes unused using declarations in
src/node_worker.cc.

PR-URL: #29883
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
PR-URL: #29845
Refs: #29842
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Closing a FileHandle almost never fails, so it was hard to
notice before that `stream.emit(err)` would not emit an error
event due to the missing event name.

Destroying the stream with the error seems like the right thing
to do in that scenario.

PR-URL: #29884
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
PR-URL: #29823
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Read up to 32 directory entries in one batch when `dir.readSync()`
or `dir.read()` are called.

This increases performance significantly, although it introduces
quite a bit of edge case complexity.

                                                                 confidence improvement accuracy (*)    (**)    (***)
     fs/bench-opendir.js mode='async' dir='lib' n=100                  ***    155.93 %      ±30.05% ±40.34%  ±53.21%
     fs/bench-opendir.js mode='async' dir='test/parallel' n=100        ***    479.65 %      ±56.81% ±76.47% ±101.32%
     fs/bench-opendir.js mode='sync' dir='lib' n=100                           10.38 %      ±14.39% ±19.16%  ±24.96%
     fs/bench-opendir.js mode='sync' dir='test/parallel' n=100         ***     63.13 %      ±12.84% ±17.18%  ±22.58%

PR-URL: #29893
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #29893
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #29909
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
During the addition of macOS support for large pages, a `memcpy` ended
up under the wrong preprocessor directive. As a result, the newly
allocated large pages were not initialized with a copy of the text
section.

Thanks to Suresh Srinivas <suresh.srinivas@intel.com>!

PR-URL: #29914
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Fixes: #29904

PR-URL: #29913
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
* Fix sorting of sections.
* Fix sorting of bottom references.
* Wrap lines at 80 chars.
* Fix a heading level.

PR-URL: #29906
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
The two-week wait period for merging npm releases is one of those rule
exceptions that would be great to get rid of (in my opinion at least).
There are too many exceptions to our rules and they tend to be scattered
across multiple documents. People don't feel confident they know the
rules, thus hampering both project velocity and Collaborator confidence.
It also means I (and perhaps others?) get lots of pings about whether
this or that can land, etc.

This particular issue has come up a few times lately, and is
specifically calling for an exception-to-the-exception so that the
latest version of npm can be released along with Node.js 13.0.0.
Refs: #29885 (comment)

I propose here reducing the wait period from two weeks to one week. If,
after some amount of time, there seems to be no problems caused by this
change, we can consider further reducing the wait period to 48 hours to
align it with all other change requests. Even if you think that is going
too far, hopefully we can at least get it reduced to a week, as the
second week of the waiting period is usually just the PR sitting around
with an occasional ping from someone about whether/when it can land.

PR-URL: #29922
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
This removes `require('util')` from the `domain` module. There was
only a single simple type check used from the `util` module which
is now inlined instead.

PR-URL: #29825
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Since worker threads are complete Node.js environments, including the
ability to load native addons, and since those native addons can
allocate resources to be freed when objects go out of scope, and since,
upon worker thread exit, the engine does not invoke the weak callbacks
responsible for freeing resources which still have references, this
modification introduces tracking for weak references such that a list
of outstanding weak references is maintained. This list is traversed
during environment teardown. The callbacks for the remaining weak
references are called.

This change is also relevant for Node.js embedder scenarios, because in
those cases the process also outlives the `node::Environment` and
therefore weak callbacks should also be rendered as environment cleanup
hooks to ensure proper cleanup after native addons. This changes
introduces the means by which this can be accomplished.

A benchmark is included which measures the time it takes to execute the
weak reference callback for a given number of weak references.

Re: tc39/proposal-weakrefs#125 (comment)
PR-URL: #28428
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Refs: nodejs/node-gyp#1820
Refs: nodejs/node-gyp#1843

PR-URL: #29897
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
If GYP finds a string variable that can be converted to an integer,
it will do it when the variable is expanded. Use "0.0" instead of "0"
to force strings and be able to use comparison operations such as
`gas_version >= "2.26"` in Python 3.

PR-URL: #29897
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
PR-URL: #29897
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
PR-URL: #29956
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Make `distutils.version.StrictVersion` available as a helper to
gyp expressions so they can do proper version checks and update
the gyp files accordingly.

Caveat emptor: `StrictVersion` does *not* like empty strings so
this commit adds truthiness guards. The helper could deal with
those but I felt it better to make it explicit.

Fixes: #29927

PR-URL: #29931
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Without the line lengths of in-memory transpiled source, it's not
possible to convert from byte ofsets to line/column offsets.

PR-URL: #29863
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Add missing option "mode". Alphabetize options for easier maintenance.

PR-URL: #29967
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Prepare the final few documents that haven't been updated to always use
`[]` with reference links and to escape `[` and `]` for things that
aren't links in markdown files.

PR-URL: #29963
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Doctool tests have been failing a lot in CI on Win2008 R2. It appears
async functions and callback-based functions are being used in
combination such that the callback-based function cannot guarantee that
it will invoke its callback. Convert the callback-based functions to
async functions so we have one paradigm and reliable results.

PR-URL: #29979
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #29962
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
PR-URL: #29961
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
PR-URL: #29961
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Provides some missing test coverage.

PR-URL: #29970
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Multiple sections may be marked as "r-xp" and with the executable's
path. We use the location of the `__nodetext` symbol added by the linker
script to ensure that the range we retrieve from the maps file does
indeed contain the Node.js text section.

Thanks to Suresh Srinivas <suresh.srinivas@intel.com>!

PR-URL: #29973
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
PR-URL: #29950
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Currently `--es-module-specifier-resolution=node` has an alternative
resolution order than the default in common.js, this causes
inconsistencies. As discussed in @nodejs/modules we want to preserve
resolution order between implementations.

PR-URL: #29974
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member Author

targos commented Nov 14, 2019

@targos
Copy link
Member Author

targos commented Nov 14, 2019

npm requires crypto support and cannot be loaded without it.

PR-URL: #30265
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Notable changes:

* Experimental support for building Node.js with Python 3 is improved.
* ICU time zone data is updated to version 2019c. This fixes the date
  offset in Brazil.

PR-URL: #30352
@targos
Copy link
Member Author

targos commented Nov 14, 2019

Added af544ab to fix CI.

@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member Author

targos commented Nov 14, 2019

@nodejs/lts

@sam-github
Copy link
Contributor

one failure:

01:07:18  > git checkout -f d58857c15e4cbde56cf1e99e91de3f89bacc24da # timeout=30
01:37:18 ERROR: Timeout after 30 minutes
01:37:19 FATAL: Could not checkout d58857c15e4cbde56cf1e99e91de3f89bacc24da
01:37:19 hudson.plugins.git.GitException: Command "git checkout -f d58857c15e4cbde56cf1e99e91de3f89bacc24da" returned status code 143:

143 is killed with SIGTERM.

I am doing the checkout manually in the ws, and it is indeed very, very slow. Got to about 2020/3123 and its now at about 1 per 2 seconds, so it'll be more than 30 minutes to complete. I'm looking into why.

@sam-github
Copy link
Contributor

Almost certainly because no ramdisks are mounted.

@sam-github
Copy link
Contributor

@targos I mounted /home/iojs/build onto a ramdisk, should be faster, can you confirm?

@targos
Copy link
Member Author

targos commented Nov 15, 2019

@targos targos merged commit 525fd9c into v12.x Nov 19, 2019
targos added a commit that referenced this pull request Nov 19, 2019
targos added a commit that referenced this pull request Nov 19, 2019
Notable changes:

* Experimental support for building Node.js with Python 3 is improved.
* ICU time zone data is updated to version 2019c. This fixes the date
  offset in Brazil.

PR-URL: #30352
targos added a commit to nodejs/nodejs.org that referenced this pull request Nov 19, 2019
@targos targos deleted the v12.13.1-proposal branch November 19, 2019 10:55
targos added a commit to nodejs/nodejs.org that referenced this pull request Nov 19, 2019
@targos targos added release Issues and PRs related to Node.js releases. and removed 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. npm Issues and PRs related to the npm client dependency or the npm registry. labels Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Issues and PRs related to Node.js releases.
Projects
None yet
Development

Successfully merging this pull request may close these issues.