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 |