Zoffix | for crying out loud... "No such method 'slurp-rest' for invocant of type 'Any' in sub at /home/zoffix/services/geth/lib/Geth/GitHub/Hooks.pm6" | 00:12 | |
Why did I restart it :() | |||
Geth_ | roast: bcaaad4d42 | (Zoffix Znet)++ | 2 files Consolidate bug cover tests |
00:16 | |
Zoffix | I see. If you precompile modules with FileSystem CUR and then uninstall a module with zef, then the precomp files still remain untouched, using whatever version was installed | ||
Geth_ | rakudo/nom: f72be0f130 | (Zoffix Znet)++ | t/spectest.data Remove dir-stress test file It was consolidated with other lone bug cover stresstests in github.com/perl6/roast/commit/bcaaad4d42 |
00:17 | |
Zoffix | m: use Test; for (NaN..NaN) => NaN, (Inf..0) => Inf, (-Inf..0) => -Inf -> (:key($range), :value($expected)) { .map: { next unless $++ > 1050; is-deeply $_, $expected, "$_ range keeps producing $expected ad infinitum"; last } } | 00:45 | |
camelia | ( no output ) | ||
Zoffix | dahell? | ||
Ah, missing $range | 00:46 | ||
Geth_ | roast: bfc143e0ac | (Zoffix Znet)++ | S02-types/range.t no floating point drifts in degenerate Ranges RT#125336: rt.perl.org/Ticket/Display.html?id=125336 |
00:51 | |
synopsebot | RT#125336 [new]: rt.perl.org/Ticket/Display.html?id=125336 Weird infinite ranges: Inf..0, -Inf..0 and NaN..0 | ||
AlexDaniel- | Zoffix++ | 00:59 | |
\o/ | 02:37 | ||
OK I think I got regressionable working | 02:38 | ||
bisect: ("foo" => 10000000000000000000).BagHash.grab(1) | 04:51 | ||
bisectable6 | AlexDaniel-, Bisecting by exit code (old=2015.12 new=f72be0f). Old exit code: 0 | ||
AlexDaniel- | I think somebody mentioned this a few days ago | ||
bisectable6 | AlexDaniel-, bisect log: gist.github.com/bf8346742ef61f4d59...0077131430 | ||
AlexDaniel-, (2017-05-11) github.com/rakudo/rakudo/commit/87...c382797fc9 | |||
AlexDaniel- | aw yeah, that's exactly how I found out about it | 04:52 | |
haā¦ almost submitted a ticket asking to unfix a bug | 06:11 | ||
.tell Zoffix should this fail like it does now? ļ½¢-> (*%_) {dd %_}(Failure.new)ļ½£ gist.github.com/41a100146ec62ece32...33060faf00 | 06:14 | ||
yoleaux | AlexDaniel-: I'll pass your message to Zoffix. | ||
AlexDaniel- | .tell lizmat this does not look right, but the commit seems to be innocent. What do you think? Let me know if I have to rakudobug it gist.github.com/AlexDaniel/de2c5f1...ca10c82f50 | 06:23 | |
yoleaux | AlexDaniel-: I'll pass your message to lizmat. | ||
AlexDaniel- | c: releases Blob[num32].new(2e0) | 06:24 | |
committable6 | AlexDaniel-, gist.github.com/3a8ab6ac823d9b2549...a99934471d | ||
AlexDaniel- | bisect: old=2017.06 Blob[num32].new(2e0) | 06:25 | |
bisectable6 | AlexDaniel-, Bisecting by output (old=2017.06 new=f72be0f) because on both starting points the exit code is 1 | ||
AlexDaniel-, bisect log: gist.github.com/41e9da04c8065b7888...534a3e98fb | |||
AlexDaniel-, (2017-06-26) github.com/rakudo/rakudo/commit/82...3ed693f9cb | |||
AlexDaniel- | c: 823011281^,823011281 Blob[num32].new(2e0) | 06:27 | |
committable6 | AlexDaniel-, gist.github.com/72544ca121a3ff6c7f...78b8885689 | ||
AlexDaniel- | bisect: old=823011281 Blob[num32].new(2e0) | 06:28 | |
bisectable6 | AlexDaniel-, Bisecting by output (old=8230112 new=f72be0f) because on both starting points the exit code is 1 | ||
AlexDaniel-, bisect log: gist.github.com/9dcb237e77570f9dcc...8fa6b963ec | |||
AlexDaniel-, (2017-07-08) github.com/rakudo/rakudo/commit/1a...2246c4c26c | |||
AlexDaniel- | c: 1a4d94930^,1a4d94930 Blob[num32].new(2e0) | ||
committable6 | AlexDaniel-, gist.github.com/70e94426796265d621...e0c1df36b1 | ||
AlexDaniel- | actually I thought that I'm on #whateverable | ||
so nevermind this spam | |||
.tell MasterDuke fwiw RT #132285 | 06:31 | ||
yoleaux | AlexDaniel-: I'll pass your message to MasterDuke. | ||
synopsebot | RT#132285 [new]: rt.perl.org/Ticket/Display.html?id=132285 [REGRESSION] Error message talks about null strings when trying to create a num32 Blob (Blob[num32].new(2e0)) | ||
AlexDaniel- | by the way, why the numbers suddenly became bigger? | 06:36 | |
mch: my $x; my $y := $x; $x = (42,$y); say $x | |||
committable6 | AlexDaniel-, Ā¦2015.12: Ā«(\List_82832240 = (42 List_82832240))Ā» Ā¦HEAD(f72be0f): Ā«(\List_93951630447504 = (42 List_93951630447504))Ā» | ||
moritz | Zoffix: I think I can connect to the box on which geth runs... now what? | 06:37 | |
AlexDaniel- | .tell Zoffix also maybe see irclog.perlgeek.de/perl6/2017-10-03#i_15250701 which does not work anymore | 06:44 | |
yoleaux | AlexDaniel-: I'll pass your message to Zoffix. | ||
jnthn | AlexDaniel: Those numbers are just memory addresses (or, likely, future memory addresses), which tend to be unpredictable by design. | 08:58 | |
Zoffix | moritz: Dunno. I can connect too, but thought maybe you know more about it. For some reason connections to it are blocked. `telnet geth.niner.name 8888` fails for example | 09:12 | |
yoleaux | 06:14Z <AlexDaniel-> Zoffix: should this fail like it does now? ļ½¢-> (*%_) {dd %_}(Failure.new)ļ½£ gist.github.com/41a100146ec62ece32...33060faf00 | ||
06:44Z <AlexDaniel-> Zoffix: also maybe see irclog.perlgeek.de/perl6/2017-10-03#i_15250701 which does not work anymore | |||
Zoffix | AlexDaniel: yes and yes | 09:13 | |
First one documented in docs.perl6.org/type/Failure#method_Capture second on in docs.perl6.org/type/Str#method_Capture | 09:19 | ||
m: try ļ½¢sub c(:&funk! (:(:$param))) { &funk(:parrrrrrrrrrrrrrrrrrrrrrrrrrrrram('finally. Zoffix++')) }; c( funk => {say $:parrrrrrrrrrrrrrrrrrrrrrrrrrrrram} );ļ½£.EVAL; say $!.^name | |||
camelia | X::Cannot::Capture | ||
Zoffix | yup | 09:20 | |
AlexDaniel: heh: irclog.perlgeek.de/perl6/2017-10-13#i_15297137 | 09:22 | ||
That's one of the bot evals | |||
Geth_ | roast: 11fb4415f0 | (Zoffix Znet)++ | S32-io/open.t .open error does not incorrectly complain about malformed UTF-8 RT#125813: rt.perl.org/Ticket/Display.html?id=125813 |
09:30 | |
synopsebot | RT#125813 [open]: rt.perl.org/Ticket/Display.html?id=125813 [UNI] Malformed UTF-8 (string out of bounds) with āsay ('a' x 10000).IO.openā | ||
Geth_ | roast: c8ebdbbf89 | (Zoffix Znet)++ | S03-smartmatch/any-num.t can smartmatch num64 against Num RT#126927: rt.perl.org/Ticket/Display.html?id=126927 |
09:35 | |
synopsebot | RT#126927 [open]: rt.perl.org/Ticket/Display.html?id=126927 [BUG] Spurious warning when trying to smartmatch num64 against Num in Rakudo | ||
Geth_ | roast: 809abcd5a7 | (Zoffix Znet)++ | S06-advanced/return.t Can return from parameter defaults RT#122345: rt.perl.org/Ticket/Display.html?id=122345 |
09:39 | |
synopsebot | RT#122345 [open]: rt.perl.org/Ticket/Display.html?id=122345 [TESTNEEDED] Returning from a parameter list default counts as returning 'outside of any Routine' in Rakudo on Moar | ||
Geth_ | roast: e0d3c550f1 | (Zoffix Znet)++ | S32-io/open.t Make test more specific |
09:45 | |
[TuxCM] | This is Rakudo version 2017.09-383-gf72be0f13 built on MoarVM version 2017.09.1-594-gb9d3f6da | 09:50 | |
csv-ip5xs 1.154 - 1.182 | |||
test 9.424 - 9.481 | |||
test-t 3.094 - 3.156 | |||
csv-parser 0.837 - 0.862 | |||
Geth_ | rakudo/nom: 969853c673 | (Zoffix Znet)++ | t/05-messages/01-errors.t Use actual test plan |
10:03 | |
Zoffix | Actually, startup boost likely had a much better improvement to spectest/stresstest, since we have a ton of `is_run` tests that fire up an $*EXECUTABLE | 10:05 | |
timotimo | oh, neat :) | 10:10 | |
Geth_ | rakudo/nom: fb7abf064e | (Zoffix Znet)++ | t/05-messages/01-errors.t Test sane error when missing variables with my and where RT#125902: rt.perl.org/Public/Bug/Display.html?id=125902 |
10:15 | |
synopsebot | RT#125902 [open]: rt.perl.org/Ticket/Display.html?id=125902 [LTA] error message āType 'Str' is not declaredā (my Str where 'foo' $test;) | ||
Geth_ | rakudo/nom: 975fcf6cfd | (Elizabeth Mattijsen)++ | src/core/Rakudo/QuantHash.pm Fix for RT #132279 Cannot be sure we have int64 range on BagHash |
11:25 | |
synopsebot | RT#132279 [new]: rt.perl.org/Ticket/Display.html?id=132279 [REGRESSION][9999] BagHash.grab with huge values (("foo" => 10000000000000000000).BagHash.grab(1)) | ||
AlexDaniel | Zoffix: ok | 11:36 | |
and yeah, I knew this would've happened (bot evals) | 11:37 | ||
maybe I should've put some irc keywords into the blacklist, hmmā¦ | 11:39 | ||
ā13721 snippets tested, 3983 are known to be unstableā | 11:40 | ||
ZofBot: when are you going to be smart enough to do this work? | |||
ZofBot | AlexDaniel, IO::Spec subclasses The following methods should be provided by the "IO::Spec" subclasses, or may be inherited from another class | ||
AlexDaniel | ohā¦ right. it printed āHi, blubā 5 times because it tests every snippet on 5 revisionsā¦ | 11:44 | |
Zoffix | imma move Geth to hack tonight | ||
And update all the hooks in perl6 org to use that | |||
AlexDaniel | and it does it in random order so we're going to see more of these soon I think | ||
Zoffix | "NotZoffix joined #perl6" | 11:58 | |
Zoffix squints | |||
ZofBot: EXTERMINATE! | |||
ZofBot | Zoffix, html?id=132128 Link: rt | ||
Zoffix | buggable: christmas | 12:10 | |
buggable | Zoffix, Christmas is in 10 weeks, 2 days, 11 hours, 49 minutes, and 22 seconds | ||
Zoffix | \o/ | ||
Geth_ | roast: bitrauser++ created pull request #347: Add test for RT#131490 | Crash in Junction:D.BUILDALL `This type (Scalar) does not support elems` |
12:17 | |
synopsebot | RT#131490 [open]: rt.perl.org/Ticket/Display.html?id=131490 [TESTNEEDED] Crash in Junction:D.BUILDALL `This type (Scalar) does not support elems` | ||
Geth_ | roast: 5ebf6f9303 | Joel++ (committed by Zoffix Znet) | S03-junctions/misc.t Add test for RT#131490 (#347) |
12:19 | |
roast: 1b0d85a39c | (Zoffix Znet)++ (committed using GitHub Web editor) | S03-junctions/misc.t Fix up test description |
12:21 | ||
Zoffix | Hm, it was fixed in github.com/rakudo/rakudo/commit/61...307fa94da6 so presumably test already existed :/ | ||
ah nope. Tests were added but they don't catch the bug in the ticket: github.com/perl6/roast/blob/master...#L526-L530 | 12:23 | ||
Geth_ | roast: c2788cce70 | (Zoffix Znet)++ (committed using GitHub Web editor) | S03-junctions/misc.t Consolidate related tests |
12:24 | |
MasterDuke | arg, how do i get the message out of a BOOTException? | 12:53 | |
Zoffix | nqp::getmessage or nqp::getpayload maybe? Some ops are listed here: github.com/perl6/nqp/blob/master/d...al-opcodes | 12:54 | |
MasterDuke | getmessage doesn't work, but maybe i haven't tried getpayload | 12:55 | |
Geth_ | rakudo: MasterDuke17++ created pull request #1189: Correctly explain why couldn't instantiate a role |
13:30 | |
rakudo/nom: b3bb8c406a | MasterDuke17++ | src/Perl6/Metamodel/ParametricRoleHOW.nqp Correctly explain why couldn't instantiate a role This broke when Exceptions were changed to not create their message unless .gist'ed. Fixes RT #132285. |
13:31 | ||
synopsebot | RT#132285 [new]: rt.perl.org/Ticket/Display.html?id=132285 [REGRESSION] Error message talks about null strings when trying to create a num32 Blob (Blob[num32].new(2e0)) | ||
Geth_ | rakudo/nom: f4f36915f5 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/Perl6/Metamodel/ParametricRoleHOW.nqp Merge pull request #1189 from MasterDuke17/RT132285 Correctly explain why couldn't instantiate a role |
||
Zoffix grumbles about using RT for dick measuring contests | 13:45 | ||
.ask AlexDaniel what do you think of this ticket? rt.perl.org/Ticket/Display.html?id...xn-1493785 The problem is swept under the rug and it the cause appears to be just that we shuttle a capture around from Unicode ops to ASCII ones, so that issue will be resolved on its own. So should the ticket be closed? What about tests? | 13:46 | ||
yoleaux | Zoffix: I'll pass your message to AlexDaniel. | ||
Zoffix | *people using | ||
MasterDuke | i'd almost think we should have spectest, stresstest, and perftest | 13:47 | |
and perftest would get things like that | 13:48 | ||
Zoffix | Yeah. Definitely | ||
.ask AlexDaniel also this one can prolly be closed entirely? rt.perl.org/Ticket/Display.html?id...et-history | |||
yoleaux | Zoffix: I'll pass your message to AlexDaniel. | ||
MasterDuke | are those moarvm benchmarks still generated daily? | ||
Zoffix shrugs | |||
oh oops. Modules.perl6.org hasn't updated since oct 9 | 14:04 | ||
yoleaux | Zoffix: make separate t/spectest.data for each language version, bruh | 14:06 | |
Zoffix | yoleaux: dun it long time ago, bruh | ||
modules.perl6.org@www:/tmp$ ls -1 | wc -l | 14:07 | ||
3339 | |||
yu | |||
Looks like File::Temp doesn't delete the temp dirs :/ | |||
oh hahaha | 14:12 | ||
RE: irclog.perlgeek.de/perl6/2017-10-09#i_15276445 | 14:13 | ||
I have the answer: it'll break the entire build :P | |||
.tell ugexe this may have been a misnamed upload for zef? Dunno if fixed version was uploaded already. modules.perl6.org will ignore it due to lack of name in the filename itself that it uses to figure this stuff out: irclog.perlgeek.de/perl6/2017-10-09#i_15276439 | 14:15 | ||
yoleaux | Zoffix: I'll pass your message to ugexe. | ||
Zoffix | And yeah, theoretically it could figure it out from the .meta file, I guess, but then this kinda makes the build process depend on processing all the dists of the user before knowing which dists are what. | 14:17 | |
Didn't consider this during making of original version of the CPAN dist source | |||
(this = that filenames could be different than what the dist is) | 14:18 | ||
ugexe | it kind has to eventually, because dists can have dashes and version like strings in their name | 14:22 | |
yoleaux | 14:15Z <Zoffix> ugexe: this may have been a misnamed upload for zef? Dunno if fixed version was uploaded already. modules.perl6.org will ignore it due to lack of name in the filename itself that it uses to figure this stuff out: irclog.perlgeek.de/perl6/2017-10-09#i_15276439 | ||
ugexe | Eg cant reliably decode module name + version from a file name alone | 14:24 | |
Zoffix | Right now it treats `-([^-]+)$` as version part (sans file extension). So basically everything after last dash | ||
m: say Version.new: "42-x" | 14:25 | ||
camelia | v42.x | ||
Zoffix scratches head | 14:26 | ||
MasterDuke | Zoffix: whoops, completely missed that you commented on 132285 | ||
Zoffix | m: say Version.new: "-" | ||
camelia | v | ||
Zoffix | m: Version.new("-").perl.say | ||
camelia | Version.new | ||
ugexe | App::ecogen handles it ok I believe | 14:30 | |
Zoffix | MasterDuke: "Not sure where tests should go, roast or rakudo?" It'd go to rakudo | ||
MasterDuke: if the new version threw some useful typed exception, then it could go to roast, but I see both buggy and non-buggy versions throw X::AdHoc and the non-buggy says it's NYI, so there's nothing really we can spec to be language behaviour, so the tests to cover the content of error would go to t/05-messages/errors.t instead of the roast | 14:31 | ||
MasterDuke | Zoffix: and then we're fine removing those tests if/when the NYI behavior is implemented? | 14:32 | |
Zoffix | Right, t/* tests can be modified whenever we need to. | 14:33 | |
MasterDuke | or should the test be something like: if exception was thrown, it has "NYI" and "num32" in it? | ||
Zoffix | Yeah, prolly | ||
c: 2017.09,HEAD use Test; throws-like ļ½¢Blob[num32].new: 2e0ļ½£, Exception, :message{.contains: ļ½¢not yet implementedļ½£ & ļ½¢num32ļ½£ and not .contains: ļ½¢got nullļ½£}, 'sane error for num32 Blob' | 14:35 | ||
committable6 | Zoffix, gist.github.com/80672fba1cba63a1cb...ff053488b4 | ||
Zoffix & | 14:37 | ||
Geth_ | rakudo: MasterDuke17++ created pull request #1190: Test RT132285, correct error for Blob with num32 |
14:41 | |
synopsebot | RT#132285 [open]: rt.perl.org/Ticket/Display.html?id=132285 [REGRESSION] Error message talks about null strings when trying to create a num32 Blob (Blob[num32].new(2e0)) | ||
Geth_ | rakudo/nom: e611978f54 | MasterDuke17++ | t/05-messages/01-errors.t Test RT132285, correct error for Blob with num32 |
14:45 | |
rakudo/nom: e475ae8e05 | (Zoffix Znet)++ (committed using GitHub Web editor) | t/05-messages/01-errors.t Merge pull request #1190 from MasterDuke17/test_for_RT132285 Test RT132285, correct error for Blob with num32 |
|||
MasterDuke | Zoffix++ | 14:53 | |
Zoffix | .tell tyil the "my App::MPD::AutoQueue still appears twice, even though I removed it from the META.list" was now fixed. If CPAN modules are missing from mp6o while there generated PAUSE .meta files for them, let me know. If PAUSE isn't generating .meta for some dists. Try to track down PAUSE people and ask them why | 15:42 | |
yoleaux | Zoffix: I'll pass your message to tyil. | ||
Zoffix received a sad private message about RT | 15:59 | ||
Would be nice if new comers were | 16:00 | ||
Would be nice if new comers were not ridiculed/attacked in ticket responses. You don't have to be Einstein to file a ticket and if there's some misconception about user's ticket, just clarify it, if the clarification is needed to resolve the ticket, or just leave it alone. Writing miles of text arguing who is most right ain't helpingāespecially ain't helping the person who'll be fixing the ticket later | 16:01 | ||
on and has to read through miles of pointless dialogue | |||
(I'm not an angel myself; but we can all aim towards some ideal) | 16:02 | ||
ZofBot: so when will the robots kill all humans? | 16:04 | ||
ZofBot | Zoffix, ^name}"; } } X::Range::InvalidArg Thrown when an argument to "Range | ||
Zoffix | :( I need to reset your brain. | 16:05 | |
ZofBot: aho matey | 16:10 | ||
ZofBot | Zoffix, Think you, but that I know our state secure, I would be so triumphant as I am? STANLEY | ||
Zoffix | Ah, I love robots. | ||
Seems we need some `make extratest` that runs extra rakudo-only tests that aren't spec. Like tests for this ticket would need installation fo the debugger module, for example: rt.perl.org/Ticket/Display.html?id...et-history | 16:29 | ||
Also, I imagine the bug-cover error message tests isn't really something end users need to run, so that can be shoved to make extratest too. Make it run on travis | |||
also need `make dangeroustest` that will run dangerous roast tests. | 16:30 | ||
s:g/'make extratest'/make xtest/; | |||
And stuff like RT#127671 would go to dangeroustest | 16:32 | ||
synopsebot | RT#127671 [open]: rt.perl.org/Ticket/Display.html?id=127671 [EXOTICTEST] ļ½¢dirļ½£ dies if weird unicode sequences are encountered (dir;) | ||
Zoffix | That'd be run only by people using some destructable VM and stuff | ||
Geth_ | rakudo/nom: 4473591b0a | (Zoffix Znet)++ | t/05-messages/01-errors.t Add NYI to test description So that when it is implemented the failure of the test would be more informative as to why it's failing. |
16:53 | |
rakudo/nom: 80cee36223 | (Zoffix Znet)++ | t/05-messages/01-errors.t Test callframe.my throws sane NYI error message RT#77754: rt.perl.org/Ticket/Display.html?id=77754 |
16:55 | ||
synopsebot | RT#77754 [open]: rt.perl.org/Ticket/Display.html?id=77754 [BUG] Null PMC access when calling .perl on callframe.my in Rakudo | ||
buggable | New CPAN upload: Propius-0.1.0.tar.gz by ATROXAPER cpan.metacpan.org/authors/id/A/AT/...1.0.tar.gz | 17:10 | |
New CPAN upload: TimeUnit-0.1.2.tar.gz by ATROXAPER cpan.metacpan.org/authors/id/A/AT/...1.2.tar.gz | |||
Geth_ | roast: 522e3da803 | (Zoffix Znet)++ | S02-types/mix.t Test Mix.roll with fractional weights does not crash RT#131855: rt.perl.org/Ticket/Display.html?id=131855 |
17:15 | |
synopsebot | RT#131855 [open]: rt.perl.org/Ticket/Display.html?id=131855 [REGRESSION] Mix.roll doesn't work with fractional weights | ||
travis-ci | Rakudo build failed. Zoffix Znet 'Merge pull request #1190 from MasterDuke17/test_for_RT132285 | 17:28 | |
travis-ci.org/rakudo/rakudo/builds/287594754 github.com/rakudo/rakudo/compare/f...75ae8e0542 | |||
buggable | [travis build above] ā All failures are due to: GitHub connectivity (1 failure). | ||
Geth_ | roast: 2a45053f2b | (Zoffix Znet)++ | S17-procasync/kill.t Test Proc::Async.kill doesn't hang If killing immediately after .start RT#125653: rt.perl.org/Ticket/Display.html?id=125653 |
17:30 | |
synopsebot | RT#125653 [new]: rt.perl.org/Ticket/Display.html?id=125653 [CONC] A promise returned by Proc::Async.start will never be kept or broken if the process is killed by the current process | ||
Geth_ | roast: 7f52f7a4e4 | (Zoffix Znet)++ | S17-procasync/kill.t Simplify tests - Use a temp file routine instead of rolling the feature manually - Use a heredoc instead of poorly idented string |
17:37 | |
Zoffix | RT#132258 is making me cry. Not the security part, but the more I research the issue the more it seems there's no sane way to pass args on Windows if you don't know what you're passing them to. | 17:59 | |
synopsebot | RT#132258 [new]: rt.perl.org/Ticket/Display.html?id=132258 [SECURITY][WINDOWS] `run "perl6" ...` can be made to execute shell commands | ||
Zoffix | And we have a bunch of Proc::Async.new: $*EXECUTABLE in roast, which basically will remain LTA, as even with some :cmd-windows-quoting arg to Proc::Async, the tests will work only for implementatinos that use a batch file for perl6 like we do and will break for those that don't :| | 18:00 | |
Geth_ | rakudo/nom: 3acde358c1 | (Samantha McVey)++ | src/core/Int.pm Show hex value as well as dec in char() error message When char() is requested to too large a value, also show the hex value and not just the decimal one. |
18:19 | |
rakudo/nom: 7bea3a2ddd | (Samantha McVey)++ | src/core/IO/Path.pm Add .parent(Int) for getting up more than one level .parent only gets the path one level up. Now .parent(Int) will get the nth parent of the requested directory. This is different than adding ../ to the path as when there are links it acts differently. In addition: `Path::IO.add('../').absolute ne Path::IO.parent.absolute` |
|||
Zoffix | FWIW we have UInt type for non-negative Ints | 18:30 | |
ugexe | almost feels like parent should do cleanup | 18:35 | |
[Coke] | Zoffix++ #UInt | 18:40 | |
ugexe: isn't that add? | |||
ugexe | add() adds a path part to a path, parent(1) returns the path with 1 path part removed | 18:41 | |
samcv | should I change it to UInt? | 18:42 | |
ugexe | e.g. "foo/../bar".IO.parent does the programmer usually want "foo/" or "foo/../" | 18:43 | |
foo/../ or "" whatever | |||
samcv | ugexe, depends on if they want to remove part of the path or not. ../ will follow links | 18:44 | |
most of the time people also don't want to see /foo/bar/baz/raz/this/../../../../../ | 18:45 | ||
but if you want to follow links on the way one level up, then you should use ../ | |||
Zoffix | samcv: IMO yeah. It'd unify the error with the rest of our errors rather than having an Ad::Hoc with a custom message (that can't really be specced properly) and moves the logic defining acceptable params into signature, so it can be inspected, for example with .cando() | 18:46 | |
samcv | ok | ||
don't need a full error message. just need a more generic error than negative .parent's make no sense :P | |||
Zoffix | m: sub (Int) {}.cando(\(-42)).so.say; sub (UInt) {}.cando(\(-42)).so.say | 18:47 | |
camelia | True False |
||
samcv | cool :) | ||
Zoffix | Wonder why Block don't got a .cando | ||
Zoffix & | |||
Geth_ | rakudo: dwarring++ created pull request #1191: optimize recently added AT-POS for typed pointers |
20:12 | |
rakudo/nom: bc5fbfcbf5 | (David Warring)++ | lib/NativeCall/Types.pm6 optimize recently added AT-POS for typed pointers inline the add method and subsequent calls to nativecast and Int and add methods. Improves performance by 4-5x. |
21:41 | ||
rakudo/nom: a54da11aa6 | (Zoffix Znet)++ (committed using GitHub Web editor) | lib/NativeCall/Types.pm6 Merge pull request #1191 from dwarring/pointer-optimize optimize recently added AT-POS for typed pointers |
|||
tyil | Zoffix: alright, thanks | 22:00 | |
yoleaux | 15:42Z <Zoffix> tyil: the "my App::MPD::AutoQueue still appears twice, even though I removed it from the META.list" was now fixed. If CPAN modules are missing from mp6o while there generated PAUSE .meta files for them, let me know. If PAUSE isn't generating .meta for some dists. Try to track down PAUSE people and ask them why | ||
Zoffix | :) | 22:01 | |
Geth_ | rakudo/nom: 012c80f336 | (Nick Logan)++ (committed using GitHub Web editor) | src/core/ThreadPoolScheduler.pm Remove unused variable |
22:06 | |
timotimo | my disappearance (of json::fast) from the ecosystem made JSON::Tiny get the #1 spot on the module citation site :S | 22:27 | |
travis-ci | Rakudo build passed. Samantha McVey 'Add .parent(Int) for getting up more than one level | 22:35 | |
travis-ci.org/rakudo/rakudo/builds/287676452 github.com/rakudo/rakudo/compare/8...ea3a2dddcf | 22:36 | ||
Geth_ | roast: 2800632eb0 | (Zoffix Znet)++ | S02-types/baghash.t Can .grab() a BagHash key with weight larger than 64 bits RT#132279: rt.perl.org/Ticket/Display.html?id=132279 |
22:53 | |
synopsebot | RT#132279 [open]: rt.perl.org/Ticket/Display.html?id=132279 [REGRESSION][9999] BagHash.grab with huge values (("foo" => 10000000000000000000).BagHash.grab(1)) | ||
Geth_ | rakudo: MasterDuke17++ created pull request #1192: Convert INTERPOLATE into multis |
23:42 |