🦋 Welcome to the IRC channel of the core developers of the Raku Programming Language (raku.org #rakulang). This channel is logged for the purpose of history keeping about its development | evalbot usage: 'm: say 3;' or /msg camelia m: ... | log inspection situation still under development | For MoarVM see #moarvm
Set by lizmat on 22 May 2021.
00:02 reportable6 left 00:05 reportable6 joined
ugexe so doing a diff with `RAKUDO_MODULE_DEBUG=1 raku -Ilib t/spec/S10-packages/basic.t 2> new.out` with and without the commit does show its precompiling more with my change 00:15
that uses 5 `use lib ...`
gist.github.com/ugexe/63267181f024...-change-L9 00:18
in the rakudo code before my changes that line says 1 RMD: Settings CORE.c loaded 00:20
00:59 vrurg left 01:10 kjp left 01:20 vrurg joined
ugexe i suppose its quite possible it was cheating before and its doing the right thing now but im not doing a very good job of confirming or denying that 01:22
02:20 evalable6 left, linkable6 left 02:23 linkable6 joined 03:00 Xliff_ left 04:14 frost joined 04:21 evalable6 joined 05:58 linkable6 left, quotable6 left, nativecallable6 left, bisectable6 left, releasable6 left, notable6 left, bloatable6 left, unicodable6 left, evalable6 left, statisfiable6 left, greppable6 left, shareable6 left, benchable6 left, squashable6 left, committable6 left, sourceable6 left, reportable6 left, tellable6 left, coverable6 left, releasable6 joined 05:59 notable6 joined, unicodable6 joined, coverable6 joined, nativecallable6 joined 06:00 benchable6 joined, greppable6 joined, committable6 joined, squashable6 joined 06:01 tellable6 joined, bisectable6 joined 06:04 reportable6 joined
Geth rakudo: usev6++ created pull request #4576:
[JVM] Provide clean fix for broken build
06:35
06:51 summerisle is now known as eof 06:58 quotable6 joined 07:00 linkable6 joined 07:01 sourceable6 joined 07:21 dogbert17 left 07:29 dogbert17 joined 07:59 evalable6 joined, shareable6 joined 08:15 dogbert17 left 09:15 evalable6 left, linkable6 left 09:17 linkable6 joined 09:24 dogbert17 joined
Geth rakudo: 8fc643ec95 | (Christian Bartolomäus)++ | 2 files
[JVM] Provide clean fix for broken build

There was no need to add the special cases to src/Perl6/World.nqp.
The adjustment to p6captureouters2 was inspired by 02c13fad8d. Again thanks to jnthn++ for the pointer.
09:35
10:00 bloatable6 joined, statisfiable6 joined 10:02 raydiak left
Geth rakudo/revert-curfs-fixes: 810b953e36 | (Elizabeth Mattijsen)++ | src/core.c/CompUnit/Repository/FileSystem.pm6
Revert CurFS fixes for now

Until we can figure out why this is causing the spectest to take about 2x as long (from 357 -> 616 wallclock for yours truly)
10:06
rakudo: lizmat++ created pull request #4577:
Revert CurFS fixes for now
11:17 evalable6 joined 12:03 reportable6 left 12:05 reportable6 joined 12:06 squashable6 left 12:08 squashable6 joined 12:46 squashable6 left 13:21 frost left
ugexe github.com/Raku/roast/blob/40756ae...sic.t#L278 this test is surprisingly not agonizingly slow 14:17
14:21 linkable6 left, evalable6 left, evalable6 joined 14:45 patrickb joined 14:47 squashable6 joined 15:47 evalable6 left 16:27 Kaiepi left, Kaiepi joined 17:17 Xliff joined 17:21 linkable6 joined 17:24 Kaipi joined 17:25 Kaiepi left 17:28 Kaipi left, Kaipi joined
Geth rakudo/new-disp-nativecall: 11 commits pushed by (Stefan Seifert)++
review: github.com/rakudo/rakudo/compare/7...54b6a7c207
17:36
17:48 evalable6 joined 18:03 reportable6 left 18:06 reportable6 joined
Geth rakudo/ugexe-patch-3: 7f3aaf8da1 | (Nick Logan)++ (committed using GitHub Web editor) | src/core.c/CompUnit/Repository/FileSystem.pm6
Ensure all methods use the updated comp-unit-id

In a recent change the distribution id was added to the module name when generating a comp-unit-id for `need` to allow multiple versions of the same name to be loaded. However,`need` and `resolve` need to generate identical comp-unit-ids for a given module name since `resolve` will be called by CUR::PrecompilationRepository when `need`ing a module.
18:34
rakudo: ugexe++ created pull request #4578:
Ensure all methods use the updated comp-unit-id
18:55 Kaipi left, Kaipi joined
MasterDuke ugexe: is ^^^ "just" a correctness fix, or does it also address the slowdown? 18:59
releasable6 Next release in ≈4 days and ≈23 hours. 1 blocker. Please log your changes in the ChangeLog: github.com/rakudo/rakudo/wiki/ChangeLog-Draft 19:00
ugexe both. i dunno how much faster the spectest is yet but it was faster for the one `use lib` heavy test i was using to benchmark. fwiw i would expect my changes to be slightly slower than before, but not more than slightly 19:04
19:06 sena_kun left, sena_kun joined 19:10 Merfont joined, Kaipi left 19:18 Kaipi joined, Merfont left
MasterDuke i just tested it and a spectest is now back around what it was before the curfs fix (i.e., after the new-disp merge) at ~175s 19:20
ugexe cool, i'll merge it then 19:27
MasterDuke (and the spectest also passed) 19:28
Geth rakudo: bc2cac1366 | (Nick Logan)++ (committed using GitHub Web editor) | src/core.c/CompUnit/Repository/FileSystem.pm6
Ensure all methods use the updated comp-unit-id (#4578)

In a recent change the distribution id was added to the module name when generating a comp-unit-id for `need` to allow multiple versions of the same name to be loaded. However,`need` and `resolve` need to generate identical comp-unit-ids for a given module name since `resolve` will be called by CUR::PrecompilationRepository when `need`ing a module.
19:32
19:32 linkable6 left 19:35 linkable6 joined 19:42 Merfont joined, Kaipi left 19:47 Kaipi joined 19:48 Merfont left 20:09 patrickb left 20:25 Kaipi left, Kaipi joined 20:27 Merfont joined, Kaipi left 21:19 Merfont left, Merfont joined 22:19 evalable6 left, linkable6 left 22:59 Merfont left 23:00 Merfont joined 23:20 linkable6 joined 23:21 Kaipi joined 23:22 Merfont left 23:49 Merfont joined 23:50 Kaipi left 23:58 Merfont left