00:00:01  * ircretaryquit (Remote host closed the connection)
00:00:09  * ircretaryjoined
00:00:15  <tjfontaine>ok, so time to test this addon build on windows
00:01:00  <tjfontaine>once windows finishes building
00:01:41  <indutny>do you have windows sources? :)
00:01:56  <tjfontaine>haha, once node finishes building on windows ;)
00:02:08  <tjfontaine>but maybe azure woudl cough them up :D
00:02:17  * defunctzombie_zzchanged nick to defunctzombie
00:02:19  <indutny>:)
00:02:20  <indutny>ok
00:02:21  <indutny>ttyl
00:02:22  <indutny>time to sleep
00:02:25  <tjfontaine>enjoy
00:02:33  <indutny>hm...
00:02:36  <indutny>about try write
00:02:36  <tjfontaine>indutny: thanks fro all your help today
00:02:38  <tjfontaine>ok?
00:02:39  <indutny>np
00:02:50  <indutny>if you are ok with it - could you please comment on it without landing?
00:02:57  <tjfontaine>yup
00:02:58  <indutny>I'm going to look at it once again tomorrow
00:03:04  <indutny>and land it if you are fine withi t
00:03:06  <indutny>with it*
00:03:09  <indutny>great
00:03:12  <indutny>cheers :P
00:03:14  <tjfontaine>cheers!
00:03:16  * mikolalysenkojoined
00:03:27  * dshaw_quit (Quit: Leaving.)
00:08:03  * mikolalysenkoquit (Ping timeout: 245 seconds)
00:09:23  <MI6>nodejs-master: #898 UNSTABLE linux-x64 (1/700) osx-x64 (1/700) linux-ia32 (3/700) osx-ia32 (1/700) smartos-ia32 (2/700) smartos-x64 (3/700) http://jenkins.nodejs.org/job/nodejs-master/898/
00:11:51  * kpdeckerjoined
00:12:45  * kpdeckerquit (Client Quit)
00:13:50  * octetcloudjoined
00:14:35  <tjfontaine>indutny: addons don't build at the moment for seemingly unrelated reasons: https://gist.github.com/anonymous/8660026/raw/edd93ebde2ff4438365f15e6cc3bcf6651905ff9/gistfile1.txt
00:16:01  <MI6>nodejs-master-windows: #684 UNSTABLE windows-x64 (7/700) windows-ia32 (11/700) http://jenkins.nodejs.org/job/nodejs-master-windows/684/
00:16:56  <tjfontaine>seems like we just need a toplevel stdint.h I guess
00:18:17  <tjfontaine>oh maybe ...
00:31:14  * dshaw_joined
00:41:41  * Kakeraquit (Ping timeout: 252 seconds)
00:42:51  * AvianFlujoined
00:45:26  * Benviequit (Remote host closed the connection)
00:45:53  * Benviejoined
00:57:05  * octetcloudquit (Ping timeout: 252 seconds)
01:02:05  * kazuponjoined
01:04:06  * mikolalysenkojoined
01:08:14  * abraxasjoined
01:09:09  * mikolalysenkoquit (Ping timeout: 265 seconds)
01:12:51  * abraxasquit (Ping timeout: 252 seconds)
01:31:04  * paulfryzelquit (Remote host closed the connection)
01:31:23  * thlorenzjoined
01:32:58  * kazuponquit (Remote host closed the connection)
01:35:40  * abraxasjoined
01:37:59  * dshaw_quit (Quit: Leaving.)
01:38:15  * paulfryzeljoined
01:39:36  * bradleymeckjoined
01:44:05  * kazuponjoined
01:46:32  * dap_quit (Quit: Leaving.)
01:48:52  * paulfryzelquit (Read error: Connection reset by peer)
01:49:24  * paulfryzeljoined
02:04:15  * hzquit
02:05:13  * nrajlichquit (Quit: ["Textual IRC Client: www.textualapp.com"])
02:05:14  * paulfryzelquit (Remote host closed the connection)
02:15:32  <MI6>joyent/node: Timothy J Fontaine master * 2f5e77f : test: make abort-fatal-error more robust (+1 more commits) - http://git.io/GC0O_A
02:23:07  * mikolalysenkojoined
02:25:26  <MI6>nodejs-master: #899 UNSTABLE linux-x64 (1/700) osx-x64 (1/700) linux-ia32 (2/700) osx-ia32 (1/700) smartos-ia32 (2/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/899/
02:31:24  <MI6>nodejs-master-windows: #685 UNSTABLE windows-x64 (7/700) windows-ia32 (9/700) http://jenkins.nodejs.org/job/nodejs-master-windows/685/
02:31:52  * c4milo_quit (Remote host closed the connection)
02:39:29  * mikolalysenkoquit (Ping timeout: 240 seconds)
02:46:12  * rmgquit (Remote host closed the connection)
02:48:05  * AWinterm_quit (Remote host closed the connection)
02:50:51  * brsonquit (Quit: leaving)
03:05:57  * mikolalysenkojoined
03:06:32  * paulfryzeljoined
03:10:57  * mikolalysenkoquit (Ping timeout: 265 seconds)
03:10:59  * paulfryzelquit (Ping timeout: 240 seconds)
03:11:23  * abraxasquit (Remote host closed the connection)
03:18:27  * rmgjoined
03:21:25  * octetcloudjoined
03:23:20  * mikolalysenkojoined
03:24:13  <MI6>joyent/node: Timothy J Fontaine master * 114bff4 : test: use logical and not or in abort-fatal-error - http://git.io/o7MbwQ
03:25:43  * abraxasjoined
03:37:21  <MI6>nodejs-master: #900 UNSTABLE linux-ia32 (2/700) osx-ia32 (1/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/900/
03:41:27  <MI6>nodejs-master-windows: #686 FAILURE windows-ia32 (12/700) http://jenkins.nodejs.org/job/nodejs-master-windows/686/
03:43:44  <tjfontaine>fuck you jenkins
03:44:09  <tjfontaine>we are so damn close to a green board
03:50:59  * rmgquit (Ping timeout: 240 seconds)
03:56:56  * kazuponquit (Remote host closed the connection)
03:57:50  * paulfryzeljoined
03:59:26  * paulfryzelquit (Read error: Connection reset by peer)
03:59:40  * paulfryzeljoined
04:01:05  * abraxasquit (Remote host closed the connection)
04:01:12  <MI6>nodejs-master-windows: #687 UNSTABLE windows-x64 (9/700) windows-ia32 (11/700) http://jenkins.nodejs.org/job/nodejs-master-windows/687/
04:03:59  * paulfryzelquit (Ping timeout: 240 seconds)
04:09:01  * defunctzombiechanged nick to defunctzombie_zz
04:27:38  * kazuponjoined
04:29:27  * kazuponquit (Read error: Connection reset by peer)
04:29:54  * kazuponjoined
04:34:13  * kazuponquit (Ping timeout: 248 seconds)
04:39:20  * thlorenzquit (Remote host closed the connection)
04:39:52  * thlorenzjoined
04:44:18  * thlorenzquit (Ping timeout: 245 seconds)
04:50:15  * mralephjoined
04:58:37  * kazuponjoined
05:00:17  * paulfryzeljoined
05:01:59  * octetcloudquit (Ping timeout: 272 seconds)
05:04:29  * paulfryzelquit (Ping timeout: 240 seconds)
05:05:04  * kenperkinsquit (Quit: Computer has gone to sleep.)
05:16:13  * abraxasjoined
05:29:30  * octetcloudjoined
05:30:04  * jmar777quit (Remote host closed the connection)
05:42:12  * dshaw_joined
05:47:35  * rmgjoined
05:48:41  * thlorenzjoined
05:51:06  * thlorenzquit (Remote host closed the connection)
05:51:26  * awreecejoined
05:52:00  * kazuponquit (Remote host closed the connection)
05:52:23  * rmgquit (Ping timeout: 272 seconds)
05:52:56  <awreece>I'm skeptical this is the right channel, but the best google searches seem to be logs from this channel - I'm getting the following build error when I try to build the latest release node v0.10.25 on illumos: failed to resolve V8DBG_OFF_FP_CONTEXT: Unknown variable name
05:53:07  <awreece>is this known broken?
05:53:11  <awreece>is there a known way to fix?
05:54:55  * kazupon_joined
05:55:51  * guybrushquit (Excess Flood)
05:56:06  * guybrushjoined
05:59:19  <othiym23>awreece: it's either this channel, #joyent, or #smartos if you want help building Node on illumos, but the people who are going to be able to help you are probably away for the evening
05:59:29  * awreecenods
06:00:28  <othiym23>awreece: are you building on SmartOS, OmniOS, or...?
06:00:35  <awreece>I'm on Omni
06:02:53  * paulfryzeljoined
06:03:15  <othiym23>awreece: I think you want to track down Dave Pacheco (dapsays on Twitter) and ask him
06:03:34  <awreece>is he on irc?
06:03:54  <othiym23>this sounds like it's probably something dtrace / mdb related that may be a divergence between SmartOS and other Illumos platforms
06:04:11  <awreece>i hope not
06:04:12  <othiym23>he is (as dap), but not right now
06:04:17  <awreece>alright
06:04:53  <awreece>thanks, I'll be back tomorrow
06:05:26  <othiym23>sure thing, sorry I didn't have anything for you right now
06:05:42  <awreece>yeah, np
06:05:56  * c4milojoined
06:07:29  * paulfryzelquit (Ping timeout: 240 seconds)
06:08:20  * mikolalysenkoquit (Ping timeout: 245 seconds)
06:20:42  * c4milo_joined
06:23:25  * c4miloquit (Ping timeout: 252 seconds)
06:28:31  * janjongboomjoined
06:34:22  <MI6>nodejs-v0.10-windows: #466 FAILURE windows-ia32 (5/610) http://jenkins.nodejs.org/job/nodejs-v0.10-windows/466/
06:35:06  * m76joined
06:44:57  * octetcloudquit (Ping timeout: 252 seconds)
07:03:51  * paulfryzeljoined
07:05:51  * abraxasquit (Remote host closed the connection)
07:07:59  * paulfryzelquit (Ping timeout: 240 seconds)
07:10:07  * janjongboomquit (Quit: My MacBook has gone to sleep. ZZZzzz…)
07:14:27  * mikolalysenkojoined
07:19:47  * mikolalysenkoquit (Ping timeout: 272 seconds)
07:22:13  * rendarjoined
07:23:27  * AvianFluquit (Ping timeout: 252 seconds)
07:25:40  * abraxasjoined
07:26:21  * daviddiasjoined
07:47:32  * bajtosjoined
07:53:05  * c4milo_quit (Remote host closed the connection)
08:04:26  * paulfryzeljoined
08:08:59  * paulfryzelquit (Ping timeout: 240 seconds)
08:10:56  * daviddiasquit (Read error: Connection reset by peer)
08:11:44  * daviddiasjoined
08:16:14  * ryahquit (Read error: Operation timed out)
08:16:22  * ryahjoined
08:18:09  * indutnyquit (Read error: Operation timed out)
08:18:21  * wolfeidauquit (Ping timeout: 265 seconds)
08:19:31  * dshaw_quit (Quit: Leaving.)
08:20:41  * indutnyjoined
08:20:52  * dshaw_joined
08:21:52  * wolfeidaujoined
08:25:09  * dshaw_quit (Ping timeout: 252 seconds)
08:25:39  <indutny>hey people
08:25:41  <indutny>tjfontaine: hey man
08:25:45  <indutny>got any opinion at https://github.com/joyent/node/pull/6902 ?
08:38:41  <rendar>indutny, thats interesting, with that function you can try to write, if you can't you'll try again later, without allocating a new "pending" write?
08:38:50  <indutny>yes
08:39:05  <indutny>but try again later should be probably done with `uv_write()`
08:39:22  <indutny>since it knowns when this "later" happens
08:40:16  <MI6>joyent/node: Fedor Indutny master * eaf7664 : node: explicitly include `sys/types.h` for size_t - http://git.io/r-RYwA
08:40:30  <rendar>indutny, right
08:46:10  * zz_karupanerurachanged nick to karupanerura
08:49:45  <MI6>nodejs-master: #901 UNSTABLE linux-ia32 (1/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/901/
08:50:07  * dshaw_joined
08:54:29  * dshaw_quit (Ping timeout: 248 seconds)
08:56:02  <MI6>nodejs-master-windows: #688 FAILURE windows-x64 (9/700) http://jenkins.nodejs.org/job/nodejs-master-windows/688/
08:57:19  * janjongboomjoined
09:05:08  * paulfryzeljoined
09:09:29  * paulfryzelquit (Ping timeout: 240 seconds)
09:20:04  * dshaw_joined
09:24:53  * dshaw_quit (Ping timeout: 248 seconds)
09:31:22  * piscisaureusjoined
09:33:28  * kazupon_quit (Remote host closed the connection)
09:36:48  * kazuponjoined
09:38:47  * hzjoined
09:41:55  * c4milojoined
09:42:16  * kazuponquit (Remote host closed the connection)
09:46:27  * c4miloquit (Ping timeout: 252 seconds)
09:58:04  * karupanerurachanged nick to zz_karupanerura
09:59:59  * piscisaureusquit (Ping timeout: 260 seconds)
10:05:58  * paulfryzeljoined
10:10:29  * paulfryzelquit (Ping timeout: 240 seconds)
10:21:19  * dshaw_joined
10:22:21  * kazuponjoined
10:26:15  * dshaw_quit (Ping timeout: 272 seconds)
10:43:55  * rmgjoined
10:46:46  <MI6>nodejs-v0.10: #1745 UNSTABLE osx-x64 (7/610) linux-ia32 (2/610) smartos-x64 (8/610) smartos-ia32 (8/610) linux-x64 (2/610) osx-ia32 (8/610) http://jenkins.nodejs.org/job/nodejs-v0.10/1745/
10:48:47  * rmgquit (Ping timeout: 272 seconds)
10:50:35  * mraleph1joined
10:50:54  * mralephquit (Read error: Connection reset by peer)
10:52:43  * rendarquit (Quit: Leaving)
10:55:36  * abraxasquit (Remote host closed the connection)
10:56:12  * abraxasjoined
10:58:41  * abraxasquit (Remote host closed the connection)
11:02:30  * kazuponquit (Remote host closed the connection)
11:02:57  * kazuponjoined
11:05:25  * Kakerajoined
11:06:43  * paulfryzeljoined
11:07:13  * kazuponquit (Ping timeout: 252 seconds)
11:09:50  * kazuponjoined
11:10:59  * paulfryzelquit (Ping timeout: 240 seconds)
11:20:55  * dshaw_joined
11:21:26  * bajtosquit (Quit: bajtos)
11:22:17  * kazuponquit (Remote host closed the connection)
11:22:44  * kazuponjoined
11:25:31  * dshaw_quit (Ping timeout: 272 seconds)
11:27:15  * kazuponquit (Ping timeout: 260 seconds)
11:29:55  * daviddiasquit (Read error: Connection reset by peer)
11:30:06  * rendarjoined
11:30:12  * c4milojoined
11:35:11  * c4miloquit (Ping timeout: 260 seconds)
11:49:51  * m76quit (Ping timeout: 272 seconds)
11:52:53  * m76joined
11:53:04  * kazuponjoined
11:57:49  * m76quit (Ping timeout: 272 seconds)
12:07:28  * paulfryzeljoined
12:09:20  * kazuponquit (Remote host closed the connection)
12:09:47  * kazuponjoined
12:10:10  * dcrostajoined
12:11:59  * paulfryzelquit (Ping timeout: 240 seconds)
12:13:57  * kazuponquit (Ping timeout: 248 seconds)
12:19:12  * kazuponjoined
12:20:25  * dshaw_joined
12:24:37  * dshaw_quit (Ping timeout: 248 seconds)
12:32:19  * kazuponquit (Remote host closed the connection)
12:35:45  * c4milojoined
12:45:11  * piscisaureusjoined
12:52:55  * c4miloquit (Ping timeout: 245 seconds)
12:58:11  * c4milojoined
12:59:30  * abraxasjoined
13:01:44  * m76joined
13:02:34  * c4milo_joined
13:04:13  * abraxasquit (Ping timeout: 252 seconds)
13:05:02  * c4milo_quit (Remote host closed the connection)
13:05:24  * c4miloquit (Read error: Connection reset by peer)
13:05:47  * c4milojoined
13:05:58  * thlorenzjoined
13:07:19  * c4milo_joined
13:08:16  * paulfryzeljoined
13:11:19  * c4miloquit (Ping timeout: 260 seconds)
13:12:04  * c4milojoined
13:12:29  * paulfryzelquit (Ping timeout: 240 seconds)
13:13:11  * c4milo_quit (Ping timeout: 260 seconds)
13:20:21  * dshaw_joined
13:20:34  * dcrostaquit (Quit: Textual IRC Client: www.textualapp.com)
13:23:42  * bajtosjoined
13:25:29  * dshaw_quit (Ping timeout: 272 seconds)
13:32:00  * thlorenzquit (Remote host closed the connection)
13:33:19  * AvianFlujoined
13:43:10  * kazuponjoined
13:47:29  * kazuponquit (Ping timeout: 241 seconds)
13:52:01  * AndreasMadsenjoined
13:53:02  * piscisaureusquit (Ping timeout: 264 seconds)
13:58:16  * AndreasMadsenquit
14:01:50  * piscisaureusjoined
14:09:10  * paulfryzeljoined
14:11:46  * pachetjoined
14:11:46  * pachetquit (Changing host)
14:11:46  * pachetjoined
14:13:45  * paulfryzelquit (Ping timeout: 252 seconds)
14:20:27  * dshaw_joined
14:25:01  * dshaw_quit (Ping timeout: 272 seconds)
14:33:30  * kenperkinsjoined
14:38:00  <indutny>hey people
14:45:19  * jmar777joined
14:53:46  * c4miloquit (Remote host closed the connection)
14:54:24  * c4milojoined
14:59:01  * c4miloquit (Ping timeout: 265 seconds)
15:01:01  * dshaw_joined
15:05:47  * dshaw_quit (Ping timeout: 265 seconds)
15:07:06  * piscisaureusquit (Ping timeout: 252 seconds)
15:09:11  * kenperkinsquit (Quit: Computer has gone to sleep.)
15:09:45  * paulfryzeljoined
15:13:35  * kenperkinsjoined
15:13:59  * paulfryzelquit (Ping timeout: 240 seconds)
15:16:24  * defunctzombie_zzchanged nick to defunctzombie
15:17:35  <MI6>nodejs-master: #902 UNSTABLE osx-x64 (1/700) linux-ia32 (2/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/902/
15:18:02  * mikolalysenkojoined
15:19:04  * thlorenzjoined
15:19:46  * piscisaureusjoined
15:20:20  * AvianFluquit (Remote host closed the connection)
15:20:20  * dshaw_joined
15:25:07  * dshaw_quit (Ping timeout: 265 seconds)
15:25:30  * c4milojoined
15:30:07  * jmar777quit (Read error: Connection reset by peer)
15:30:33  * jmar777joined
15:31:17  * bradleymeckquit (Quit: bradleymeck)
15:39:07  * bradleymeckjoined
15:42:23  * kpdeckerjoined
15:46:19  * defunctzombiechanged nick to defunctzombie_zz
15:56:40  * paulfryzeljoined
16:03:14  * paulfryzelquit (Read error: Connection reset by peer)
16:03:52  * paulfryzeljoined
16:04:27  * mikolalysenkoquit (Ping timeout: 260 seconds)
16:04:50  * mikolalysenkojoined
16:05:07  * octetcloudjoined
16:05:48  * defunctzombie_zzchanged nick to defunctzombie
16:07:28  * AvianFlujoined
16:10:19  * kpdeckerquit (Quit: Leaving.)
16:11:40  * kenperkinsquit (Remote host closed the connection)
16:12:22  * kenperkinsjoined
16:14:28  * bajtosquit (Quit: bajtos)
16:19:47  * Kakeraquit (Read error: Operation timed out)
16:20:37  * mikolalysenkoquit (Ping timeout: 252 seconds)
16:22:39  * Kakerajoined
16:31:25  <tjfontaine>indutny: morn
16:31:59  <tjfontaine>indutny: we broke the windows build :)
16:32:08  <AlexisMocha_>was about to say that
16:32:23  <LeftWing>Not like anybody uses that old thing anyway. ;P
16:32:32  <tjfontaine>lemme see what we landed in that patch exactly
16:32:43  <tjfontaine>sonly 32bit windows
16:33:00  <tjfontaine>https://github.com/joyent/node/compare/114bff467eb6...eaf76648a6ba
16:33:17  <AlexisMocha_>i think maybe the patch should apply only to VS 2013
16:33:20  <tjfontaine>LeftWing: we're so close to a green board for windows though :)
16:33:28  <tjfontaine>AlexisMocha_: it was failing on 2010
16:33:40  <tjfontaine>AlexisMocha_: it's not failing to build node itself, but failing for people trying to build addons
16:34:08  <AlexisMocha_>mmm can we revert the checkin and then try to re-address the issue?
16:34:20  <AlexisMocha_>i'll be happy to look into it
16:34:23  * felixge_joined
16:35:19  <tjfontaine>ya, we can, it's interesting that we're only breaking it for 32bit
16:35:38  * felixgequit (Ping timeout: 264 seconds)
16:35:39  * felixge_changed nick to felixge
16:36:05  * mikolalysenkojoined
16:36:09  * AlexisMocha_quit (Read error: Connection reset by peer)
16:36:37  * AlexisMochajoined
16:37:24  <indutny>AlexisMocha: hey man
16:37:30  <indutny>tjfontaine: and fixed?
16:37:54  <indutny>oooh
16:37:56  <indutny>I forgot to push
16:38:15  <indutny>tjfontaine: AlexisMocha : please review https://gist.github.com/indutny/71c98a9d907b5edee7f5
16:38:18  <indutny>should be working fine
16:40:05  * felixgequit (Ping timeout: 248 seconds)
16:41:43  <tjfontaine>ok let me try, if AlexisMocha doesn't get to it first
16:42:02  * felixgejoined
16:42:02  * felixgequit (Changing host)
16:42:02  * felixgejoined
16:42:13  <AlexisMocha>i am looking into it
16:42:37  <AlexisMocha>...to find out what the "right way" is
16:43:00  <piscisaureus>indutny: AlexisMocha: SIZE_T should not be a typedef but a #define
16:43:25  <piscisaureus>you could also do
16:43:28  <piscisaureus>#define _SSIZE_T_
16:43:32  <piscisaureus>_SSIZE_T_DEFINED_
16:43:40  <piscisaureus>er, #define _SSIZE_T_DEFINED_
16:44:34  * AvianFlu_joined
16:44:46  <piscisaureus>AlexisMocha, indutny: also, why not include "uv.h" in node.h -- it defines ssize_t
16:44:56  <indutny>have you seen patch?
16:44:58  <indutny>https://gist.github.com/indutny/71c98a9d907b5edee7f5
16:45:06  <tjfontaine>I think we split them up for a reason once before
16:45:11  <indutny>that's what I am propisng
16:45:13  <indutny>proposing*
16:46:29  <AlexisMocha>my gut feeling tells me that we dont' want to define those things ourselves
16:46:39  <AlexisMocha>but rather include the right header
16:46:44  * rmgjoined
16:47:26  <AlexisMocha>according to MSDN, it's supposed to be defined in CRTDEFS.H , across all versions
16:47:44  * AvianFluquit (Disconnected by services)
16:47:47  * AvianFlu_changed nick to AvianFlu
16:47:57  <AlexisMocha>have you tried including that?
16:48:30  * bajtosjoined
16:53:18  <AlexisMocha>works on 2010 32 and 64 bit
16:53:32  * daviddiasjoined
16:55:59  * piscisaureusquit (Ping timeout: 265 seconds)
16:59:19  <AlexisMocha>https://github.com/joyent/node/pull/6984
17:00:24  * kpdeckerjoined
17:02:57  <tjfontaine>lemme put that in a feature branch
17:03:19  <tjfontaine>that way it will actually go through CI, but I have confidence
17:03:32  <tjfontaine>AlexisMocha: can you also try and build one of hte addons?
17:04:21  <tjfontaine>AlexisMocha: cd test/addons/hello-world, ..\..\..\Release\node.exe ../../../deps/npm/node_modules/node-gyp/bin/node-gyp --nodedir=..\..\..\ rebuild
17:04:24  <tjfontaine>should be sufficient
17:04:52  <AlexisMocha>thanks for the instructions... gonna do it now
17:05:16  <tjfontaine>some pathing may be off slightly
17:05:30  <tjfontaine>I need to refactor the Makefile into a python script that is cross platform ...
17:05:30  <tjfontaine>anyway
17:06:14  <MI6>joyent/node: Alexis Campailla merge-review * f8870f3 : windows: define size_t in node.h (+17 more commits) - http://git.io/12IDeA
17:06:52  <tjfontaine>http://jenkins.nodejs.org/job/node-review-windows/15/ is the build we're watching
17:07:46  <AlexisMocha>mmmm.. still missing ssize_t
17:08:46  <tjfontaine>that's what we were trying to fix with this patch, I'm not opposed to including uv.h in node.h
17:15:22  <AlexisMocha>what if we didn't use ssize_t? it's non standard and the two places where we use it really invoke functions that return size_t
17:19:55  <indutny>so
17:19:59  <indutny>you guys figured it out? :)
17:20:13  <indutny>tjfontaine: doesn't my patch work?
17:20:25  <indutny>sorry, I'm flickering in and out today
17:20:29  <indutny>not following whole conversation
17:21:09  <MI6>node-review-unix: #164 UNSTABLE smartos-ia32 (1/700) osx-ia32 (1/700) osx-x64 (4/700) http://jenkins.nodejs.org/job/node-review-unix/164/
17:22:21  <AlexisMocha>indutny: work in progress :)
17:23:06  * dshaw_joined
17:23:30  <MI6>node-review-windows: #15 UNSTABLE windows-x64 (8/700) windows-ia32 (12/700) http://jenkins.nodejs.org/job/node-review-windows/15/
17:25:39  * dap_joined
17:27:44  * TooTallNatejoined
17:28:16  <indutny>ok
17:29:09  * defunctzombiechanged nick to defunctzombie_zz
17:30:40  * bradleymeckquit (Quit: bradleymeck)
17:33:44  * AWintermanjoined
17:51:06  * daviddiasquit (Read error: Connection reset by peer)
17:53:08  <MI6>libuv-master: #455 FAILURE http://jenkins.nodejs.org/job/libuv-master/455/
17:53:21  * hzquit
17:55:26  * octetcloudquit (Ping timeout: 264 seconds)
18:01:49  <indutny>TooTallNate: hey man
18:02:01  <indutny>TooTallNate:: I'm trying to use `node-ref`
18:02:05  <indutny>it is super cool :)
18:02:09  <tjfontaine>heh
18:02:11  <indutny>however I'm thinking about one thing
18:02:15  <indutny>that I really miss pretty much
18:02:23  <indutny>right now I'm doing
18:02:27  <indutny>buf.slice(8).ref()
18:02:47  <indutny>but it is producing a new object everytime I'm doing it
18:03:02  <indutny>TooTallNate: if you could introduce `.ref(offset)` - it would be super cool
18:03:04  <indutny>I think
18:03:07  <indutny>or something like this
18:03:11  <indutny>also
18:03:17  <indutny>`.deref(offset)`
18:03:18  <indutny>:)
18:03:29  <tjfontaine>oh that reminds me
18:03:30  <indutny>which should add offset to the pointer and deref it
18:05:18  * brsonjoined
18:05:49  * Kakeraquit (Read error: Connection reset by peer)
18:09:38  * Kakerajoined
18:17:38  <trevnorris>hello all
18:17:39  <TooTallNate>indutny: deref(offset) might work
18:17:51  <indutny>great!
18:17:56  <indutny>that would simplify my life :)
18:18:03  <indutny>using it in jit.js and js.js now
18:18:13  <indutny>only waiting for `mmap` module author to accept my changes
18:18:16  <indutny>in order to push out versions
18:18:27  <trevnorris>tjfontaine: looking at your rebase of ben's work now
18:18:29  <indutny>TooTallNate: does ref() sounds good too?
18:19:49  <trevnorris>tjfontaine: replace this: binding.startGarbageCollectionTracking(this.emit.bind(this, 'gc'));
18:19:49  <trevnorris>with this: var self = this; binding.startGarbageCollectionTracking(function() { self.emit.call('gc'); });
18:19:55  <trevnorris>erm
18:20:15  <tjfontaine>this was just a pure port, if you want I will do a PR and you can comment on that so I have tracking
18:20:25  <trevnorris>great. sounds good
18:20:33  <tjfontaine>moment
18:24:01  <tjfontaine>trevnorris: https://github.com/joyent/node/pull/6985
18:24:10  <trevnorris>great. thanks
18:25:19  <tjfontaine>brb
18:25:22  <trevnorris>k
18:27:17  <AlexisMocha>tjfontaine: indutny: I updated the PR. This seems like the most portable solution and the addon builds fine with it. https://github.com/MSOpenTech/node/commit/ff1d918af282275292e083857f0fa9449e58a071
18:28:07  <indutny>yoou must be kidding
18:31:04  * bajtosquit (Quit: bajtos)
18:31:26  <AlexisMocha>why?
18:32:54  <indutny>because it is using size_t and ptrdiff_t
18:33:00  <indutny>i'd rather stick to intptr_t
18:33:14  <indutny>and uintptr_t
18:33:28  <indutny>or
18:33:30  <indutny>intptr_t
18:33:31  <indutny>and
18:33:32  <indutny>ptrdiff_t
18:33:39  <indutny>but it is quite lame
18:33:48  <indutny>also this is a public header
18:34:02  <AlexisMocha>size_t, ptrdiff_t are standard types, all defined in stddef
18:34:02  <indutny>and changing types is the last thing that I'd want to do
18:34:23  <AlexisMocha>we shouldn't be in the business of definining non-portable types when there is a perfectly suitable standard equivalent
18:35:29  <indutny>ok
18:35:48  <indutny>if it works :)
18:36:35  * TooTallNatequit (Quit: Computer has gone to sleep.)
18:36:48  * TooTallNatejoined
18:41:52  <MI6>nodejs-master-windows: #689 FAILURE windows-x64 (8/700) http://jenkins.nodejs.org/job/nodejs-master-windows/689/
18:44:02  * rosskjoined
18:44:38  * AvianFluquit (Remote host closed the connection)
18:47:24  * indexzerojoined
18:54:51  * mikealjoined
18:58:53  * AvianFlujoined
19:00:44  * bajtosjoined
19:00:50  * dshaw_quit (Quit: Leaving.)
19:02:29  * abraxasjoined
19:03:49  * bajtosquit (Client Quit)
19:04:57  * brsonquit (Ping timeout: 272 seconds)
19:07:29  * abraxasquit (Ping timeout: 272 seconds)
19:10:37  * indexzeroquit (Quit: indexzero)
19:15:32  * octetcloudjoined
19:16:17  <tjfontaine>AlexisMocha, indutny: weighing our options here, I'm slightly worried about incompatible sizes between addons and libuv, which would be dangerous
19:20:27  * AvianFluquit (Remote host closed the connection)
19:21:53  <indutny>+1 for size_t
19:21:57  <indutny>and ssize_t
19:22:00  <indutny>if we could get them work
19:28:05  * Kakeraquit (Ping timeout: 248 seconds)
19:32:28  * janjongboomquit (Quit: My MacBook has gone to sleep. ZZZzzz…)
19:32:48  <AlexisMocha>tjfontaine: can you elaborate on "incompatible sizes"?
19:33:25  <tjfontaine>I dunno, I'm just worried about using different types and definitions among node/uv that could lead to problems for people writing addons
19:33:25  <AlexisMocha>ssize_t, intptr_t, ptrdiff_t all have the same size
19:34:05  <tjfontaine>well, I worry that there might be a difference across compilers?
19:34:16  <indutny>AlexisMocha: why not #define them
19:34:17  <indutny>or typedef
19:34:25  <AlexisMocha>which is why the path of least resistence is to use standard types
19:34:56  <tjfontaine>yes, I concur that stddef is a safer include, does uv.h return or consume ssize_t as well?
19:35:09  <indutny>it does consume, I think
19:35:13  * Kakerajoined
19:35:24  <indutny>and ssize_t too
19:35:26  <AlexisMocha>if you define them, then you still need to make sure the underlying types are compatible, plus now you have to worry about the defines
19:36:21  <AlexisMocha>those types will be compatible, no need to change things if it only consumes them
19:36:39  <AlexisMocha>until we run into situations like today
19:38:30  <indutny>well
19:38:35  <indutny>that's why it may be a good idea to include uv.h
19:38:37  * janjongboomjoined
19:38:54  <indutny>so that it'll be compatible with it
19:40:56  <tjfontaine>indeed, the "issue" with that is that now addons are also parsing libuv headers even though they may not give a crap and ever use libuv, hopefully that doesn't also cause header ordering issues in other places
19:41:17  <AlexisMocha>uv.h includes stddef
19:42:36  <AlexisMocha>and also gets a ssize_t definition
19:42:39  * mikolalysenkoquit (Ping timeout: 252 seconds)
19:43:26  * janjongboomquit (Ping timeout: 264 seconds)
19:43:58  <tjfontaine>the way uv.h does it seems reasonable enough for us to duplicate?
19:44:09  <tjfontaine>uv-win.h
19:47:08  * dshaw_joined
19:47:32  <AlexisMocha>you could certainly do that
19:47:34  * mikolalysenkojoined
19:47:36  <AlexisMocha>however it's not standard
19:47:57  <AlexisMocha>so it might conflict with who knows what other headers out there
19:49:44  <AlexisMocha>what is the value you see in exposing a definition for ssize_t?
19:50:49  * AvianFlujoined
19:55:16  * defunctzombie_zzchanged nick to defunctzombie
19:55:18  * c4miloquit (Ping timeout: 265 seconds)
19:56:41  <indutny>AlexisMocha: many are relying on it anyway
19:56:48  <indutny>I mean addon authors
19:58:59  * AvianFluquit (Ping timeout: 240 seconds)
20:00:00  * jmar777quit (Read error: Connection reset by peer)
20:00:25  * jmar777joined
20:03:55  * AvianFlujoined
20:17:32  * kenperkins_joined
20:21:25  <trevnorris>tjfontaine: have a quick explanation of how you think MDC should be refactored?
20:21:25  * kenperkinsquit (Ping timeout: 248 seconds)
20:28:35  <tjfontaine>trevnorris: I dunno, but the amount of code duplication there feels painful
20:31:07  <trevnorris>tjfontaine: yeah, I agree. was hoping all that could have been removed using AL in domains, but since that got killed by hapi tests...
20:31:26  <tjfontaine>AlexisMocha, indutny: this whole conversation is scaring me further about our module api/abi problem :)
20:31:55  <trevnorris>feel the same about _tickDomainCallback, but it did have a noticeable perf impact.
20:32:30  <tjfontaine>trevnorris: right, I'm wondering if we could abstract some of the load/unload stuff into some static methods AW::TriggerBefore AW::TriggerAfter
20:32:49  <tjfontaine>and let the compiler DoTheRightThing for us
20:34:37  <tjfontaine>it doesn't need to be fixed before 0.12 in that regard, it's just one of those things, if I start refactoring MDC ... I should touch all the call sites then
20:49:38  <indutny>tjfontaine:there are no abi problem :)
20:50:32  <indutny>at least
20:50:37  <indutny>while we are not distributing binaries
20:53:18  * rainabbajoined
20:54:00  <rainabba>There a library/function/utility out there that can sanitize directory/files names?
20:54:36  <rainabba>nm :P
20:56:29  <indutny>wrong room :)
20:56:33  <indutny>channel*
20:56:34  <indutny>whatever
20:57:45  <tjfontaine>indutny: there are abi problems, because we load shared objects that may be compiled with different toolchains
20:58:09  <tjfontaine>indutny: we have a simple module version check, but it's very obtuse in its usage
20:58:28  <tjfontaine>and c++ compilers are notorious for being unsafe at any speed for abi
20:58:38  <indutny>wait a minute
20:58:40  <indutny>what shared objects?
20:58:45  <indutny>aaah
20:58:47  <tjfontaine>node binary addons
20:58:49  <indutny>you mean size_t and stuff
20:58:53  * dshaw_quit (Quit: Leaving.)
20:58:55  <tjfontaine>right
20:58:59  <indutny>well
20:59:07  <indutny>this problem should not exist if the `node.h` is properly defined
20:59:10  <tjfontaine>it's all very scary, and often working by accident
20:59:15  <indutny>but yes
20:59:16  <indutny>you are right
20:59:39  <tjfontaine>moving locations here so I'm not picking up interrupts from real life
21:03:15  * abraxasjoined
21:06:10  * dshaw_joined
21:07:49  * abraxasquit (Ping timeout: 248 seconds)
21:08:21  * bradleymeck_joined
21:08:27  * bradleymeck_quit (Client Quit)
21:09:19  <tjfontaine>AlexisMocha: if intptr_t and ssize_t are the same on windows, what are the dangers with following that pattern for node.h on windows?
21:09:27  <tjfontaine>*same size
21:10:45  <tjfontaine>where at all possible I want to make sure we're doing it the RightWay, so if there's commentary from inside MS that uv-win.h is wrong here as well, then lets investigate a portable solution
21:10:55  <tjfontaine>that can be applicable across our platform
21:11:10  * hzjoined
21:17:03  * __rockbot___joined
21:17:38  * indexzerojoined
21:18:06  * dshaw_quit (Quit: Leaving.)
21:22:02  * dshaw_joined
21:22:25  <tjfontaine>this post depresses me, but its place in the conversation is slightly tangental http://blogs.msdn.com/b/david_leblanc/archive/2008/09/02/ptrdiff-t-is-evil.aspx
21:25:40  * defunctzombiechanged nick to defunctzombie_zz
21:28:03  * __rockbot___changed nick to __rockbot__
21:29:31  * kpdeckerquit (Quit: Leaving.)
21:30:20  * hueniversejoined
21:31:23  * mikolalysenkoquit (Ping timeout: 245 seconds)
21:31:58  * kazuponjoined
21:31:59  * __rockbot__quit (Quit: __rockbot__)
21:33:57  * jmar777quit (Remote host closed the connection)
21:41:30  <MI6>libuv-master-windows: #35 FAILURE http://jenkins.nodejs.org/job/libuv-master-windows/35/
21:45:40  <AlexisMocha>tjfontaine: no problem in typedef'ing or converting ssize_t to intptr_t o ptrdiff_t per se
21:47:24  <AlexisMocha>tjfontaine: what's not portable is the definition itself. Should it be a #define or typedef? depend on _SSIZE_T_DEFINED? Or the inclusion of some specific header? Given that it's non-standard, the definition more easily might clash with other definitions out there. That's all really.
21:49:04  <tjfontaine>so your preferred course of action here is to include stddef.h and modify to ptrdiff_t for just this part of the api?
21:50:56  <MI6>libuv-v0.10-windows: #24 FAILURE http://jenkins.nodejs.org/job/libuv-v0.10-windows/24/
21:52:17  <AlexisMocha>That would be my call, yes. The alternative is acceptable too. Both might break some users. But if we remove the dependency from the non-standard type, we know this issue is closed for good. If we keep the non-standard type, it might come and bite us again in the future.
21:53:16  <tjfontaine>the fix for the non-standard type though, really involves touching the api for libuv as well?
21:53:26  <tjfontaine>we really need to audit this whole portion of our surface area
21:53:37  <tjfontaine>but I am not sure we're going to be able to do that before 0.12 :)
21:54:24  <AlexisMocha>well, there appear to be no need to fix it for libuv right now...
21:54:39  <tjfontaine>ok
22:00:26  * defunctzombie_zzchanged nick to defunctzombie
22:13:40  <Kakera>pretty sure I've already signed the CLA...
22:13:48  <tjfontaine>Kakera: ya google is being mean today
22:14:04  <MI6>joyent/node: Timothy J Fontaine merge-review * 462bfda : src: only define ssize_t on windows if undefined - http://git.io/1Ib2Aw
22:16:52  * dshaw_quit (Quit: Leaving.)
22:17:02  * rmgquit (Remote host closed the connection)
22:18:06  * dshaw_joined
22:19:04  * bradleymeck_joined
22:19:18  <tjfontaine>blah this of course helps break lint
22:19:30  <tjfontaine>and appeasing lint breaks the build
22:24:06  * bradleymeck_changed nick to bradleymeck
22:28:16  * mikolalysenkojoined
22:28:59  <MI6>node-review-unix: #165 UNSTABLE smartos-ia32 (1/700) linux-ia32 (1/700) osx-ia32 (3/700) osx-x64 (2/700) http://jenkins.nodejs.org/job/node-review-unix/165/
22:29:47  <MI6>node-review-windows: #16 UNSTABLE windows-x64 (8/700) windows-ia32 (10/700) http://jenkins.nodejs.org/job/node-review-windows/16/
22:30:18  * rendarquit (Quit: Leaving)
22:31:18  * bradleymeckquit (Quit: bradleymeck)
22:32:59  * mikolalysenkoquit (Ping timeout: 240 seconds)
22:33:44  * pachet_joined
22:36:50  * pachetquit (Ping timeout: 264 seconds)
22:41:25  * brsonjoined
22:44:33  <indutny>tjfontaine: so except few nits
22:44:38  <indutny>tjfontaine: does try write LGTY?
22:45:02  <indutny>just want to make sure
22:45:05  <tjfontaine>yup
22:47:01  * dshaw_quit (Quit: Leaving.)
22:47:43  <tjfontaine>HOLY SHIT GUYS, WE CAN ASSIGN LABELS AND MILESTONES AND ASSIGNEES IN PRS
22:47:43  <LOUDBOT>NOBODY WANTS TO THINK ABOUT THOSE THINGS
22:48:51  <indutny>yes :)
22:50:10  * superjoejoined
22:52:39  * mikolalysenkojoined
22:52:43  * dshaw_joined
22:54:26  <tjfontaine>so
22:54:34  <tjfontaine>how to fix lint and the build
22:55:58  * indexzeroquit (Quit: indexzero)
22:57:02  <indutny>em?
22:57:12  <indutny>lint is quite easy, right?
22:57:23  <indutny>you just need to move it below
22:57:32  <indutny>right after `namespace node {} `
22:57:40  <indutny>what's wrong with build?
22:58:01  * mikolalysenkoquit (Ping timeout: 272 seconds)
22:58:06  <tjfontaine>that won't have it defined for Decode* will it?
22:58:15  <tjfontaine>or is it delayed because of cpp multiple passes?
22:58:50  <tjfontaine>oh I guess it is further down int he file
22:58:53  <tjfontaine>ignore me :)
23:01:14  * paulfryz_joined
23:03:03  <MI6>joyent/node: Fedor Indutny master * 9836a4e : stream_wrap: use `uv_try_write` where possible - http://git.io/QlmhAg
23:03:05  <indutny>finally
23:03:10  <tjfontaine>:)
23:03:34  <tjfontaine>are you ok with that patch I landed above if I move it to right before Decode*?
23:03:48  * m76quit (Read error: Connection reset by peer)
23:03:59  * paulfryzelquit (Ping timeout: 240 seconds)
23:04:38  <indutny>where is it?
23:04:50  <indutny>I can see only my patch
23:04:51  <indutny>in log
23:05:10  <tjfontaine>pushed to a feature branch, https://github.com/joyent/node/compare/f8870f312a97...462bfdaa1b81
23:05:24  <indutny>oh
23:05:34  <indutny>you need to move it below
23:05:41  <indutny>but yeah
23:05:42  <indutny>seems to be fine
23:05:50  <tjfontaine>ok, I'm going to push it then
23:06:38  <indutny>so
23:06:44  <indutny>I'm going to push another commit
23:06:49  <indutny>unless you will hurry a bit :)
23:06:53  <indutny>3
23:06:55  <indutny>2
23:06:57  <tjfontaine>push both of thoms together
23:06:57  <indutny>1
23:07:02  <indutny>ok
23:07:06  <tjfontaine>https://github.com/joyent/node/pull/6983
23:07:54  <MI6>joyent/node: Timothy J Fontaine master * ab54e32 : src: only define ssize_t on windows if undefined - http://git.io/qBOo3Q
23:08:10  <indutny>yay!
23:08:11  <indutny>:)
23:08:12  <MI6>joyent/node: Thom Seddon master * 657cd2c : lib: fix unnecessary coercion in lib/net.js (+1 more commits) - http://git.io/Pw64LA
23:08:28  <tjfontaine>:)
23:08:47  * kazuponquit (Remote host closed the connection)
23:09:14  * kazuponjoined
23:10:32  * paulfryz_quit (Remote host closed the connection)
23:10:37  <indutny>[email protected]
23:11:09  * paulfryzeljoined
23:12:34  <MI6>nodejs-master: #903 UNSTABLE osx-x64 (1/700) linux-ia32 (1/700) osx-ia32 (1/700) http://jenkins.nodejs.org/job/nodejs-master/903/
23:12:56  <tjfontaine>so close to green board!
23:13:09  * hueniversequit (Ping timeout: 248 seconds)
23:13:41  * kazuponquit (Ping timeout: 248 seconds)
23:14:10  <indutny>tjfontaine: haha :)
23:14:23  <indutny>if we will land that close(0) patch
23:14:37  <indutny>huhh http://jenkins.nodejs.org/job/nodejs-master/903/DESTCPU=ia32,label=linux/tapTestReport/test.tap-251/ ?
23:14:39  <tjfontaine>oh stdio you mean
23:14:50  <tjfontaine>ya I'm not sure what's up with that one
23:14:52  <indutny>ah
23:14:59  <indutny>we haven't updated parser
23:15:00  <indutny>yet
23:15:07  <tjfontaine>right
23:15:11  * AWintermanquit (Ping timeout: 260 seconds)
23:15:14  <indutny>let me do a release
23:15:23  <tjfontaine>sorry hadn't gotten to it yet
23:15:43  * drsjoined
23:15:46  <indutny>np
23:15:46  <tjfontaine>we have trywrite that's a ncie feature for this unstable release
23:15:53  <indutny>well, let's fix parser
23:15:57  <indutny>that's quite quick
23:16:00  <indutny>just give me a 5-10 minutes
23:16:03  <tjfontaine>k
23:16:07  * paulfryzelquit (Ping timeout: 272 seconds)
23:16:38  <tjfontaine>we are seeing a new test failure though
23:16:41  * hueniversejoined
23:16:50  <drs>so question for you guys, uv_accept where server is a UV_TCP and client is a UV_PIPE does not work on Windows.
23:17:15  * rmgjoined
23:17:27  <drs>and that's because uv_accept casts the UV_PIPE to a uv_tcp where handle->socket is not initialized
23:17:43  <drs>so it fails an assertion
23:17:55  <indutny>tjfontaine: which one?
23:18:09  <indutny>drs: why should it work?
23:18:19  <tjfontaine>indutny: https://gist.github.com/tjfontaine/8678691/raw/5848e06492b999cec17187ca0d12cbde86c7be20/test-stdin-from-file.js.txt
23:18:25  <indutny>that's it
23:18:26  <indutny>close stdio
23:18:35  <indutny>that PR will fix it
23:18:40  <tjfontaine>oh ok
23:18:59  <MI6>nodejs-master-windows: #690 FAILURE windows-x64 (9/700) http://jenkins.nodejs.org/job/nodejs-master-windows/690/
23:19:08  <indutny>yeah
23:19:11  <tjfontaine>next round on windows will work for that build
23:19:12  <drs>I guess what I'm trying to do is do that file descriptor passing magic on Windows
23:19:25  <tjfontaine>indutny: that assertion doesn't blow on v0.10?
23:19:26  <drs>i just assumed it had worked before at some point
23:19:30  <indutny>drs: it should not be a tcp server
23:19:37  <indutny>drs: it should be an ipc pipe
23:19:46  <indutny>and you should uv_accept(ipc_pipe, received_pipe)
23:19:54  <indutny>and work with received_pipe
23:20:03  <indutny>tjfontaine: yes, it happens after try write patch
23:20:03  <drs>but what about a new socket connection
23:20:11  <indutny>tjfontaine: can't tell you exact reason of it
23:20:14  <indutny>perhaps timing
23:20:28  <indutny>drs: well, uv_accept(ipc_pipe, tcp_conn)
23:20:33  <tjfontaine>indutny: oh, seems like timing, are we firing an eof much sooner than we were before because we're consuming it faster?
23:20:41  <indutny>probably
23:20:45  <drs>oh.. maybe that's it.
23:20:52  <indutny>drs: yeah
23:20:55  <indutny>you receive a handle type
23:20:59  <indutny>as the last argument to read2_cb
23:21:08  <indutny>use it to determine which type of handle you should accept
23:21:13  <indutny>but, btw,
23:21:20  <indutny>I think only tcp connections are working on windows
23:21:30  <indutny>I mean receiving them over IPC pipe
23:21:42  <indutny>tjfontaine: running tests after http_parser update
23:21:44  <tjfontaine>k
23:21:50  <drs>indutny: yes, I think i read that somewhere, and that's fine
23:21:52  <indutny>tjfontaine: looks good so far
23:22:06  <MI6>nodejs-master: #904 UNSTABLE linux-x64 (1/700) osx-x64 (1/700) linux-ia32 (1/700) osx-ia32 (1/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/904/
23:22:19  <indutny>quick question
23:22:22  <indutny>what do we think about https://github.com/joyent/node/issues/6986 ?
23:22:27  <indutny>I'm fine with it
23:22:35  <indutny>but I spend not too much time writing js
23:22:38  <tjfontaine>heh
23:23:03  <indutny>will it break Stability - 3
23:23:05  <indutny>Stable
23:23:29  <indutny>guess it is a minor change
23:23:31  <indutny>and should be fine
23:23:39  <tjfontaine>I'm not sure, my preference is for explicit notation in all cases, but people are different
23:23:56  <tjfontaine>I think the question is if it round trips ok
23:24:23  <tjfontaine>url.format(url.parse('http://foo'))
23:24:34  <MI6>joyent/node: Fedor Indutny master * 597eb6a : deps: update http_parser to 2.2.1 - http://git.io/BPePoQ
23:24:54  <tjfontaine>on v0.10
23:24:54  <tjfontaine>> url.format(url.parse('http://foo'))
23:24:54  <tjfontaine>'http://foo/'
23:25:07  <indutny>yeah
23:25:14  <indutny>and if you'll add port - it will preserve it
23:25:15  <tjfontaine>> url.format(url.parse('http://foo'))
23:25:16  <tjfontaine>'http://foo/'
23:25:27  <tjfontaine>ya, i think preservation is key
23:25:42  <indutny>ok
23:25:45  <indutny>I'll comment
23:25:48  <tjfontaine>thanks
23:26:00  * dshaw_quit (Quit: Leaving.)
23:26:22  <tjfontaine>so, stdio close I'm still not sure I'm sold on for v0.10, but it's clearly now a must for master
23:26:42  <tjfontaine>I'd like that in there before I cut the v0.11 release
23:26:49  <indutny>ok
23:26:50  * defunctzombiechanged nick to defunctzombie_zz
23:26:50  <indutny>let's do it
23:26:51  <tjfontaine>isaacs: you have time to comment on this?
23:26:58  <tjfontaine>indutny: we can always backport
23:27:01  <indutny>indeed
23:27:03  <indutny>I'll keep it open
23:27:06  <tjfontaine>thanks
23:28:29  * rmgquit (Ping timeout: 240 seconds)
23:29:09  <indutny>this is going to be pretty big v0.11 release
23:29:15  <indutny>are we going to freeze features after it?
23:29:31  <indutny>I don't think that there are any features left in my inbox
23:29:42  <indutny>except this https://github.com/joyent/node/issues/2754#issuecomment-32244205
23:29:46  <indutny>but I haven't even read it
23:29:55  <tjfontaine>I dont' think you do, but I have one you could take off my plate while I finish the tracing stuff
23:30:47  <tjfontaine>but I can talk to you about it tomorrow, I'm going to do a v0.11 release tonight and then go heads down for a while
23:30:58  <indutny>good
23:31:06  <MI6>joyent/node: Fedor Indutny master * fc26fd6 : node: do not ever close stdio - http://git.io/x0bfZQ
23:31:23  <indutny>-1
23:31:24  <indutny>:)
23:31:26  <tjfontaine>:P
23:32:19  <tjfontaine>indutny: you mean you want to integrate pty.js?
23:32:46  <indutny>I have no idea
23:32:50  <indutny>reading the issue now
23:33:53  <MI6>nodejs-master: #905 UNSTABLE osx-x64 (1/700) linux-ia32 (1/700) osx-ia32 (1/700) smartos-ia32 (1/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/905/
23:35:24  <MI6>nodejs-master-windows: #691 UNSTABLE windows-x64 (9/700) windows-ia32 (11/700) http://jenkins.nodejs.org/job/nodejs-master-windows/691/
23:35:45  <indutny>hm...
23:35:57  <indutny>cluster-disconnect
23:36:03  <indutny>racey
23:36:30  <tjfontaine>even still I landed a partial fix
23:37:21  <tjfontaine>ah 1.0
23:37:23  <tjfontaine>er timeotu
23:37:32  <tjfontaine>indutny: see https://github.com/joyent/node/pull/6840
23:37:41  <tjfontaine>indutny: specifically the second commit from AlexisMocha
23:38:20  <indutny>ok
23:38:26  <indutny>I'm sleepy :)
23:38:27  <indutny>sorry
23:38:39  <indutny>perhaps in next release
23:38:42  <tjfontaine>nod
23:39:02  <tjfontaine>it may be that cluster-disconnect is all we really need for the test :)
23:39:10  * rmgjoined
23:39:55  * thlorenzquit (Remote host closed the connection)
23:40:28  * dshaw_joined
23:40:58  <MI6>joyent/node: Wyatt Preul v0.10 * 8c05570 : docs: clarify origin in agent.maxSockets section - http://git.io/e-HkAQ
23:41:12  <indutny>time to land some oldies
23:41:13  <indutny>:)
23:43:06  <MI6>nodejs-master: #906 UNSTABLE linux-ia32 (1/700) smartos-x64 (1/700) http://jenkins.nodejs.org/job/nodejs-master/906/
23:43:09  <indutny>https://github.com/joyent/node/pulse
23:43:12  <indutny>we are mostly deleting stuff
23:43:27  <tjfontaine>good :)
23:43:31  <indutny>. On master, 360 files have changed and there have been 3,671 additions and 6,645 deletions.
23:43:41  <tjfontaine>ya but v8 ...
23:43:49  <tjfontaine>git diff --stat -- src lib
23:44:12  <tjfontaine>oh you need v0.10
23:44:32  <tjfontaine> 138 files changed, 26804 insertions(+), 20270 deletions(-)
23:44:52  <indutny>mostly writing :)
23:44:55  <indutny>bad
23:44:56  <tjfontaine>just libuv
23:44:57  <tjfontaine> 217 files changed, 15832 insertions(+), 8476 deletions(-)
23:45:00  <drs>is there a good example of passing tcp connections over uv_pipes that runs on Windows?
23:45:19  <tjfontaine>drs: the test should have one?
23:45:45  <indutny>https://github.com/joyent/libuv/blob/master/test/test-ipc.c
23:45:48  <indutny>or
23:45:48  <indutny>https://github.com/joyent/libuv/blob/master/test/test-ipc-send-recv.c
23:45:57  <indutny>tjfontaine: so good change :)
23:46:01  <indutny>that we haven't documented now
23:46:06  <indutny>but which you should probably include in release notes
23:46:09  <tjfontaine>ya, frustrating
23:46:10  <indutny>is a tcp dual stack
23:46:14  <tjfontaine>I'm hoping we get better
23:46:19  <tjfontaine>brb
23:48:42  <indutny>oh actually
23:49:23  * pachet_quit (Quit: leaving)
23:49:39  <MI6>nodejs-v0.10-windows: #467 UNSTABLE windows-ia32 (5/610) windows-x64 (6/610) http://jenkins.nodejs.org/job/nodejs-v0.10-windows/467/
23:49:47  <MI6>nodejs-v0.10: #1746 UNSTABLE osx-x64 (5/610) linux-ia32 (2/610) smartos-x64 (8/610) smartos-ia32 (8/610) linux-x64 (2/610) osx-ia32 (7/610) http://jenkins.nodejs.org/job/nodejs-v0.10/1746/
23:50:17  * mikolalysenkojoined
23:51:15  <indutny>looks like it was working
23:51:16  <indutny>:)
23:51:17  <indutny>before it
23:51:19  <indutny>hahaha
23:52:08  <indutny>I'm not sure
23:52:13  <indutny>could you please verify it somewhere?
23:52:37  <MI6>nodejs-master-windows: #692 UNSTABLE windows-x64 (9/700) windows-ia32 (12/700) http://jenkins.nodejs.org/job/nodejs-master-windows/692/
23:55:54  * kenperkins_quit (Quit: Computer has gone to sleep.)
23:58:04  <Kakera>are there any modules where an exception with custom properties gets thrown from C++-land?
23:58:49  <MI6>libuv-master-gyp: #413 UNSTABLE smartos-ia32 (3/204) smartos-x64 (3/204) http://jenkins.nodejs.org/job/libuv-master-gyp/413/