| Geth | roast: 919a2ddde6 | (Daniel Green)++ | S02-types/native.t Add test for RT #127973 Native strings can be as large as regular strings. |
04:09 | |
| synopsebot | RT#127973 [open]: rt.perl.org/Ticket/Display.html?id=127973 [BUG] native strings not allowed to be as big as regular strings | ||
| buggable | New CPAN upload: v0.1.30.tar.gz by UGEXE cpan.metacpan.org/authors/id/U/UG/....30.tar.gz | 04:32 | |
| Zoffix | "Stage parse : 101.480" | 05:04 | |
| Zoffix weeps | |||
| "Stage parse : 74.669" | 05:52 | ||
| Zoffix unweeps :) | |||
| bartolin | . | 06:03 | |
| yoleaux | 8 Oct 2017 17:47Z <jnthn> bartolin: github.com/perl6/nqp/blob/master/s....nqp#L2895 is how it's defined on Moar, so 1 means evaluate to argument 1 (zero-based) | ||
| bartolin | jnthn: thanks! so (if i don't misunderstand) it would be ok to change the op to return $RT_INT (since we call nqp::killprocasync with an Int as the second argument). i'll take another look later today. | 06:06 | |
| Zoffix | m: use Test; my ($a, $b); $b = 'b'; :(:$a) := $b; is $a, 'b', 'can bind to single pair'; | 06:13 | |
| camelia | not ok 1 - can bind to single pair # Failed test 'can bind to single pair' # at <tmp> line 1 # expected: 'b' # got: (Mu) |
||
| Zoffix | This throws X::Cannot::Capture with my patch... It's a TODOed test. | 06:14 | |
| m: use Test; my ($a, $b); $b = 'b'; :(:$a) := \(:a($b)); is $a, 'b', 'can bind to single pair'; | 06:16 | ||
| camelia | ok 1 - can bind to single pair | ||
| Zoffix | I'm calling bullshit on it. Even if Capture would not throw and return the self, it'd throw "too many positionals" error. And it's a TODO 'cause Str.Capture returns an empty Capture, so $a ends up as Mu | 06:17 | |
| Geth | roast: 4bf399347c | (Zoffix Znet)++ | S06-other/pairs-as-lvalues.t Fix and unfudge incorrect Pair binding test The current version attempts to bind a Str variable to a Signature with one named arg in it. The test is fudged, as :$a gets a Mu value instead of $b's value, and it doesn't outright die only because Str.Capture returns an empty capture. Fix by binding a proper Capture, with $b passed as a named :a arg. |
06:22 | |
| roast: 3cbeaea94d | (Zoffix Znet)++ | S06-other/pairs-as-lvalues.t Actually remove the fudge Part of github.com/perl6/roast/commit/4bf399347c |
06:23 | ||
| Zoffix | oh damn, it fails now >_< I thought I did run it | 06:25 | |
| I suck | 06:26 | ||
| Geth | roast: 0a6daaac25 | (Zoffix Znet)++ | S06-other/pairs-as-lvalues.t Fix the bind test Part of fix described in github.com/perl6/roast/commit/4bf399347c If actual binding is expected, the $a needs to be is raw. |
06:27 | |
| roast/6.c-errata: 6c6385a162 | (Zoffix Znet)++ | S06-other/pairs-as-lvalues.t Fix and unfudge incorrect Pair binding test The current version attempts to bind a Str variable to a Signature with one named arg in it. The test is fudged, as :$a gets a Mu value instead of $b's value, and it doesn't outright die only because Str.Capture returns an empty capture. In addition, if actual bind is wanted, the :$a needs to be received raw. Fix by binding a proper Capture, with $b passed as a named :a arg. |
06:30 | ||
| [Tux] | This is Rakudo version 2017.09-350-g12fcece49 built on MoarVM version 2017.09.1-576-g64b5dc03 | 06:50 | |
| csv-ip5xs 1.247 - 1.352 | |||
| test 10.373 - 10.969 | |||
| test-t 3.199 - 3.292 | |||
| csv-parser 0.944 - 1.006 | |||
| Zoffix | Gonna add Signature.ACCEPTS: Mu $topic that does self.ACCEPTS: $topic.Capture | ||
| Gonna add Signature.ACCEPTS: Mu $topic that does (try self.ACCEPTS: $topic.Capture) || False | 06:51 | ||
| Geth | rakudo/nom: 4ba12ff17c | (Zoffix Znet)++ | src/core/Exception.pm Implement X::Cannot::Capture To be thrown from .Capture methods of certain types per irclog.perlgeek.de/perl6/2017-03-07#i_14221839 |
06:52 | |
| [Tux] | This is Rakudo version 2017.09-350-g12fcece49 built on MoarVM version 2017.09.1-576-g64b5dc03 | ||
| csv-ip5xs 1.244 - 1.256 | |||
| test 9.988 - 10.312 | |||
| test-t 3.161 - 3.173 | |||
| csv-parser 0.861 - 0.878 | |||
| (second double run) | |||
| looking good | 06:53 | ||
| Geth | rakudo/nom: bad9fefdd7 | (Zoffix Znet)++ | 8 files Make .Capture of certain core types throw To avoid supersticious, performance-reducing parens; e.g. -> ($x) {} where -> $x {} was meant. Discussion: irclog.perlgeek.de/perl6/2017-03-07#i_14221839 |
||
| Zoffix | buggable: speed 100 | ||
| buggable | Zoffix, ▄▄▅▃▄▄↑▅▅▅▅↑▇↑▆▄▅▅▅▆▄▅▄▄▅▇▆▅↑↑▅▅▅▅▄▄▄▅▄▄▄▄▅▄▄▆▄▄▃▄▄▄▄▄▅▅▄▅▄▃▆↑█▆▃▂▁▂▂▁▂▂▂▂▃▅▂▃▂▃▆▂▁▁▁▂▂▂▂▂▃▂▂▂▃▃▁▃▁▁ dates: 2017-09-05–2017-10-09; range: 3.154s–4.237s; speed: 10% faster | ||
| Zoffix | \o/ | ||
| ZOFFLOP: t/spec/integration/precompiled.t | 07:03 | ||
| Geth | rakudo/nom: 1ea3297b21 | (Zoffix Znet)++ | src/core/Range.pm Block out Mu.new from Range To avoid throwing incorrect error that Range.new takes only named args |
07:04 | |
| roast: 6ac1f272f4 | (Zoffix Znet)++ | S02-types/range.t Test Range.new with wrong args Does not claim it takes only named args Rakudo fix: github.com/rakudo/rakudo/commit/1ea3297b21 |
07:07 | ||
| lizmat | Files=1227, Tests=75620, 343 wallclock secs (14.66 usr 5.17 sys + 2413.72 cusr 208.00 csys = 2641.55 CPU) | 07:41 | |
| yoleaux | 8 Oct 2017 17:43Z <jnthn> lizmat: iirc NQP uses some codes that aren't used in Rakudo, and Rakudo supported them for that reason. I think you asked me about them a while back 'cus you were considering removing them from Mu.BUILDALL. | ||
| 8 Oct 2017 17:44Z <jnthn> lizmat: If you changed the meanings of the integers and didn't align NQP too then it's possible something coulda broken. | |||
| 8 Oct 2017 17:48Z <jnthn> lizmat: auto-generating BUILD_LEAST_DERIVED would make mixins cheaper; note it's only worth doing for mixin types, but I think you have that information available to you in method compose, or via some .is_mixin method or some such. Can't remember off the top of my head, but I'm sure it's available. So I'd only generate them in that case, not for all classes where it'd be useless | |||
| jnthn | bartolin: Yes, fine to change that. Thanks for working on it. :-) | 08:14 | |
| Zoffix | m: say 'x'.match(/./).^mro | 08:25 | |
| camelia | ((Match) (Capture) (Cool) (Any) (Mu)) | ||
| Zoffix | TIL a Match is a Capture :o | ||
| Weird... | 08:37 | ||
| m: -> (Str() $a, Str() :$b) { say [$a, $b] }('abc'.match(/(.) $<b>=. ./)) | |||
| camelia | Cannot find method 'gist': no method cache and no .^find_method in block <unit> at <tmp> line 1 |
||
| Zoffix | m: -> ($a, :$b) { say [$a.Str, $b.Str] }('abc'.match(/(.) $<b>=. ./)) | ||
| camelia | [a b] | ||
| Zoffix | m: -> ($a, :$b) { say [$a.^name, $b.^name] }('abc'.match(/(.) $<b>=. ./)) | ||
| camelia | [Match Match] | ||
| Zoffix | Looks like signature's Str() coercer results in a different value | 08:38 | |
| m: use Test; is-deeply set(42, <70>).Capture, \("42" => True, "70" => True) | 08:54 | ||
| camelia | not ok 1 - # Failed test at <tmp> line 1 # expected: \("42" => Bool::True, "70" => Bool::True) # got: \("42" => Bool::True, "70" => Bool::True) |
||
| Zoffix | Expected and Got are the same... wtf | ||
| m: dd (\("42" => True, "70" => True)).Capture::hash | 08:57 | ||
| camelia | Map.new(()) | ||
| Zoffix | Hm, these end up as positionals :\ | ||
| Ah, OK, it's meant to. | 09:00 | ||
| ZOFLOP: t/spec/S17-procasync/kill.t | 09:11 | ||
| ZOFVM: Files=1277, Tests=152640, 150 wallclock secs (20.83 usr 3.28 sys + 3264.45 cusr 169.26 csys = 3457.82 CPU) | |||
| Geth | roast: ca273503b6 | (Zoffix Znet)++ | S02-types/capture.t Spec .Capture on core types Discussions: irclog.perlgeek.de/perl6/2017-03-07#i_14221839 irclog.perlgeek.de/perl6-dev/2017-...i_15275349 irclog.perlgeek.de/perl6-dev/2017-...i_15274676 irclog.perlgeek.de/perl6-dev/2017-...i_14629113 |
09:18 | |
| rakudo/nom: cd5864cfc1 | (Zoffix Znet)++ | 5 files Implement .Capture on types per spec Spec: github.com/perl6/roast/commit/ca273503b6 Discussions: irclog.perlgeek.de/perl6/2017-03-07#i_14221839 irclog.perlgeek.de/perl6-dev/2017-...i_15275349 irclog.perlgeek.de/perl6-dev/2017-...i_15274676 irclog.perlgeek.de/perl6-dev/2017-...i_14629113 |
09:19 | ||
| Zoffix | So... caller's language version. A var like $/ to store it in sounds good for it? This stuff's a bit over my head, but I'd like to get 6.d moving | 09:40 | |
| Actually, I'm not 100% sure why $*PERL.version has to be wrong. | 09:44 | ||
| ZOFFLOP: t/spec/S17-promise/stress.t | 09:54 | ||
| stmuk | m: use v6.d.PREVIEW; say $*PERL.version; | 10:17 | |
| camelia | v6.d | ||
| Zoffix | Looks like t/spec/S17-channel/stress.t hung altogether now. | 10:19 | |
| Oh maybe it finished... Pressing CTRL+C didn't react right away, but when it was done it said all tests pass, with ridiculous runtime Files=1277, Tests=152645, 741 wallclock secs (21.19 usr 3.45 sys + 4085.28 cusr 179.47 csys = 4289.39 CPU) | |||
| stmuk | m: use v6.c; { use v6.d.PREVIEW }; say $*PERL.version; | 10:20 | |
| camelia | v6.d | ||
| Zoffix | That will throw. You can only specify one language version per comp unit | 10:22 | |
| But $*PERL.version still ain't working right when modules and precompilation is involved. github.com/perl6/6.d-prep/blob/mas...s-language | |||
| Geth | rakudo/nom: 27131ed8d1 | (Zoffix Znet)++ | src/core/Signature.pm Make Signature.ACCEPTS coerce args to Capture - Instead of relying to Any.ACCEPTS to always give False - We already did do this for Positional and Associative arguments |
10:24 | |
| roast: 48d5d39e2b | (Zoffix Znet)++ | S03-smartmatch/capture-signature.t Spec Signature.ACCEPTS with non-Signature/Capture args Rakudo impl: github.com/rakudo/rakudo/commit/27131ed8d1 |
10:25 | ||
| roast: 064707a320 | (Zoffix Znet)++ | S32-num/expmod.t Add .expmod with negative power tests RT#130713: rt.perl.org/Ticket/Display.html?id=130713 |
11:46 | ||
| synopsebot | RT#130713 [open]: rt.perl.org/Ticket/Display.html?id=130713 42.expmod(-1,1) hangs | ||
| Zoffix observes that writing a test while fixing a bug takes a lot less time than writing one for a fix that happened awhile back. While fixing you know what the cause it, so you can cover that cause right away. | 11:53 | ||
| buggable: tag testneeded | |||
| buggable | Zoffix, There are 55 tickets tagged with TESTNEEDED; See fail.rakudo.party/t/TESTNEEDED for details | ||
| Zoffix | I'm surprised so many tickets are left over... I was sure they'd go like hot cakes during squashathon | ||
| Geth | rakudo/nom: 56eef6967b | (Elizabeth Mattijsen)++ | 2 files Fix for RT #132249 |
12:00 | |
| synopsebot | RT#132249 [new]: rt.perl.org/Ticket/Display.html?id=132249 [LTA] use lib ‘’ spills guts (use lib ‘’) | 12:01 | |
| ggoebel | zoffix: have you added fail.rakudo.party/t/TESTNEEDED to hacktoberfest? | 12:15 | |
| jnthn | Just build HEAD and t/02-rakudo-repl.t is very faily | 13:51 | |
| *built | 13:54 | ||
| MasterDuke_ | anyone mind merging github.com/rakudo/rakudo/pull/1184 ? | 14:48 | |
| Geth | rakudo/nom: 16c2a15767 | (Daniel Green)++ | src/Perl6/Grammar.nqp Use cached line numbers for token label |
14:49 | |
| rakudo/nom: f28875f109 | (Jonathan Worthington)++ (committed using GitHub Web editor) | src/Perl6/Grammar.nqp Merge pull request #1184 from MasterDuke17/use_cached_line_numbers_for_token_label Use cached line numbers for token label |
|||
| jnthn | Looks sensible enough to me :) | ||
| MasterDuke_ | jnthn: thanks | ||
| ugexe | that test is failing regularly on windows now as well | 14:58 | |
| all windows failures started on the full-make-test merge, so maybe that is the cause of it elsewhere too | 14:59 | ||
| oh those tests probably werent run before at all | 15:00 | ||
| Geth | nqp/master: 4 commits pushed by pmurias++ | 15:15 | |
| travis-ci | NQP build failed. pmurias 'Test nqp::setdimensions return value' | 15:31 | |
| travis-ci.org/perl6/nqp/builds/285636737 github.com/perl6/nqp/compare/943f7...4207bd4fc4 | |||
| Zoffix | jnthn: "faily" means what? You always see it fail or does it flop? | 15:39 | |
| jnthn | Zoffix: Fails some of its tests, reliably. | 15:40 | |
| Seems like the same ones | |||
| Zoffix | jnthn: can you pastebin the output? | 15:41 | |
| jnthn | Zoffix: gist.github.com/jnthn/f0bab3cce7ef...b7ee4fdeed | 15:42 | |
| Zoffix | Thanks. | ||
| jnthn | I guess if you're asking, you ain't seeing them? | ||
| Zoffix | jnthn: nope. And on my box, the stuff REPL is fed with isn't part of the tested output, while on your box it is. | 15:43 | |
| jnthn | Odd | 15:44 | |
| Zoffix | jnthn: what OS is that? | ||
| jnthn | Linux. Ubuntu 16.04 | ||
| So, about as boring as possible. :) | |||
| Zoffix | I'm Bodhi Linux, based off Ubuntu 14.04 | 15:45 | |
| jnthn | fwiw, the repl seems to function correctly if I enter it | ||
| Zoffix | jnthn: do you Readline/Lineoise installed? | ||
| jnthn | Hmm | ||
| `use Readline` works | 15:46 | ||
| So seems I have that one | |||
| `use Linenoise` doesn't | |||
| Zoffix tries installing it | |||
| jnthn | Though I should check if that's what the module is called | ||
| Yeah, it is. So, just Readline | 15:47 | ||
| And I don't get the text suggesting that I install Readline | |||
| Which I guess suggests it's being used | |||
| gfldex | zoffix++ # for closing my tickets | 15:55 | |
| ugexe | multi sub is-run-repl isn't closing handles fwiw... maybe thats why windows is failing that test | 15:56 | |
| can't test atm | |||
| Geth | rakudo/nom: 1ce3a36df4 | (Zoffix Znet)++ | t/packages/Test/Helpers.pm6 Implement :$line-editor option - Sets REPL_LINE_EDITOR for duration of the test - Default to `none`, which prevents spurious failures for users who have one of the recommended line editors installed. |
16:01 | |
| rakudo/nom: 973f88dc65 | (Zoffix Znet)++ | t/packages/Test/Helpers.pm6 Document is-run-repl sets ERROR_COLOR option |
|||
| rakudo/nom: 484f987259 | (Zoffix Znet)++ | t/packages/Test/Helpers.pm6 Close pipes in is-run-repl; ugexe++ |
16:04 | ||
| Zoffix backlogs #perl6 a bit | 16:08 | ||
| jnthn | Zoffix: That looks to have helped, nice :) | ||
| Zoffix | lizmat: dam. looks like all the Cool method that take and pass a Capture are uber slower | ||
| m: .contains: 'meow' for ^10000_00; say now - INIT now | |||
| camelia | 11.0881977 | ||
| Zoffix | m: .Str.contains: 'meow' for ^10000_00; say now - INIT now | ||
| camelia | 0.6811970 | ||
| Zoffix | And there's lots of them | 16:09 | |
| lizmat | yeah, if you look at the profile | ||
| Zoffix | m: say 11/.68 | ||
| camelia | 16.176471 | ||
| lizmat | you'll see that it cannot cache the method lookup | ||
| Zoffix | :( | ||
| lizmat | Routine.find_best_dispatchee being called all the time | 16:11 | |
| taking 30% of time | |||
| which also implies that the proto for Str.contains gets hammered, which takes another 30% | 16:12 | ||
| which then selects Cool.contains, taking another 15% of time | 16:13 | ||
| to the actual workhorse, which takes 7% of time (and which somehow isn't jitted) | |||
| jnthn | Yeah, flattening things is not yet well optimized. | 16:14 | |
| I've been pondering how we might handle those a bit better down in the VM, so the benefit of both multi cache and spesh | |||
| *for the benefit | |||
| lizmat | jnthn: can it be that a multi-candidate doesn't get jitted because its proto doesn't get jitted ? | 16:15 | |
| jdv79 | yup. 600M isnt enough mem to build. OOMed | 16:16 | |
| jnthn | lizmat: No, they'd be independent, and often protos never get jitted | ||
| lizmat | ah ok | ||
| jnthn | 'cus they get bypassed entirely by a cache lookup | ||
| Zoffix | jdv79: you need ~1.2GB | ||
| jdv79 | hopefully 1.5G will be enough | ||
| jnthn | So the call is directly into the candidate | ||
| jdv79 | ok. itll just take all day since 1G of that is on a SD card | 16:17 | |
| jnthn | But the cache doesn't handle flattening | ||
| lizmat | another weird thing: in that example I also see "add_to_cache" being called 1M times | 16:18 | |
| you would think that once is enough ? | |||
| jnthn | It'd be better named try_add_to_cache :) | 16:19 | |
| If there's flattening then it won't actually add it | |||
| fwiw, I'm doing a bit of work on hyper/race this week. Locally for now, will hopefully land something in the repo by the end of the week | 16:21 | ||
| MasterDuke_ | very cool | 16:22 | |
| gfldex | jnthn: that is very welcome because I already got a blog post title and the title tends to be the hard part. | 16:25 | |
| jnthn | :P | ||
| Trick will be not getting distracted debugging something else | 16:26 | ||
| :) | |||
| Anyways, home/dinner time now, I think :) | |||
| Zoffix | At least some of the failures of t/02-rakudo/repl.t on windows is 'cause $*OUT.tell always returns 0 | 16:34 | |
| buggable | New CPAN upload: App-Mi6-0.0.6.tar.gz by SKAJI cpan.metacpan.org/authors/id/S/SK/...0.6.tar.gz | 17:02 | |
| jdv79 | Stage parse : 3731.644 :) | 17:23 | |
| Zoffix | :) | 17:26 | |
| MasterDuke_ | that handily beats my ~2400 with a debugging, unoptimized moarvm in valgrind | 17:29 | |
| timotimo | i have received a rather confusing comment on the weekly blog today o_O | 17:30 | |
| in the moderation queue | 17:31 | ||
| MasterDuke | oh? | 17:32 | |
| Zoffix | What was it? | 17:33 | |
| timotimo | about keys for hashes | 17:34 | |
| i'll let it go up on the site, though i can't help but feel it's the wrong place for the question? | |||
| i think it went onto the weekly blog's comment section because it linked to the thread "Tip: assign a value to a hash using a variable as a key" on the users list | |||
| (because the comment explicitly references it) | |||
| MasterDuke | forward to the mailing list? | ||
| Zoffix | Well... I was wondering how long it'd take: "No more Texas operators? Political correctness infected Perl 6, too? Sad!" | 17:35 | |
| timotimo | i can't take any person seriously who ends their comments in "Sad!" | ||
| gfldex | i'm tempted to feed the troll :-> | 17:38 | |
| Zoffix wouldn't | 17:39 | ||
| timotimo | i'd prefer no troll feeding on the weekly blog comments section | 17:40 | |
| Zoffix | Let them feel sad that no one read their comment :) | ||
| buggable | New CPAN upload: App-Mi6-0.0.7.tar.gz by SKAJI cpan.metacpan.org/authors/id/S/SK/...0.7.tar.gz | 17:42 | |
| gfldex | timotimo: you are feeding that troll by proving him with a platform. Esp. as he got nothing else to say. | 17:43 | |
| timotimo | well, it was let through and now it'd seem rather petty to delete it after-the-fact | 17:44 | |
| Zoffix | toss it :) | ||
| [Coke] | it's your site, you can be a little petty | 17:45 | |
| gfldex | being able to change your mind when presented with more arguments is no sign of weekness | ||
| m: sub strong(**@arguments){ say ‚I'm Strong!‘ }; strong(|(1..∞)); | 17:46 | ||
| ok, I was wrong :D | 17:47 | ||
| camelia | (timeout) | ||
| Zoffix | m: sub strong(**@arguments){ say ‚I'm Strong!‘ }; strong(1..∞); | 17:48 | |
| camelia | I'm Strong! | ||
| gfldex | that's just one argument, even though it is quite a big one | ||
| m: sub strong(*@arguments){ say ‚I'm Strong!‘ }; strong(|(1..∞)); | 17:51 | ||
| that leaves the question if a slurpy should explode a Slip | |||
| camelia | (timeout) | ||
| timotimo | [Coke]: fwiw, i'd say it's liz' site by now :) | 17:52 | |
| Zoffix | It likely hangs before it gets to any slurpies | ||
| m: (1..∞).FLATTENABLE_LIST | |||
| camelia | (timeout) | ||
| Geth | nqp: 0b6fa59726 | pmurias++ | src/vm/js/nqp-runtime/reprs.js [js] Treat a non primitive type to multidim arrays as an object |
18:51 | |
| nqp: 01429c3c15 | pmurias++ | src/vm/js/nqp-runtime/reprs.js [js] Make nqp::atposnd return null when there is a missing element |
|||
| nqp: ce60c2bf38 | pmurias++ | t/nqp/102-multidim.t Fix test count |
18:52 | ||
| nqp: faa67169a3 | pmurias++ | t/nqp/094-clone.t More multidim tests |
|||
| AlexDaniel_ | .tell jnthn you told me to ping you later, so here it is. Plz give Zoffix or me more privileges for Rakudo and MoarVM repos so that we can manage webhooks. Alternatively, go and remove webhooks to 94.23.219.181 | 18:55 | |
| yoleaux | AlexDaniel_: I'll pass your message to jnthn. | ||
| jnthn | Ah, something my tired brane might actually usefull be able to do tonight... | 18:56 | |
| yoleaux | 18:55Z <AlexDaniel_> jnthn: you told me to ping you later, so here it is. Plz give Zoffix or me more privileges for Rakudo and MoarVM repos so that we can manage webhooks. Alternatively, go and remove webhooks to 94.23.219.181 | ||
| travis-ci | NQP build failed. pmurias '[js] Make nqp::atposnd return null when there is a missing element' | 19:05 | |
| travis-ci.org/perl6/nqp/builds/285733487 github.com/perl6/nqp/compare/6b420...429c3c1566 | |||
| jnthn | AlexDaniel_: I've tried it for the MoarVM repo (so you and zoffix should have invites for that); let me know if you can get to the webhooks page | 19:06 | |
| (Also, I removed the one you meneionted since I was there anyway) | |||
| Provided I got it right I'll do the same for the Rakudo one :) | |||
| Zoffix | Joined in, but no webhook access | ||
| AlexDaniel_ | same | 19:07 | |
| jnthn | d'oh | ||
| try again now? | |||
| AlexDaniel_ | that works | 19:08 | |
| jnthn | \o/ | ||
| And should have it for Rakudo too :) | 19:09 | ||
| Zoffix | jnthn++ | ||
| jnthn: that also gave me commit bit to MoarVM. What's the policy for merging/committing stuff? | |||
| jnthn | Yeah, you'll both have quite a few extra perms as a result of this... | 19:10 | |
| For MoarVM - typo fixes and other stuff nobody could argue with, just push/merge | |||
| AlexDaniel_ | mhm github allows more refined control of privileges for access tokens, but that's per user | 19:11 | |
| jnthn | For more serious changes, I'd start out with PR-ing things. It's C, with a GC running around, and a missed MVMROOT for MVM_ASSIGN_REF can means hours of frustrating bug hunt later... | 19:13 | |
| s/for/or/ | |||
| Zoffix | OK. I'll then merge github.com/MoarVM/MoarVM/pull/723 once Travis/Appveyor clear it | ||
| 92AAB6NK0 | NQP build failed. pmurias 'More multidim tests' | 19:14 | |
| travis-ci.org/perl6/nqp/builds/285733646 github.com/perl6/nqp/compare/01429...a67169a349 | |||
| AlexDaniel_ | uh… that was a very interesting nickname | ||
| jnthn | Zoffix: Yeah, that looks fine | 19:17 | |
| jnthn bbiab | 19:20 | ||
| Geth | roast: 63950b4b52 | (Zoffix Znet)++ | S32-io/tell.t Test .tell on TTY handles gives faked out values RT#132254: rt.perl.org/Ticket/Display.html?id=132254 MoarVM fix: github.com/MoarVM/MoarVM/pull/723 |
19:25 | |
| synopsebot | RT#132254 [new]: rt.perl.org/Ticket/Display.html?id=132254 [WIN32] $*OUT.tell always gives 0 | ||
| AlexDaniel_ | unicodable6: hug | 20:46 | |
| unicodable6 | AlexDaniel_, U+0F19 TIBETAN ASTROLOGICAL SIGN SDONG TSHUGS [Mn] ( ༙) | ||
| AlexDaniel_, U+D6C5 Hangul Syllable HUG [Lo] (훅) | |||
| AlexDaniel_, 5 characters in total: gist.github.com/e3393fdfded96991d9...b9e697d4b4 | |||
| lizmat | and another Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2017/10/09/...r-empathy/ | 20:48 | |
| Zoffix | woooo | 20:49 | |
| AlexDaniel_ | lizmat: have you seen irclog.perlgeek.de/perl6/2017-10-08#i_15275712 ? | 20:52 | |
| Zoffix | lizmat++ good weekly | 20:54 | |
| bartolin | yes, very much lizmat++ | 20:55 | |
| AlexDaniel_ | cool weekly. lizmat++ | 20:57 | |
| lizmat: “More than 250 commits” did you mean 150? | 20:58 | ||
| also, maybe it's worth not mentioning it given that Zoffix++ doubled that number unintentionally | 20:59 | ||
| teatime | haha | ||
| Zoffix | (if you're going in to mend the weekly, there's an unclosed paren after `(such as @a[3..*]` ) | 21:00 | |
| lizmat | AlexDaniel_ : oops, my math skills need some tuning :-) | 21:01 | |
| AlexDaniel_ Zoffix fixed and fixed :-) | 21:02 | ||
| AlexDaniel_ | lizmat: the actual number is 110 | 21:03 | |
| so over 100 is fair to say I guess :) | 21:04 | ||
| lizmat | well, I just did a git describe on the last commit of last Monday, and the one now | ||
| AlexDaniel_ | lizmat: (it's because of commits like this: github.com/rakudo/rakudo/commit/1e...f763759972 ) | ||
| lizmat | and the difference is more than 150 | ||
| AlexDaniel_ | yeah but 40 commits show up twice… once normal and once rebased :S | 21:05 | |
| lizmat | AlexDaniel_: lalala I can't hear you :) | ||
| AlexDaniel_ | :D | ||
| lizmat | (fixed :-) | 21:06 | |
| lizmat steps away from the keyboard for a bit | |||
| bartolin | hmm, we have SIGHUP as the default for method kill: github.com/rakudo/rakudo/blob/484f...nc.pm#L409 | ||
| wouldn't it make more sense to use SIGTERM there? | 21:07 | ||
| AlexDaniel_ | bartolin: have you tried sending SIGTERM to a process group? | 21:08 | |
| bartolin | AlexDaniel_: no, i didn't. i didn't think about a process group until now :-) | 21:09 | |
| AlexDaniel_ | bartolin: what happens sometimes is that the PID you're sending it too exits, but everything else stays alive | ||
| so in theory we'd want to have something like, errr, -SIGTERM | 21:10 | ||
| or better some named argument to do the same | |||
| bartolin: but so far I don't even think that we have a ticket for that :S | 21:11 | ||
| bartolin is trying to implement nqp::killprocasync for the jvm backend and the provided way to kill a started process there is destroy (what calls SIGTERM under the hood). docs.oracle.com/javase/8/docs/api/...#destroy-- | |||
| i googled a bit (no experiments, yet), but didn't see specific advices against sending SIGTERM to a process group. however, SIGHUP looks a bit surprising to me | 21:23 | ||
| AlexDaniel_ | it's not that it's bad or anything, it's just that in some cases it won't kill much | 21:24 | |
| bartolin | yes, and also both the command and the shell 'kill' builtin default to SIGTERM afaik | 21:27 | |
| at least there are not tests in roast wrt SIGHUP | 21:28 | ||
| ugexe | i wonder if its related to windows not generating sigterm | 22:13 | |
| teatime | does windows do signals at all | 22:15 | |
| ugexe | docs.libuv.org/en/v1.x/signal.html | ||
| the first part is "here is how windows is handled" | 22:16 | ||
| jnthn | lizmat++ # weekly :) | 22:25 | |
| Geth | nqp: a65dbe0c77 | (Zoffix Znet)++ | tools/build/MOAR_REVISION Bump MoarVM |
22:59 | |
| ¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...0-g631b3bf | |||
| rakudo/nom: e11f4ea266 | (Zoffix Znet)++ | tools/build/NQP_REVISION Bump NQP |
|||
| ¦ rakudo/nom: version bump brought these changes: github.com/perl6/nqp/compare/2017....5-ga65dbe0 | |||
| Zoffix | ZOFVM: Files=1277, Tests=152647, 150 wallclock secs (21.11 usr 3.34 sys + 3248.13 cusr 169.64 csys = 3442.22 CPU) | 23:11 | |
| travis-ci | NQP build failed. Zoffix Znet 'Bump MoarVM' | 23:14 | |
| travis-ci.org/perl6/nqp/builds/285823905 github.com/perl6/nqp/compare/faa67...5dbe0c7775 | |||
| timotimo | jnthn: i think cro-http would be a good candidate to get a HTTP tag in its meta info | 23:52 |