Skip to content

Commit f255180

Browse files
joshgavMylesBorins
authored andcommitted
doc: add CTC meeting minutes 2016-07-06
PR-URL: #7570 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Evan Lucas <[email protected]>
1 parent b60473f commit f255180

File tree

1 file changed

+150
-0
lines changed

1 file changed

+150
-0
lines changed

doc/ctc-meetings/2016-07-06.md

+150
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,150 @@
1+
# Node Foundation CTC Meeting 2016-07-06
2+
3+
## Links
4+
5+
* **Audio Recording**: TBP
6+
* **GitHub Issue**: https://github.com/nodejs/node/issues/7558
7+
* **Minutes Google Doc**: <https://docs.google.com/document/d/1NWIKwYxDTApvc9Xbq5JTMorRPKIBuBKAA0zcjm8K_II>
8+
* _Previous Minutes Google Doc: <https://docs.google.com/document/d/1a19l2ICQHCw2w6cM4AayRM8QX8cIrgnHXCtzmuxfWuM>_
9+
10+
## Present
11+
12+
* Сковорода Никита Андреевич @ChALkeR (CTC)
13+
* Colin Ihrig @cjihrig (CTC)
14+
* Evan Lucas @evanlucas (CTC)
15+
* Jeremiah Senkpiel @Fishrock123 (CTC)
16+
* James Snell @jasnell (CTC)
17+
* John-David Dalton @jdalton (observer/Microsoft)
18+
* Josh Gavant @joshgav (observer/Microsoft)
19+
* Michael Dawson @mhdawson (CTC)
20+
* Brian White @mscdex (CTC)
21+
* Alexis Campailla @orangemocha (CTC)
22+
* Shigeki Ohtsu @shigeki (CTC)
23+
* Trevor Norris @trevnorris (CTC)
24+
* Rich Trott @Trott (CTC)
25+
26+
## Standup
27+
28+
* Сковорода Никита Андреевич @ChALkeR (CTC)
29+
* Nothing significant.
30+
* Colin Ihrig @cjihrig (CTC)
31+
* Reviewed issues. Reviewed pull requests. Opened a couple PRs
32+
* Evan Lucas @evanlucas (CTC)
33+
* Commented on a few issues, not much else
34+
* Jeremiah Senkpiel @Fishrock123 (CTC)
35+
* v6.3.0 Release, most difficult to date, lots of merge conflicts.
36+
* Various PR / Issue review
37+
* John-David Dalton @jdalton (observer/Microsoft)
38+
* Another round of questions with Chakra dev about Unambiguous JS Grammar confirming difficulty level of API is not significant
39+
* Updated Unambiguous JS Grammar based on feedback
40+
* Josh Gavant @joshgav (observer/Microsoft)
41+
* Out for July 4th holiday.
42+
* Michael Dawson @mhdawson (CTC)
43+
* Added linuxOne to regression tests/watched for issues, all looks good, next step is to add to nightly releases
44+
* Chasing down AIX build breaks test failures
45+
* Continued work on ABI stable module API and scheduling of next WG meeting
46+
* misc PR/Issue reviews/pull requests
47+
* post-mortem work and scheduling of next meeting
48+
* small update to benchmark graphs
49+
* keeping up to date with issues
50+
* Brian White @mscdex (CTC)
51+
* Started the process of rebasing the old js http parser onto master to see how it compares to the current http implementation
52+
* Reviewed PRs, commented on issues
53+
* Alexis Campailla @orangemocha (CTC)
54+
* Nothing to report (was out on vacation)
55+
* Shigeki Ohtsu @shigeki (CTC)
56+
* Reviewed only one PR for crypto doc
57+
* Trevor Norris @trevnorris (CTC)
58+
* Updated AsyncWrap EP
59+
* fs.realpath() deep symlink PR
60+
* Rich Trott @Trott (CTC)
61+
* Onboarded @bzoz
62+
* Fixing test-net-write-slow on FreeBSD (PR #7555)
63+
* James M Snell @jasnell (CTC)
64+
* WHATWG http implementation
65+
* Closing up other business before going on vacation for a week
66+
67+
## Agenda
68+
69+
Extracted from **ctc-agenda** labelled issues and pull requests from the **nodejs org** prior to the meeting.
70+
71+
### nodejs/node
72+
73+
* build: drop support for VS 2013 in v7 [#7484](https://github.com/nodejs/node/issues/7484)
74+
* Node 6 fs.realpath behavior changes [#7175](https://github.com/nodejs/node/issues/7175)
75+
76+
### nodejs/node-eps
77+
78+
* Replace section 5.1 with unambiguous JavaScript grammar. [#33](https://github.com/nodejs/node-eps/pull/33)
79+
* AsyncWrap public API proposal [#18](https://github.com/nodejs/node-eps/pull/18)
80+
81+
## Previous Meeting
82+
83+
* New Buffer API backport to v4.x [#7475](https://github.com/nodejs/node/pull/7475)
84+
85+
* New PR: https://github.com/nodejs/node/pull/7562
86+
87+
* child_process: validate fork/execFile arguments [#7399](https://github.com/nodejs/node/pull/7399)
88+
* Node 6 fs.realpath behavior changes [#7175](https://github.com/nodejs/node/issues/7175)
89+
* repl: Default `useGlobal` to false in CLI REPL. [#5703](https://github.com/nodejs/node/pull/5703)
90+
* Seek legal advice on LICENSE and copyright blocks in code [#3979](https://github.com/nodejs/node/issues/3979)
91+
* Mention odd-number release has eight months support as Current. [#113](https://github.com/nodejs/LTS/issues/113)
92+
* AsyncWrap public API proposal [#18](https://github.com/nodejs/node-eps/pull/18)
93+
94+
## Minutes
95+
96+
### build: drop support for VS 2013 in v7 [#7484](https://github.com/nodejs/node/issues/7484) (@jasnell)
97+
98+
Consider v6 and v7 separately. All agree on moving to VS 2015 for v7. Only question is v6 prior to LTS. We need more info to make decision on v6.
99+
100+
If we stay with 2013 for v6 we’ll have to support it for the entire lifetime of v6 LTS. But it needs to still be possible to build native addons with 2013.
101+
102+
@jasnell - Our recommendation is to build native addons with VS 2015 unless it doesn’t work (e.g. in [node-sass#1283](https://github.com/sass/node-sass/issues/1283).
103+
104+
@orangemocha - No problem building `node.exe` with VS 2015 in v7. But dropping support for 2013 would break some modules (breaking change) due to ABI incompatibilities, so it might not be appropriate for v6. Need more time to assess impact.
105+
106+
### Node 6 fs.realpath behavior changes [#7175](https://github.com/nodejs/node/issues/7175)
107+
108+
@trevnorris - fix is working, test in development. Comment further in the PR.
109+
110+
### Replace section 5.1 with unambiguous JavaScript grammar. [#33](https://github.com/nodejs/node-eps/pull/33)
111+
112+
@jdalton added line stating explicitly that the .mjs proposal is still possible.
113+
114+
Vote taken in favor of merging this PR. Alexis officially abstained.
115+
116+
### AsyncWrap public API proposal [#18](https://github.com/nodejs/node-eps/pull/18)
117+
118+
See also [#7565](https://github.com/nodejs/node/issues/7565), which questions whether AsyncWrap should exist (from @bnoordhuis).
119+
120+
@jasnell - Error handling should be addressed.
121+
122+
@trott - A vote for the EP means “*if* we implement, this is what the JavaScript API will look like” and not necessarily “we intend to implement this.”
123+
124+
@trevnorris - Correct. Confirm the API, then more work is needed on implementation, e.g. to include info on promises.
125+
126+
@trevnorris - biggest missing chunk is public C++ API. Trouble is that it requires exposing some things which are not currently in public headers.
127+
128+
@trevnorris - Goal is to include this as experimental in v6 before LTS. But still waiting on additional hooks into promises from V8.
129+
130+
@Fishrock123 - AsyncWrap is best bet for CLS across async activities, replaces deprecated domains.
131+
132+
Vote: 7-8 ayes, 0 opposed, 0 abstentions.
133+
134+
135+
## Q/A on public fora
136+
137+
None.
138+
139+
## Next Meeting
140+
141+
* Move to VS 2015 for release build in v6? Is this a breaking change?
142+
143+
## Upcoming TLP & WG Meetings
144+
145+
* CTC: 2016-07-13
146+
* TSC: 2016-07-14
147+
* Diagnostics: 2016-07-13
148+
* Post-Mortem: [nodejs/post-mortem#31](https://github.com/nodejs/post-mortem/issues/31)
149+
* LTS:
150+
* Build:

0 commit comments

Comments
 (0)