| MasterDuke | samcv: did you see this? github.com/MoarVM/MoarVM/pull/501#...w-16889758 | 00:16 | |
| samcv | MasterDuke, uhm not all the types have that pointer type? | 00:17 | |
| but if you make that change does it supress all warnings? | |||
| there's MVMUnicodeGraphemeNameRegistry too | 00:18 | ||
| MasterDuke | it surpresses the one warning during a moarvm compile | ||
| samcv | yeah | ||
| I wouldn't make this change. i mean | |||
| better to fix the actual source than just suppress the warning | |||
| or maybe have one MVMUnicodeXXX and cast both the other ones to that | 00:19 | ||
| than to cast to a type which it is not | |||
| or maybe there's some way to do it to have the macro HASH_ADD_KEYPTR wants to be something that accepts both of them, i'm not sure | 00:21 | ||
| let me look for a sec | |||
| MasterDuke | well, UnicodeNameRegistry and UnicodeGraphemeNameRegistry are pretty much identical | ||
| samcv | they aren't the same though | 00:22 | |
| don't they have different types in the containing items? | |||
| yeah they do | |||
| MasterDuke | yeah, but i think the one is just a typedef of the other | ||
| samcv | no they are not | ||
| MasterDuke | (the differing ones) | ||
| samcv | see unicode.h | 00:23 | |
| here github.com/MoarVM/MoarVM/blob/mast...code.h#L14 | |||
| one holds a codepoint value and one holds an index of another struct | |||
| so I really don't like the idea of making both the same type | 00:24 | ||
| even though the size of the values inside the struct is the same | |||
| MasterDuke | typedef MVMint32 MVMGrapheme32; | ||
| src/6model/reprs/MVMString.h:19: | 00:25 | ||
| i have no strong feelings about what the correct solution is, i just made a (what seemed reasonable at the time without knowing anything about the code) change that silenced the warning | 00:27 | ||
| samcv | your other changes seem fine. but I don't like changing that one unless we have a better solution | ||
| MasterDuke | so feel free to do or suggest something else/better | ||
| won't hurt my feelings | 00:28 | ||
| samcv | we could always use a union tbh | 00:29 | |
| i think. i am thinking | |||
| or maybe this is the opposite of what i want. i'll get back to when I figure out what will be best :) | 00:30 | ||
| MasterDuke | sure | 00:32 | |
| samcv | yeah we can use a union | 00:33 | |
| but that may require more changes else where. so not a good change to just make quickly | 00:34 | ||
| i'm gonna look at the macro | 00:35 | ||
| MasterDuke | do you mind replaying in the comment thread? | 00:36 | |
| *replying | |||
| samcv | ye | ||
| MasterDuke | (no need to replay the whole conversation) | ||
| thanks | |||
| samcv | MasterDuke, can you try casting as (keyptr) | 00:39 | |
| see if that works | 00:40 | ||
| er wait that's wrong. ah | 00:41 | ||
| Geth | ast: 5d837a7a26 | (Jonathan Scott Duff)++ | S03-operators/range-basic.t Add tests for RT#130554 |
05:44 | |
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130554 | ||
| [Tux] | This is Rakudo version 2016.12-337-g7f970357b built on MoarVM version 2016.12-115-ged2df9ac | 07:09 | |
| csv-ip5xs 2.889 | |||
| test 12.366 | |||
| test-t 5.024 | |||
| csv-parser 13.224 | |||
| lizmat | Files=1163, Tests=56135, 186 wallclock secs (11.01 usr 4.49 sys + 1102.06 cusr 118.87 csys = 1236.43 CPU) | 08:25 | |
| samcv: the reason I repeated the string in the text, is that some browser may actually not display the emoji correctly | 08:26 | ||
| samcv | kk | ||
| though it seemed redundant. "\c[charname]" (charname) | 08:29 | ||
| that was all | |||
| lizmat | yeah, I know, but having the emoji only would have been lost on many | 08:35 | |
| and just the text after it would look strange to me without the parens | |||
| hmmm... is there a reason why the @cycle on a .rotor() cannot be lazy (apart from implementation complexity?) | 09:14 | ||
| |Tux| | I had some git mayhem, so I am unsure if this last timing was caused by that. Just to make sure, here is a new one ⦠| 09:30 | |
| This is Rakudo version 2016.12-351-g0cd921e35 built on MoarVM version 2016.12-115-ged2df9ac | |||
| csv-ip5xs 2.803 | |||
| test 12.007 | |||
| test-t 4.878 | |||
| csv-parser 13.332 | |||
| 337 ā 351 | 09:31 | ||
| lizmat | 337 ā 351 ? | ||
| ah, git commits | |||
| a new low! | 09:32 | ||
| :-) | |||
| DrForr | I'd better update :) | 09:34 | |
| lizmat should take more tea: the cycle on a rotor *can* be lazy, and *is* made lazy internally anyway :-) | 09:35 | ||
| afk for a few hours& | 09:41 | ||
| brokenchicken | lizmat: there's this bug tho: rt.perl.org/Ticket/Display.html?id...et-history | 11:57 | |
| lizmat | ok, will take care of that | ||
| :-) | |||
| well, of the rotor case, not necessarily the slurpy arg case | 11:58 | ||
| hmmm... or maybe I must :-( | |||
| m: sub a(*@a) { say @a.iterator.is-lazy }; a(1...*) # underlying issue | 12:01 | ||
| camelia | rakudo-moar 0cd921: OUTPUTĀ«Falseā¤Ā» | ||
| lizmat | that should have said True | ||
| jnthn | m: sub a(+@a) { say @a.iterator.is-lazy }; a(1...*) # curious | 12:03 | |
| camelia | rakudo-moar 0cd921: OUTPUTĀ«Trueā¤Ā» | ||
| ilmari | what's the difference betwen *@a and +@a? | 12:04 | |
| ah, docs.perl6.org/type/Signature#Sing...ule_Slurpy | 12:05 | ||
| shouldn't that be s/(engage)d/$0s/? | 12:07 | ||
| lizmat | ilmari: there's github.com/perl6/doc and PR's :-) | 12:09 | |
| stackoverflow.com/questions/4169657...-in-perl-6 # why do we need the inner eager ??? | 12:25 | ||
| that feels like a bug to me ? | |||
| timotimo | is that just "last statement not being sunk correctly"? | 12:26 | |
| i.e. if we put a "say 'finished'" at the end, will it work? | |||
| lizmat | trying | 12:27 | |
| yup | 12:28 | ||
| that's it | |||
| timotimo | i'm glad my intuition was right :) | ||
| lizmat | yeah, adding another statement will make the inner map sink | 12:29 | |
| stackoverflow.com/questions/4169657...in-perl-6/ # answered | 12:39 | ||
| timotimo: basically, all is as it should be, it's just that a Seq of .Seqs getting sunk, doesn't iterate over the inner .Seqs | 12:40 | ||
.oO( I'm so glad I don't have to read that aloud :-) |
12:41 | ||
| timotimo | OK! | ||
| lizmat | and I don't think it should | ||
| jnthn looks but suspects not a bug | |||
| Yeah, no bug | 12:42 | ||
| lizmat | yeah, the outer Seq is sunk, and that doesn't run the inners Seqs | ||
| jnthn | But if you're just using map for the side-effects, why not write it with `for`? | ||
| timotimo | lizmat: just pronounce it like "seek" | ||
| jnthn | The point of map is you're mapping something to something else :P | ||
| lizmat | well, there's that :-) | ||
| timotimo | because after all it's short for Sequence | ||
| jnthn | .oO( I just mapped Seqs, and it felt so good... ) |
12:43 | |
| lizmat | well, I'm not sure I like being chased by men in turbans :-) | 12:44 | |
| with their kirpans | |||
| 12:45 | |||
| oops, wrong paste | |||
.oO( en.wikipedia.org/wiki/Sikh ) |
|||
| timotimo | jnthn: an array let me put my items inside it? | 12:46 | |
| jnthn | :) | ||
| brokenchicken | On the topic of slurpies there's also this ticket: rt.perl.org/Ticket/Display.html?id...et-history | 13:16 | |
| buggable: tag regression | |||
| buggable | brokenchicken, There are 22 tickets tagged with REGRESSION; See perl6.fail/t/REGRESSION for details | ||
| brokenchicken | NeuralAnomaly: status | 13:17 | |
| NeuralAnomaly | brokenchicken, [ā] Next release will be in 3 days and 16 hours. Since last release, there are 89 new still-open tickets (89 unreviewed and 0 blockers) and 329 unreviewed commits. See perl6.fail/release/stats for details | ||
| samcv | the newest atom beta that came out a week ago is so much faster brokenchicken | 13:29 | |
| lizmat | brokenchicken: RT #130566 is fixed | 13:30 | |
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130566 | ||
| samcv | though your computer is probably faster than mine. I can actually open big files and it won't go all funny, and opening any file is like 2x faster | ||
| lizmat | brokenchicken: also fixed: RT #130513 | ||
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130513 | ||
| lizmat | and RT #130510 | 13:31 | |
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130510 | ||
| brokenchicken | cool | 13:32 | |
| timotimo | that sounds great! (and much needed) | ||
| samcv | what I said? or? | ||
| the slurpy | |||
| timotimo | no, atom performance | 13:33 | |
| samcv | ah yeah | ||
| jnthn | Yeah, I moved to Vim largely because I got tired of <open large file> <hang> | 13:37 | |
| Doubt I'll move back, alas. | |||
| Really liking how decluttered things are in Vim. Me. The code I'm working on. That's all. | 13:38 | ||
| timotimo | vim has *big* trouble when a file contains a line like # ----------------------------------- | 13:39 | |
| it seems like | |||
| haven't golfed it yet | |||
| but at least you can ctrl-c and it'll stop the syntax highlighter and you can just scroll away | |||
| brokenchicken | .ask geekosaur you know a lot about this topic. Mind looking at rt.perl.org/Ticket/Display.html?id...et-history ? Is that even a bug? | 13:40 | |
| yoleaux2 | brokenchicken: I'll pass your message to geekosaur. | ||
| jnthn | brokenchicken: I looked at that and thought "is it really reasonable to expect Perl 6 to do that" too, tbh. But yeah, geekosaur++ will likely know better on it :) | 13:42 | |
| timotimo | ah, yeah, pty support would be good indeed | 13:43 | |
| nine | The pty command sounds useful for that. Maybe one should just push that in between Perl 6 and the program it runs? | 13:47 | |
| yoleaux2 | 6 Jan 2017 17:20Z <notviki> nine: is this meant to work? gist.github.com/zoffixznet/b6cb79c...ad08deb68d Seems to only use the sub from first loaded version and keeps using it that version even if I reverse the loading order, unless I also nuke the .precomp | ||
| 6 Jan 2017 17:26Z <notviki> nine: here are the contents of the files in that tree: gist.github.com/zoffixznet/ccdd17f...c06be8afbe | |||
| 6 Jan 2017 17:30Z <notviki> nine: err, `cat v2/Foo.pm6` has { say "two"; } instead of { say "one" } (and yeah, I then changed it to say "ones" and the code kept saying "one" until I nuked precomp | |||
| 9 Jan 2017 04:11Z <japhb> nine: Which phasers are expected to happen *during* precompilation, which ones are expected to happen at need time, and which ones during import time? | |||
| nine | .tell notviki the FileSystem repository is meant for development mostly and therefore ignores all version, auth and api requirements and just matches the name of the module. | 13:56 | |
| yoleaux2 | nine: I'll pass your message to notviki. | ||
| lizmat | nine: I guess you want to leave that for brokenchicken :-) | 13:57 | |
| Geth | ast: ronaldxs++ created pull request #223: Test for RT #130562 |
13:58 | |
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130562 | ||
| Geth | ast: 89c51acfb9 | (Ronald Schmidt)++ | S05-metasyntax/longest-alternative.t Test for RT #130562, ltm alternation only sees first sequential alternative. |
14:02 | |
| ast: e5efaec7ba | lizmat++ | S05-metasyntax/longest-alternative.t Merge pull request #223 from ronaldxs/mixed-ltm-sequential-alternation Test for RT #130562 |
|||
| synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130562 | ||
| nine | brokenchicken: the FileSystem repository is meant for development mostly and therefore ignores all version, auth and api requirements and just matches the name of the module. | 14:05 | |
| lizmat wonders whether synopsebot6 should be taught not to mention the same ticket more than once a minute | 14:11 | ||
| brokenchicken | hmm | 14:17 | |
| $ perl6 -e 'use lib </tmp/>; use B; use A' | |||
| ===SORRY!=== | |||
| Failed to open dir: 13 | |||
| mc: use lib </home/bisectable/test>; use B; use A | 14:19 | ||
| committable6 | brokenchicken, ¦«2015.12»: «exit signal = SIGSEGV (11)» | ||
| brokenchicken | bisect: use lib </home/bisectable/test>; use B; use A | ||
| bisectable6 | brokenchicken, On both starting points (old=2015.12 new=0cd921e) the exit code is 0, exit signal is 11 (SIGSEGV) and the output is identical as well | ||
| brokenchicken, Output on both points: | |||
| brokenchicken | k | ||
| That was rt.perl.org/Ticket/Display.html?id...et-history seems we had the issue for ages | 14:20 | ||
| nine | 13 is permission denied | ||
| brokenchicken | Ah | ||
| nine always loves the scavenger hunt for ERRNO constants in /usr/include | 14:21 | ||
| errno.h -> bits/errno.h -> linux/errno.h -> asm/errno.h -> asm-generic/errno.h -> asm-generic/errno-base.h | 14:22 | ||
| ilmari | moreutils has errno(1) | 14:34 | |
| Geth | kudo/nom: 7d5bbefe7f | (Zoffix Znet)++ | docs/ChangeLog Fix formatting |
14:35 | |
| ilmari | $ errno ENOSPC | ||
| ENOSPC 28 No space left on device | |||
| $ errno 37 | |||
| ENOLCK 37 No locks available | |||
| etc | |||
| brokenchicken | lol | 14:39 | |
| $ errno 25 | |||
| ENOTTY 25 /* Not a typewriter */ | |||
| lizmat | afk& | 14:41 | |
| ilmari | brokenchicken: the modern wording for that one is «Inappropriate ioctl for device» | 14:42 | |
| [Coke] | samcv: any plans to add a routine to provide a runtime version of "\c[]" ? | 14:48 | |
| samcv | hmm? | 14:49 | |
| oh. | |||
| that that could be nice | |||
| nine | ilmari: wow, I'd have probably needed every single program contained in moreutils at one point or another. Thanks for pointing out that gem! | ||
| ilmari | nine: yeah, it's very handy | ||
| samcv | moreutils is great | 14:50 | |
| ilmari | I use sponge the most | ||
| samcv | same | ||
| [Coke] | samcv: without a routine, you end up with: github.com/coke/p6-unicode-mangler...gle.p6#L82 | 14:51 | |
| samcv | [Coke], how does unifromname sound? though idk maybe makes it sound like it returnrs a Uni object | ||
| but uniname works only on strings or codes not on Uni so | |||
| there's that | |||
| [Coke] | I mean, uniname is the right name, but it got stolen for the other direction. :) | 14:53 | |
| I don't like unifromname, but don't have a great alternative. from-unicode-name? | |||
| samcv | sure | 14:56 | |
| idk. we need a poll | |||
| [Coke] | I mean, they all suck. | 14:57 | |
| samcv | yeah | ||
| but one of them sucks the least | |||
| also i need to sleep | |||
| i think from-unicode-name could be ok i guess | 14:59 | ||
| brokenchicken | Seems this month will have the biggest changelog that we had for a while... | 16:33 | |
| [Coke] | should we have a Uni.Range to show the range of codepoints? | 16:42 | |
| timotimo | btw do we expose our supported unicode version somewhere? | 16:43 | |
| $*VM.uni-version or something could be used | |||
| [Coke] | timotimo: been asking for that since christmas, nope. | 16:47 | |
| timotimo | oh | ||
| [Coke] | even better: make it settable! | ||
| what's the highest uni ord? | 16:48 | ||
| timotimo | mirror mirror on the wall .. | 16:50 | |
| brokenchicken | m: 0x10FFFF.chr.say | 16:53 | |
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«\x{10FFFF}ā¤Ā» | ||
| nine | timotimo++ # reminding me of Blind Guardian which I haven't listened to in ages. | ||
| ilmari | 'Supplementary Private Use Area-B' ends at U+10FFFF | ||
| brokenchicken | m: 0x10FFFFFFFFFF.chr.say | 16:54 | |
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«chr codepoint cannot be negative⤠in block <unit> at <tmp> line 1ā¤ā¤Ā» | ||
| brokenchicken | heh | ||
| timotimo | nine: when i think "The Mirror", i'm reminded of Dream Theater | 16:56 | |
| nine | timotimo: :) | 17:05 | |
| TimToady | samcv: I'd just call it "uni", since it'll usually be obvious from the name what the input is :) | 17:14 | |
| s/name/argument | |||
| and it's perfectly fine to name something by what it produces, generally | 17:15 | ||
| hmm, if she's on california time, looks like she coded all night and then collapsed in a heap | 17:17 | ||
| TimToady fondly remembers being young enough to do that... | |||
| [Coke] | TimToady: I miss those days myself. | ||
| geekosaur | re rt.perl.org/Ticket/Display.html?id...et-history they are expecting Proc::Async to be an Expect implementation, essentially. although it might be nice if it did provide the option to use a pty instead of a pipe (this should not be default as it's higher overhead and ptys are a limited resource compared to pipes) | 17:45 | |
| yoleaux2 | 13:40Z <brokenchicken> geekosaur: you know a lot about this topic. Mind looking at rt.perl.org/Ticket/Display.html?id...et-history ? Is that even a bug? | ||
| b2gills .o( I wonder if Proc::Asyc.new could have a :tty flag added ) | 17:55 | ||
| geekosaur | the real question is likely whether libuv provides it, since iirc Proc::Async defers to libuv for most stuff | 18:05 | |
| timotimo | it should | 18:09 | |
| jnthn | Seems not | 18:10 | |
| timotimo | huh, seems like | 18:11 | |
| jnthn | At least, the searches I did for libuv pty suggested not | 18:12 | |
| timotimo | it mean: seems like not | 18:16 | |
| geekosaur | actually, looks like it does but it doesn't connect the dots for you. docs.libuv.org/en/v1.x/tty.html | 18:20 | |
| nor does it appear to have an API specifically for getting a pty; it just lets you specify one. (uv_process_t *does* support bidirectional "pipes", which only makes sense with a pty or on freebsd) | 18:22 | ||
| Geth | kudo/nom: e041b0f827 | (Pawel Murias)++ | src/Perl6/World.nqp Fix f67df8ad9d4f920bb63ea40501f3b1db9d8e661d. |
20:25 | |
| lizmat | pmurias: is that a revert ? | 20:26 | |
| Ah, it is... | 20:27 | ||
| too bad.. :-) | |||
| pmurias | lizmat: it's not a revert | ||
| lizmat | ah, good ! | 20:28 | |
| b2gills | That looks like it is an infinite loop github.com/rakudo/rakudo/commit/e0...330fcR1528 | 20:34 | |
| pmurias | sorry | 20:36 | |
| b2gills | You can't learn from mistakes that you don't make | 20:38 | |
| lizmat | jnthn: would you mind terribly if I added a .perl method to IterationBuffer ? | 20:39 | |
| m: my $l := IterationBuffer.new; $i.push(42); dd $i | |||
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«5===SORRY!5=== Error while compiling <tmp>ā¤Variable '$i' is not declaredā¤at <tmp>:1ā¤------> 3my $l := IterationBuffer.new; 7ā5$i.push(42); dd $iā¤Ā» | ||
| lizmat | m: my $l := IterationBuffer.new; $l.push(42); dd $l | 20:40 | |
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«IterationBuffer.newā¤Ā» | ||
| lizmat | :-( | ||
| pmurias | do we loose anything by adding a .perl method to IterationBuffer? | 20:49 | |
| Geth | kudo: usev6++ created pull request #996: [JVM] Don't use --nqp-lib=blib for evalserver |
20:50 | |
| lizmat | pmurias: well, I'm not 100% sure: jnthn was pretty adamant about keeping it as simple as possible | 20:52 | |
| multi method perl(IterationBuffer:D:) { nqp::p6bindattrinvres(nqp::create(List),List,'$!reified',self) } | 20:53 | ||
| should do the trick | |||
| jnthn | A .perl is probably only going to help people debug stuff | 21:18 | |
| So long as we don't change the REPR or add any validation to the methods it should be fine. | 21:19 | ||
| I don't see any negatives to adding a .perl method :) | |||
| I guess the "should do the trick" example is missing a .perl though, and that's not really correct even | 21:20 | ||
| Because it doesn't actually create the IterationBuffe | |||
| *IterationBuffer | |||
| It'd create a List | |||
| lizmat | multi method perl(IterationBuffer:D:) { nqp::p6bindattrinvres(nqp::create(List),List,'$!reified',self).perl } | 21:29 | |
| it would convert the IterationBuffer to a List and then .perl it | |||
| that should work, no? | 21:30 | ||
| timotimo | doesn't round-trip :) | 21:31 | |
| Geth | kudo/nom: 39bf63f574 | usev6++ | tools/build/create-jvm-runner.pl [JVM] Don't use --nqp-lib=blib for evalserver ... and perl6-debug-j. That makes them work after 'make install'. (They still fail to work otherwise.) |
21:32 | |
| kudo/nom: 71fd4178bf | lizmat++ | tools/build/create-jvm-runner.pl Merge pull request #996 from usev6/jvm_build_blib [JVM] Don't use --nqp-lib=blib for evalserver |
|||
| lizmat | timotimo: don't care | ||
| actually, it would, as the reified normally *is* an IterationBuffer | |||
| well, in a way :-) | 21:33 | ||
| timotimo | hm | ||
| jnthn | :P | 21:36 | |
| Maybe it should .perl to `do { given IterationBuffer.new { .push(1); .push(2); ... $_ } } :P | 21:37 | ||
| But yeah, if we add a better .perl it should try to round-trip | 21:38 | ||
| lizmat | FLATTENABLE_LIST ? :-) | 21:39 | |
| anyways, we can't build until pmurias has fixed World.nqp, right ? | 21:43 | ||
| bartolin | lizmat: btw, it looks like r-j has the same problem with nqp::null/Mu with $!next in R::It.CrossIterables and R::It.CrossIterablesMap as it had in R::It.OneValue :-/ | 21:48 | |
| lizmat: I'll try a spectest with a similiar workaround tomorrow | |||
| lizmat | I feared it would and hoped the nqp::null / Mu issue could be solved at a deeper level | ||
| the thing is that checking for nqp::null is about 2x as fast as checking for Mu | 21:49 | ||
| and these are always hot paths, so I would like to keep that :-) | |||
| bartolin | yeah, it definitely should -- but I don't know how to fix it | ||
| lizmat | ok, from now on I will add jvm specific code in the new stuff I'm writing | 21:50 | |
| bartolin will keep searching for a fix in nqp-j | 21:51 | ||
| lizmat | maybe jnthn has some ideas | 21:52 | |
| jnthn | lizmat: No, that's an internal detail that may go away some day :) | ||
| (to FLATTENABLE_LIST) | |||
| lizmat | yeah... and it even doesn't do what I wanted it to do (on a List) | 21:58 | |
| jdv79 | so... my laptop froze 2 times in a row running make install | 22:11 | |
| wondering if i should try again | 22:12 | ||
| as in kernel crash | |||
| mst | make *install* crashed it | 22:16 | |
| how bizarre | |||
| bartolin | .ask [Coke] do you have a suggestion whom I could contact about my CLA (cmp irclog.perlgeek.de/perl6-dev/2017-...i_13920361 ) | 22:17 | |
| yoleaux2 | bartolin: I'll pass your message to [Coke]. | ||
| jdv79 | actually no | ||
| lizmat | jdv79: maybe it ates all the memory ? | ||
| jdv79: also, do you have pmurias' last commit as well ? | |||
| jdv79 | that seems more plausible. box was dead but if i wait a few minutes it comes back | ||
| yes | |||
| lizmat | ah, then you need to revert that locally for now | ||
| jdv79 | oh cool. thanks! | 22:18 | |
| sortiz | Commit e041b0f broke rakudo's build. | ||
| lizmat | pmurias: are you ok with a revert of e041b0f, or are you close to a solution ? | 22:19 | |
| travis-ci | Rakudo build errored. Pawel Murias 'Fix f67df8ad9d4f920bb63ea40501f3b1db9d8e661d.' | 22:21 | |
| travis-ci.org/rakudo/rakudo/builds/192811365 github.com/rakudo/rakudo/compare/7...41b0f8276b | |||
| buggable | [travis build above] ā Did not recognize some failures. Check results manually. | ||
| geekosaur | even travis piling on :p | 22:24 | |
| lizmat | m: say [3] ~~ Cool # huh ??? | 22:25 | |
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«Trueā¤Ā» | ||
| lizmat | m: say [Nil] ~~ Cool # huh ??? | ||
| camelia | rakudo-moar 7d5bbe: OUTPUTĀ«Trueā¤Ā» | ||
| sortiz | The problem with the commit is that the new build_container_add_to_sc method (line 1527) calls itself, a typo imo | 22:26 | |
| Geth | kudo/nom: 5 commits pushed by lizmat++ | ||
| kudo/nom: c6662d14be | (Elizabeth Mattijsen)++ | src/Perl6/World.nqp Revert "Fix f67df8ad9d4f920bb63ea40501f3b1db9d8e661d." This reverts commit e041b0f8276b2abfbeefc3746c6fd26be33ef598. |
22:27 | ||
| lizmat | hmmm... apparently List is Cool ??? | 22:29 | |
| that is a bit of a WAT to me | |||
| Geth | kudo/nom: 92e704b4e7 | (Elizabeth Mattijsen)++ | src/core/Rakudo/Iterator.pm Introducing R:It.Rotor, the new .rotor workhorse |
22:40 | |
| sortiz | lizmat, the simple change in gist.github.com/salortiz/ceccd0c18...9d3b04ef89 fix pmurias commit _and_ fix DBIish also. | 22:44 | |
| b2gills .oO( Maybe I should go and get a commit bit, because if I had one I would have done ^ over two hours ago when I pointed the problem out ) | 22:49 | ||
| brokenchicken | Nil and Failure being Cool are more of a wat to me :/ | 22:51 | |
| sortiz | b2gills, sure. As pmurias acknowledged the problem I was only waiting for the fix... | ||
| brokenchicken | But yeah, lists are. Recall that debate on stringy methods doing weird things with Lists, like .contains() almost works. | 22:52 | |
| Geth | kudo/nom: d7b82149d3 | (Elizabeth Mattijsen)++ | src/core/List.pm Make List.rotor between 15x and 20x faster By either using the new R:It.Batch or R:It.Rotor iterators |
22:57 | |
| lizmat is testing sortiz' patch | 22:59 | ||
| pmurias | sorry, forgot I didn't push the commit :/ | 23:00 | |
| lizmat | *sigh* | 23:03 | |
| b2gills | lizmat: I tested it ( or rather I copied it by hand, and tested that ) Spec test passes | ||
| lizmat | well, I've learned not to push unless I have done a "make spectest" the hard way | ||
| Geth | kudo/nom: 703799ec24 | (Elizabeth Mattijsen)++ | src/Perl6/World.nqp Restore pmurias++ work, sortiz++, b2gills++ |
23:07 | |
| lizmat | and all of a sudden,. t/spec/S12-construction/destruction.t has 2 TODOd tests passing | ||
| that file hasn't basically been touched since 2009! | |||
| not sure whether that is a good thing or not | 23:08 | ||
| brokenchicken | lizmat++ # dat speed gain :o | 23:11 | |
| pmurias | do we have a 'make spectests' like thing that tests that the important modules in the ecosystem aren't broken? | 23:12 | |
| brokenchicken | none that I know of | 23:13 | |
| sortiz | Well, I can confirm DBIish is now fixed | 23:14 | |
| lizmat | whee! | ||
| brokenchicken | pmurias: but that's a good idea. At the very least should be part of pre-release checks | 23:15 | |
| lizmat | well, that's called "cpan-testers" more or less :-) | ||
| maybe mst has some ideas about that :-) | |||
| pmurias | cpan-testers is awesome | 23:27 |