MasterDuke | jnthn, timotimo, lizmat: any thoughts on github.com/rakudo/rakudo/pull/1317 ? | 00:15 | |
jnthn | MasterDuke: I suspect more effective would be a set of candidates like (Slip:D), (Any) for one-item, and (Slip:D, Slip:D), (Slip:D, Any), (Any, Slip:D) and (Any, Any) for the two argument case | 00:25 | |
Those would deal with a lot more cases | |||
And specialize decently into the Int/Int and Str/Str ones | 00:26 | ||
m: say Slip ~~ Cool | 00:28 | ||
camelia | True | 00:29 | |
jnthn | lizmat: ^^ | ||
(wrt the comment you just put on the ticket) | |||
lizmat | hehe... there's no escaping, I'm being watched :-) | ||
would have been nice if the commit message had mentioned the reason :-) | |||
lizmat is back from travelling for 3 days and goes to get some rest in a good bed for a change | 00:30 | ||
timotimo | have a good one! | ||
jnthn | Rest well | 00:31 | |
lizmat | thanks all! | ||
jnthn | As I noted above, though, adding candidates for handling Slip would help | ||
timotimo | mostly for handling anything but slips? | 00:37 | |
jnthn | timotimo: If there's a candidate for Slip, it's tighter than the one for Any, so it means the Any ones know there's no Slip | 00:40 | |
timotimo | yeah | 00:41 | |
jnthn | 'night o/ | 00:45 | |
AlexDaniel | releasable6: next | 02:05 | |
releasable6 | AlexDaniel, Next release in 6 days and ≈16 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Changelog for this release was not started yet | ||
AlexDaniel, Details: gist.github.com/0bb2234029a2a03e69...3a5a9e1f14 | |||
jstuder | timotimo: I updated the statevar init PR (github.com/rakudo/rakudo/pull/1542) with the spectest info we spoke about earlier. It appears that the test did not actually fail but I read the results wrong. | 02:55 | |
samcv | nice! i got open() working with a replacement character when an unencodable character is encountered for windows-1252/1 | 07:13 | |
pmurias | geekosaur: It's still does a dynamic lookup, just that we have extra fallback to a global semantics when it fails | 08:50 | |
nine | m: multi foo() { say "none" }; multi foo(|args) { say "many" }; multi foo(*@args, *%args) { say "named" }; foo(a => 1) | 10:35 | |
camelia | Unexpected named argument 'a' passed in sub foo at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
nine | jnthn: what's going on here ^^^? | 10:36 | |
MasterDuke | .tell pmurias i added that @*comp_line_directives stuff when i was adding support for `#line 2 SETTING::src/core/core_prologue.pm6` (so error message gave the real file instead of gen/moar/CORE.setting). but i barely knew what i was doing, so feel free to do something better | 11:33 | |
yoleaux | MasterDuke: I'll pass your message to pmurias. | ||
AlexDaniel | Zoffix: fwiw just as a thought, maybe eventually we should start recommending to install rakudo from the official distro repos. Ubuntu Artful (17.10) and Bionic (upcoming in a month) have 2017.06, but their LTS version has 2015.11 (oh no!), Debian stable has 2016.12 which is not bad at all, Debian testing has 2017.06. Fedora should have at least 2017.06 starting from f26. But all versions of Linux Mint have 2015.11 I think… | 12:16 | |
so I guess eventually = not now, but perhaps we should keep in mind that this will be possible soon | |||
or, I don't know, am I the only one who likes when the download page says “just do `apt install thiscoolthing`”? | 12:17 | ||
and, um… I guess there's no good way to detect a distro using JS | 12:18 | ||
nine | AlexDaniel: openSUSE Tumbleweed currently ships 2018.01 and so does the upcoming openSUSE Leap 15.0 | 12:25 | |
Geth | nqp: a6b9e845f4 | usev6++ | src/vm/jvm/runtime/org/perl6/nqp/io/FileHandle.java [JVM] Support opening files with :read and :append Makes some more test in S32-io/open.t pass. Compare also RT #131145. Thanks to cygx++ for giving a pointer on how this could be done. |
12:32 | |
synopsebot | RT#131145 [open]: rt.perl.org/Ticket/Display.html?id=131145 [JVM] Modes for IO::Handle.open need clarification | ||
Geth | features: JJ++ created pull request #7: Eliminates niecza refs closes #6 when accepted |
13:00 | |
MasterDuke | jnthn and anyone else: i would have thought this patch would work, but there are a lot of test failures. gist.github.com/MasterDuke17/86c7c...29cec3d9e8 | 13:18 | |
timotimo | MasterDuke: you're not slipping the slips at all there | 14:31 | |
m: say (|(1, 2, 3), |(9, 9, 9)).perl | |||
camelia | (1, 2, 3, 9, 9, 9) | ||
timotimo | oh, d'oh | ||
you are binding a reifier there | |||
i see a problem with the second candidate, though | 14:32 | ||
you're binding b to the second slot, but what if the slip in a has more than one element? | 14:33 | ||
MasterDuke: could that be the problem? | 14:34 | ||
MasterDuke | i didn't correctly follow what was happening (in the original code) when there was a Slip, so yeah, i definitely need to change something | 14:36 | |
but i don't think the number of elements in the slip matter, right? | |||
m: use nqp; sub a(Any \b, Slip:D \c) { say nqp::elems(nqp::p6argvmarray()) }; my @d = <a b c>; a(4, @d.Slip) | 14:37 | ||
camelia | 2 | ||
MasterDuke | m: use nqp; sub a(Any \b, Slip:D \c) { say nqp::elems(nqp::p6argvmarray()) }; my @d = <a b c>; a(4, |@d) | ||
camelia | Too many positionals passed; expected 2 arguments but got 4 in sub a at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
timotimo | well, you're binding into slot 2 of the reification target | 14:39 | |
when the slip gets slipped, its contents will spill into the reification target, too | |||
MasterDuke | hm, i guess the `Slip:D \a, Any \b` candidate shouldn't actually bind b at all | 14:42 | |
timotimo | i agree | 14:43 | |
Geth | nqp/master: 7 commits pushed by pmurias++
|
15:37 | |
AlexDaniel | hmm, how can I make synopsebot bot #whateverable? | 17:34 | |
join* | |||
MasterDuke | synopsebot: help | 17:35 | |
synopsebot | MasterDuke, See: github.com/perl6/synopsebot | ||
MasterDuke | just add #whateverable here? github.com/perl6/synopsebot/blob/m...bot.p6#L15 | 17:36 | |
Geth | synopsebot: d02bfb7932 | (Aleks-Daniel Jakimenko-Aleksejev)++ | synopsebot.p6 Add #whateverable to the list |
17:37 | |
timotimo | i'm not sure synopsebot has any "check out the source regularly" mechanisms in place; didn't when i last touched it, which may have been sufficiently long ago for this message to not matter at all | 17:38 | |
AlexDaniel | `ssh hack.p6c.org` – hmm, can I have access to that? | 17:39 | |
Geth | features: 5ad23566e3 | (JJ Merelo)++ | 3 files Eliminates niecza refs closes #6 when accepted |
17:48 | |
features: ee7fbb17da | (Moritz Lenz)++ (committed using GitHub Web editor) | 3 files Merge pull request #7 from JJ/master Eliminates niecza refs closes #6 |
|||
roast: ronaldxs++ created pull request #399: Fix attribute mutator tests and add comment discouraging use |
17:50 | ||
mr_ron | From what I can figure out a fix for the attribute mutator roast test could have been done a long time ago with subs but there was a hangup over using methods. Why is it not OK to solve with subs until we understand exactly how methods fit this case? | 18:25 | |
AlexDaniel | timotimo: maybe you can just follow the steps here for me? github.com/perl6/synopsebot#pullin...ng-the-bot | 19:09 | |
or anybody else who has enough privs | |||
Zoffix | Yeah, I can | 19:10 | |
AlexDaniel | thanks! | ||
Zoffix | AlexDaniel: actually, I'll create an account and you can do it yourself. What username/key? | 19:11 | |
AlexDaniel | Zoffix: these two keys: github.com/AlexDaniel.keys | 19:12 | |
Zoffix: username “alex” if available | |||
alexdaniel otherwise | |||
Zoffix | AlexDaniel: I added you to sudoers and msg'ed you the pass. I didn't setup the key. I think you'd be able to set it up on your own. | 19:14 | |
Are you able to log in and pull-restart synopsebot? | 19:15 | ||
AlexDaniel: note that perms on your home dir are drwxr-xr-x. That's the default adduser makes but I changed mine to be 0 for the world | 19:16 | ||
AlexDaniel | Zoffix: OK, that works! Thank you very much! | 19:17 | |
Zoffix | ok | 19:18 | |
Geth | geth: be7c31d75e | (Zoffix Znet)++ (committed using GitHub Web editor) | README.md Add info on webhook/code locations |
19:28 | |
MasterDuke | m: use nqp; sub a(\b) { say nqp::elems(b) }; sub c(|) { a(nqp::list(nqp::p6argvmarray())) }; c(1,2) | 20:02 | |
camelia | This type (List) does not support elems in sub a at <tmp> line 1 in sub c at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
MasterDuke | i am thinking very slowly today. how do it make ^^^ work? changing \b to $b or @b doesn't help | 20:03 | |
Geth | nqp: 46fbb5e9d8 | usev6++ | src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java [JVM] Let seekfh return null, not file handle This avoids errors in S32-io/seek.t caused by missing method 'sink' for BOOTIO when nqp::seekfh is the last command in a block. This mirrors the behaviour of MoarVM. |
20:09 | |
Zoffix | MasterDuke: looks like it gets hllized when it goes from one sub to another. | 20:13 | |
m: use nqp; sub a(\b) { say nqp::elems(nqp::getattr(b, List, '$!reified')) }; sub c(|) { a nqp::p6argvmarray }; c 1,2 | |||
camelia | 2 | ||
Zoffix | Don't know how solid that is, to poke into List guts that way (and assuming it's reified) | ||
timotimo | you can get around hllization with some trickery; not sure what exactly, perhaps involving findmethod | ||
Zoffix | m: use nqp; sub a(\b) { say nqp::elems(b) }; sub c(|) { nqp::callstatic(&a, nqp::p6argvmarray) }; c 1,2 | 20:14 | |
camelia | This type (List) does not support elems in sub a at <tmp> line 1 in sub c at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
timotimo | oh, right, not a method | 20:19 | |
i wonder if what i'm thinking of is even still relevant or if it's already years old | |||
MasterDuke | Zoffix++, timotimo++ | 20:28 | |
timotimo | what if you type the argument Mu? | ||
MasterDuke | same | ||
Geth | rakudo: MasterDuke17++ created pull request #1608: Make 2 elem infix:<,> special cases more generic |
20:50 | |
AlexDaniel | releasable6: status | 22:05 | |
releasable6 | AlexDaniel, Next release in 5 days and ≈20 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Changelog for this release was not started yet | ||
AlexDaniel, Details: gist.github.com/613eeadedea306d077...0b03b7dacd | |||
Kaiepi | where would be the appropriate place to put tests for Perl6::Metamodel::Naming? | 22:35 | |
yoleaux | 10 Mar 2018 06:42Z <Zoffix> Kaiepi: not 100% sure, but this seems to do the trick, without regexes: gist.github.com/zoffixznet/da3b0ce...f8483b2f05 | ||
Kaiepi | would writing some for rakudo's test suite be needed, or just ones for roast? | 22:39 | |
Zoffix | Kaiepi: just for roast. Tests in rakudo's suite are just for stuff we don't want to be part of the language spec. | 23:01 | |
Maybe in S14-roles/mixin.t, unless you already found a good place. Seems there's already a test for a similar thing: S14-roles/mixin.t:229: "mixing in a role from a deeper namespace doesn't clobber the targets shortname"; |