[00:36] *** Kaiepi left [00:56] *** benjif left [00:59] *** ZzZombo_ joined [01:02] *** ZzZombo left [01:02] *** ZzZombo_ is now known as ZzZombo [01:50] *** softmoth left [02:14] *** softmoth joined [02:33] *** Kaiepi joined [03:27] *** ZzZombo_ joined [03:30] *** ZzZombo left [03:30] *** ZzZombo_ is now known as ZzZombo [03:32] *** Kaiepi left [05:31] *** Kaiepi joined [05:46] *** Kaiepi left [05:51] *** Kaiepi joined [06:13] *** softmoth left [06:21] *** [Tux] left [06:49] vrurg: that sounds promising! The only thing making me nervous is that the comment above that method talks about precomp and our minimal test case doesn't actually use precompilation. But the comment may just be misleading [07:39] *** ZzZombo_ joined [07:42] *** ZzZombo left [07:43] *** ZzZombo_ is now known as ZzZombo [08:11] *** ZzZombo left [08:30] ¦ problem-solving: JJ self-assigned Generic name for FALLBACK and CALL-ME https://github.com/perl6/problem-solving/issues/119 [08:32] *** ufobat_ is now known as ufobat [10:06] Files=1276, Tests=109517, 202 wallclock secs (28.33 usr 7.31 sys + 2893.61 cusr 213.88 csys = 3143.13 CPU) [10:06] 2019-10-18T08:38:24Z #moarvm lizmat I know, I pushed those accidentally, then couldn't push back, or didn't want to [10:06] 2019-10-18T09:08:20Z #moarvm lizmat I'm decently sure that I fixed *those* bugs now [10:13] ¦ nqp: 0d7250f00d | (Elizabeth Mattijsen)++ | tools/templates/MOAR_REVISION [10:13] ¦ nqp: Bump NQP to get latest fixes [10:13] ¦ nqp: review: https://github.com/perl6/nqp/commit/0d7250f00d [10:19] .tell tbrowder looks like your changes broke code at $project [10:19] lizmat, I'll pass your message to tbrowder [10:21] .tell tbrowder looks like an empty #| at the start of a line in a sequence of #| comments, breaks compilation [10:21] lizmat, I'll pass your message to tbrowder [10:25] m: #| foo [10:25] #| [10:25] #| bar [10:25] rakudo-moar 75d705f19: ( no output ) [10:25] hmmm [10:29] ¦ rakudo: 33192b5e93 | (Elizabeth Mattijsen)++ | tools/templates/NQP_REVISION [10:29] ¦ rakudo: Bump NQP to get latest MoarVM fixes [10:29] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/33192b5e93 [10:31] bisectable6: "#| foo\n#|\n#| bar".EVAL [10:31] lizmat, I'll pass your message to bisectable6 [10:31] hmmm... looks like we lost bisectable ? [10:32] .tell tbrowder https://github.com/rakudo/rakudo/issues/3242 [10:32] lizmat, I'll pass your message to tbrowder [10:44] ¦ roast: f39e92ac2f | (Elizabeth Mattijsen)++ | S26-documentation/why-leading.t [10:44] ¦ roast: Add test for R#3242 [10:44] ¦ roast: review: https://github.com/perl6/roast/commit/f39e92ac2f [10:58] m: "#| foo\n#|\n#| bar".EVAL [10:58] rakudo-moar 33192b5e9: OUTPUT: «5===SORRY!5=== Error while compiling /home/camelia/EVAL_0␤Opening bracket required for #| declarator block␤at /home/camelia/EVAL_0:2␤------> 3#|7⏏5␤» [11:19] *** MasterDuke left [11:44] *** pmurias joined [11:47] *** ufobat left [11:49] lizmat: oops! i'll see if i can fix that. good catch. [11:50] 2019-10-18T10:19:51Z #raku-dev tbrowder looks like your changes broke code at $project [11:50] 2019-10-18T10:21:29Z #raku-dev tbrowder looks like an empty #| at the start of a line in a sequence of #| comments, breaks compilation [11:50] 2019-10-18T10:32:22Z #raku-dev tbrowder https://github.com/rakudo/rakudo/issues/3242 [12:02] *** MasterDuke joined [12:08] *** pmurias left [12:09] *** pmurias joined [12:16] *** pmurias left [12:17] *** pmurias joined [12:22] *** pmurias left [12:25] .ask lizmat can you revert my merge? [12:25] tbrowder, I'll pass your message to lizmat [12:27] *** [TuxCM] joined [12:41] <[TuxCM]> Rakudo version 2019.07.1-414-g33192b5e9 - MoarVM version 2019.07.1-283-ge594fbcbd [12:41] <[TuxCM]> csv-test-xs-20 0.417 - 0.428 [12:41] <[TuxCM]> csv-ip5xs 0.746 - 0.756 [12:41] <[TuxCM]> test-t --race 0.780 - 0.781 [12:41] <[TuxCM]> test-t 1.730 - 1.748 [12:41] <[TuxCM]> csv-ip5xs-20 6.357 - 6.371 [12:42] <[TuxCM]> test 7.195 - 7.228 [12:42] <[TuxCM]> test-t-20 --race 9.157 - 9.236 [12:42] <[TuxCM]> csv-parser 20.835 - 20.851 [12:42] <[TuxCM]> test-t-20 29.152 - 29.358 [12:44] *** Kaiepi left [12:44] *** pmurias joined [12:53] *** Kaiepi joined [13:11] *** bisectable6 joined [13:13] tbrowder: how do you write your commit messages? [13:13] tbrowder: do you use git from the command line or some tool? [13:18] *** ufobat joined [13:20] ¦ rakudo: Kaiepi self-assigned signal has odd signature https://github.com/rakudo/rakudo/issues/3035 [13:20] nine: it wasn't even close. Just spotted a potential issue. [13:28] git -m at cli [13:34] nine: just for the record: what I currently surmise is that core.setting is not serialized with a unit sc. But when unit itself is deserialized it fixes this with load deps task. But if an object from that unit is repossessed – there is nobody to attach correct core to its outers. [13:42] ¦ rakudo: 9f1794cf36 | (Aleks-Daniel Jakimenko-Aleksejev)++ | 2 files [13:42] ¦ rakudo: Revert changes to #| comment blocks [13:42] ¦ rakudo: [13:42] ¦ rakudo: This reverts commits: [13:42] ¦ rakudo: * 24bd36bca3869d1b5529c36c34a3523e6b1fd81c. [13:42] ¦ rakudo: * 03fb9e6ed1179acd665d0f6f2ac4382e7b5e50d2. [13:42] ¦ rakudo: * 749831b2d20855b6d07ee8d288ec9bbf77c7d0bb. [13:42] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/9f1794cf36 [13:44] AlexDaniel: nice work, thnx! [13:44] tbrowder: don't use -m [13:45] tbrowder: https://github.com/rakudo/rakudo/blob/master/CONTRIBUTING.md#commit-messages [13:45] tbrowder: first line is a title, then an empty line, then a useful description [13:45] realistically there's no easy way to do that if you use -m [13:46] if you don't use -m, git will open a text editor where you can write the commit message properly [13:46] which text editor it opens is configurable, in my case for example that's emacs because that's what I use [13:49] what i've done in the past is use "git commit -F file.log" [13:53] with this pr i had to commit in pieces because of various mistakes. but it was to my branch and i've seen git allows an overall merge msg so i thought that was satisfactory. [13:53] -F – Yeah, well, that's a hard way to do it [13:54] as for fixing mistakes in your branch `git commit --amend` and `git rebase -i` [13:54] in the future, on my end, should i incrementally merge master into my branch during work for the pr? [13:54] no [13:55] i'm really sorry about the snafu. i usually do a full spec test before the pr but got sloppy. [13:56] what you can do is rebase your commits on top of master [13:56] but generally you don't need that [13:57] well, you do if your commits are very old… [13:59] to visualize, these are your commits: [13:59] * AlexDaniel` uploaded an image: image.png (2KB) < https://matrix.org/_matrix/media/r0/download/alexdaniel.org/HaXnAGYYPqsmDirIhZffKoJw > [13:59] that blue lines goes back several *months* [13:59] and everything else looks confusing too [14:00] functionally it doesn't matter, but if everyone starts doing that our git history will look like something from nightmares :) [14:00] aarg! [14:01] tbrowder: I think this was a nice resource, probably still is: https://learngitbranching.js.org/ [14:01] there are a few great excercises on how to rebase and juggle with commits [14:02] ok, but the -F log seems easier than the interactive--keep notes as one goes along. [14:04] tbrowder: perhaps, but I never needed that because I can always just look at the diff of what I've done [14:04] `git diff` to see what was changed and `git diff --staged` to see what I'm about to commit [14:04] and after commiting `git show` to see the last commit before I push [14:06] yeah, i use the git diff regularly, but haven't used --staged [14:06] vrurg: also my message was bogus anywhere. Your bug is all about precompilation. I just confused it with the other issue I'm looking into [14:08] I should not forget to commit the fix later anyway. [14:15] *** pmurias left [14:16] If I'm correct about core setting not been re-attached to resposessed closures, then it means same bug must happen if closure is referring to a symbol from another module. Need to check it when have time... [14:21] *** pmurias joined [14:23] kawaii: any news? :) [14:24] *** lucasb joined [14:29] ¦ nqp/truffle: 4 commits pushed by (Paweł Murias)++ [14:29] ¦ nqp/truffle: e3f3010ee5 | [truffle] Teach the DSL to how to pass in a GlobalContext argument [14:29] ¦ nqp/truffle: 694bd63fb9 | [truffle] Move file to right dir [14:29] ¦ nqp/truffle: 547d7f50bd | [truffle] Fix sixmodel bootstrap code [14:29] ¦ nqp/truffle: 594db86843 | [truffle] Finish implementing nqp::knowhow and nqp::knowhowattr [14:29] ¦ nqp/truffle: review: https://github.com/perl6/nqp/compare/caf08863f35c...594db86843b4 [14:29] For you, Alex? Always. [14:29] https://usercontent.irccloud-cdn.com/file/9Gz9Uq1j/blin-output-2019-10-18.tar.gz [14:29] AlexDaniel: ^ 🥳 [14:30] kawaii: holy moly [14:30] ¦ roast: Kaiepi++ created pull request #584: Add tests for R#3035 [14:30] ¦ roast: review: https://github.com/perl6/roast/pull/584 [14:38] *** pmurias left [14:50] *** pmurias joined [14:51] *** [TuxCM] left [14:58] *** pmurias left [15:06] *** Guest13443 left [15:29] *** ufobat left [15:46] ¦ problem-solving: JJ assigned to AlexDaniel Issue Wikipedia and Wikidata curation https://github.com/perl6/problem-solving/issues/120 [16:24] *** softmoth joined [16:33] *** [Tux] joined [17:00] kawaii: btw I looked through these and filed https://github.com/rakudo/rakudo/issues/3243 and https://github.com/rakudo/rakudo/issues/3244 [17:00] everything else is probably fine [17:40] *** Kaiepi left [17:45] *** benjif joined [18:04] *** Kaiepi joined [18:21] Finally! [18:25] The new magic incantation that fixes things is: $*W.context().sub_id_to_code_object(){nqp::getcodecuid(nqp::getattr(self, Code, '$!do'))} := -> {} if $*W; [18:41] AlexDaniel: this is about the NativeCall regression ^^^ [18:42] So what's happening is that while we replace the routine's $!do (the actual code body) when we apply the native trait, $*W.compile_in_context will replace it again with the original routine. compile_in_context is used for evaluating BEGIN blocks and also role bodies during compilation. [18:43] The line I just posted deflects that into the empty code block, so our replacement $!do stays intact. [19:14] *** ggoebel left [19:24] ¦ rakudo: e45bb3418d | (Stefan Seifert)++ | 2 files [19:24] ¦ rakudo: Fix native subs declared in BEGIN blocks and role bodies [19:24] ¦ rakudo: [19:24] ¦ rakudo: When a native sub was declared within a BEGIN block or a role body, the [19:24] ¦ rakudo: replacement code body did not stick but was replaced with the original stub [19:24] ¦ rakudo: when the containing block was compiled dynamically. [19:24] ¦ rakudo: [19:24] ¦ rakudo: Fix by removing the code object from the map of objects needing fixup. [19:24] ¦ rakudo: Fixes GH #3235 and #3243 [19:24] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/e45bb3418d [19:24] AlexDaniel: ^^^ [19:25] *** Kaiepi left [19:26] *** Kaiepi joined [19:44] see rakudo issue #2332 for my original proposal [19:51] Anyone else seeing t/08-performance/05-processkeys.t failing during make test? Can't make it fail when run individually but it reproducibly fails in make test [19:53] nine: nope, can't reproduce, even under load [19:53] 2019-10-18T12:25:02Z #raku-dev lizmat can you revert my merge? [19:53] tbrowder: looks like AlexDaniel beat me to it [19:54] yep, sorry for the snafu [19:56] if you get a chance i've proposed a change to handling leading declarator blocks over on #raku [20:00] nine: not happening for me [20:00] but i think i've seen some random fails in t/02-rakudo/repl.t [20:05] *** pmurias joined [20:16] nine, can't reproduce either [20:17] ¦ problem-solving: JJ assigned to jnthn Issue Check MoarVM out using OSS-fuzz https://github.com/perl6/problem-solving/issues/121 [20:29] *** pmurias left [20:30] ¦ roast: Altai-man++ created pull request #585: Test Parameter subclassing [20:30] ¦ roast: review: https://github.com/perl6/roast/pull/585 [20:31] *** pmurias joined [20:40] ¦ roast: Altai-man++ created pull request #586: Test runtime-created classes being attributes [20:40] ¦ roast: review: https://github.com/perl6/roast/pull/586 [20:54] ¦ rakudo: Altai-man++ created pull request #3245: Test rakudo-specific warning for shape specifier [20:54] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/3245 [21:01] *** Kaiepi left [21:01] ¦ nqp: xkr47++ created pull request #585: Update README.pod: irc #perl6 -> #raku [21:01] ¦ nqp: review: https://github.com/perl6/nqp/pull/585 [21:02] *** Kaiepi joined [21:05] ¦ roast: Altai-man++ created pull request #587: Test hyperop results in a slip [21:05] ¦ roast: review: https://github.com/perl6/roast/pull/587 [21:29] *** softmoth left [21:52] ¦ nqp/truffle: c1706c6733 | (Jonas Berlin)++ (committed using GitHub Web editor) | README.pod [21:52] ¦ nqp/truffle: Update README.pod [21:52] ¦ nqp/truffle: [21:52] ¦ nqp/truffle: #perl6 -> #raku [21:52] ¦ nqp/truffle: review: https://github.com/perl6/nqp/commit/c1706c6733 [21:52] ¦ nqp/truffle: 12a1676ba8 | (Jonathan Worthington)++ (committed using GitHub Web editor) | README.pod [21:52] ¦ nqp/truffle: Merge pull request #585 from xkr47/patch-1 [21:52] ¦ nqp/truffle: [21:52] ¦ nqp/truffle: Update README.pod: irc #perl6 -> #raku [21:52] ¦ nqp/truffle: review: https://github.com/perl6/nqp/commit/12a1676ba8 [22:07] nine: niiice! [22:07] .seen cygx [22:07] AlexDaniel, I saw cygx 2019-08-24T08:52:30Z in #perl6-dev: Well, there are still some moving pieces, but in contrast to my last attempt a couple of years ago, I have a better idea what needs to happen [22:09] vrurg: any news on https://github.com/rakudo/rakudo/issues/3190 ? [22:11] ¦ rakudo: AlexDaniel assigned to jnthn Issue Regression in Cro (fetching a json file eats RAM) https://github.com/rakudo/rakudo/issues/3223 [22:12] *** ggoebel joined [22:17] *** ggoebel left [22:18] ¦ rakudo/new-extensions: daa16b9c92 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/core.c/CompUnit/Repository/FileSystem.pm6 [22:18] ¦ rakudo/new-extensions: Add support for .rakumod extension [22:18] ¦ rakudo/new-extensions: review: https://github.com/rakudo/rakudo/commit/daa16b9c92 [22:18] ¦ rakudo: AlexDaniel++ created pull request #3246: Add support for .rakumod extension [22:18] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/3246 [22:26] *** [TuxCM] joined [22:34] *** [TuxCM] left [22:54] *** pmurias left [23:39] AlexDaniel: no news. Frankly, I forgot about it. Installing openbsd now... Again... :(