Geth | rakudo: FCO++ created pull request #1273: Fixes #1258 |
00:59 | |||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1273 [open]: github.com/rakudo/rakudo/pull/1273 Fixes #1258 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: c8370f21f0 | (Fernando Correa de Oliveira)++ | src/core/Supply.pm Fix Supply.throttle() |
01:16 | |||||||||||||||||||||||||||||||||||||
rakudo: a9ec5f9249 | lizmat++ (committed using GitHub Web editor) | src/core/Supply.pm Merge pull request #1271 from FCO/fix-throttle Fix Supply.throttle() |
|||||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1271 [closed]: github.com/rakudo/rakudo/pull/1271 Fix Supply.throttle() | ||||||||||||||||||||||||||||||||||||||
roast: 994394c95a | (Fernando Correa de Oliveira)++ | S17-supply/throttle.t Test if Supply.throttle($, $second) become done when its source become done |
|||||||||||||||||||||||||||||||||||||||
roast: c05cf6f458 | lizmat++ (committed using GitHub Web editor) | S17-supply/throttle.t Merge pull request #359 from FCO/fix-throttle Test if Supply.throttle($, $second) become done when its source becom… |
|||||||||||||||||||||||||||||||||||||||
synopsebot | SPEC#359 [closed]: github.com/perl6/roast/pull/359 Test if Supply.throttle($, $second) become done when its source becom… | ||||||||||||||||||||||||||||||||||||||
roast: FCO++ created pull request #360: Issue 1258 |
01:46 | ||||||||||||||||||||||||||||||||||||||
synopsebot | SPEC#360 [open]: github.com/perl6/roast/pull/360 Issue 1258 | ||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. lizmat 'Merge pull request #1271 from FCO/fix-throttle | 02:37 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/307324182 github.com/rakudo/rakudo/compare/4...ec5f92494e | |||||||||||||||||||||||||||||||||||||||
buggable | [travis build above] ☠ Did not recognize some failures. Check results manually. | ||||||||||||||||||||||||||||||||||||||
yoleaux | AlexDaniel`: RT/GH weekly | 07:18 | |||||||||||||||||||||||||||||||||||||
bartolin | hmm, after merging rakudo's PR#1240 we also need to merge PR#353 for roast? | 08:41 | |||||||||||||||||||||||||||||||||||||
my nightly spectests were quite unhappy | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: aa3c2c5b04 | (Fernando Correa de Oliveira)++ | src/core/Supply.pm Fixes #1258 |
09:44 | |||||||||||||||||||||||||||||||||||||
rakudo: 80dfb30ca3 | (Fernando Correa de Oliveira)++ | src/core/Supply.pm change $.foo to self.foo |
|||||||||||||||||||||||||||||||||||||||
rakudo: b285aaf350 | lizmat++ (committed using GitHub Web editor) | src/core/Supply.pm Merge pull request #1273 from FCO/issue-1258 Fixes #1258 |
|||||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1258 [closed]: github.com/rakudo/rakudo/issues/1258 Supply.list returns a Seq | ||||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1273 [closed]: github.com/rakudo/rakudo/pull/1273 Fixes #1258 | ||||||||||||||||||||||||||||||||||||||
roast/master: 4 commits pushed by (Fernando Correa de Oliveira)++, lizmat++ | |||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. lizmat 'Merge pull request #1273 from FCO/issue-1258 | 10:31 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/307425555 github.com/rakudo/rakudo/compare/a...85aaf35084 | |||||||||||||||||||||||||||||||||||||||
lizmat | decommute& | 10:39 | |||||||||||||||||||||||||||||||||||||
nine | Oh boy, did we just lose ugexe? :( | 10:52 | |||||||||||||||||||||||||||||||||||||
yoleaux | 24 Nov 2017 04:04Z <geekosaur> nine: colabti.org/irclogger/irclogger_log...11-24#l119 possible precomp bug? (cf colabti.org/irclogger/irclogger_log...1-24#l156) | ||||||||||||||||||||||||||||||||||||||
nine | .tell geekosaur I've had a quick look at it yesterday and believe it to be a type mismatch in p5_av_top_index. But I'm not entirely sure yet. | 10:54 | |||||||||||||||||||||||||||||||||||||
yoleaux | nine: I'll pass your message to geekosaur. | ||||||||||||||||||||||||||||||||||||||
nine | .tell geekosaur Oooh...no, it's probably not as mundane as that. An issue with JIT compilation of native calls is more consistent with the facts. It's probably not correctly expanding signed return values correctly on some platforms. | 11:05 | |||||||||||||||||||||||||||||||||||||
yoleaux | nine: I'll pass your message to geekosaur. | ||||||||||||||||||||||||||||||||||||||
bartolin | nine: there is some background and a very worthy comment from jnthn here github.com/MoarVM/MoarVM/pull/748 | 11:31 | |||||||||||||||||||||||||||||||||||||
tbrowder | .tell AlexDaniel roast PR #353 adds additional tests for rakudo PR #1240 so it should be merged also | 11:58 | |||||||||||||||||||||||||||||||||||||
yoleaux | tbrowder: I'll pass your message to AlexDaniel. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | . | 12:57 | |||||||||||||||||||||||||||||||||||||
yoleaux | 11:58Z <tbrowder> AlexDaniel: roast PR #353 adds additional tests for rakudo PR #1240 so it should be merged also | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | SPEC#1240 | ||||||||||||||||||||||||||||||||||||||
um | |||||||||||||||||||||||||||||||||||||||
oops | 12:58 | ||||||||||||||||||||||||||||||||||||||
SPEC#353 | |||||||||||||||||||||||||||||||||||||||
synopsebot | SPEC#353 [open]: github.com/perl6/roast/pull/353 Fix rt 128221 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | well, I'd have to wake up first :) | ||||||||||||||||||||||||||||||||||||||
Geth | roast: 0ef7f132a6 | usev6++ | S17-supply/zip.t [jvm] Skip newly added test |
14:14 | |||||||||||||||||||||||||||||||||||||
[Tux] |
|
14:15 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: fbf7becac3 | usev6++ | src/vm/jvm/runtime/org/perl6/rakudo/RakudoContainerSpec.java [jvm] Skip type check if expected type is Mu That's what we do in 'type_check_store' on MoarVM as well. |
14:21 | |||||||||||||||||||||||||||||||||||||
lizmat waves from under The Channel | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: 694f534ad7 | (Elizabeth Mattijsen)++ | src/core/Rakudo/Iterator.pm Introducing R:I.Toggle, the .toggle workhorse |
14:26 | |||||||||||||||||||||||||||||||||||||
rakudo: 78caeb6bce | (Elizabeth Mattijsen)++ | src/core/Any-iterable-methods.pm Implement .toggle - takes a number of Callables and an :off flag indicating initial state - (1..15).toggle(* < 5, * > 10, * < 15) i # 1 2 3 4 11 12 13 14 - (1..15).toggle(:off, * > 2, * < 5, * > 10, * < 15) # 3 4 11 12 13 14 |
|||||||||||||||||||||||||||||||||||||||
lizmat | jnthn: hope I got the intended semantics right | 14:27 | |||||||||||||||||||||||||||||||||||||
jnthn | lizmat: Cool, will have a play with it when I get a spare moment :) | 14:28 | |||||||||||||||||||||||||||||||||||||
lizmat | the 1 callable cases re-use the R:I.While and R:I.Until iterators I made earlier | 14:29 | |||||||||||||||||||||||||||||||||||||
that could be considered overkill, not sure | |||||||||||||||||||||||||||||||||||||||
dogbert17 | the documentation for CallFrame lists the existence of a method name 'level'. Perhaps I'm blind but I can't find it. Has it been removed/misplaced? docs.perl6.org/type/CallFrame#method_level | 15:14 | |||||||||||||||||||||||||||||||||||||
m: say CallFrame.^methods() | |||||||||||||||||||||||||||||||||||||||
camelia | (SET-SELF new line file code callframe annotations gist my BUILDALL) | ||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build errored. usev6 '[jvm] Skip type check if expected type is Mu | 15:19 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/307491535 github.com/rakudo/rakudo/compare/b...f7becac3ef | |||||||||||||||||||||||||||||||||||||||
buggable | [travis build above] ☠ Did not recognize some failures. Check results manually. | 15:20 | |||||||||||||||||||||||||||||||||||||
bartolin | ^^ travis fails to install the build environment | 15:23 | |||||||||||||||||||||||||||||||||||||
Geth | roast: f1692c05cd | usev6++ | 14 files [jvm] Unskip a bunch of now working tests |
15:55 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: I left some comments in a review | 21:40 | |||||||||||||||||||||||||||||||||||||
tbrowder | AlexDaniel: I answered your comments. | 21:59 | |||||||||||||||||||||||||||||||||||||
Hope they are adequate. | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: yeah. There are two more unanswered comments in outdated diffs though | 22:03 | |||||||||||||||||||||||||||||||||||||
you have to click “Show outdated” to see them | |||||||||||||||||||||||||||||||||||||||
github is weird :S | |||||||||||||||||||||||||||||||||||||||
(so what if it's a comment on an intermediate commit, why hide them?) | 22:04 | ||||||||||||||||||||||||||||||||||||||
tbrowder | I’ll have to answer those tomorrow, sorry...got to go. | 22:08 | |||||||||||||||||||||||||||||||||||||
bartolin | j: say "alive" | 22:09 | |||||||||||||||||||||||||||||||||||||
camelia | Error while reading '/home/camelia/p6eval-token': No such file or directory at /home/camelia/rakudo-j-inst/bin/eval-client.pl line 10. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | now that we have rakudo/issues open, perhaps it's time to do this: irclog.perlgeek.de/perl6-dev/2016-...i_13123382 | 22:23 |