dalek ast: c3923e8 | (Zoffix Znet)++ | S06-other/main-usage.t:
Unfudge now-passing test for RT#127977

sub MAIN constraint checks must not leak to the user of the program.
Fixed by one of these commits:
  github.com/rakudo/rakudo/compare/7...ad497dd0cb
00:03
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127977
[Coke] . 01:15
BrokenRobot: There is a presaved query at rakudo.org/tickets/ 01:16
that links to tickets that need docs. I'll try to include the link the next time I mention it.
MasterDuke [Coke]: do you happen to have some sort of admin access to rt.perl.org? 01:17
.seen timotimo 01:22
yoleaux2 I saw timotimo 5 Jul 2016 20:18Z in #perl6-dev: <timotimo> ooooh
jdv79 [Coke] is an admin or more iirc 01:24
at least for the p6 stuff
Zoffix [Coke], out of all those links, only the testsneeded URL is working. The rest give query error 01:26
MasterDuke excellent, then i hope he returns soon
Zoffix And I'm surprised the tests one doesn't give the CSFR error 01:27
Actually all of them have CSRF errors if I try another browser (not logged in there) 01:29
" There was an error parsing your search query: Wrong query, expecting a VALUE in 'Queue=%>27perl6<--here%27 AND Status=%27new%27'. Your RT admin can find more information in the error logs. "
[Coke] awwaiid - added you on RT 01:31
MasterDuke: I have admin rights to the perl6 bugadmin queue.
Zoffix That error sounds a lot like the one I was getting a few days ago when I was trying to use RT's REST API. This URL worked after lots of twiddling: github.com/zoffixznet/Rele6sr/blob...RT.pm6#L44 01:32
MasterDuke [Coke]: hmm, would you be able to fix my account?
Zoffix ¯\_(ツ)_/¯
MasterDuke i can't see/edit tickets when i'm logged in
[Coke] For the errors, we can try to find a URL that works now, or we can open a ticket with the perl rt admins. 01:33
MasterDuke: All i can do is give you admin privs to perl6 tickets.
If you're having other issues, you need to talk to the perl rt admins. 01:34
MasterDuke ok. i tried in irc.perl.org:#rt and irc.freenode.net:#rt a couple days ago, but haven't heard anything from either 01:36
and i think part of my problem is my emails aren't getting through 01:37
Zoffix Register a throw-away email on gmail.com and try that? Try creating a new RT account?
¯\_(ツ)_/¯
MasterDuke i did try another gmail account that i never use for anything 01:38
Zoffix No idea, sorry.
timotimo i've spoken up against migrating away from RT in the past, but a few of us are getting absolutely screwed over by that piece of ... software 01:40
[Coke] tbrowder: your rt question about "top 10 tickets" isn't something I can help with 01:41
MasterDuke: what email address did you try to send the report from? 01:42
and to? (privmsg me)
psch hm, do we get any info to the user about post constraints on MAIN with what fixed #127977? 05:38
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=127977
psch 'cause that's kinda the coolest bit that happened in the branch tadzik linked in the ticket
alright, the getextype problem is because we're actually not creating a VMException in throwcatdyn_c 07:30
which seems somewhat wrong, considering getextype needs one
but i think we can just create one in throwcatdyn_c, so i'm gonna try that 07:31
[Tux] This is Rakudo version 2016.06-150-g41b685e built on MoarVM version 2016.06-9-g8fc21d5 07:44
test 16.005
test-t 9.593
csv-parser 16.777
psch humm, creating a VMException there seems to really blow up compilation time o.o 08:01
'bout 20 minutes now, and it's only at stage2/NQP.nqp 08:02
nine You sure it's still doing something usefull? 08:05
psch yeah, it's still going, at Perl6-Actions.nqp now 08:11
i mean, driving total compilation time up by some 300% is a bit much to explain with "throwcatdyn_c is a really hot method", but it's not completely impossible vOv 08:12
otoh, compared to previously, it now allocates a VMExceptionInstance *and* a Throwable for the Backtrace 08:13
and i think especially the Backtrace is probably rather expensive 08:14
nine What's it doing with the VMExceptionInstance anyway? 08:16
psch the handler for lexical &return needs a VMExceptionInstance 08:17
nine But what for? 08:18
psch to know what kind of Exception it is?
&EXCEPTION in src/core/Exception.pm:250 for example
nine How do you create the VMExceptionInstance? 08:20
psch pretty much exactly as die_s_c does 08:21
that is, get the exType from the hllConfig, call allocate on the STable
well, the REPR
then set field
+s 08:22
i'm pretty sure the biggest performance impact is the Backtrace, and i'm not sure we actually need it
nine Can't you leave out the nativeTrace in throwcatdyn_c? Considering that we got by without a VMExceptionInstance at all previously, we may not actually need everything.
psch right, i'm still gonna let the build finish, can't be longer than another 20 to 30 minutes... :) 08:23
nine All other fields should be dirt cheap
psch gotta get more coffee and some breakfast anyway
nine coffee++
psch also i'm kinda curious what stage parse comes out as heh 08:27
nine You must be a very patient man 08:28
psch well, i'm doing other things on the side, so it's not so bad 08:37
not even 9 minutes for stage parse, not so bad :P 08:41
...but over 20 minutes stage jast, which i am now fed up with hah 09:05
jnthn psch: I suspect constructing a new Throwable on every single return/next/last etc. could easily do it 10:02
psch: That's really expensive 'cus it snapshots the entire call stack
psch jnthn: right, which is why i took that out. but apparently even allocating the VMExceptionInstance takes long enough for an unbearable impact 10:03
as in, i've been running Configure.pl without the new Throwable for almost an hour now
not what else to try there, tbh, i don't think leaving a static VMExceptionInstance around would work 10:04
jnthn iirc, on Moar we only construct the VMException instance if the target handler actually needs it 10:05
Handlers that are just "goto"-ish don't need it
And you can figure that you from the handler metadata, I believe.
Same should be possible on JVM
psch i don't think i know all of the handler fields 10:06
jnthn In fact must be, otherwise we'd not know to invoke the handler on the top of the call stack.
psch i *think* handlerInfo[1] is the extype, but does that tell me if it needs an vmExObj..?
jnthn /* Exception handler kinds. */ 10:07
public static final int EX_UNWIND_SIMPLE = 0;
public static final int EX_UNWIND_OBJECT = 1;
public static final int EX_BLOCK = 2;
It's whatever is getting compared agaisnt those constants
psch ah, okay
oh ffs 10:08
i actually commented the new Throwable in die_s_c, not throwcatdyn_c ._.
at least that easily explains why it didn't seem faster than the compile before... 10:09
Zoffix timotimo, the problem I see with all the calls to migrate from RT is no one's offering to be the person to do the enormous amount of work involved. 10:32
IIRC MadcapJake offered to make a better interface for bug reporting, but I'm not aware of even that being completed. Actually migrating would be orders of magnitude more work. 10:33
AlexDaniel Zoffix: migrate where? 10:56
if given enough permissions to work on stuff, I can do it. And I think that there are others who will happily help with it. The problem is that we can't decide what we should use instead of RT 10:57
Zoffix nine++ with RT#128457 gone I can now write my Mojolicious + Perl 6 article \o/ 11:06
synopsebot6 Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128457
Zoffix Based on all the likes my peak-preview of the Perl 6 Mojo app got, it should be a well-read post.
*peek
nine Yeah! 11:08
psch well, i've got the getextype fixed with always creating a VMExceptionInstance, and that shows we still get the BEGIN EVAL problem 11:13
i'll probably change the getextype to "only create if the handler needs it" as jnthn++ suggested, though, so no commit yet 11:14
nine psch: the last thing rakudo-j needs is slower build times :) 11:15
psch well, allocating a VMExceptionInstance doesn't really seem to impact it
clean build still at around 13min
nine oh
So it really was just those backtraces
psch as mentioned, i commented the new Throwable in die_s_c instead of throwcatdyn_c
yeah
dalek kudo/nom: c843682 | lizmat++ | src/core/Array.pm:
Make Array.push|append about 40% faster

  - remove ensure-allocated, integrate in code where needed
   This is really the big win: the ensure-allocated method for some
   reason did *not* get JITted.
  - rewrite hot code completely using nqp ops
