00:13:38  * Something12joined
02:30:34  * DarkGodquit (Ping timeout: 240 seconds)
06:53:57  * SkyRocknRolljoined
07:57:45  * tlhunterquit (Ping timeout: 260 seconds)
08:25:37  * DarkGodjoined
10:04:20  * Something12quit (Ping timeout: 276 seconds)
13:16:29  * KennethWilkejoined
13:26:06  * SkyRocknRollquit (Ping timeout: 240 seconds)
13:32:21  * SkyRocknRolljoined
13:42:33  * SkyRocknRoll_joined
14:23:46  * SkyRocknRollquit (Quit: Leaving)
14:24:04  <rphillips>morning
14:28:34  * travis-cijoined
14:28:35  <travis-ci>luvit/luvi#852 (master - b5d543d : Ryan Phillips): The build passed.
14:28:35  <travis-ci>Change view : https://github.com/luvit/luvi/compare/414fbd9cbdbf...b5d543d55dce
14:28:35  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107782375
14:28:35  * travis-cipart
14:30:10  * KennethWilkequit (Remote host closed the connection)
14:35:59  <rphillips>luvi 2.6.0 tagged
14:36:16  * travis-cijoined
14:36:17  <travis-ci>luvit/luvi#853 (master - 5df4345 : Ryan Phillips): The build passed.
14:36:17  <travis-ci>Change view : https://github.com/luvit/luvi/compare/b5d543d55dce...5df4345b034a
14:36:17  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107783990
14:36:17  * travis-cipart
14:37:25  * travis-cijoined
14:37:26  <travis-ci>luvit/luvi#854 (v2.6.0 - 5df4345 : Ryan Phillips): The build passed.
14:37:26  <travis-ci>Change view : https://github.com/luvit/luvi/compare/v2.6.0
14:37:26  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107784074
14:37:26  * travis-cipart
15:13:50  <rphillips>creationix: could you help me get lit updated for luvi 2.6.0?
15:21:30  * SkyRocknRoll_quit (Remote host closed the connection)
15:25:12  <creationix>rphillips, morning
15:42:47  * therebelrobotjoined
15:48:04  <rphillips>might have gotten it
15:50:19  <rphillips>creationix: https://github.com/luvit/lit/pull/169/files
15:50:22  <rphillips>look ok?
15:53:55  * therebelrobotquit (Ping timeout: 240 seconds)
15:54:18  <creationix>rphillips, I think I'd rather just keep the old behavior in coro-fs
15:54:29  <creationix>no need to make breaking changes all the way up the chain
15:54:33  <rphillips>k
15:54:39  <creationix>there are lots more places in lit where it would break
15:54:59  <creationix>I'm justing testing now to make sure my code works the same on both versions of luvi
15:55:06  <rphillips>cool
15:57:06  <creationix>https://github.com/luvit/lit/commit/5d3ae15413c6f20a6356438f9ff584a4aa9345b3
15:57:31  <creationix>later once the new luvi is updated everwhere (maybe a few months) we can look at pushing the change through
15:57:45  <creationix>at least now people can drop down to using luv directly if the object allocation is a problem for them
15:58:06  <rphillips>nice
16:00:07  <rphillips>should I PR the version bumps? or are you in the process of doing that?
16:01:08  <rphillips>https://github.com/luvit/lit/pull/170
16:03:37  <creationix>that's fine, I just did enough to test it against both versions
16:05:31  <creationix>ok, published new lit
16:06:01  <creationix>hmm, lit update is broken
16:06:07  <creationix>rphillips, does it work for you?
16:06:42  <rphillips>it worked for me
16:08:37  <rphillips>on OSX
16:10:06  <creationix>try it again
16:10:13  <creationix>after you're already up to date
16:11:04  <rphillips>didn't error out... says up to doate
16:11:06  <rphillips>date*
16:13:21  <creationix>weird
16:18:36  * travis-cijoined
16:18:37  <travis-ci>luvit/luvit#2758 (fixes/bump_to_luvi_1_8_0-3 - 9d6d5a1 : Ryan Phillips): The build has errored.
16:18:37  <travis-ci>Change view : https://github.com/luvit/luvit/compare/2ee2341006d1...9d6d5a1c710b
16:18:37  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107807482
16:18:37  * travis-cipart
16:21:54  <creationix>ok, I've got a patch for luvit, I'm testing against old and new luvi
16:22:00  <rphillips>cool
16:23:06  * travis-cijoined
16:23:07  <travis-ci>luvit/luvit#2759 (fixes/bump_to_luvi_1_8_0-3 - bfac948 : Ryan Phillips): The build has errored.
16:23:07  <travis-ci>Change view : https://github.com/luvit/luvit/compare/9d6d5a1c710b...bfac94803125
16:23:07  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107808044
16:23:07  * travis-cipart
16:25:09  * travis-cijoined
16:25:10  <travis-ci>luvit/luvit#2760 (fixes/bump_to_luvi_1_8_0-3 - 91c6d03 : Ryan Phillips): The build has errored.
16:25:10  <travis-ci>Change view : https://github.com/luvit/luvit/compare/bfac94803125...91c6d03880d5
16:25:10  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107808298
16:25:10  * travis-cipart
16:25:15  <creationix>rphillips https://github.com/luvit/luvit/pull/876
16:27:42  <creationix>I just tested manually with both versions of luvi, looks good
16:29:03  * travis-cijoined
16:29:04  <travis-ci>luvit/luvit#2762 (fixes/tap_handle_closing - 6511174 : Ryan Phillips): The build has errored.
16:29:04  <travis-ci>Change view : https://github.com/luvit/luvit/compare/ddb10171ce4b...651117438ab7
16:29:04  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107809312
16:29:04  * travis-cipart
16:30:18  * travis-cijoined
16:30:19  <travis-ci>luvit/luvit#2764 (master - ddee359 : Ryan Phillips): The build passed.
16:30:19  <travis-ci>Change view : https://github.com/luvit/luvit/compare/bd442b8be65c...ddee35921517
16:30:19  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107809580
16:30:19  * travis-cipart
16:30:26  * travis-cijoined
16:30:27  <travis-ci>luvit/luvit#2763 (luv-change-adapt - 96e3a62 : Tim Caswell): The build passed.
16:30:27  <travis-ci>Change view : https://github.com/luvit/luvit/commit/96e3a625cef7
16:30:27  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107809559
16:30:27  * travis-cipart
16:32:46  <creationix>ok, new luvit published to lit
16:33:24  <rphillips>hmm. the readdir test failed
16:33:27  * travis-cijoined
16:33:28  <travis-ci>luvit/luvit#2766 (luv-change-adapt - 7910169 : Tim Caswell): The build has errored.
16:33:28  <travis-ci>Change view : https://github.com/luvit/luvit/compare/96e3a625cef7...7910169de24b
16:33:28  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107810453
16:33:28  * travis-cipart
16:34:16  <rphillips>ah, that must have been a previous commit
16:35:13  <creationix>I wonder why only linux failed
16:38:52  * travis-cijoined
16:38:53  <travis-ci>luvit/luvit#2768 (luv-change-adapt - e74d149 : Tim Caswell): The build passed.
16:38:53  <travis-ci>Change view : https://github.com/luvit/luvit/compare/7910169de24b...e74d1499112a
16:38:53  <travis-ci>Build details : https://travis-ci.org/luvit/luvit/builds/107811901
16:38:53  * travis-cipart
16:43:22  <creationix>must have been bad commit, works on my linux and now travis is green
16:43:34  <rphillips>yep. thanks!
16:44:03  <creationix>still having trouble updating luvit using lit update
16:45:25  <creationix>Thanks for building the binaries, I'll add the 3 32-bit linux variants now
16:45:45  <rphillips>cool. thanks
16:50:42  <creationix>building on linux-32 failed during luajit
16:50:51  <creationix>trying again hoping it's transient
16:52:35  <creationix>still got link error, missing various math libraries
16:53:15  <rphillips>maybe it needs -lm now :/
16:55:23  <creationix>where would that go?
16:56:20  <rphillips>target_link_libraries(mylib m) in luajit.cmake perhaps
16:56:42  <rphillips>in a unix or linux only stanza
16:57:36  <creationix>hmm, cmake tried to detect if it should be added
16:58:08  <creationix>https://github.com/luvit/luv/blob/master/deps/luajit.cmake#L91-L94
17:01:30  <rphillips>i just hit the same problem with my 32bit build on centos
17:02:16  <creationix>I tweaked the cmakefile in deps/luv/deps to unconditionally include libm and it still had the same error
17:02:46  <creationix>I'm going to try building luv via luarocks on that machine
17:02:55  <rphillips>creationix: it's missing lj_vmmath.c
17:03:00  <creationix>armv7 worked fine btw
17:03:47  <rphillips>https://github.com/luvit/luv/pull/215
17:04:30  <rphillips>darn :/
17:07:38  <rphillips>creationix: did it fix it on your 32bit build?
17:07:44  <creationix>still building
17:09:07  <creationix>I guess on most platforms luajit uses optimized assembly for that math, but some fallback to using the match library
17:09:15  * travis-cijoined
17:09:16  <travis-ci>luvit/luv#186 (fixes/lj_vmmath - 3dd2f22 : Ryan Phillips): The build passed.
17:09:16  <travis-ci>Change view : https://github.com/luvit/luv/commit/3dd2f229f21a
17:09:16  <travis-ci>Build details : https://travis-ci.org/luvit/luv/builds/107819023
17:09:16  * travis-cipart
17:09:23  <creationix>hmm, still build error
17:09:37  <creationix>buildvm_x86.dasc:(.text+0x84b): undefined reference to `lj_wrap_pow'
17:09:56  <rphillips>hmm. lj_wrap_pow is vmmath.c
17:10:05  <rphillips>is in*
17:10:30  <rphillips>building on our 32bit box
17:11:01  <creationix>https://github.com/LuaJIT/LuaJIT/blob/v2.1/src/lj_vmmath.c#L18-L35 maybe __PIC__ isn't set?
17:11:13  <rphillips>yeah
17:12:54  <creationix>oh, one fun thing about updating to new luajit
17:12:59  <creationix>I added binary parsing to luajit 2.1
17:13:01  <creationix>0b10010
17:13:07  <creationix>that works now in the repl
17:13:15  <creationix>couldn't talk mike into adding octal though :/
17:13:32  <rphillips>testing a patch on my box
17:16:34  <rphillips>creationix: https://github.com/luvit/luv/pull/215/files
17:16:36  <rphillips>works now
17:16:59  <creationix>so it was missing PIC then
17:17:03  <rphillips>yep
17:17:46  <creationix>so do we want PIC?
17:17:59  <rphillips>on 32bit builds we should
17:18:05  <rphillips>libuv sets it
17:18:13  <creationix><command-line>:0:0: warning: "__PIC__" redefined [enabled by default]
17:18:15  <creationix>and then segfault
17:18:21  <rphillips>hmm
17:18:26  <creationix>building just luv, not luvi
17:19:21  <creationix>I think we need to somehow set -fpic in gcc and not set the var directly
17:19:33  <rphillips>yeah
17:22:23  <creationix>hmm, if I remove the `&& __PIC__` guard, it also compiled, but segfaults (with your pic patch reverted)
17:24:21  * travis-cijoined
17:24:22  <travis-ci>luvit/luv#188 (fixes/lj_vmmath - e8f1961 : Ryan Phillips): The build passed.
17:24:22  <travis-ci>Change view : https://github.com/luvit/luv/compare/3dd2f229f21a...e8f19611b449
17:24:22  <travis-ci>Build details : https://travis-ci.org/luvit/luv/builds/107821931
17:24:22  * travis-cipart
17:33:32  <creationix>fwiw, luajit build just fine using the official makefile
17:33:36  <creationix>*builds
17:48:11  <rphillips>:/
17:48:15  <rphillips>not that easy to get it built the same
18:03:38  * therebelrobotjoined
18:51:10  <creationix>I wonder if anyone has gotten luajit 2.1 working with cmake on i386
18:51:22  <creationix>all I can find is people hitting the same issue and going back to luajit 2.0.x
18:52:11  <creationix>I wish I knew how many people actually use luvi in 32-bit x86 linux
19:14:28  * Haragethjoined
19:55:01  <creationix>rphillips, I added `-fPIC` to CMAKE_C_FLAGS manually, and the amalg build of luajit is taking ages to complete
19:55:12  <rphillips>hmm
19:55:17  <creationix>CPU pegged 100% (in virtualbox)
19:55:36  <rphillips>ah, i am probably at the same place... executable doesn't crash, but hangs
19:55:46  <creationix>yep
20:04:11  <creationix>strace shows nothing when hung
20:07:50  <creationix>hmm, torch has a 2.1 cmakefile that comes from the luadist one https://raw.githubusercontent.com/torch/luajit-rocks/974453600c56aac6612396ceb895e87344d679a8/luajit-2.1/CMakeLists.txt
20:07:57  <creationix>doesn't ours come from the same origins?
20:08:06  <rphillips>yeah
20:09:57  <creationix>hmm, that one just says "architecture not supported"
20:11:17  <creationix>upstream luadist hasn
20:11:27  <creationix>hasn't updated luajit since 2.0.3
20:11:33  <creationix>(2 years ago)
20:13:34  <creationix>maybe we should try add_custom_command to use the official makefile to build luajit
20:15:34  * Haragethquit (Remote host closed the connection)
20:25:51  <creationix>hmm, using native makefile on my mac causes clang to crash
20:26:10  <creationix>nothing like segfaulting your C compiler
20:26:41  <creationix>oh, nevermind, I'm using a custom clang
20:27:32  * Haragethjoined
20:36:38  <rphillips>creationix: oh man... you will not believe this one! \o/
20:37:15  <creationix>find the problem?
20:37:19  <rphillips>yep
20:37:27  <creationix>I was about to rewrite luajit.cmake from scratch
20:38:32  <rphillips>PR'ing it
20:40:20  <rphillips>creationix: https://github.com/luvit/luv/pull/216/files
20:40:28  <rphillips>give that a try. I kid you not
20:40:54  <creationix>lol
20:42:26  <rphillips>i'll PR it on luajit-rocks as well
20:46:03  * travis-cijoined
20:46:04  <travis-ci>luvit/luv#190 (support_32bit_luajit_2_1 - 4cb335d : Ryan Phillips): The build passed.
20:46:04  <travis-ci>Change view : https://github.com/luvit/luv/commit/4cb335d49e25
20:46:04  <travis-ci>Build details : https://travis-ci.org/luvit/luv/builds/107863807
20:46:04  * travis-cipart
20:51:18  <rphillips>going to merge
20:51:41  * Something12joined
21:40:51  <rphillips>https://github.com/luvit/luvi/pull/139
21:49:57  * travis-cijoined
21:49:57  <travis-ci>luvit/luvi#855 (fixes/luvi_1_8_0_4 - e7c39ce : Ryan Phillips): The build passed.
21:49:57  <travis-ci>Change view : https://github.com/luvit/luvi/commit/e7c39ced40bd
21:49:57  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107877797
21:49:57  * travis-cipart
22:18:20  <daurnimator>creationix: why do you need octal?
22:56:36  * travis-cijoined
22:56:37  <travis-ci>luvit/luvi#857 (master - 572b199 : Ryan Phillips): The build passed.
22:56:37  <travis-ci>Change view : https://github.com/luvit/luvi/compare/5df4345b034a...572b199b7b9c
22:56:37  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107894706
22:56:37  * travis-cipart
23:35:26  <creationix>daurnimator, file permissions of course :)
23:35:42  <creationix>mostly for implementing git in lua, the tree entry literals use octal
23:36:15  <daurnimator>creationix: there's not really any agreement on octals anyway
23:36:23  <daurnimator>leading 0 would be too big of a change
23:36:32  <daurnimator>and not that much uses 0o755
23:36:35  <creationix>I proposed 0o755
23:36:41  <daurnimator>it's unfamiliar enough to not be helpful IMO
23:36:41  <creationix>with optional capital o
23:36:51  <creationix>that's what JS recently added along with binary
23:36:55  <daurnimator>o"755" is good enough, and doesn't require changes.
23:37:11  <creationix>sure, but then you have the runtime overhead of passing a string to a function
23:37:13  <creationix>and don't get constant folding
23:37:16  * travis-cijoined
23:37:17  <travis-ci>luvit/luvi#858 (master - fa81852 : Ryan Phillips): The build passed.
23:37:17  <travis-ci>Change view : https://github.com/luvit/luvi/compare/572b199b7b9c...fa8185289619
23:37:17  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107902660
23:37:17  * travis-cipart
23:37:53  * travis-cijoined
23:37:54  <travis-ci>luvit/luvi#859 (v2.6.1 - fa81852 : Ryan Phillips): The build passed.
23:37:54  <travis-ci>Change view : https://github.com/luvit/luvi/compare/v2.6.1
23:37:54  <travis-ci>Build details : https://travis-ci.org/luvit/luvi/builds/107902685
23:37:54  * travis-cipart
23:38:10  <creationix>so what I have to do is put in the decimal version with a comment showing the octal version
23:38:19  <daurnimator>creationix: how often do you have octal literals on a hot path?
23:38:46  <creationix>semi-hot path in lua-git based stuff
23:39:09  <creationix>but there is also file I/O involved, so it's probably not significant
23:45:34  <rphillips>creationix: can you try the 32 bit luvi builds?