github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
00:40 anatofuz left 00:43 anatofuz joined 00:50 anatofuz left 01:01 anatofuz joined 01:07 anatofuz left 01:12 anatofuz joined 01:23 anatofuz left 01:27 anatofuz joined 01:32 anatofuz left 01:34 lucasb left 01:49 camelia left 01:51 camelia joined 02:31 anatofuz joined 03:21 AlexDani` joined 03:25 AlexDaniel left 03:42 anatofuz left 03:43 anatofuz joined 03:48 anatofuz left 05:06 AlexDani` is now known as AlexDaniel, AlexDaniel left, AlexDaniel joined 05:09 anatofuz joined, anatofuz left, anatofuz joined, robertle left 05:48 anatofuz left 06:16 anatofuz joined 06:25 anatofuz left 06:30 anatofuz joined, anatofuz left 06:31 anatofuz joined 06:37 domidumont joined 07:13 anatofuz_ joined, anatofuz_ left 07:15 anatofuz left, anatofuz_ joined 07:24 anatofuz_ left 07:30 anatofuz joined 07:47 zakharyas joined 08:05 sena_kun joined 08:08 nebuchadnezzar left 08:13 nebuchadnezzar joined 08:16 robertle joined 08:36 anatofuz_ joined 08:39 anatofuz left 08:47 anatofuz_ left, anatofuz joined 08:59 anatofuz left 09:02 anatofuz joined 09:07 anatofuz left 09:08 robertle left 09:09 robertle joined 09:11 anatofuz joined 09:15 anatofuz left 09:17 anatofuz joined 09:43 anatofuz left 10:00 anatofuz joined 10:02 anatofuz left 10:03 anatofuz joined 10:18 anatofuz left 10:20 anatofuz joined 10:21 robertle left 10:23 anatofuz left
Guest23744 not much action here atm :) 10:24
10:25 anatofuz joined, moritz_ is now known as moritz, robertle joined 10:26 anatofuz left 10:29 anatofuz joined 10:31 anatofuz left
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
10:36 anatofuz joined
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
11:04 Geth joined 11:15 anatofuz left, zakharyas left 11:17 anatofuz joined 11:21 anatofuz left 11:30 anatofuz joined
AlexDaniel timotimo: so what's this commit about? github.com/perl6/nqp/commit/1ff309...04f0e43d80 11:31
11:33 robertle left
timotimo github.com/MoarVM/MoarVM/commit/3f...7f27158e7d 11:34
github.com/MoarVM/MoarVM/commit/20...aad78bb1db
11:35 robertle joined
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
12:05 anatofuz left 12:06 anatofuz joined
timotimo oh, also 12:09
the spesh mvmevent piece isn't yet in there with the commit in question
12:10 anatofuz left
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
12:42 robertle left 12:44 robertle joined 12:58 lucasb joined 13:02 anatofuz joined 13:18 zakharyas joined 13:19 anatofuz_ joined 13:20 anatofu__ joined 13:21 anatofu__ left 13:22 anatofuz left 13:24 anatofuz_ left
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
15:22 robertle left
lizmat and another Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2019/09/23/...else-itch/ 15:28
15:39 domidumont left 15:56 zakharyas left 16:11 robertle joined 17:03 dogbert11 joined
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
18:10 domidumont joined, domidumont left 18:11 domidumont joined 18:21 domidumont left
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
18:34 Ven`` joined
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
18:41 brrt joined
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
20:23 brrt left
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
20:48 patrickb joined
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
21:05 Ven`` left, Ven`` joined 21:06 Ven`` left
timotimo patrickb: maybe under platform/ ? 21:11
patrickb timotimo: That looks good. Thanks! 21:12
21:22 sena_kun left 21:27 MasterDuke left 21:50 anatofuz joined 22:18 patrickb left 22:25 anatofuz left 22:50 anatofuz joined 23:07 anatofuz_ joined, anatofuz left, anatofuz_ left 23:10 anatofuz joined, anatofuz left 23:44 anatofuz joined