11:46
lizmat hopes for some good test-t effects 11:52
yoleaux2 5 Jul 2016 21:41Z <AlexDaniel> lizmat: fwiw, numeric overflow changes were before 2015.12, but you can still find it with bisectable: bisect: good=2015.10 2 ** 99999999999999999999999999999999999
BrokenRobot wow! lizmat++ that's awesome! 12:06
Looks like test-t soon (already?) will be in the 8s rnage \o/ 12:07
lizmat hopes so 12:11
travis-ci Rakudo build passed. Elizabeth Mattijsen 'Make Array.push|append about 40% faster 12:41
travis-ci.org/rakudo/rakudo/builds/142733653 github.com/rakudo/rakudo/compare/4...43682f3b58
[Coke] goes back to work today! (bah!) 13:10
gfldex jnthn: i'm getting «Internal error: invalid thread ID 111254976 in GC work pass» when use Gumbo;. Is there any easy way to provide you with debug info? 13:31
DrForr For those of you needing an excuse to get cheap Guinness - conferences.oreilly.com/oscon/open-...tail/53512 13:36
nine DrForr: in your bio "for Parrot née Perl 6" looks like a typo? 13:38
DrForr The ' née ' bit? I thought that was the proper way to spell that, with the accent. 13:39
perlpilot it is
nine Oh, then I just learned a new English word :) 13:40
DrForr++
DrForr www.wordnik.com/words/nee # The second definition.
It's pretty rare, but when you're looking for the right words... 13:43
gfldex please update Pod::To::HTML on docs.perl6.org to get github.com/perl6/Pod-To-HTML/commi...478f36407c 13:48
Skarsnik Skarsnik> Hello, I am not sure if someone remember my leak issue with rakudo/moar and the XML module. I was curious if it's finally fixed, but it's still here (even if it's smaller than before) gist.github.com/Skarsnik/03b970d2a4b827ba1e1d It was fixed for some revision it seem according to this gist 13:50
timotimo at least one thing in there has been fixed since 13:52
i remember reversing the order of nfg and fsa destroying
Skarsnik Let me add an updated output 13:53
timotimo because it used to destroy the FSA first, then try to free the NFG thing which had been allocated inside the FSA
that'd be good 13:54
Skarsnik I added a 2a - recent 13:55
I don't remember how to get the valgrind output thou
timotimo valgrind --leak-check=full, and don't forget to --full-cleanup for moarvm 13:59
psch well, perl6-valgrind-m exists
timotimo right, but i'm not sure it has --full-cleanup by default
and it also doesn't allow you to give valgrind options 14:00
really, someone who knows a bit about shell scripting ought to have a quick look at that thing
psch yeah
Skarsnik I need to rebuild moarvm ?
timotimo (that's how i spell "not me")
no, you do not
Skarsnik or just edit the option
timotimo yeah, just put it into the shell script
psch okay this is just weird 14:01
the handler that gets invoked doesn't take any arguments, according to ArgsExpectetation
but it apparently invokes something that needs the vm_ex
...how does that work
and how does the handler pass on the vm_ex when i create it in throwcatdyn_c 14:02
timotimo doesn't it get passed through the throwing mechanism? 14:03
psch 'cause according to the backtrace it's the invokeDirect call in ExceptionHandling.java:215 that calls the handler
timotimo just guessing
psch and that calls something that doesn't take args, the insightfully named method B9E5B486DDF0CC0DE86076CC049FDA9257D5E15D.qb_12146
Skarsnik what 14:04
root@testperl6:~/piko# head /root/.rakudobrew/bin/perl6-valgrind-m
#!/usr/bin/env perl
# I AM RAKUDOBREW. ALSO POSSIBLY SPARTACUS. OUTLOOK CLOUDY, ASK AGAIN LATER.
use strict;
psch oh. both hll routines that could refer to get their args with p6argvmarray 14:05
BrokenRobot 0.o
tadzik it is self aware
sorry
BrokenRobot :)
psch okay, yeah, i'm gonna stick with the "just always build a VMExceptionInstance" approach 14:06
timotimo Skarsnik: are 2a recent and 2a - recent the same file?
psch it works, it doesn't seem to significantly impact performance, and i kinda understand it vOv
Skarsnik Yeah timotimo, I though it does not added the file or I am blind and add it again x) 14:07
timotimo hehe.
Skarsnik hm, using the correct perl6-valgrind-m (and adding the full-cleanup stuff and co) I get Unhandled exception: failed to load library 'dynext/libperl6_ops_moar.so' 14:11
at <unknown>:1 (/root/.rakudobrew/moar-nom/perl6.moarvm:<dependencies+deserialize>)
jnthn gfldex: Probably, file as good reproduction instructions as you can unless you fancy further hunting. In the latter case, you can go to your MoarVM folder, and in src/gc/debug.h turn the MVM_GC_DEBUG define to 1, make clean, make install, and see if you get a different error. If yes, then a run under gdb (or your favorite tool) with a breakpoint set in MVM_panic and then getting the backtrace would also perhaps give some clues. Depends how mu 14:12
dalek p: 32ad2eb | peschwa++ | src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java:
Always create a VMExceptionInstance in throwcatdyn_c.

