Kaiepi | BeastieBot, build | 01:04 | |||||||||||||||||||||||||||||||||||||
BeastieBot, help | |||||||||||||||||||||||||||||||||||||||
PufferBot, help | 01:05 | ||||||||||||||||||||||||||||||||||||||
oh forgot to actually use the rakudo plugin with irc::client after the huge refactor | 01:06 | ||||||||||||||||||||||||||||||||||||||
lol | |||||||||||||||||||||||||||||||||||||||
BeastieBot, help | 01:07 | ||||||||||||||||||||||||||||||||||||||
BeastieBot | Kaiepi, address me with "build", "test", or "spectest" to test building Rakudo, running tests, and running Roast's suite respectively on freebsd. Address me with "all" to attempt to run all three sequentially. | ||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, build | 01:10 | |||||||||||||||||||||||||||||||||||||
BeastieBot, help | |||||||||||||||||||||||||||||||||||||||
BeastieBot | Kaiepi, address me with "build", "test", or "spectest" to test building Rakudo, running tests, and running Roast's suite respectively on freebsd. Address me with "all" to attempt to run all three sequentially. | ||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, test | 01:11 | |||||||||||||||||||||||||||||||||||||
odd | 01:12 | ||||||||||||||||||||||||||||||||||||||
PufferBot, build | |||||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: were those returns always there? | 01:59 | |||||||||||||||||||||||||||||||||||||
yoleaux | 15 Mar 2018 20:22Z <cfa> Zoffix: /perl6/roast/issues/402 | ||||||||||||||||||||||||||||||||||||||
Zoffix | m: sub foo { my $p = start { return 42 }; await $p; $p }; say foo | ||||||||||||||||||||||||||||||||||||||
camelia | MoarVM panic: Internal error: Unwound entire stack and missed handler | ||||||||||||||||||||||||||||||||||||||
Zoffix | I don't think you can return from a method from within a promise and if you meant to return from the .protected bit only, then you need to make that a sub, not a block. | 02:00 | |||||||||||||||||||||||||||||||||||||
Also, what's sending output to IRC? | |||||||||||||||||||||||||||||||||||||||
Kaiepi | the methods within the promises | ||||||||||||||||||||||||||||||||||||||
Zoffix | FWIW returning a Promise from IRC::Client method sends return value to IRC when promise completes | ||||||||||||||||||||||||||||||||||||||
Kaiepi | is there a way to make it so it doesn't address the user that called the command when you return a string from the method? | 02:02 | |||||||||||||||||||||||||||||||||||||
Zoffix | Ah. No | ||||||||||||||||||||||||||||||||||||||
BeastieBot, source | 02:03 | ||||||||||||||||||||||||||||||||||||||
BeastieBot, help | |||||||||||||||||||||||||||||||||||||||
Where's the full source at? | |||||||||||||||||||||||||||||||||||||||
Kaiepi | github.com/Kaiepi/p6-RakudoBot/blo...Rakudo.pm6 | ||||||||||||||||||||||||||||||||||||||
Zoffix | "No such method 'irc' for invocant of type 'IRC::Client::Plugin::Rakudo'" | 02:09 | |||||||||||||||||||||||||||||||||||||
"Attempt to return outside of immediatelly-enclosing Routine (i.e. `return` execution is outside the dynamic scope of the Routine where `return` was used)" | 02:10 | ||||||||||||||||||||||||||||||||||||||
Kaiepi | hm | ||||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: so you need to add ` does IRC::Client::Plugin;` to the class IRC::Client::Plugin::Rakudo line and, as I mentioned, change those blocks given to .protect() into sub {} | ||||||||||||||||||||||||||||||||||||||
Kaiepi | ah | 02:11 | |||||||||||||||||||||||||||||||||||||
gimme a sec, i gotta set up my vim plugins on my vps before i can test | |||||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: and I found that out by sticking CATCH { default { say "Error: $_" } } inside those `start` blocks. Also, IRC::Client.new can take a `:debug` flag that prints what it's doing (you can also install Terminal::ANSIColor to get preety colors) | 02:12 | |||||||||||||||||||||||||||||||||||||
Kaiepi | yeah, i'm adding a debug option to config.json for that | 02:13 | |||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: here's the modifications I made. Seems to do things now and no errors show up in `build` routine. Still need to change blocks to subs in all the other protected blocks that try to return: gist.github.com/zoffixznet/3453f53...b3498b55a4 | ||||||||||||||||||||||||||||||||||||||
Zoffix & | |||||||||||||||||||||||||||||||||||||||
Kaiepi | thanks | 02:14 | |||||||||||||||||||||||||||||||||||||
PufferBot, build | 02:26 | ||||||||||||||||||||||||||||||||||||||
PufferBot, source | 03:17 | ||||||||||||||||||||||||||||||||||||||
PufferBot | Kaiepi, github.com/Kaiepi/p6-RakudoBot.git | ||||||||||||||||||||||||||||||||||||||
Kaiepi | PufferBot, build | 03:18 | |||||||||||||||||||||||||||||||||||||
BeastieBot, test | 03:51 | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Testing Rakudo... | ||||||||||||||||||||||||||||||||||||||
[freebsd] | t/04-nativecall/02-simple-args.t (Wstat: 256 Tests: 14 Failed: 1) | 03:53 | ||||||||||||||||||||||||||||||||||||||
[freebsd] Tests failed. See the output at fpaste.scsys.co.uk/575920 | |||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: Kaiepi++ created pull request #1620: Skip NativeCall TakeUint16 test on FreeBSD |
06:53 | |||||||||||||||||||||||||||||||||||||
nine_ | .tell jnthn On the issue of `composalizable` and `inheritalizable` I'm firmly in the "brilliant use of English derivational morphology" camp. But then, I'm native German speaker, so I'm used to torturing words ;) | 06:58 | |||||||||||||||||||||||||||||||||||||
yoleaux | nine_: I'll pass your message to jnthn. | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 51c4d4d89c | (Stefan Seifert)++ | lib/NativeCall.pm6 Fix precomp files of NativeCall users containing absolute paths By storing the compiler object in a variable scoped to the Native role, it gets serialized into precomp files. This compiler object contains a lot of information including the path to the source file of the currently compiling module (the NativeCall user) and the path of the target bytecode file. ... (5 more lines) |
08:52 | |||||||||||||||||||||||||||||||||||||
nqp: 00b1f84f35 | (Stefan Seifert)++ | src/QRegex/NFA.nqp Remove partially commented out timing debug code Since this code went in in 2014 we have gained a profiler, rendering custom timing code pretty much obsolete. The global variable initialized to the current time however prevents reproducible builds. |
09:31 | ||||||||||||||||||||||||||||||||||||||
[TuxCM] |
|
09:57 | |||||||||||||||||||||||||||||||||||||
nine_ | .tell jnthn What do you thing about this patch giving us reproducible nqp builds? With it a simple make clean && make install without changes won't break an installed rakudo anymore. gist.github.com/niner/c82676810ce4...e0ef086ef6 | 10:47 | |||||||||||||||||||||||||||||||||||||
yoleaux | nine_: I'll pass your message to jnthn. | ||||||||||||||||||||||||||||||||||||||
lizmat | Files=1236, Tests=76268, 315 wallclock secs (15.01 usr 5.36 sys + 2170.44 cusr 215.50 csys = 2406.31 CPU) | 11:32 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | PufferBot: all | 11:53 | |||||||||||||||||||||||||||||||||||||
BeastieBot: all | |||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] The current branch has uncommitted changes. Please tell Kaiepi to commit or reset any changes made before running your command again. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel, done! | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | Kaiepi: please commit or reset any changes | ||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, all | 11:54 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Configuring Rakudo... | ||||||||||||||||||||||||||||||||||||||
[freebsd] Building Rakudo... | 11:55 | ||||||||||||||||||||||||||||||||||||||
[freebsd] Build failed. See the output at fpaste.scsys.co.uk/575922 | 11:58 | ||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, build | 11:59 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Configuring Rakudo... | ||||||||||||||||||||||||||||||||||||||
[freebsd] Building Rakudo... | |||||||||||||||||||||||||||||||||||||||
Kaiepi | PufferBot, all | 12:00 | |||||||||||||||||||||||||||||||||||||
PufferBot, build | 12:01 | ||||||||||||||||||||||||||||||||||||||
ok weird, it should be using the same code beastiebot is | |||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Build failed. See the output at fpaste.scsys.co.uk/575924 | 12:15 | |||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, build | 13:01 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Configuring Rakudo... | ||||||||||||||||||||||||||||||||||||||
[freebsd] Building Rakudo... | |||||||||||||||||||||||||||||||||||||||
Kaiepi | PufferBot, all | 13:03 | |||||||||||||||||||||||||||||||||||||
PufferBot | [openbsd] Configuring Rakudo... | ||||||||||||||||||||||||||||||||||||||
[openbsd] Building Rakudo... | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | Kaiepi: so what's up with these failures? | ||||||||||||||||||||||||||||||||||||||
does it mean that rakudo is broken? | 13:04 | ||||||||||||||||||||||||||||||||||||||
how do I interpret this result correctly? | |||||||||||||||||||||||||||||||||||||||
nine | I'd guess it's running out of memory | ||||||||||||||||||||||||||||||||||||||
yoleaux | 12:13Z <jnthn> nine: Goodness, that's a tiny patch to achieve such a thing! Can't think of an objection :) | ||||||||||||||||||||||||||||||||||||||
Kaiepi | the failures don't seem to happen when i run make manually | 13:05 | |||||||||||||||||||||||||||||||||||||
PufferBot | [openbsd] Build successful! | ||||||||||||||||||||||||||||||||||||||
[openbsd] Testing Rakudo... | |||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Build failed. See the output at fpaste.scsys.co.uk/575932 | ||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
PufferBot | [openbsd] Tests passed! | 13:06 | |||||||||||||||||||||||||||||||||||||
[openbsd] Running Roast test suite... (this will take a while) | |||||||||||||||||||||||||||||||||||||||
Kaiepi | i'm either running out of memory or running out of swap space | 13:08 | |||||||||||||||||||||||||||||||||||||
pid 32318 (moar), uid 1001, was killed: out of swap space | 13:09 | ||||||||||||||||||||||||||||||||||||||
nine | AlexDaniel: I guess I should only commit gist.github.com/niner/c82676810ce4...e0ef086ef6 after the release? | 13:10 | |||||||||||||||||||||||||||||||||||||
Zoffix | AlexDaniel: commited some changes to changelog: github.com/rakudo/rakudo/wiki/Chan...5562c0d9ec | 13:11 | |||||||||||||||||||||||||||||||||||||
ZofBot: who logs changes the changelog? | 13:12 | ||||||||||||||||||||||||||||||||||||||
ZofBot | Zoffix, But it will get better | ||||||||||||||||||||||||||||||||||||||
nine | jnthn: looks like I was rather close to reproducibility when I worked on eliminating those uninitialized padding bytes in serialization back in August. If only I had known how close :) | 13:13 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | nine: for me it's a bit hard to judge if this is going to affect anything, so maybe after the release? | 13:17 | |||||||||||||||||||||||||||||||||||||
PufferBot | [openbsd] | t/spec/S16-io/print.t (Wstat: 512 Tests: 15 Failed: 2) | ||||||||||||||||||||||||||||||||||||||
[openbsd] | t/spec/S32-io/IO-Socket-Async.t (Wstat: 256 Tests: 17 Failed: 0) | |||||||||||||||||||||||||||||||||||||||
[openbsd] Roast tests failed. See the output at fpaste.scsys.co.uk/575933 | |||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | Zoffix: this is awesome, thank you! | ||||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: looks like it failed to update roast repo: "*** Error 1 in target 'spectest_update' (ignored)" | 13:19 | |||||||||||||||||||||||||||||||||||||
Kaiepi: and the S16-io/print.t failures look to be exactly due to that; not recent enough roast | 13:20 | ||||||||||||||||||||||||||||||||||||||
nine | AlexDaniel: it could be big news for distro packagers. But no, let's not rush things. The change could cause weird issues that only surface in real world usage. | ||||||||||||||||||||||||||||||||||||||
Kaiepi | does make spectest not update roast on its own? | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | ZofBot: the right answer was github.com/perl6/whateverable/issues/208 | ||||||||||||||||||||||||||||||||||||||
ZofBot | AlexDaniel, hmmm... running a --profile on the same .hypering code, produces wildly differently sized .html files (60MB - 342MB) # timotimo jnthn | ||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, all | 13:21 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Configuring Rakudo... | ||||||||||||||||||||||||||||||||||||||
nine | AlexDaniel: but damn am I looking forward to getting this in :) | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Building Rakudo... | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | huggable: star | ||||||||||||||||||||||||||||||||||||||
huggable | AlexDaniel, Estimated Rakudo Star releases for 2017: .01, .04, .07 & .10 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | nine: I think it'll go in at the right time ↑ :) | ||||||||||||||||||||||||||||||||||||||
Zoffix | Kaiepi: it does, but in your case there was an error. | ||||||||||||||||||||||||||||||||||||||
Kaiepi | ah | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | assuming releases that have star based on them are somewhat better | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Build successful! | 13:23 | |||||||||||||||||||||||||||||||||||||
[freebsd] Testing Rakudo... | |||||||||||||||||||||||||||||||||||||||
[freebsd] | t/04-nativecall/02-simple-args.t (Wstat: 256 Tests: 14 Failed: 1) | 13:26 | ||||||||||||||||||||||||||||||||||||||
[freebsd] Tests failed. See the output at fpaste.scsys.co.uk/575934 | |||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
Kaiepi | BeastieBot, spectest | 13:27 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Running Roast test suite... (this will take a while) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | BeastieBot: fwiw there are alse #whateverable and #zofbot that are meant for bot abuse | 13:36 | |||||||||||||||||||||||||||||||||||||
also | |||||||||||||||||||||||||||||||||||||||
* | |||||||||||||||||||||||||||||||||||||||
maybe we should start #robotabuse | 13:42 | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] | t/spec/S17-supply/syntax.t (Wstat: 9 Tests: 69 Failed: 0) | 14:01 | |||||||||||||||||||||||||||||||||||||
[freebsd] | t/spec/S17-promise/basic.t (Wstat: 256 Tests: 41 Failed: 1) | |||||||||||||||||||||||||||||||||||||||
[freebsd] Roast tests failed. See the output at fpaste.scsys.co.uk/575935 | |||||||||||||||||||||||||||||||||||||||
Kaiepi, done! | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | BeastieBot: spectest | 14:12 | |||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Running Roast test suite... (this will take a while) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | flappers? | ||||||||||||||||||||||||||||||||||||||
lizmat | those look like familiar flappers to me | 14:13 | |||||||||||||||||||||||||||||||||||||
notable6: weekly www.0racle.info/articles/pick_and_c...part1-half | 14:23 | ||||||||||||||||||||||||||||||||||||||
notable6 | lizmat, Noted! | ||||||||||||||||||||||||||||||||||||||
lizmat | notable6: weekly www.0racle.info/articles/pick_and_choose_part2 | 14:25 | |||||||||||||||||||||||||||||||||||||
notable6 | lizmat, Noted! | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] | t/spec/S17-supply/syntax.t (Wstat: 9 Tests: 69 Failed: 0) | 14:47 | |||||||||||||||||||||||||||||||||||||
[freebsd] | t/spec/S17-promise/at.t (Wstat: 256 Tests: 4 Failed: 1) | |||||||||||||||||||||||||||||||||||||||
[freebsd] | t/spec/S17-promise/basic.t (Wstat: 512 Tests: 41 Failed: 2) | |||||||||||||||||||||||||||||||||||||||
[freebsd] Roast tests failed. See the output at fpaste.scsys.co.uk/575936 | |||||||||||||||||||||||||||||||||||||||
AlexDaniel, done! | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | so at.t flapped but syntax.t and basic.t are failing consistently? | 14:55 | |||||||||||||||||||||||||||||||||||||
Geth | roast/post-release-2018.03: 05c07809fb | (Zoffix Znet)++ | S32-io/open.t Do not rely on EVAL's :context in tests |
15:09 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | nine: ↑ you can put it into the post-release branch | 15:15 | |||||||||||||||||||||||||||||||||||||
BeastieBot: spectest | 15:24 | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] Running Roast test suite... (this will take a while) | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo/post-release-2018.03: 18eda3ea69 | (Zoffix Znet)++ | src/core/IO/Handle.pm6 Clear up nqp code in IO::Handle.open - &&/|| already compile to nqp::if/nqp::unless - No functional changes |
15:42 | |||||||||||||||||||||||||||||||||||||
rakudo: 0194ef46a3 | (Elizabeth Mattijsen)++ | src/core/Rakudo/Internals/HyperToIterator.pm6 Streamline HyperToIterator.consume-batch - makes it about 4x faster - makes say ^Inf .hyper.grep( *.is-prime ).skip(9999).head about 2% faster |
15:51 | ||||||||||||||||||||||||||||||||||||||
BeastieBot | [freebsd] | t/spec/S17-supply/syntax.t (Wstat: 9 Tests: 69 Failed: 0) | 15:57 | |||||||||||||||||||||||||||||||||||||
[freebsd] | t/spec/S17-promise/nonblocking-await.t (Wstat: 256 Tests: 4 Failed: 0) | |||||||||||||||||||||||||||||||||||||||
[freebsd] | t/spec/S17-promise/basic.t (Wstat: 256 Tests: 41 Failed: 1) | |||||||||||||||||||||||||||||||||||||||
[freebsd] Roast tests failed. See the output at fpaste.scsys.co.uk/575937 | |||||||||||||||||||||||||||||||||||||||
AlexDaniel, done! | |||||||||||||||||||||||||||||||||||||||
nine | Looks like I also got a patch for rakudo for making builds reproducible, i.e. no more timestamps in path names or hidden in sha hashes. I can now run Inline::Perl5's tests, do a make install in rakudo (re-compiling NativeCall), run Inline::Perl5 again and don't have to wait for it to re-compile Inline::Perl5 | 16:09 | |||||||||||||||||||||||||||||||||||||
lizmat | whee! | 16:12 | |||||||||||||||||||||||||||||||||||||
nine | Apart from this convenience and much more rare "Missing or wrong version of dependency" messages, this will most of all help packagers. Because a simple rebuild of one package (nqp, rakudo or some module package) will no longer invalidate depending packages. | 16:13 | |||||||||||||||||||||||||||||||||||||
lizmat | nine++ | 16:14 | |||||||||||||||||||||||||||||||||||||
japhb | Definitely nine++ | ||||||||||||||||||||||||||||||||||||||
jnthn | nine++ indeed | 16:16 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 21997b6256 | (Jonathan Worthington)++ | src/Perl6/Actions.nqp Don't use typevar inappropriately It instructs optimizers that if they are holding the type of the first argument (in a method, "the invocant") constant, then the lookup of the lexical can also be assumed constant also. However, we were using it in nested blocks, where a first argument type could be held constant but it was totally unrelated to any notion of "invocant". ... (5 more lines) |
17:03 | |||||||||||||||||||||||||||||||||||||
roast: 953f8b8638 | (Jonathan Worthington)++ | S12-methods/private.t Test to cover buggy private method lookup caching |
17:04 | ||||||||||||||||||||||||||||||||||||||
nqp: db6801437b | pmurias++ | src/vm/js/nqp-runtime/ctx.js [js] Throw an exception when accessing a missing variable with nqp::atkey |
17:27 | ||||||||||||||||||||||||||||||||||||||
nqp: 199dd708b1 | pmurias++ | 4 files [js] Implement getcodelocation It's done in a very inefficent manner, needs to be more optimized for performance important use |
|||||||||||||||||||||||||||||||||||||||
AlexDaniel | nine: that's awesome! | ||||||||||||||||||||||||||||||||||||||
jnthn | m/sg camelia m: class C { multi method m(Int $x) { samewith ~$x }; multi method m(Str) { }; multi method call() { self.m(42) } }; C.call for ^100000 | 17:29 | |||||||||||||||||||||||||||||||||||||
bah :) | |||||||||||||||||||||||||||||||||||||||
I have a fix for that blocker too | 17:30 | ||||||||||||||||||||||||||||||||||||||
#1573 | |||||||||||||||||||||||||||||||||||||||
Golfs to | 17:34 | ||||||||||||||||||||||||||||||||||||||
$ MVM_SPESH_NODELAY=1 MVM_SPESH_BLOCKING=1 ./perl6-m -e 'class C { multi method m(Int $x) { samewith $x.Str }; multi method m(Str) { }; multi method call() { self.m(42) } }; C.call for ^10000' | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 6232d29ebc | (Elizabeth Mattijsen)++ | src/core/Rakudo/Internals/HyperToIterator.pm6 Streamline HyperToIterator.pull-one a bit - remove unnecessary variable - bind rather than assign, removing need for deconts - make sure it is "is raw" (apparently we have no tests for containers) |
||||||||||||||||||||||||||||||||||||||
Zoffix | jnthn: FWIW, there're two blockers; another one is R#1566 (mentioning, just in case) | 17:37 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1566 [open]: github.com/rakudo/rakudo/issues/1566 [regression][⚠ blocker ⚠] multi built-ins are not assignable to Callable | ||||||||||||||||||||||||||||||||||||||
Zoffix | That's the Callable-is-mixed-in-before-it's-composed thing | 17:38 | |||||||||||||||||||||||||||||||||||||
jnthn | Yeah, I won't have time for that | ||||||||||||||||||||||||||||||||||||||
Not this side of the weekend, anyways | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: e12e305a6b | (Jonathan Worthington)++ | src/Perl6/Actions.nqp Make sure `samewith`-using routines aren't inlined We already did it for callsame, nextsame, etc. but forgot to put samewith in the list. |
17:39 | |||||||||||||||||||||||||||||||||||||
Zoffix | jnthn: OK, then I'll stick a couple of .^compose after Callable is composed, just to unblock the blocker. So that any new routines that got this bug this release would not have it. There's &max, &min, and probably a couple more. | 17:40 | |||||||||||||||||||||||||||||||||||||
dogbert11 | jnthn++, that second one was quick | ||||||||||||||||||||||||||||||||||||||
Zoffix | m: say &max ~~ Callable; &max.^compose; say &max ~~ Callable | ||||||||||||||||||||||||||||||||||||||
camelia | False True |
||||||||||||||||||||||||||||||||||||||
Geth | roast: e6b0872ef7 | (Jonathan Worthington)++ | S12-methods/defer-next.t Test to cover Rakudo issue #1573 |
17:41 | |||||||||||||||||||||||||||||||||||||
jnthn | Zoffix: Yeah, that's probably best. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | Zoffix++ | 17:47 | |||||||||||||||||||||||||||||||||||||
jnthn | Alrightly, well, that's a couple of icky bugs squished at least... | 17:59 | |||||||||||||||||||||||||||||||||||||
jnthn wanders home | |||||||||||||||||||||||||||||||||||||||
Geth | roast/post-release-2018.03: 37aa7c3d7f | (Zoffix Znet)++ | S32-io/open.t Expand/clarify spec for .open mode arguments Closes R#1614 github.com/rakudo/rakudo/issues/1614 Closes R#1615 github.com/rakudo/rakudo/pull/1615 - Add 183 new tests (plan count went down due to use of subtests) - Tests all of the "approved" mode combinations - Deliberately leaves unapproved mode combinations implementation-dependent, to keep docs simpler and to not lock down implementations to a particular set of options and order of checking them |
18:02 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1614 [open]: github.com/rakudo/rakudo/issues/1614 .open with :truncate/:create violates "read only" default open mode | ||||||||||||||||||||||||||||||||||||||
R#1615 [open]: github.com/rakudo/rakudo/pull/1615 Throw when opening read-only files with truncate and create flags | |||||||||||||||||||||||||||||||||||||||
Zoffix | k, looking through all the commits since last release, looks like only &min, &max, &minmax regressed for not being Callable anymore | 19:39 | |||||||||||||||||||||||||||||||||||||
ZOFFLOP: t/spec/S02-types/baghash.t | 19:40 | ||||||||||||||||||||||||||||||||||||||
(some tests that assume ordering?) | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo/post-release-2018.03: 87e43c4aea | (Zoffix Znet)++ | src/core/operators.pm6 Fix infix:<o> identity function Failure.self explodes Failures, so we can't use it for identity routine. |
19:41 | |||||||||||||||||||||||||||||||||||||
roast/post-release-2018.03: eacaf276f8 | (Zoffix Znet)++ | S03-operators/composition.t Test infix:<o>() with Failures Rakudo fix: github.com/rakudo/rakudo/commit/87e43c4aea |
19:44 | ||||||||||||||||||||||||||||||||||||||
Zoffix | will fix the &min, &max, &minmax in ~2h | 19:48 | |||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Jonathan Worthington 'Make sure `samewith`-using routines aren't inlined | 20:48 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/354430390 github.com/rakudo/rakudo/compare/6...2e305a6b9e | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | samcv: hey, how is it going? I think rakudo side will ready tomorrow | 21:22 | |||||||||||||||||||||||||||||||||||||
be* | |||||||||||||||||||||||||||||||||||||||
releasable6: status | |||||||||||||||||||||||||||||||||||||||
releasable6 | AlexDaniel, Next release in ≈21 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Changelog for this release was not started yet | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | yea, in that amount of time | ||||||||||||||||||||||||||||||||||||||
samcv | AlexDaniel: going good. sounds good to me! | 21:49 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: 4513c279d4 | (Zoffix Znet)++ | src/core/Cool.pm6 Workaround regression in Routines not doing Callable Removes blocker status from R#1566 github.com/rakudo/rakudo/issues/1566 The actual bug is that Callable role gets mixed in into routines before it's composed, and when it is composed, the routines end up not "doing" `Callable` role, even though they do. There are many more routines suffering this issue, but these three regressed since last release and we don't have the time to fix the primary bug before the release, so in this fudge goes. |
21:53 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1566 [open]: github.com/rakudo/rakudo/issues/1566 [regression][⚠ blocker ⚠] multi built-ins are not assignable to Callable | ||||||||||||||||||||||||||||||||||||||
roast: 154a22c963 | (Zoffix Znet)++ | S03-smartmatch/any-callable.t Test for glitches in Routines not doing Callable role R#1566: github.com/rakudo/rakudo/issues/1566 Rakudo workaround for select routines: github.com/rakudo/rakudo/commit/4513c279d4 |
|||||||||||||||||||||||||||||||||||||||
Zoffix | releasable6: status | 21:58 | |||||||||||||||||||||||||||||||||||||
releasable6 | Zoffix, Next release in ≈21 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Changelog for this release was not started yet | ||||||||||||||||||||||||||||||||||||||
Zoffix, Details: gist.github.com/071eb3f0b0963c700a...03dc3a7f1d | |||||||||||||||||||||||||||||||||||||||
Zoffix | .tell AlexDaniel the workaround for R#1566 went in. There are no more known release blockers. | ||||||||||||||||||||||||||||||||||||||
yoleaux | Zoffix: I'll pass your message to AlexDaniel. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | oh well, that's awesome | ||||||||||||||||||||||||||||||||||||||
yoleaux | 21:58Z <Zoffix> AlexDaniel: the workaround for R#1566 went in. There are no more known release blockers. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | thanks |