github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
Guest23744 not much action here atm :) 10:24
Guest23744 jnthn: how's your throat, any improvements? 10:32
jnthn Yeah, I can talk again :)
Guest23744 yay :)
anyone interested in merging a trivial PR? github.com/MoarVM/MoarVM/pull/1164 10:34
jnthn Hm, I'm sure that should be declared `static` too 10:37
Geth_ MoarVM: 40fcdb7e88 | (Jan-Olof Hendig)++ | src/spesh/inline.c
Fix incorrect return type in get_effective_size

Probably a typo. The code makes certain the the return value can't be negative.
MoarVM: 92dd055be2 | (Jonathan Worthington)++ (committed using GitHub Web editor) | src/spesh/inline.c
Merge pull request #1164 from dogbert17/fix-returntype

Fix incorrect return type in get_effective_size
Guest23744 jnthn: thx, do you want a new PR with static added?
jnthn Yeah, it seems there's a few things in inline.c that need it 10:39
Guest23744 aha
nwc10 good *, jnthn's throat!
Guest23744 oh hello nwc10
jnthn Basically, anything without an MVM_ prefix wants it
Guest23744 cool, I can probably fix that unless someone beats me to it 10:40
I'm also thinking of creating a PR for M#1172. That's a copy pasto if I ever saw one 10:41
synopsebot M#1172 [open]: github.com/MoarVM/MoarVM/issues/1172 possible uid/gid confusion
Guest23744 also something needs to be rooted round github.com/MoarVM/MoarVM/blob/mast...ps.c#L496. I have seen the mutex become zero after the call to 'close_stdin' which I believe can allocate 10:46
*around
AlexDaniel timotimo: so what's this commit about? github.com/perl6/nqp/commit/1ff309...04f0e43d80 11:31
timotimo github.com/MoarVM/MoarVM/commit/3f...7f27158e7d 11:34
github.com/MoarVM/MoarVM/commit/20...aad78bb1db
AlexDaniel timotimo: ok and how could it be causing this? github.com/rakudo/rakudo/issues/31...-534059341 11:44
I mean… looking at the commits I can totally see how it could… 11:45
but can you look at it?
timotimo oh shit, i was pinged and didn't notice 11:48
frickin github notifications being absolutely useless
the vmevent stuff is supposed to not have any effect when it's not turned on 11:52
timotimo oh, also 12:09
the spesh mvmevent piece isn't yet in there with the commit in question
timotimo i wonder if is_coordinator is simply wrong 12:11
it does say that only one thread would have GCWhatToDo_All set 12:13
Geth_ MoarVM/deadlock-fix-attempt: c5a600799b | (Timo Paulssen)++ | 2 files
remove most event subscription stuff

to test a hypothesis why an installation deadlocks		 14:09
lizmat and another Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2019/09/23/...else-itch/ 15:28
Geth_ MoarVM: dogbert17++ created pull request #1180:
Fix for M#1172		 17:18
synopsebot M#1172 [open]: github.com/MoarVM/MoarVM/issues/1172 possible uid/gid confusion
Geth_ MoarVM: dogbert17++ created pull request #1181:
Add a few missing static keywords before a few functions		 17:38
MoarVM: 6926a492ee | (Jan-Olof Hendig)++ | src/io/fileops.c
Fix for M#1172

Probably a copy paste error. The result of getegid() should of course be compared to the statbuf.gid field and not its uid field. The fix is spectest clean.
  robertle++ for finding this bug		 17:43
synopsebot M#1172 [open]: github.com/MoarVM/MoarVM/issues/1172 possible uid/gid confusion
MoarVM: fc78a13b6f | (Jonathan Worthington)++ (committed using GitHub Web editor) | src/io/fileops.c
Merge pull request #1180 from dogbert17/fix-m-1172

Fix for M#1172
Kaiepi now that the release is out, can someone please review github.com/MoarVM/MoarVM/pull/1166 ? it blocks the rest of the work for the grant 18:32
lizmat Kaiepi: there's a release ??? 18:36
Kaiepi releasable6, status 18:37
releasable6 Kaiepi, Next release in ≈26 days and ≈0 hours. R6 is down. At least 5 blockers. Unknown changelog format
Kaiepi, Details: gist.github.com/8f070a682597c77289...69f86b6c79
Kaiepi oh, didn't notice the blockers
Kaiepi when would be a good time for someone to review it? 18:42
brrt review what? 18:54
#1166? 18:55
Kaiepi yeah 19:06
brrt i'll have a look :-) 19:17
Kaiepi sweet, thanks 19:37
brrt Kaiepi: it's a bit large and I don't know the context very well, but it looks sane enough 20:23
Kaiepi yeah, i got kinda carried away with the changes towards the end 20:26
i'll try to make pullreqs have less changes made all at once so they're easier to review
patrickb o/ 20:48
tellable6 2019-09-23T02:46:19Z #perl6-dev <vrurg> patrickb Could you, pls, test R#3194? It's another big revamp, for NQP this time. It coincided with your NQP_HOME changes and caused collisions which I had to resolve manually and wanna make sure I missed nothing.
synopsebot R#3194 [closed]: github.com/rakudo/rakudo/pull/3194 [WIP] Further improvements to the build process
patrickb I'd like to write three functions to wrap open/_wopen, fopen/_wfopen and stat/_wstat and use them throughout the code. Idea is to use the versions that do proper encoding on Windows. Where would I put such wrappers? 20:51
src/io/io.c?
When do I need to use MVMROOT? Those functions will use MVM_malloc and MVM_free to reserve a local variable of appropriate size to hold an encoded path. 20:53
timotimo patrickb: maybe under platform/ ? 21:11
patrickb timotimo: That looks good. Thanks! 21:12

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!