Otherwise we have cases in Rakudo where we expect a control exception, but get null instead. There's probably a more specific place where this could be solved, but it eludes me.
14:13
kudo/nom: 0295443 | peschwa++ | / (2 files):
Add two debug ENV variables.

Those are RAKUDO_JDB_PORT in perl6-jdb-server for setting the jdb port and RAKUDO_PRECOMP_NESTED_JDB, which, when set, uses perl6-jdb-server with increasing ports for precompilation.
kudo/nom: d505a8c | peschwa++ | tools/build/NQP_REVISION:
Bump NQP_REVISION, for a getextype fix.

This prevents reaching nqp::getextype calls without a valid argument.
Skarsnik this take forever to run in valgrind x) 14:27
timotimo of course it does :)
but valgrind still has a very good pay-off for its cost
Skarsnik a libpath was removed for some reason in the valgrind script btw 14:29
timotimo huh.
Skarsnik I edited the wrong file, I need to rerun this thing again x) 14:30
timotimo oh
Skarsnik I mean the wrong perl6 script 14:31
not the valgrind shell script
--libpath="/root/.rakudobrew/moar-nom/install/share/perl6/runtime" this was missing 14:32
or whatever is generated from
timotimo is a bit wary of rakudobrew's generated forwarder scripts 14:34
Skarsnik timotimo, gist.github.com/Skarsnik/03b970d2a...rind-trace 14:43
timotimo ugh, no line numbers
[Coke] RT: 1340; CONC: 7; GLR: 6; JVM: 70; LHF: 1; LTA: 73; NYI: 28; OSX: 6; PERF: 16; POD: 3; PRECOMP: 4; RFC: 22; SEGV: 22; STAR: 1; TESTNEEDED: 14; TODO: 9; UNI: 5; WEIRD: 3 14:47
lizmat afk& 14:48
sno [Tux]: do you have a moment wrt. DBD::File? (query?) 14:53
gfldex moritz: please update Pod::To::HTML on docs.perl6.org to get github.com/perl6/Pod-To-HTML/commi...478f36407c 15:10
moritz gfldex: doing it now 15:11
... done 15:13
gfldex thanks
Tux_ sno, yes, but I don't think this channel is the right spot 15:40
sno Tux_: that's why I suggested a query
BrokenRobot ENODALEK 16:18
[Tux] This is Rakudo version 2016.06-153-gd505a8c built on MoarVM version 2016.06-9-g8fc21d5 19:02
test 16.186
test-t 9.331
csv-parser 17.269
dalek ast: 98952f9 | (Zoffix Znet)++ | S06-traits/is-readonly.t:
Fix broken plan
19:05
travis-ci Rakudo build errored. Pepe Schwarz 'Bump NQP_REVISION, for a getextype fix. 20:40
travis-ci.org/rakudo/rakudo/builds/142773456 github.com/rakudo/rakudo/compare/c...05a8c4ad46
lizmat m: say 0.rand # sorta expected to see Nil there 21:29
camelia rakudo-moar d505a8: OUTPUT«0␤»
timotimo that'd be a strange edge-case, IMO 21:35
lizmat well, it's an edge case that perhaps needs to be highlighted somehow 21:36
because usually it means that the value you want a rand of, was not initialized
or somehow lost its value 21:37
not really important, though
timotimo that sounds more like you want to warn when you use an actually undefined value
i can see having the invocant be a number that varies between 0 and whatever
dalek kudo/nom: 55c359e | lizmat++ | src/core/ (2 files):
Make Array use List.is-lazy

