| gfldex | m: sub s(&c:(:$p)){ c() }; sub f(:p(:$a)){say $a }; f(:p<♥>); s(&f) | 00:28 | |||||||||||||||||||||||||||||||||||||
| camelia | ♥ Constraint type check failed in binding to parameter '&c'; expected anonymous constraint to be met but got Sub (sub f (:p(:$a)) { #`(...) in sub s at <tmp> line 1 in block <unit> at <tmp> line 1 |
||||||||||||||||||||||||||||||||||||||
| gfldex | m: sub s(&c:(:$p)){ c() }; sub f(:a(:$p)){say $p }; f(:p<♥>); s(&f) | ||||||||||||||||||||||||||||||||||||||
| camelia | ♥ Constraint type check failed in binding to parameter '&c'; expected anonymous constraint to be met but got Sub (sub f (:a(:$p)) { #`(...) in sub s at <tmp> line 1 in block <unit> at <tmp> line 1 |
||||||||||||||||||||||||||||||||||||||
| gfldex | I'm leaning towards bug. | ||||||||||||||||||||||||||||||||||||||
| gfldex goes to sleep over it | 00:29 | ||||||||||||||||||||||||||||||||||||||
| travis-ci | Rakudo build passed. Zoffix Znet 'Revert "Actually use new-from-git-describe in Compiler.new" | 00:38 | |||||||||||||||||||||||||||||||||||||
| travis-ci.org/rakudo/rakudo/builds/337352106 github.com/rakudo/rakudo/compare/a...4d6b461662 | |||||||||||||||||||||||||||||||||||||||
| Geth | rakudo: e8c6c259ce | (Zoffix Znet)++ | src/Perl6/Actions.nqp Fix postfix whatever curry with another curry inside Fixes RT#130859: rt.perl.org/Ticket/Display.html?id=130859 When we enter the currier, we use arity to figure out how many kids of the QAST to examine for currying. However, in the case of `*.[*-1]`, the remaining kids have a call to a Callable, the QAST::Block for which we never migrate, since we never examine them. Fix by looping from $e to $qast.elems to migrate them. |
00:43 | |||||||||||||||||||||||||||||||||||||
| synopsebot | RT#130859 [new]: rt.perl.org/Ticket/Display.html?id=130859 [WEIRD] WhateverCode subscript inside WhateverCode inside block doesn't properly close over lexical | ||||||||||||||||||||||||||||||||||||||
| roast: 501f51b56c | (Zoffix Znet)++ | S02-types/whatever.t Test postfix curry with another curry inside RT#130859: rt.perl.org/Ticket/Display.html?id=130859 Rakudo fix: github.com/rakudo/rakudo/commit/e8c6c259ce |
00:44 | ||||||||||||||||||||||||||||||||||||||
| Zoffix | m: dd (:$_, :$_) | 03:47 | |||||||||||||||||||||||||||||||||||||
| camelia | (:_(Any), :_(Any)) | ||||||||||||||||||||||||||||||||||||||
| Zoffix | m: dd (:$_ :$_) | ||||||||||||||||||||||||||||||||||||||
| camelia | ===SORRY!=== Unknown QAST node type NQPMu |
||||||||||||||||||||||||||||||||||||||
| Zoffix | Are these supposed to be equivalent or should the latter throw? | ||||||||||||||||||||||||||||||||||||||
| m: class Foo { method bar (|c){ dd c } }.bar(:foo :bar :ber) | 03:50 | ||||||||||||||||||||||||||||||||||||||
| camelia | \(:bar, :ber, :foo) | ||||||||||||||||||||||||||||||||||||||
| Zoffix | m: class Foo { method bar (|c){ dd c } }.bar(42, :foo :bar :ber) | ||||||||||||||||||||||||||||||||||||||
| camelia | \(42, :foo) | ||||||||||||||||||||||||||||||||||||||
| Zoffix | this case falls into the same category... | ||||||||||||||||||||||||||||||||||||||
| m: dd %(:42a:70b:80c) | 03:52 | ||||||||||||||||||||||||||||||||||||||
| camelia | Hash % = {:a(42)} | ||||||||||||||||||||||||||||||||||||||
| Zoffix | and this one too | ||||||||||||||||||||||||||||||||||||||
| The extra pairs end up as named args to Pair.new constructing the earlier Pair, because it thinks we're doing `$foo.bar():adverbs` call. | 03:53 | ||||||||||||||||||||||||||||||||||||||
| gonna try to make it work and see if anything explodes | 03:57 | ||||||||||||||||||||||||||||||||||||||
| samcv: you around? What should be put into weekly about windows-1251 encoding? I see some reverts... but don't know if everything was reverted or not | 05:05 | ||||||||||||||||||||||||||||||||||||||
| samcv | Zoffix: windows-1251 (Cyrillic) support has been added (we didn't have it before) | 05:06 | |||||||||||||||||||||||||||||||||||||
| also bugs in windows-1252 have been fixed | |||||||||||||||||||||||||||||||||||||||
| Zoffix | Thanks. | ||||||||||||||||||||||||||||||||||||||
| samcv | such as certain codepoints not being properly decoded | ||||||||||||||||||||||||||||||||||||||
| s/decoded/encoded | 05:09 | ||||||||||||||||||||||||||||||||||||||
| but also possibly decoded too | 05:10 | ||||||||||||||||||||||||||||||||||||||
| let me check | |||||||||||||||||||||||||||||||||||||||
| Zoffix: ok. the problem was only encoding, not with decoding | 05:25 | ||||||||||||||||||||||||||||||||||||||
| Zoffix | cool | ||||||||||||||||||||||||||||||||||||||
| samcv | m: "8250|732|339|382|376|8482|353".split('|')».uniname.say | ||||||||||||||||||||||||||||||||||||||
| camelia | (DIGIT EIGHT DIGIT SEVEN DIGIT THREE DIGIT THREE DIGIT THREE DIGIT EIGHT DIGIT THREE) | ||||||||||||||||||||||||||||||||||||||
| samcv | uh | ||||||||||||||||||||||||||||||||||||||
| m: "8250|732|339|382|376|8482|353".split('|').chrs.uninames.say | |||||||||||||||||||||||||||||||||||||||
| camelia | (SINGLE RIGHT-POINTING ANGLE QUOTATION MARK SMALL TILDE LATIN SMALL LIGATURE OE LATIN SMALL LETTER Z WITH CARON LATIN CAPITAL LETTER Y WITH DIAERESIS TRADE MARK SIGN LATIN SMALL LETTER S WITH CARON) | ||||||||||||||||||||||||||||||||||||||
| samcv | m: "8250|732|339|382|376|8482|353".split('|').chrs.uninames.perl.say | 05:26 | |||||||||||||||||||||||||||||||||||||
| camelia | ("SINGLE RIGHT-POINTING ANGLE QUOTATION MARK", "SMALL TILDE", "LATIN SMALL LIGATURE OE", "LATIN SMALL LETTER Z WITH CARON", "LATIN CAPITAL LETTER Y WITH DIAERESIS", "TRADE MARK SIGN", "LATIN SMALL LETTER S WITH CARON").Seq | ||||||||||||||||||||||||||||||||||||||
| samcv | m: "8250|732|339|382|376|8482|353".split('|')».chr.join(', ').say | 05:27 | |||||||||||||||||||||||||||||||||||||
| camelia | ›, ˜, œ, ž, Ÿ, ™, š | ||||||||||||||||||||||||||||||||||||||
| samcv | should be able to copy ^ into the changelog for the ones where encoding is fixed | ||||||||||||||||||||||||||||||||||||||
| Zoffix: oh and make sure to also add, that windows-1251 also now works on jvm too | 05:29 | ||||||||||||||||||||||||||||||||||||||
| that was much easier though, since it just involved adding it to the encodings list :) | |||||||||||||||||||||||||||||||||||||||
| Zoffix | :) | ||||||||||||||||||||||||||||||||||||||
| This weeks Perl 6 Weekly, hot off the press: p6weekly.wordpress.com/2018/02/05/...-squashed/ | 06:29 | ||||||||||||||||||||||||||||||||||||||
| lizmat++ for doing so many of them | 06:30 | ||||||||||||||||||||||||||||||||||||||
| Geth | rakudo: niner++ created pull request #1482: Remove irrelevant and wrong type constraint |
06:48 | |||||||||||||||||||||||||||||||||||||
| samcv | Zoffix++ | 07:10 | |||||||||||||||||||||||||||||||||||||
| nine | Zoffix: sorry for breaking Travis yet again. At least I now think I know what's wrong. | 07:35 | |||||||||||||||||||||||||||||||||||||
| yoleaux | 4 Feb 2018 23:55Z <Zoffix> nine: I reverted "new-from-git-describe in Compiler.new" because it was breaking Travis builds with `MVMArray: Can't pop from an empty array`: travis-ci.org/rakudo/rakudo/jobs/3...5453#L1468 | ||||||||||||||||||||||||||||||||||||||
| nine | Zoffix: it seems like on Travis the compiler version does not contain any '.' | 07:36 | |||||||||||||||||||||||||||||||||||||
| jnthn: oh, that looks quite outdated. I closed it | 07:37 | ||||||||||||||||||||||||||||||||||||||
| Yep, indeed, the version string on Travis is empty: Version: '' | 08:38 | ||||||||||||||||||||||||||||||||||||||
| I'd guess Travis does not fetch git tags and gen-version.pl does not check if it found any, thus overwriting the $version from the VERSION file with an empty string | 08:49 | ||||||||||||||||||||||||||||||||||||||
| Geth | rakudo: 43859b54d5 | (Stefan Seifert)++ | src/core/Compiler.pm Actually use new-from-git-describe in Compiler.new |
09:36 | |||||||||||||||||||||||||||||||||||||
| rakudo: 382024e607 | (Stefan Seifert)++ | tools/build/gen-version.pl Fix empty compiler version when the git repo doesn't have tags |
|||||||||||||||||||||||||||||||||||||||
| rakudo: bdb4d34d3b | (Stefan Seifert)++ | src/core/core_epilogue.pm Initialize the Perl object at BEGIN time to save runtime |
|||||||||||||||||||||||||||||||||||||||
| nine | Zoffix: finally nailed it :) | 09:37 | |||||||||||||||||||||||||||||||||||||
| Zoffix | Great, though I would've written it as `$version = $git_version || "$version-<unknown-commit>"` or something. Just so a failure to fetch version from git differs from a released version. | 09:44 | |||||||||||||||||||||||||||||||||||||
| nine | Feel free to improve :) | 09:50 | |||||||||||||||||||||||||||||||||||||
| samcv | Zoffix: what's your thoughts about throwing when invalid codepoints are encountered. when decoding windows-1252. currently we don't throw and pass through invalid single byte codepoints | 10:08 | |||||||||||||||||||||||||||||||||||||
| Zoffix | No idea, sorry | ||||||||||||||||||||||||||||||||||||||
| Zoffix drops to bed | |||||||||||||||||||||||||||||||||||||||
| samcv | and the question is, once i implement the MoarVM ops to make that configurable (so we can have it loose, or strict) | ||||||||||||||||||||||||||||||||||||||
| Zoffix: do we wait til 6.d to make it strict? | 10:09 | ||||||||||||||||||||||||||||||||||||||
| is the real question i'm trying to ask you | |||||||||||||||||||||||||||||||||||||||
| since we have 6.c tests which test that it decodes and encodes invalid codepoints | |||||||||||||||||||||||||||||||||||||||
| or anyone elses thoughts on when to change the default for this | 10:10 | ||||||||||||||||||||||||||||||||||||||
| AlexDaniel | reportable6: 2018-01-29T00:00:00Z 2018-02-05T00:00:00Z | 11:20 | |||||||||||||||||||||||||||||||||||||
| reportable6 | AlexDaniel, OK, working on it! This may take up to 40 seconds | ||||||||||||||||||||||||||||||||||||||
| AlexDaniel, gist.github.com/261b983377a0c79801...97dd5f9ede | 11:21 | ||||||||||||||||||||||||||||||||||||||
| AlexDaniel | Zoffix++ awesome weekly | 11:23 | |||||||||||||||||||||||||||||||||||||
| pmurias | samcv: re changing the default, that seems like a clear violation of the 6.c tests | 13:20 | |||||||||||||||||||||||||||||||||||||
| the tests where testing for laxness explicitly so it's not like we are changing some accidental behavior | 13:22 | ||||||||||||||||||||||||||||||||||||||
| Geth | nqp: 364d9b65fb | pmurias++ | 3 files [js] Refactor the decoder to use the codec api |
14:07 | |||||||||||||||||||||||||||||||||||||
| nqp: e6fe8810da | pmurias++ | 2 files [js] Using utf8-c8 in decoders works |
|||||||||||||||||||||||||||||||||||||||
| nqp: 6ad811c24d | pmurias++ | 7 files [js] Stop using the grapheme-breaker package and use our own stuff |
|||||||||||||||||||||||||||||||||||||||
| AlexDaniel` | samcv: yes, it does seem like something for v6.d, at least with my first uneducated look at it | 14:49 | |||||||||||||||||||||||||||||||||||||
| |Tux| |
|
15:42 | |||||||||||||||||||||||||||||||||||||
| 8.582 - 9.139 | |||||||||||||||||||||||||||||||||||||||
| Grrrrr | 15:44 | ||||||||||||||||||||||||||||||||||||||
| Rakudo version 2018.01-117-gbdb4d34d3 - MoarVM version 2018.01-77-g9a029b408 | |||||||||||||||||||||||||||||||||||||||
| timotimo | oops, what happened there? | 15:54 | |||||||||||||||||||||||||||||||||||||
| nine | Looks like interleaved double pasting | 15:55 | |||||||||||||||||||||||||||||||||||||
| timotimo | mhm | ||||||||||||||||||||||||||||||||||||||
| |Tux| | My mouse bounces :( | 16:00 | |||||||||||||||||||||||||||||||||||||
| Zoffix | c: HEAD, HEAD~100 say $*PERL.compiler.version eqv $*PERL.compiler.version.perl.EVAL | 21:29 | |||||||||||||||||||||||||||||||||||||
| committable6 | Zoffix, ¦HEAD(bdb4d34): «False» ¦HEAD~100: «True» | ||||||||||||||||||||||||||||||||||||||
| Zoffix | The Version.new-from-git-describe doesn't split the version correctly. Also, that really shouldn't have been a public method, but rather something stuffed into Rakudo::Internals | 21:30 | |||||||||||||||||||||||||||||||||||||
| Based on the commit, it was added only because we can't use regexes early enough where it's needed. I'm gonna rewrite Version.new without regexes and nix .new-from-git-describe | 21:36 |