Geth | roast: e3d5196640 | (Samantha McVey)++ | 2 files Add additional tests for Unassigned characters (Cn) and <:C> Also untodo the now passing tests. These tests are for RT#124863 |
00:02 | |||||||||||||||||||||||||||||||||
synopsebot | RT#124863 [open]: rt.perl.org/Ticket/Display.html?id=124863 [UNI] Roast rakudo skip/todo test:./S05-mass/properties-general.t line:735 reason: Unions of properties of non-existent codepoints | ||||||||||||||||||||||||||||||||||
Zoffix | Missed the "wait" :) | 00:04 | |||||||||||||||||||||||||||||||||
All tests successful. | 00:05 | ||||||||||||||||||||||||||||||||||
Files=1283, Tests=152832, 153 wallclock secs (21.64 usr 3.23 sys + 3282.18 cusr 167.58 csys = 3474.63 CPU) | |||||||||||||||||||||||||||||||||||
Result: PASS | |||||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. Zoffix Znet 'Bump MoarVM' | 00:09 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296545753 github.com/perl6/nqp/compare/b8c85...f7bb81776a | |||||||||||||||||||||||||||||||||||
Zoffix | jvm stuff fails | 00:15 | |||||||||||||||||||||||||||||||||
Oh, I guess it's missing the nqp::boxI_I op? | |||||||||||||||||||||||||||||||||||
Dunno; fails in t/nqp/014-while.t and t/nqp/044-try-catch.t | |||||||||||||||||||||||||||||||||||
jnthn | sleep & | 00:18 | |||||||||||||||||||||||||||||||||
samcv | o/ jnthn | 00:27 | |||||||||||||||||||||||||||||||||
Geth | nqp: 31844c5f88 | MasterDuke17++ | 2 files Implement nqp::fromI_I for JVM |
00:49 | |||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. MasterDuke17 'Implement nqp::fromI_I for JVM' | 01:13 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296563077 github.com/perl6/nqp/compare/a3f7b...844c5f88d4 | |||||||||||||||||||||||||||||||||||
Zoffix | t/nqp/014-while.t and t/nqp/044-try-catch.t | 01:18 | |||||||||||||||||||||||||||||||||
MasterDuke | nqp-m: my $x := 0; repeat until $x >= 2 -> $another_x {if $x == 0 { nqp::say(nqp::isnull($another_x)) } else { nqp::say($another_x) }; $x := $x + 1; } | 04:15 | |||||||||||||||||||||||||||||||||
camelia | 1 0 |
||||||||||||||||||||||||||||||||||
MasterDuke | nqp-j: my $x := 0; repeat until $x >= 2 -> $another_x {if $x == 0 { nqp::say(nqp::isnull($another_x)) } else { nqp::say($another_x) }; $x := $x + 1; } | ||||||||||||||||||||||||||||||||||
well, that prints 0, 0 for nqp-j (t/nqp/014-while.t) | 04:16 | ||||||||||||||||||||||||||||||||||
Geth | nqp: 72487f7d81 | usev6++ | 2 files [jvm] Skip two newly added tests that fail on JVM |
06:31 | |||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. usev6 '[jvm] Skip two newly added tests that fail on JVM' | 06:58 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296639149 github.com/perl6/nqp/compare/31844...487f7d81fa | |||||||||||||||||||||||||||||||||||
bartolin | hmm, looks like the travis job for jvm didn't run to completion ^^ | 07:02 | |||||||||||||||||||||||||||||||||
how does one bump nqp? does it suffice to update rakudo's tools/build/NQP_REVISION with the output of 'git describe' (from the nqp repo)? | 07:07 | ||||||||||||||||||||||||||||||||||
Geth | nqp: 5f9e354ccf | (Christian BartolomƤus)++ (committed using GitHub Web editor) | t/nqp/014-while.t [jvm] Fix skip command (we skip two tests here) |
07:14 | |||||||||||||||||||||||||||||||||
travis-ci | NQP build passed. Christian BartolomƤus '[jvm] Fix skip command (we skip two tests here)' | 07:31 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296648915 github.com/perl6/nqp/compare/72487...9e354ccfcf | |||||||||||||||||||||||||||||||||||
bartolin | \o/ | 07:32 | |||||||||||||||||||||||||||||||||
[Tux] |
|
07:37 | |||||||||||||||||||||||||||||||||
pmurias | The hint when getting attributes on moarvm and jvm isn't always correct? | 08:50 | |||||||||||||||||||||||||||||||||
jnthn | pmurias: It's invalid in the presence of MI | 10:06 | |||||||||||||||||||||||||||||||||
pmurias | jnthn: the get attribute code is expected to verify if the hint is correct? | 10:09 | |||||||||||||||||||||||||||||||||
jnthn | It's expected to ignore it in the case of MI; see the MoarVM impl where that's probably clearest | 10:13 | |||||||||||||||||||||||||||||||||
Zoffix | bartolin: yeah, that's the way to do it. FWIW, here are my bumping bash aliases: gist.github.com/zoffixznet/8274ecd...72d496c604 | 10:20 | |||||||||||||||||||||||||||||||||
pmurias | jnthn: thank you, I see it now | ||||||||||||||||||||||||||||||||||
Zoffix | (the second one is for manual push for when stresstest flops) | 10:21 | |||||||||||||||||||||||||||||||||
patrickz | Hey! I'm trying to get my first doc commit ready. I'm struggling with `=begin code` blocks, which seem to indent further with every line in them (somehow even spanning multiple blocks). | 10:37 | |||||||||||||||||||||||||||||||||
This is how the code looks: gist.github.com/patzim/4b6d2c83132...9f828904bf | |||||||||||||||||||||||||||||||||||
This is the result: imgur.com/a/lXLGi | 10:38 | ||||||||||||||||||||||||||||||||||
Any ideas? | |||||||||||||||||||||||||||||||||||
bartolin | Zoffix: thanks! | 10:39 | |||||||||||||||||||||||||||||||||
Zoffix | patrickz: unsure what you mean. Got a screenshot/markup pastebin of problematic rendering? | 10:46 | |||||||||||||||||||||||||||||||||
patrickz | Zoffix: imgur.com/a/lXLGi < I added a second image with the spaces I'm worried about marked. | 10:49 | |||||||||||||||||||||||||||||||||
Zoffix | patrickz: and what's the resultant markup? | 10:52 | |||||||||||||||||||||||||||||||||
That's not the render I get and I don't see how you end up with your version with the markup I get | |||||||||||||||||||||||||||||||||||
patrickz | Ah, the images are the html/perl6.xhtml file opened directly in firefox. | 10:53 | |||||||||||||||||||||||||||||||||
Is that at least somehow representative how it will end up on the website? | 10:54 | ||||||||||||||||||||||||||||||||||
Zoffix | patrickz: then I would ignore it and just commit and see what it comes out as on the website. html/perl6.xhtml has some escaping bugs ( github.com/perl6/doc/issues/1285 ) so it wouldn't surprise me if it also generates invalid markup that causes your issue | 10:58 | |||||||||||||||||||||||||||||||||
patrickz: personally, I just push docs as is and don't worry much about rendering. It can be fixed later, if needed and this method lets me write all the docs quicker, without spending time to render them. | 10:59 | ||||||||||||||||||||||||||||||||||
Zoffix & | |||||||||||||||||||||||||||||||||||
patrickz | Will do. Thanks! | ||||||||||||||||||||||||||||||||||
lizmat | blog.toggl.com/world-created-programmer/ # made me lol | 12:13 | |||||||||||||||||||||||||||||||||
Zoffix | This rings too true to be funny :) "To create darkness, you need to hide the land, but that's OK cause creatures won't be added until later" | 12:15 | |||||||||||||||||||||||||||||||||
moritz | lizmat: me too. Thanks! | 12:51 | |||||||||||||||||||||||||||||||||
Geth | nqp/master: 5 commits pushed by pmurias++ | 13:05 | |||||||||||||||||||||||||||||||||
MasterDuke | anyone have an opinion on github.com/rakudo/rakudo/pull/1221/? if it is going to be merged, it'll be easier to do it now rather than later, since it touches so many lines it's fragile to conflicts from other changes | 13:26 | |||||||||||||||||||||||||||||||||
pmurias: i added the new nqp::fromI_I recently for moar and jvm, but i don't know javascript enough to do it for that backend | 13:29 | ||||||||||||||||||||||||||||||||||
perlpilot | MasterDuke: my opinion: +1 :) | 13:30 | |||||||||||||||||||||||||||||||||
MasterDuke | cool | 13:34 | |||||||||||||||||||||||||||||||||
Zoffix | MasterDuke: is that just for consistency? | 13:38 | |||||||||||||||||||||||||||||||||
Or will `{ * }` stop working soon and this is to avoid that breakage? I thougth I saw something along those lines mentioned somewhere | |||||||||||||||||||||||||||||||||||
MasterDuke | `{ * }` breaks if you add scope around it | 13:39 | |||||||||||||||||||||||||||||||||
Zoffix | Ah | ||||||||||||||||||||||||||||||||||
MasterDuke | i don't think it's intended to break otherwise | ||||||||||||||||||||||||||||||||||
Geth | rakudo: 0edd0cc9c1 | MasterDuke17++ | 23 files Normalize proto bodies Convert all `{ * }` to `{*}` for consistency. |
13:41 | |||||||||||||||||||||||||||||||||
rakudo: 733d50c03d | MasterDuke17++ (committed using GitHub Web editor) | 23 files Merge branch 'master' into normalize_proto_bodies |
|||||||||||||||||||||||||||||||||||
rakudo: 2cadb2bc01 | (Zoffix Znet)++ (committed using GitHub Web editor) | 23 files Merge pull request #1221 from MasterDuke17/normalize_proto_bodies Normalize proto bodies |
|||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1221 [closed]: github.com/rakudo/rakudo/pull/1221 Normalize proto bodies | ||||||||||||||||||||||||||||||||||
MasterDuke | thanks | 13:42 | |||||||||||||||||||||||||||||||||
perlpilot | MasterDuke: I even think that `proto sub foo(|) {*}` would be better written as `proto sub foo(|) { {*} }` but that might just be some foolish consistency on my part :) | 13:43 | |||||||||||||||||||||||||||||||||
(I think the latter helps remind that there can be other code in the proto) | |||||||||||||||||||||||||||||||||||
and {*} makes a nice visual pill | 13:44 | ||||||||||||||||||||||||||||||||||
MasterDuke | perlpilot: that does have its appeal...but i wonder if the extra scope would noticeably slow down parsing and/or execution? | 13:46 | |||||||||||||||||||||||||||||||||
Zoffix | A quick glance at optimized AST tells me `{ {*} }` might not be considered an "only star" proto and misses some optimizations | 13:47 | |||||||||||||||||||||||||||||||||
perlpilot | Then, yeah, it's a foolish consistency on my part :) | 13:48 | |||||||||||||||||||||||||||||||||
Zoffix | Don't see any dif in time measurements tho | 13:49 | |||||||||||||||||||||||||||||||||
(or maybe it's just drowned by noise of the bench | |||||||||||||||||||||||||||||||||||
) | |||||||||||||||||||||||||||||||||||
Ah nah | 13:50 | ||||||||||||||||||||||||||||||||||
m: proto z(|) { * }; multi z ($) { }; for ^1000_000 { $ = z 42; }; say now - INIT now | |||||||||||||||||||||||||||||||||||
camelia | 0.1827956 | ||||||||||||||||||||||||||||||||||
Zoffix | m: proto z(|) { {*} }; multi z ($) { }; for ^1000_000 { $ = z 42; }; say now - INIT now | ||||||||||||||||||||||||||||||||||
camelia | 0.5886782 | ||||||||||||||||||||||||||||||||||
Zoffix | m: say .58867/.1827 | ||||||||||||||||||||||||||||||||||
camelia | 3.222058 | ||||||||||||||||||||||||||||||||||
MasterDuke | wow, didn't expect it'd be quite that big | 13:52 | |||||||||||||||||||||||||||||||||
jnthn isn't surprised | 13:55 | ||||||||||||||||||||||||||||||||||
The first form can skip the call to the proto entirely once it's cached | 13:56 | ||||||||||||||||||||||||||||||||||
The second form can't | |||||||||||||||||||||||||||||||||||
Geth | rakudo: fccc75152d | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Quite extensive refactor of Telemetry - just store the nqp::getrusage struct instead of already dissecting - makes Telemetry.new about 15% faster - adapt all instance method logic accordingly - moves burden to Telemetry::Period creation, but that's when we need and all of the snapping is already done ... (10 more lines) |
14:00 | |||||||||||||||||||||||||||||||||
dogbert17 | ZOFFLOP: t/spec/S11-modules/nested.t | 14:28 | |||||||||||||||||||||||||||||||||
there's something fishy with this test, it failed under spectest and when I ran it standalone I got (on the first run): Use of uninitialized value $repo-id of type Any in string context. | 14:29 | ||||||||||||||||||||||||||||||||||
Methods .^name, .perl, .gist, or .say can be used to stringify it to something meaningful. | |||||||||||||||||||||||||||||||||||
in any statement_control at blib/Perl6/Grammar.moarvm line 1 | |||||||||||||||||||||||||||||||||||
Zoffix | I observed the same behaviour for it during my flops | 14:33 | |||||||||||||||||||||||||||||||||
dogbert17 | Zoffix: are you still experiencing RT #132273 ? | 14:43 | |||||||||||||||||||||||||||||||||
synopsebot | RT#132273 [new]: rt.perl.org/Ticket/Display.html?id=132273 [SEGV] while running t/spec/S17-supply/supplier-preserving.t | ||||||||||||||||||||||||||||||||||
Zoffix | haven't tried | 14:45 | |||||||||||||||||||||||||||||||||
dogbert17 | ok, just checking :) | ||||||||||||||||||||||||||||||||||
Zoffix | |6h rt.perl.org/Ticket/Display.html?id=132273 | ||||||||||||||||||||||||||||||||||
ZofBot | Zoffix, Will remind you on 2017-11-03T16:45:22.427746-04:00 about rt.perl.org/Ticket/Display.html?id=132273 | ||||||||||||||||||||||||||||||||||
dogbert17 | will try to repro but suspect that I'll fail | 14:46 | |||||||||||||||||||||||||||||||||
lizmat | m: constant method = 1; my @a; @a[method] = 42 # DIHWIDT :-( | 14:58 | |||||||||||||||||||||||||||||||||
camelia | 5===SORRY!5=== Error while compiling <tmp> Missing block at <tmp>:1 ------> 3constant method = 1; my @a; @a[method7ā5] = 42 # DIHWIDT :-( |
||||||||||||||||||||||||||||||||||
Zoffix | m: constant method = 1; my @a; @a[::method] = 42; say @a | 15:01 | |||||||||||||||||||||||||||||||||
camelia | [(Any) 42] | ||||||||||||||||||||||||||||||||||
Zoffix | m: constant constant = 1; constant @a = constant; say @a | ||||||||||||||||||||||||||||||||||
camelia | (1) | ||||||||||||||||||||||||||||||||||
Zoffix | ^_^ | 15:02 | |||||||||||||||||||||||||||||||||
ugexe | what is List.to / List.from intended to solve? | 15:43 | |||||||||||||||||||||||||||||||||
[Coke] | |1m /me wonders what this does. | 15:44 | |||||||||||||||||||||||||||||||||
ugexe | it expects either the first or last item of a list to be a specific type (Match), and one works on the first item and one works on the last | 15:45 | |||||||||||||||||||||||||||||||||
Geth | rakudo: 1c2c7d845e | (Elizabeth Mattijsen)++ | lib/Telemetry.pm6 Added formatting capabilities to Telemetry report - :@format named parameter - expects a list with lists of: column name, method name, format, legend - format *without* the preceding '%' |
15:46 | |||||||||||||||||||||||||||||||||
[Coke] has been spending way too much time at $dayjob lately, and misses hacking on p6. keep up the good work, everyone. ;) | |||||||||||||||||||||||||||||||||||
lizmat | gfldex: ^^^ | ||||||||||||||||||||||||||||||||||
afk for a bit& | |||||||||||||||||||||||||||||||||||
ugexe | m: my @a = ("a" ~~ /a/, "bb" ~~ /b/); say @a.from; say @a.to | ||||||||||||||||||||||||||||||||||
camelia | 0 1 |
||||||||||||||||||||||||||||||||||
ugexe | m: my @a = ("a" ~~ /a/, "bb" ~~ /a/); say @a.from; say @a.to | ||||||||||||||||||||||||||||||||||
camelia | 0 No such method 'to' for invocant of type 'Any'. Did you mean 'so'? in block <unit> at <tmp> line 1 |
||||||||||||||||||||||||||||||||||
ugexe | that seems to indicate its not very robust | 15:47 | |||||||||||||||||||||||||||||||||
jnthn | ugexe: m:g | 15:48 | |||||||||||||||||||||||||||||||||
ugexe | right, but even the error message implies something strange | ||||||||||||||||||||||||||||||||||
its calling .to on a List, but the error complains about calling .to on an item in the list items | |||||||||||||||||||||||||||||||||||
jnthn | m:g will only ever produce a list of Match objects | 15:49 | |||||||||||||||||||||||||||||||||
evalable6 | (exit code 1) 04===SORRY!04=== Error while compiling /tmp/kGiCf5ENky Missingā¦ |
||||||||||||||||||||||||||||||||||
jnthn, Full output: gist.github.com/716eef5e734b1efc50...1fe93826f6 | |||||||||||||||||||||||||||||||||||
Zoffix | ugexe: IIRC it's 'cause List.to/.from are an, awful hackl | 15:50 | |||||||||||||||||||||||||||||||||
s: (), 'to', \() | |||||||||||||||||||||||||||||||||||
SourceBaby | Zoffix, Sauce is at github.com/rakudo/rakudo/blob/4fca...st.pm#L372 | ||||||||||||||||||||||||||||||||||
Zoffix | It basically *assumes* if such a method is called, the list is a list of Match objects | ||||||||||||||||||||||||||||||||||
ugexe | thats why i wonder what its useful for | ||||||||||||||||||||||||||||||||||
AlexDaniel slaps evalable6 | |||||||||||||||||||||||||||||||||||
ugexe | i assume its useful for -something- | ||||||||||||||||||||||||||||||||||
Zoffix has no idea what for and wouldn't mind the hack going away :P | 15:51 | ||||||||||||||||||||||||||||||||||
But I'm guessing it's for uniform use with $/.to and $/.from giving the right result regardless of whether the $/ has a list of Matches (e.g. from + quantifier) or just one Match | 15:52 | ||||||||||||||||||||||||||||||||||
ugexe | then maybe its not needed since the cursor rework stuff awhile back | ||||||||||||||||||||||||||||||||||
Zoffix | m: 'abc' ~~ /(.)+/; say $/[0].^name | 15:54 | |||||||||||||||||||||||||||||||||
camelia | Array | ||||||||||||||||||||||||||||||||||
Zoffix | Wouldn't it need to be here ^ | ||||||||||||||||||||||||||||||||||
m: 'abc' ~~ /(.)+/; say [.from, .to] with $/[0] | |||||||||||||||||||||||||||||||||||
camelia | [0 3] | ||||||||||||||||||||||||||||||||||
ugexe | removing it causes two spectest files to fail, but it still passed the Grammar::HTTP tests | 16:22 | |||||||||||||||||||||||||||||||||
mst | if unicode gets more than one pile of poo glyph can we have an [[:excremoji:]] matcher for it? | 16:30 | |||||||||||||||||||||||||||||||||
Zoffix | :} | ||||||||||||||||||||||||||||||||||
ilmari | \p{oo} | 16:41 | |||||||||||||||||||||||||||||||||
\poo even | |||||||||||||||||||||||||||||||||||
buggable | New CPAN upload: cro-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | 16:46 | |||||||||||||||||||||||||||||||||
New CPAN upload: cro-core-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | |||||||||||||||||||||||||||||||||||
New CPAN upload: cro-http-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | |||||||||||||||||||||||||||||||||||
New CPAN upload: cro-tls-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | |||||||||||||||||||||||||||||||||||
New CPAN upload: cro-websocket-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | |||||||||||||||||||||||||||||||||||
New CPAN upload: cro-zeromq-0.7.1.tar.gz by JNTHN cpan.metacpan.org/authors/id/J/JN/...7.1.tar.gz | |||||||||||||||||||||||||||||||||||
timotimo | nice | ||||||||||||||||||||||||||||||||||
Zoffix ponders what could be put into cro-magnon namespace... | 16:48 | ||||||||||||||||||||||||||||||||||
en.wikipedia.org/wiki/Cro-Magnon | |||||||||||||||||||||||||||||||||||
ZofBot: the humans did not appreciate the joke about their ancenstor. Abort Mission! | 16:55 | ||||||||||||||||||||||||||||||||||
ZofBot | Zoffix, When the time came for me to join the firm, I put it to dad straight | ||||||||||||||||||||||||||||||||||
Zoffix | Didn't know you had a dad, bruh. | 16:56 | |||||||||||||||||||||||||||||||||
[Coke] | is it intentional to get cpan upload info to both #perl6 & #perl6-dev ? | 16:58 | |||||||||||||||||||||||||||||||||
ilmari | Zoffix: I thought you were the dad? | 17:04 | |||||||||||||||||||||||||||||||||
Zoffix | [Coke]: yes. My logic is not everyone is present in both of those channels to select just a single one to have those reports appear in. | ||||||||||||||||||||||||||||||||||
ilmari: it's complicated! :o | |||||||||||||||||||||||||||||||||||
ilmari likes how Jeph Jacques refers to his ebooks bot as "my idiot robot son" | |||||||||||||||||||||||||||||||||||
Zoffix | heh | 17:05 | |||||||||||||||||||||||||||||||||
ilmari | twitter.com/jephjacques / twitter.com/jeph_ebooks | ||||||||||||||||||||||||||||||||||
Zoffix | [Coke]: you're the second person to ask that tho. I think I'll move #perl6-dev reports to #zofbot... | ||||||||||||||||||||||||||||||||||
[Coke] | eh. irc sucks. I get it. :) | 17:06 | |||||||||||||||||||||||||||||||||
Zoffix | Done. | 17:07 | |||||||||||||||||||||||||||||||||
El_Che | any alpine devs around? | 17:21 | |||||||||||||||||||||||||||||||||
Geth | rakudo: MasterDuke17++ created pull request #1229: Test for RT #132291 |
18:05 | |||||||||||||||||||||||||||||||||
synopsebot | RT#132291 [new]: rt.perl.org/Ticket/Display.html?id=132291 [LTA] Too few positionals passed to '' (for [:a] X [:b] -> ($i, $j) { }) | ||||||||||||||||||||||||||||||||||
RAKUDO#1229 [open]: github.com/rakudo/rakudo/pull/1229 Test for RT #132291 | |||||||||||||||||||||||||||||||||||
Geth | rakudo: cd9f66eff5 | MasterDuke17++ | t/05-messages/02-errors.t Test for RT #132291 |
18:07 | |||||||||||||||||||||||||||||||||
rakudo: 32412170b3 | (Zoffix Znet)++ (committed using GitHub Web editor) | t/05-messages/02-errors.t Merge pull request #1229 from MasterDuke17/test_for_RT132291 Test for RT #132291 |
|||||||||||||||||||||||||||||||||||
El_Che | I am working on alpine packages, and I have a failing test: gist.github.com/nxadm/f1c10be2a03b...5fbde35c72 ( t/04-nativecall/01-argless.t) | 18:11 | |||||||||||||||||||||||||||||||||
I know alpine is not the nicest for C and C++ code, but I haven't seen this one | |||||||||||||||||||||||||||||||||||
Failed test 'called argless function returning char' at t/04-nativecall/01-argless.t line 25 expected: '2' got: '94117762201346' | |||||||||||||||||||||||||||||||||||
if someone has seen it, ping me | 18:12 | ||||||||||||||||||||||||||||||||||
Zoffix | m: say 94117762201346.log: 2 | ||||||||||||||||||||||||||||||||||
camelia | 46.4195322521493 | ||||||||||||||||||||||||||||||||||
Zoffix shrugs | |||||||||||||||||||||||||||||||||||
El_Che | good ... how :) | ||||||||||||||||||||||||||||||||||
AlexDaniel | MasterDuke: :D | 18:13 | |||||||||||||||||||||||||||||||||
( rt.perl.org/Ticket/Display.html?id...xn-1504393 ) | 18:14 | ||||||||||||||||||||||||||||||||||
I hope twice is enough! :) | 18:15 | ||||||||||||||||||||||||||||||||||
MasterDuke | heh, that's one thoroughly resolved ticket | ||||||||||||||||||||||||||||||||||
El_Che | is there a 2017.10.1 coming or is it for 2017.11? | 18:16 | |||||||||||||||||||||||||||||||||
I asked becuase fpm (what I use to generate pkgs) is broken on alpine | |||||||||||||||||||||||||||||||||||
so I went into the rabbit hole to create the pkg manually | 18:17 | ||||||||||||||||||||||||||||||||||
using the tools (that downloads and compiles the stuff for you) you can get a package, but you loose a lot of flexibility | |||||||||||||||||||||||||||||||||||
I don't want to figure out how patches work on the alpine build system :) | |||||||||||||||||||||||||||||||||||
MasterDuke | i don't think there are any plans for a 2017.10.1 | 18:18 | |||||||||||||||||||||||||||||||||
El_Che | there are no alpine packages, so it's not like it's blocking or something :) | ||||||||||||||||||||||||||||||||||
Zoffix | 2017.11 is gonna be here in just 2 weeks | ||||||||||||||||||||||||||||||||||
El_Che | indeed | ||||||||||||||||||||||||||||||||||
AlexDaniel | El_Che: what's the ticket id for that issue? | 18:34 | |||||||||||||||||||||||||||||||||
releasable6: next | 18:35 | ||||||||||||||||||||||||||||||||||
releasable6 | AlexDaniel, Next release in 15 days and ā0 hours. No blockers. 0 out of 113 commits logged | ||||||||||||||||||||||||||||||||||
AlexDaniel, Details: gist.github.com/03c27e76a8a89b9c11...49b6de1969 | |||||||||||||||||||||||||||||||||||
El_Che | AlexDaniel: 13229 | 18:37 | |||||||||||||||||||||||||||||||||
AlexDaniel | RT#13229 | ||||||||||||||||||||||||||||||||||
mm | |||||||||||||||||||||||||||||||||||
? | |||||||||||||||||||||||||||||||||||
Zoffix | There ain't no such ticket. The number's missing a digit | 18:38 | |||||||||||||||||||||||||||||||||
El_Che | 132291 | 18:39 | |||||||||||||||||||||||||||||||||
sorry | |||||||||||||||||||||||||||||||||||
Zoffix | RT#132291 | ||||||||||||||||||||||||||||||||||
synopsebot | RT#132291 [resolved]: rt.perl.org/Ticket/Display.html?id=132291 [LTA] Too few positionals passed to '' (for [:a] X [:b] -> ($i, $j) { }) | ||||||||||||||||||||||||||||||||||
Zoffix | El_Che: are you sure? That's just some LTA error, why's that blocking alpine? | 18:40 | |||||||||||||||||||||||||||||||||
Geth | nqp/master: 4 commits pushed by pmurias++ | ||||||||||||||||||||||||||||||||||
El_Che | Zoffix: i meant the other way round | ||||||||||||||||||||||||||||||||||
Zoffix: as in "it's literally not important" | 18:41 | ||||||||||||||||||||||||||||||||||
I am working on a first alpine pkg | |||||||||||||||||||||||||||||||||||
so it's ok if the fix in in 2017.11 or later | |||||||||||||||||||||||||||||||||||
Zoffix is confused | |||||||||||||||||||||||||||||||||||
El_Che: fix for what? | |||||||||||||||||||||||||||||||||||
For alpine issue? Then the question is: what's that issue? What's the ticket ID for it? | 18:42 | ||||||||||||||||||||||||||||||||||
El_Che | ok, let me rephrase, read as no-sarcasm | ||||||||||||||||||||||||||||||||||
so, there is a bug that make a test fail, that is already fixed in rakudo upstream | |||||||||||||||||||||||||||||||||||
I didn't know that | 18:43 | ||||||||||||||||||||||||||||||||||
Zoffix | Ah ok | ||||||||||||||||||||||||||||||||||
El_Che | So, now I'll continue without make test | ||||||||||||||||||||||||||||||||||
in order to figure the alpine packaging out | |||||||||||||||||||||||||||||||||||
by the time I have a pkg, rakudo 2017.11 will be out and the tests will pass | |||||||||||||||||||||||||||||||||||
AlexDaniel | El_Che: OK cool. If you notice any further issues on alpine, please file a bug report and ping me the ticket ID. I'll make sure we do something about it before the release. | 18:44 | |||||||||||||||||||||||||||||||||
El_Che | I am half way in understanding how alpine building works :) | ||||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. pmurias 'Test that unhandled named arguments to subs are not allowed' | 19:12 | |||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/296926994 github.com/perl6/nqp/compare/87000...536d98a51e | |||||||||||||||||||||||||||||||||||
El_Che | ok, --gen-* is a no-no on alpine as it tries to write in the --prefix dir before make install | 19:48 | |||||||||||||||||||||||||||||||||
Geth | rakudo: 8b24bf5cd4 | usev6++ | src/vm/jvm/runtime/org/perl6/rakudo/RakudoContainerSpec.java [jvm] Also do a type check when assigning Nil Fixes GH#1225, pmurias++ |
20:15 | |||||||||||||||||||||||||||||||||
synopsebot | GH#1225 [open]: github.com/rakudo/rakudo/issues/1225 [JVM] Do a type check when resetting a default when asigning a Nil | ||||||||||||||||||||||||||||||||||
Zoffix | dogbert17: RT#132273 no longer segfaults; closed ticket | 21:17 | |||||||||||||||||||||||||||||||||
synopsebot | RT#132273 [resolved]: rt.perl.org/Ticket/Display.html?id=132273 [SEGV] while running t/spec/S17-supply/supplier-preserving.t | ||||||||||||||||||||||||||||||||||
jnthn | evening o/ | 21:50 | |||||||||||||||||||||||||||||||||
Zoffix | \o | 21:53 | |||||||||||||||||||||||||||||||||
jnthn | github.com/rakudo/rakudo/issues/1230 is the bug I ran into the other day | 22:06 | |||||||||||||||||||||||||||||||||
Filed it so we don't forget on it, but intend to work on it next week or so | |||||||||||||||||||||||||||||||||||
Did we pick a way to refer to GitHub issues from roast? | 22:08 | ||||||||||||||||||||||||||||||||||
Zoffix just used full URL so far | |||||||||||||||||||||||||||||||||||
Also makes GitHub include reference to the commit on the issue | 22:09 | ||||||||||||||||||||||||||||||||||
Like here: github.com/perl6/roast/blob/master...urn.t#L106 | 22:10 | ||||||||||||||||||||||||||||||||||
Oh the ref happens if you also include link in commit | 22:11 | ||||||||||||||||||||||||||||||||||
jnthn | Alright, will follow that convention | ||||||||||||||||||||||||||||||||||
Zoffix | or write "rakudo/1234" | ||||||||||||||||||||||||||||||||||
(in commit message I meant) | |||||||||||||||||||||||||||||||||||
jnthn | I was mostly thinking about the test file itself | 22:12 | |||||||||||||||||||||||||||||||||
But yeah, "Test for rakudo/1234" getting linked in the commit message is handy | |||||||||||||||||||||||||||||||||||
hah, phew...the heap of Travis fail notifications I got for Cro were because there was some minutes between me removing META.list entries and the CPAN additions being visible to zef | 22:14 | ||||||||||||||||||||||||||||||||||
El_Che | About travis. I now also create Ubuntu 14.04 rakudo debs (OS that travis run), so I can speedup tests of my stuff: github.com/nxadm/rakudo-pkg/blob/m...is.yml#L27 . Is there a use case for me providing debs for older rakudo releases? Maybe in a repo? Or is there no use case before rakudbrew is more flexible? | 22:20 | |||||||||||||||||||||||||||||||||
s/before/because/ | |||||||||||||||||||||||||||||||||||
jnthn | oh heck, I must update some of mine for that | 22:29 | |||||||||||||||||||||||||||||||||
El_Che: Do you do 16.04 ones, ooc? | |||||||||||||||||||||||||||||||||||
Oh wait, Travis doesn't support that yet | |||||||||||||||||||||||||||||||||||
Heh | |||||||||||||||||||||||||||||||||||
El_Che: But in terms of "older releases" - yes, for example I'd like to be able to test against the last Rakudo that was used to make an R*. | 22:30 | ||||||||||||||||||||||||||||||||||
El_Che | well, 14.04 shoould die in 2019 :) | 22:31 | |||||||||||||||||||||||||||||||||
jnthn | I don't think Travis does 16.04 yet, though. | ||||||||||||||||||||||||||||||||||
El_Che | no, travis is 14.04, hence the new 14.04 pkgs | ||||||||||||||||||||||||||||||||||
jnthn | Once there's a 6.d language release, most probably the first release of Rakudo that supported that will be an interesting thing to test against long-term too | 22:33 | |||||||||||||||||||||||||||||||||
El_Che | well, from 2017.10 onwards there will be a deb for 14.04 | 22:35 | |||||||||||||||||||||||||||||||||
it's automated :) | |||||||||||||||||||||||||||||||||||
jnthn | Ah, I see | ||||||||||||||||||||||||||||||||||
El_Che | my point is that regular modules probably wont test against random commits | 22:36 | |||||||||||||||||||||||||||||||||
jnthn | For me personally, that'll suffice | ||||||||||||||||||||||||||||||||||
El_Che | so If we can provide a matrix of debs, it will shave 10minutes of every build | ||||||||||||||||||||||||||||||||||
jnthn: cheers on cro, btw. I just had a look at the docs | 22:38 | ||||||||||||||||||||||||||||||||||
fancy site :) | |||||||||||||||||||||||||||||||||||
jnthn | :) | 22:50 | |||||||||||||||||||||||||||||||||
Site built using Cro, of course :) | |||||||||||||||||||||||||||||||||||
El_Che | I remember the crust author talking about threading problems at fosdem | 22:52 | |||||||||||||||||||||||||||||||||
you skip that middellayer? | |||||||||||||||||||||||||||||||||||
it was this talk: archive.fosdem.org/2016/schedule/event/crust/ | 22:53 | ||||||||||||||||||||||||||||||||||
jnthn | Well, between then and now there's been a lot of fixes also | ||||||||||||||||||||||||||||||||||
Cro uses IO::Socket::Async though | 22:54 | ||||||||||||||||||||||||||||||||||
It doesn't do any explicit threading; it's built entirely out of the higher-level Perl 6 constructs | |||||||||||||||||||||||||||||||||||
Meaning that a server under low load will run with just a handful of threads, and one under load will happily start some more | 22:55 | ||||||||||||||||||||||||||||||||||
Geth | roast: fd3ba22c63 | usev6++ | S32-io/io-handle.t [jvm] Fudge failing tests in S32-io/io-handle.t |
22:56 | |||||||||||||||||||||||||||||||||
jnthn | Cro so far takes the Node.js approach of "run our own HTTP server" | ||||||||||||||||||||||||||||||||||
El_Che | jnthn: or go's | 22:57 | |||||||||||||||||||||||||||||||||
jnthn | That too | ||||||||||||||||||||||||||||||||||
Or .Net or... :) | |||||||||||||||||||||||||||||||||||
That said, Cro is just a way to build async pipelines | |||||||||||||||||||||||||||||||||||
gfldex | i'm afraid of using PerlĀ 6 in a coding competion in fear of being accused of cheating. :-> | ||||||||||||||||||||||||||||||||||
jnthn | So there's no reason at all that we couldn't provide FastCGI support | 22:58 | |||||||||||||||||||||||||||||||||
El_Che | jnthn: I heard an interesting talk about the jvm and docker at oss summit last week. About how only java 9 was party aware of limit set to the container | ||||||||||||||||||||||||||||||||||
and I wondered how rakudo worked | |||||||||||||||||||||||||||||||||||
e.g. problem was that the jvm saw x cores, while it only had access to let's say 2 | 22:59 | ||||||||||||||||||||||||||||||||||
but because it saw all the cores in the machines it started X threads per core for housekeeping | |||||||||||||||||||||||||||||||||||
jnthn | We're not quite that aggressive. | ||||||||||||||||||||||||||||||||||
El_Che | so each container used way to much memory | ||||||||||||||||||||||||||||||||||
jnthn | But still somewhat vulnerable to that issue | 23:00 | |||||||||||||||||||||||||||||||||
El_Che | java 9 now can take the heap hint from the container limits, and support dedicated cores (not percentages, what most people use) | ||||||||||||||||||||||||||||||||||
jnthn | MoarVM only starts one or two internal threads | ||||||||||||||||||||||||||||||||||
El_Che | jnthn: that's ok | 23:01 | |||||||||||||||||||||||||||||||||
jnthn | So that's OK | ||||||||||||||||||||||||||||||||||
Rakudo's thread pool scheduler is more interesting. | |||||||||||||||||||||||||||||||||||
El_Che | indeed | ||||||||||||||||||||||||||||||||||
jnthn | It will add threads if it sees that there's more work than the current are rapidly chewing through | ||||||||||||||||||||||||||||||||||
Up until the CPU core count | |||||||||||||||||||||||||||||||||||
It probably, though we'd have to check this, seeing the same number the JVM was | |||||||||||||||||||||||||||||||||||
El_Che | so it can exhaust the ram | 23:02 | |||||||||||||||||||||||||||||||||
jnthn | Beyond the CPU core count it'll reluctantly add threads if it things the program is deadlocked | ||||||||||||||||||||||||||||||||||
El_Che | resulting in OOM for the container | ||||||||||||||||||||||||||||||||||
jnthn | *nod* | ||||||||||||||||||||||||||||||||||
It's probably worth factoring memory into the scheduler's decision making too | 23:03 | ||||||||||||||||||||||||||||||||||
El_Che | I think that probably 70% of the talks at linux con were about containers | ||||||||||||||||||||||||||||||||||
so that's something to keep in mind | |||||||||||||||||||||||||||||||||||
jnthn | For sure | 23:04 | |||||||||||||||||||||||||||||||||
El_Che | besides that containers make the more fragile dependency installation less of an issue | ||||||||||||||||||||||||||||||||||
kind-of levels the playfiel | |||||||||||||||||||||||||||||||||||
d | |||||||||||||||||||||||||||||||||||
jnthn | Yeah; we'll be providing Cro docker images in the near future :) | 23:05 | |||||||||||||||||||||||||||||||||
El_Che | I want to provide rakudo alpine pkgs | ||||||||||||||||||||||||||||||||||
so paranoid people like me can build their own cro images | |||||||||||||||||||||||||||||||||||
jnthn | :) | ||||||||||||||||||||||||||||||||||
I'll probably look for a base image that has an OpenSSL with ALPN support, so folks can get HTTP/2.0 support out of the box with them | 23:06 | ||||||||||||||||||||||||||||||||||
El_Che | centos 7.4? | ||||||||||||||||||||||||||||||||||
centos, debian, ubuntu are popular and sane picks | 23:07 | ||||||||||||||||||||||||||||||||||
but alpine is sooo small :) | |||||||||||||||||||||||||||||||||||
jnthn | I'm most likely to go debian or ubuntu, out of familiarity more than anything :) | ||||||||||||||||||||||||||||||||||
El_Che | I run production on centos 7 images though | ||||||||||||||||||||||||||||||||||
linux team run centos | |||||||||||||||||||||||||||||||||||
otherwise would have been debian/ubuntu | |||||||||||||||||||||||||||||||||||
I have a few things in alpine though | 23:08 | ||||||||||||||||||||||||||||||||||
small apps in prod | |||||||||||||||||||||||||||||||||||
specially good for go code | |||||||||||||||||||||||||||||||||||
I wonder how alpine will be for the perl 6 ecosystem | |||||||||||||||||||||||||||||||||||
some things are tough to compile | |||||||||||||||||||||||||||||||||||
jnthn | Yeah, we had to patch MoarVM a bit because of its small stack size | 23:09 | |||||||||||||||||||||||||||||||||
El_Che | yes, I remember that | ||||||||||||||||||||||||||||||||||
jnthn | With regards to the thread scheduler, I have the unenviable task of trying to make it sufficiently smart and also sufficiently understandable. | 23:10 | |||||||||||||||||||||||||||||||||
Because one can never make it so smart that nobody will ever thing to ask how it makes its choices :) | |||||||||||||||||||||||||||||||||||
*think | |||||||||||||||||||||||||||||||||||
El_Che | bus bus bus | ||||||||||||||||||||||||||||||||||
:) | |||||||||||||||||||||||||||||||||||
jnthn | And it used to be so stupid I could explain, "well, it just predictably does this really awful thing" :) | 23:11 | |||||||||||||||||||||||||||||||||
And now it's more like "It tries to do X, and it depends on Y and Z" :) | |||||||||||||||||||||||||||||||||||
El_Che | soon you'll need those 3D screens from science fiction movies | ||||||||||||||||||||||||||||||||||
jnthn | There are at least env vars one can set to see the data and decisions though | ||||||||||||||||||||||||||||||||||
El_Che | "I see dead threads" | 23:12 | |||||||||||||||||||||||||||||||||
Zoffix | So a wild guess: edits to issues aren't re-sent to people who only read them via email, right? | 23:38 | |||||||||||||||||||||||||||||||||
On GitHub | |||||||||||||||||||||||||||||||||||
jnthn | Zoffix: Matches what I've observed, yes | 23:39 |