Perl 6 - the future is here, just unevenly distributed

IRC log for #inline, 2014-12-10

| Channels | #inline index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:01 GitHub56 [inline-c-pm] bulk88 comment on issue #31: I \*\*\*\*ing swear I debugged it couple months ago. That is how I knew it was the ```.lock`` file. I dont know why I can't find any trace of my research on google, unless I X-ed it all out and never saved it and forgot about it like a fool. So chcp is hanged in kernel mode in ExitProcess. It already unregistered from the Win32 layer so that is why it doesn't appear in the vast majority of process manager
00:23 bulk88 for mohawk etc http://pastebin.com/JRtXaJFT
00:23 GitHub10 [inline-c-pm] mohawk2 comment on issue #31: The lock file is in order to implement what is being tested by test 26: deconflicting access to the DID directory (eg `_Inline`). It's happening in Inline and also in Inline::C. http://git.io/chbQFg
00:26 mohawk bulk88, that's NT kernel code?
00:42 bulk88 yes
00:45 bulk88 im not sure if I will be able to find out what owns the lock, KEVENT struct has no concept of owner PID unlike user mode CRITICAL_SECTION lock which includes owner PID for programmer convinence
00:46 bulk88 owner found!
00:47 bulk88 not sure if its the owner, but another blocked on handle, and its in perl.exe
00:56 mohawk bulk88, there are two pseudo-forks, it surely is one of those?
01:00 willthechill joined #inline
01:07 GitHub91 [inline-c-pm] bulk88 comment on issue #31: While looking for what could have locked the .lock file, I noticed something in the test/26fork.t process.... http://git.io/JFA2zQ
01:53 bulk88 http://www.agent31.eu/2007/01/perl-windows-and-file-locking.html
03:11 bulk88 bug solved I think
04:25 willthechill joined #inline
05:23 GitHub116 [inline-c-pm] bulk88 comment on issue #31: The bug isolated into a test case... http://git.io/zZedQQ
05:50 davido_lt joined #inline
06:47 bulk88 gonna write my own patch for 26forkt. i got all the tests in there 2 pass already even the todos
06:48 davido_lt bulk88++
06:56 mohawk bulk88++
06:57 mohawk ingy, are you content for me to merge bulk88's PR and release?
06:58 mohawk bulk88, could you possibly redo with cuddled else?
07:13 ingy with or without cuddled else?
07:20 GitHub161 [inline-c-pm] ingydotnet created PR/31 (+1 new commit): http://git.io/LsQ6Lw
07:20 GitHub161 inline-c-pm/PR/31 6bd908a Alexandr Ciornii: skip tests that hangs on Windows XP
07:21 ingy mohawk: if you mean that ^^, yes just rebase that to master and release it
07:21 GitHub112 [inline-c-pm] sisyphus comment on issue #31:
07:21 ingy mohawk: please don't use GitHub
07:21 ingy mohawk: please don't use GitHub's merge button
07:24 ingy basically I don't want any merge commits in the master history if possible
07:25 ingy PR/16 was merged, but that's the only one
07:27 ingy inline-pm history is a little messier but flat recently
07:31 bulk88 the commit on my github kis broken
07:32 ingy bulk88: did you submit a PR?
07:32 mohawk ingy, i'll hang fire for now
07:32 ingy what does that mean, mohawk
07:33 bulk88 no i didnt do a pr i almlost did but stopped b4 i lost internet
07:33 bulk88 on train now
07:33 mohawk hang fire = do nothing then go off unexpectedly ;-)
07:33 ingy bulk88: ok
07:34 ingy I'll review in the morning
07:35 ingy or whenever you get to it, and after I've slept
07:43 travis-ci ingydotnet/inline-c-pm#120 (PR/31 - 6bd908a : Alexandr Ciornii): The build passed.
07:43 travis-ci Change view : https://github.com/ingydotnet/inline-c-pm/commit/6bd908a2f879
07:43 travis-ci Build details : http://travis-ci.org/ingydotnet/inline-c-pm/builds/43564668
08:04 bulk88 pushed
08:04 bulk88 from inside a "tube"
08:06 GitHub7 [inline-c-pm] bulk88 opened pull request #32: fix hang in test/26fork.t on Win32 (master...master) http://git.io/r9RIPQ
08:28 willthechill joined #inline
09:20 GitHub131 [inline-c-pm] ingydotnet created PR/32 (+2 new commits): http://git.io/eU78eA
09:20 GitHub131 inline-c-pm/PR/32 a566ec7 bulk88: fix hang in test/26fork.t on Win32...
09:20 GitHub131 inline-c-pm/PR/32 1739b57 Ingy döt Net: Formatting fixes.
09:21 ingy mohawk: there's branch PR/32 fixed up
09:23 ingy note that I get fails using `prove -lv test/27inline_maker.t` on master (and on PR/32)
09:24 ingy if you can get test/27 fixed on PR/32, then go ahead and fast-forward onto master and release it
09:35 GitHub152 [inline-c-pm] bulk88 comment on issue #32: Updated the commit, the first one had a race condition and I saw the lock was ignored (my mistake) after I tested the locking ability with warn statements.... http://git.io/1ZPYoQ
09:40 travis-ci ingydotnet/inline-c-pm#121 (PR/32 - 1739b57 : Ingy döt Net): The build passed.
09:40 travis-ci Change view : https://github.com/ingydotnet/inline-c-pm/compare/a566ec7235cd^...1739b57fb47d
09:40 travis-ci Build details : http://travis-ci.org/ingydotnet/inline-c-pm/builds/43572439
09:47 davido___ joined #inline
10:34 GitHub57 [inline-c-pm] bulk88 comment on issue #32: Repushed. Fixed "cannot unlink file for C:\sources\inline-c-pm\_Inline_26fork.7304\lib\auto\_26fo... http://git.io/aOusOQ
10:45 GitHub194 [inline-c-pm] bulk88 comment on issue #32: Repushed, included the stuff from https://github.com/ingydotnet/inline-c-pm/commit/1739b57fb47dd05ea086b4a20ad81b3a9335ea02 which was a patch to an older removed commit. http://git.io/Jd64QA
10:54 bulk88 on https://github.com/bulk88/inline-c-pm/commit/74035c139507045761c44f29ce9a7f416d9e41d4 27inline_maker.t, if I comment out the skip_all, all the tests pass
17:30 davido___ ingy I'm here.
17:49 GitHub92 [inline-module-pm] daoswald comment on issue #12: I'm not sure on this (even though you are most likely right).  PRD is only necessary when Inline::CPP is in use.  That only happens when we're in development mode.  Developers would already have installed Inline::CPP, and in so doing, would have pulled in the PRD dependency automatically.... http://git.io/d9aeNg
17:57 ingy davido___: yo
17:57 ingy heading to office
17:57 ingy bbi25~
18:02 GitHub56 [inline-c-pm] ingydotnet deleted PR/32 at 1739b57: http://git.io/XFXO1Q
18:05 davido___ hi, ok.
18:07 ingy bulk88: there are 3 skip_all. which one?
18:20 bulk88 "#plan skip_all => "Not yet ported to MS Windows" if $^O =~ /MSWin32/i;"
18:26 GitHub167 [inline-c-pm] bulk88 comment on issue #32: It also seems I fixed https://github.com/ingydotnet/inline-c-pm/issues/19 in this patch without knowing about #19 . http://git.io/dTmwqQ
18:47 ingy bulk88: https://gist.github.com/anonymous/8fb0d0484c0d71b955fa
18:48 ingy hi davido___
18:55 davido___ hi
18:56 daoswald^ingy hi
18:56 davido___ i'm in i think.
18:56 davido___ yup
18:57 davido___ prvmsg
19:00 ingy let's fix CPP to handle using
19:01 davido___ ok, let's do that.
19:01 davido___ Also, did you see my response to issue 12?
19:02 ingy url?
19:02 ingy there's a lot of repos in play
19:02 davido___ http://git.io/d9aeNg
19:03 ingy got it
19:04 sivoais just noting this here: added some PRs to <https://github.com/ingydotnet/dist-zilla-plugin-inlinemodule-pm> to fix the issues I added a couple days ago. Should update it to use the postamble method
19:04 davido___ great sivoais.
19:04 sivoais though I didn't run `zild make update`
19:05 davido___ prvmsg again... bbiab
19:16 GitHub100 [inline-c-pm] bulk88 comment on issue #32: Repushed, changed sterilization of the mutex name per MSDN docs (no thanks to Win32::IPCs docs which havent been updated in more than a decade). The starting the name with "C:\" always failed. Adding 'Inline__C_' made it not fail, most of the time atleast, except in 27inline_maker.t where it too failed. So removing all the fwd slashes fixed 27inline_maker.t and it passes for me now (I had to uncomment t
19:32 ingy bulk88: testing
19:44 davido___ back
19:49 ingy ok
19:49 davido___ :)
19:50 davido___ I saw that... "deleted .gitignore"
19:50 ingy not committing it
19:51 davido___ yeah.
19:51 ingy just annoying me :)
19:51 davido___ I know.
19:51 davido___ It just means I have to keep the repo cleaner, and avoid git add .
19:53 davido___ Need to remember to update (or add) the nolist element in Makefile.PL for IL::CPP
19:53 davido___ noindex
19:57 davido___ I like this direction.
20:01 davido___ Yikes, I should eliminate the plan.
20:03 ingy IT EFFING WORKS!
20:03 davido___ NICE!
20:03 davido___ Release! :)
20:03 ingy first time
20:03 ingy seems crazy
20:03 davido___ Yeah.  Mostly because Inline::CPP is soooooo messy.  Hard to believe any change doesn't break something.
20:04 davido___ You may merge that into master if you like. Do we need to document anything? Probably not in I::CPP.
20:05 davido___ What's your impression on what parts of Inline::CPP should be indexed, and which parts should not?
20:09 davido___ those tests were too fast
20:09 davido___ make clean
20:09 davido___ yeah, that should do it.
20:16 davido___ sec (ins adjstr)
20:21 davido___ back
20:23 davido___ had a thought....
20:24 davido___ nm.  :) later
20:30 ingy where are those warnings from?
20:31 davido___ hm.
20:41 ingy wasn't working
20:41 davido___ yeah. :)
20:42 ingy I gotta go
20:42 davido___ ok, later?
20:42 ingy I'll push to a branch
20:42 davido___ ok.
20:43 davido___ I'll be back in the evening (need to go deal with ins. soon)
20:43 davido___ prvmsg....
20:46 GitHub114 [Inline-CPP] ingydotnet created using (+1 new commit): http://git.io/519smA
20:46 GitHub114 Inline-CPP/using 1dc4fc8 Ingy döt Net: First try

| Channels | #inline index | Today | | Search | Google Search | Plain-Text | summary