From 09e606b1afd9579dc757044cff9123f5adfdbe22 Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Wed, 25 Sep 2019 11:55:07 -0400 Subject: [PATCH 1/6] doc: add minutes for meeting 25 Sep 2019 --- meetings/2019-09-25.md | 105 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 105 insertions(+) create mode 100644 meetings/2019-09-25.md diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md new file mode 100644 index 00000000..06f72f9f --- /dev/null +++ b/meetings/2019-09-25.md @@ -0,0 +1,105 @@ +# Node.js Foundation Technical Steering Committee (TSC) Meeting 2019-09-25 + +## Links + +* **Recording**: https://www.youtube.com/watch?v=YarfwDtaANw +* **GitHub Issue**: https://github.com/nodejs/TSC/issues/758 + +## Present + +* Beth Griggs @BethGriggs (TSC) +* Christian Clauss @cclauss (guest participant) +* Сковорода Никита Андреевич @ChALkeR (TSC) +* Colin Ihrig @cjihrig (TSC) +* Gabriel Schulhof @gabrielschulhof (TSC) +* Gireesh Punathil @gireeshpunathil (TSC) +* Joyee Cheung @joyeecheung (TSC) +* Michael Dawson @mhdawson (TSC) +* Michaël Zasso @targos (TSC) +* Myles Borins @MylesBorins (TSC) +* Tobias Nießen @tniessen (TSC) + +### Announcements + +* No announcements this week. + +### CPC and Board Meeting Updates + +* Board + * Board meeting is this week. Nothing on board to be discussed with board at this time. + +* CPC + * progress on CoC proprosal + * working on project progression + +*Extracted from **tsc-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. + +### nodejs/node + +* deps: update V8 to 7.8 [#29493](https://github.com/nodejs/node/pull/29493) + * New pull request, which has 7.8 building. Only 1 more patch needed beyond where it was + * Michael updated xcode requirement for 13.x + * Myles, requesting to land SemVer major past cutoff. + * Bethany wording is that there need to be no objections to it landing after cutoff. + * Bethany will send out email asking for objections for SemVer majors, if none will include. + * No objections voiced in meeting to this one landing. + +* src,lib: make ^C print a JS stack trace [#29207](https://github.com/nodejs/node/pull/29207) + * Consensus from the 10 TSC members present is opt-in. That seems to align with previous + feedback as well as feedback from the tooling team. + +* build: ongoing list of actions for Python 3 compatibility [#25789](https://github.com/nodejs/node/issues/25789) + * [ ] tap2junit: v0.1.5 release (nodejs/tap2junit#12) to the Python Packaging Index [PyPI](https://pypi.org/project/tap2junit) + * repo moved to nodejs org, was blocking being able to run jenkins testing. PR + to make python 3 compatible. + * [ ] node-gyp: release so that npm can pick it up https://github.com/nodejs/node-gyp/issues/1791#issuecomment-533715634 + * working on doing round trip. Blocked by OSX bug Clauss is working on that. + * [ ] re-vendor npm's gyp back into nodejs/node __/deps/npm/node_modules/node-gyp__ + * [ ] Travis CI does not test ./configure --ninja test #29415 __*** help needed ***__ + * Sam, some discussion about testing configure options, not all are tested. Some tested + in docker. Question is why this one? Less vital if the tap2junit lands. + * [ ] node-gyp: Accept Python 3 by default (_semver-major_) nodejs/node-gyp#1844 (macOS fails) + +### nodejs/TSC + +* TSC representation on Moderation Team [#757](https://github.com/nodejs/TSC/issues/757) + * Myles not concerned and no barrier to entry + * No concerns from any of the TSC members. + +## Strategic Initiatives + +* Modules + * Moving forward, hit tiny hiccup. Flag caused bootstrap with flag to be different than without + flag. Removing flag caused changes and exposed some unexpected regressions + * Needed to put proxies in place for built in modules so that they can stay in sync between + CommonJS and ES6, but has performance cost. For most modules perf is ok, but for process + And buffer, not likely acceptable. Re-evaluating ways to approach this, in particular with + performance critical APIs. Pre-req to remove flag + * Gut feel is won’t be able to remove flag for 12.x LTS. hoping to land to get things perf + consistent. + +* N-API + * Working on workshop for Node+JS interactive + * Work on improving Thread safe functionality in node-addon-api + +* V8 Currency + * Talked about V8 earlier, nothing else to add + +* Open Web Standards + * Joyee: TPAC last week. Not a lot of Node.js related discussions. + * Want to move core test runner into it’s own package, then vendor in to tools so it can be + published on npm. WebCrypto project under Node.js org wants to use it. Related to running + the webplatform tests. + +* Python3 + * Covered above + +* Startup Performance + * Still working on upstream issues in V8 to unblock to be able to include loaders in snapshots. + +## Upcoming Meetings + +* **Node.js Foundation Calendar**: https://nodejs.org/calendar + +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. + From 650d097694885ad0841e534a4f084a41dcf66f09 Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Thu, 26 Sep 2019 10:06:50 -0400 Subject: [PATCH 2/6] Update meetings/2019-09-25.md Co-Authored-By: Rich Trott --- meetings/2019-09-25.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md index 06f72f9f..ec00b617 100644 --- a/meetings/2019-09-25.md +++ b/meetings/2019-09-25.md @@ -73,7 +73,7 @@ flag. Removing flag caused changes and exposed some unexpected regressions * Needed to put proxies in place for built in modules so that they can stay in sync between CommonJS and ES6, but has performance cost. For most modules perf is ok, but for process - And buffer, not likely acceptable. Re-evaluating ways to approach this, in particular with + and buffer, not likely acceptable. Re-evaluating ways to approach this, in particular with performance critical APIs. Pre-req to remove flag * Gut feel is won’t be able to remove flag for 12.x LTS. hoping to land to get things perf consistent. From 893839def782b984241fa1f664ee88bfece3ae82 Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Thu, 26 Sep 2019 10:06:58 -0400 Subject: [PATCH 3/6] Update meetings/2019-09-25.md Co-Authored-By: Rich Trott --- meetings/2019-09-25.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md index ec00b617..f96db18c 100644 --- a/meetings/2019-09-25.md +++ b/meetings/2019-09-25.md @@ -53,7 +53,7 @@ * repo moved to nodejs org, was blocking being able to run jenkins testing. PR to make python 3 compatible. * [ ] node-gyp: release so that npm can pick it up https://github.com/nodejs/node-gyp/issues/1791#issuecomment-533715634 - * working on doing round trip. Blocked by OSX bug Clauss is working on that. + * working on doing round trip. Blocked by OSX bug. Christian is working on that. * [ ] re-vendor npm's gyp back into nodejs/node __/deps/npm/node_modules/node-gyp__ * [ ] Travis CI does not test ./configure --ninja test #29415 __*** help needed ***__ * Sam, some discussion about testing configure options, not all are tested. Some tested From 6b46c9640e5da80985b1dc720e93b6167b09a59a Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Thu, 26 Sep 2019 10:07:07 -0400 Subject: [PATCH 4/6] Update meetings/2019-09-25.md Co-Authored-By: Rich Trott --- meetings/2019-09-25.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md index f96db18c..111a9f7c 100644 --- a/meetings/2019-09-25.md +++ b/meetings/2019-09-25.md @@ -89,7 +89,7 @@ * Joyee: TPAC last week. Not a lot of Node.js related discussions. * Want to move core test runner into it’s own package, then vendor in to tools so it can be published on npm. WebCrypto project under Node.js org wants to use it. Related to running - the webplatform tests. + the Web Platform Tests. * Python3 * Covered above From d0dd84bf7cd50ff319cd2674f920f6e481159913 Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Thu, 26 Sep 2019 10:07:13 -0400 Subject: [PATCH 5/6] Update meetings/2019-09-25.md Co-Authored-By: Rich Trott --- meetings/2019-09-25.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md index 111a9f7c..e6e366d3 100644 --- a/meetings/2019-09-25.md +++ b/meetings/2019-09-25.md @@ -80,7 +80,7 @@ * N-API * Working on workshop for Node+JS interactive - * Work on improving Thread safe functionality in node-addon-api + * Working on improving thread-safe functionality in node-addon-api * V8 Currency * Talked about V8 earlier, nothing else to add From b895ff77d2c1d7d157cc29a97d2792399cf15eb0 Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Thu, 26 Sep 2019 10:07:28 -0400 Subject: [PATCH 6/6] Update meetings/2019-09-25.md Co-Authored-By: Sam Roberts --- meetings/2019-09-25.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2019-09-25.md b/meetings/2019-09-25.md index e6e366d3..31d6324c 100644 --- a/meetings/2019-09-25.md +++ b/meetings/2019-09-25.md @@ -59,7 +59,7 @@ * Sam, some discussion about testing configure options, not all are tested. Some tested in docker. Question is why this one? Less vital if the tap2junit lands. * [ ] node-gyp: Accept Python 3 by default (_semver-major_) nodejs/node-gyp#1844 (macOS fails) - + * Sam: call-for-volunteers -- the nodejs/node-gyp team could use more people involved, and that the https://github.com/refack/GYP3 project does as well. https://github.com/nodejs/node-gyp/issues/1791 has more information. Without more engagement, this situation will slowly get worse. ### nodejs/TSC * TSC representation on Moderation Team [#757](https://github.com/nodejs/TSC/issues/757)