github.com/moarvm/moarvm | IRC logs at colabti.org/irclogger/irclogger_logs/moarvm
Set by AlexDaniel on 12 June 2018.
00:10 Altreus left 00:11 Altreus joined 00:33 leont left 01:18 konvertex left 04:22 leont joined 06:03 leont left
Geth_ MoarVM: 396c90039a | (Tim Smith)++ | src/core/nativecall.c
nativecall: Don't obliterate lib_name in exception text
 06:45
MoarVM: 775746944e | niner++ (committed using GitHub Web editor) | src/core/nativecall.c
Merge pull request #1304 from softmoth/adhoc-libname

nativecall: Don't obliterate lib_name in exception text
nine If there's ever a brown-paper-bag-bug it must be this ^^^ :D 06:46
nwc10 "oops" 07:03
good *, #moarvm
07:57 patrickb joined 08:56 sena_kun joined
Geth_ MoarVM: ee8f3af703 | (Patrick Böker)++ | src/io/procops.c
Always handle proc exec arguments verbatim on Windows

Together with a change in rakudo this fixes Raku/problem-solving#20.		 09:23
MoarVM: f6d04a45b9 | (Patrick Böker)++ (committed using GitHub Web editor) | src/io/procops.c
Merge pull request #1265 from patrickbkr/problem-solving20

Always handle proc exec arguments verbatim on Windows
jnthn morning o/ 09:26
sena_kun o/
nwc10 \o 09:28
MasterDuke anyone mind taking a quick look at github.com/MoarVM/MoarVM/pull/1303 ? i have another small pr in the works, but the two would conflict 09:29
patrickb jnthn: There were no blockers for the PRs except for the problem-solving issue not being resolved. I just rebased and merged the bunch. :-) 09:34
jnthn Excellent, thanks.
Geth_ MoarVM: 3fe5848762 | (Daniel Green)++ | src/6model/serialization.c
Use correct encoding for deserializing strings

A string could be a filename, which are the classic example of something that could have non-ASCII and/or non-UTF8 bytes, so use utf8-c8 to encode them.
Also, every other use of an SC's body->description is via MVM_string_utf8_encode_C_string, make that change also.		 09:35
MoarVM: 6f42c67f07 | (Jonathan Worthington)++ (committed using GitHub Web editor) | src/6model/serialization.c
Merge pull request #1303 from MasterDuke17/fix_possible_wrong_encoding_of_filenames_when_deserializing

Use correct encoding for deserializing strings
jnthn A RakuAST day today rather than dispatcher stuff. 09:36
MasterDuke thanks 09:37
jnthn patrickb: Any ideas on why all PRs to MoarVM seem to report a bunch of CI failures? 09:39
They look like repository clone failures or some such
patrickb jnthn: I think it's unable to retrieve the correct repository URL. The logs show "***" for the URL. It's the next thing on my list. :-) 09:40
MasterDuke patrickb: there's also some weirdness about a commit hash it's trying to checkout that doesn't exist 09:41
patrickb MasterDuke: That's the link you sent me, right?
jnthn patrickb: Thanks
MasterDuke yep
09:41 leont joined
Geth_ MoarVM: MasterDuke17++ created pull request #1305:
Simplify some MVM_string_ascii_encode calls		 09:57
10:02 konvertex joined 10:10 Altai-man_ joined 10:12 sena_kun left 12:10 sena_kun joined 12:12 Altai-man_ left 12:25 AlexDaniel left 12:32 Kaeipi left 12:34 Kaiepi joined 13:04 AlexDaniel joined, AlexDaniel left, AlexDaniel joined 14:10 Altai-man_ joined 14:12 leont left, sena_kun left 14:17 leont joined 14:41 zakharyas joined 15:35 Altai-man_ left 15:36 sena_kun joined
Geth_ MoarVM: 1bad1eff60 | (Daniel Green)++ | 3 files
Simplify some MVM_string_ascii_encode calls

MVM_string_ascii_encode_any just calls MVM_string_ascii_encode with NULL and 0 for the last two arguments, so use it.		 15:46
MoarVM: a8b0861559 | (Jonathan Worthington)++ (committed using GitHub Web editor) | 3 files
Merge pull request #1305 from MasterDuke17/simplify_some_MVM_string_ascii_encode_calls

Simplify some MVM_string_ascii_encode calls
15:54 konvertex left 15:59 konvertex joined 16:10 Altai-man_ joined 16:12 sena_kun left 17:47 zakharyas left 18:11 sena_kun joined 18:13 Altai-man_ left 19:09 zakharyas joined 19:24 MasterDuke left
patrickb samcv: I'm currently trying to replicate the travis CI config in azure CI. There is a LLVM coverage configuration that generates coverage reports on moarvm.github.io/coverage/ 19:40
samcv: That's looks broken since 03-07-2018. 19:42
samcv: I'd appreciate help with migrating and fixing this job. For now I'll copy the build configuration but won't touch the upload of the results. 19:44
Currently the travis configuration only builds and tests NQP, not rakudo. Do we want to also run rakudo tests? 20:08
Pro: More testing Con: Failures as soon as Rakudo master is broke. 20:09
20:10 Altai-man_ joined 20:12 sena_kun left
Geth_ MoarVM: patrickbkr++ created pull request #1306:
More azure builds		 20:19
20:29 Kaiepi left
lizmat patrickb: the more *real* failures, the better :-) 20:34
patrickb lizmat: That's an argument for both sides. Testing raku will give more failures, both real (if MoarVM breaks rakudo) and false (if rakudo is broke). 20:43
lizmat ah, in that sense... 20:45
hmmm... I guess we don't want testing rakudo then 20:46
as breaking rakudo is much more likely than MoarVM
patrickb But people did request spectesting. Hm... I'll think about this some more. 21:02
Geth_ MoarVM/master: 7 commits pushed by (Patrick Böker)++ 21:03
patrickb ^ That's a merge of several AzureCI config changes. We still have to decide what we do about rakudo / spectest builds in MoarVM. 21:04
lizmat: Thanks for your feedback. I'll ponder a little.
patrickb is off to bed. 21:05
lizmat patrickb++
patrickb Night everyone. o/
lizmat night!
21:05 patrickb left
jnthn tbh, I'd hope that breaking Rakudo *spectests* is at least relatively rare compared to the more general case of "something regressed a module" 21:12
I mean, folks are mostly good about running spectest before a commit
lizmat there's that... :-) 21:15
21:26 zakharyas left 22:11 sena_kun joined 22:12 Altai-man_ left 22:26 leont left 22:57 MasterDuke joined

Powered by Cro and the Raku® Programming Language.

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

Thank you!