No reason to special case Array
21:38
Zoffix m: say 09 22:31
camelia rakudo-moar 55c359: OUTPUT«Potential difficulties:␤ Leading 0 does not indicate octal in Perl 6.␤ Please use 0o9 if you mean that.␤ at <tmp>:1␤ ------> say 09⏏<EOL>␤9␤»
Zoffix m: say 08
camelia rakudo-moar 55c359: OUTPUT«Potential difficulties:␤ Leading 0 does not indicate octal in Perl 6.␤ Please use 0o8 if you mean that.␤ at <tmp>:1␤ ------> say 08⏏<EOL>␤8␤»
Zoffix PR to Fix: github.com/rakudo/rakudo/pull/812
And I'll do tests for rt.perl.org/Ticket/Display.html?id=119339 if/when the PR is merged.
Oh, wait. Made a booboo. 22:34
($value can be more than 1 digit) 22:35
travis-ci Rakudo build passed. Elizabeth Mattijsen 'Make Array use List.is-lazy 22:36
travis-ci.org/rakudo/rakudo/builds/142887649 github.com/rakudo/rakudo/compare/d...c359e8e448
Zoffix K, now it's good: github.com/rakudo/rakudo/pull/813 23:13
gfldex Zoffix: if you got the time please review github.com/perl6/Pod-To-HTML/commi...c87b589b26 23:33