MasterDuke what's a good way in nqp-land to know if something is an enum? 00:12
to be more precise, in World.nqp 00:14
timotimo you can .HOW things, can't you? 00:15
otherwise probably nqp::gethow
MasterDuke the .HOW.name is just the name of the enum
timotimo yeah, but what's the .HOW.HOW.name? :D 00:16
MasterDuke hm, i thought it did try that, but let me check
timotimo m: say True.HOW.HOW.name(True.HOW)
camelia Perl6::Metamodel::EnumHOW
MasterDuke well there you go. wonder what it was i tried... 00:17
AlexDaniel_ “Message body is not shown because it is too large.” *facepalm*
timotimo ;(
samcv should i just todo all jvm nqp failures? 00:26
y/n
timotimo modules.perl6.org now updated in about 55 minutes 00:27
so 20 minutes saved perhaps?
samcv AlexDaniel_, MasterDuke some of the waste that's in INTERPOLATE. i added some 'note's to nqp
gist.github.com/39c206ff7c45649e26...b9156b2323
'hi' ~~ /hi/ compared to my $var = 'hi'; 'hi' ~~ /$hi/ 00:28
3 vs 13 times called. the nqp !INTERPOLATE method
MasterDuke which is 13? 00:29
AlexDaniel_ 'hi' ~~ /$hi/
samcv ^
timotimo it looks like we can give Pithub an LWP::UserAgent object we create ourselves; if we initialize it with keep_alive => 10 or something we can save a bit of time spinning up and tearing down connections perhaps?
samcv where it outputs "literal␤" is in `method termish` btw 00:30
AlexDaniel_ samcv: by the way, why am I being pinged? What's so special about me in this particular case? :)
samcv idk
you are here
what, 20 seconds before i highlighted you :) 00:31
and you're often interested in text parsing/unicode things
AlexDaniel_ ah ok, I was just thinking that maybe something has to be bisected or whatever 00:32
samcv i've at least added to roast the issues between 'ignorecase' not working properly for INTERPOLATE. so that's decent. wasn't able to figure out the cause though. i wish i understood how nqp worked that way 00:33
may be issues with 'ignoremark' or 'ignorecase+ignoremark' but i have yet to find bigs with those when the /$thing/ is interpolated versus /not/. unless you can think up some (or remember) 00:34
MasterDuke i thought i fixed the ignore* in INTERPOLATE recently? 00:36
samcv well there are at least some bugs still
MasterDuke i just spotted a wild TimToady over in #perl6, he might have some good input 00:37
samcv github.com/perl6/roast/commit/946d2ff79 and also see what i wrote in rt.perl.org/Ticket/Display.html?id=132233
though i can't count on it. i remember it failing in all cases i tried *before* but now it only fails with interpolated haystack in cases i cannot determine (it fails in roast test) 00:38
so if you say you fixed an issue with interpolated haystack, then maybe i remember correctly and it used to *always* fail with interpolated haystack and only does sometimes. because that test succeeds when i do it in REPL but in roast, it fails
have not yet figured out WTF is going on 00:39
timotimo going to bed, seeya :)
MasterDuke c6aacafd1f6cbb069befd0236e9f246092e11640 should have fixed some things, but i guess not everything 00:40
samcv ah nice. so that was the thing that fixed some cases 00:42
though it is still broken on the nqp side
MasterDuke yeah, i don't think i looked at nqp at all when i was working on INTERPOLATE 00:43
samcv i'm guessing that may be the descrepency. since nqp seems totally broken. but on perl6 some cases it works some cases it does not with interpolate
though i mean with perl6 at least. it works and doesn't work with perl6. and i'm not sure when/why it works/doesn't just that somee cases work and some don't
but big ++ to making the situation better :) 00:44
Geth rakudo: MasterDuke17++ created pull request #1181:
Suggest enum values as types
00:59
rakudo/nom: 12774237a6 | (Daniel Green)++ | src/Perl6/World.nqp
Suggest enum values as types

Implements RT #123926, e.g., `enum E <Foo Bar>; sub x(Floo) {}` now says
  `Invalid typename 'Floo' in parameter declaration. Did you mean 'Foo'?`
01:21
rakudo/nom: f62ae60c48 | (Zoffix Znet)++ (committed using GitHub Web editor) | src/Perl6/World.nqp
Merge pull request #1181 from MasterDuke17/RT123926

Suggest enum values as types
synopsebot RT#123926 [new]: rt.perl.org/Ticket/Display.html?id=123926 [BUG] LTA error message without Levenshtein distance suggestion when mistyping enum value in signature in Rakudo
rakudo: skids++ created pull request #1182:
Warn on typical precedence errors with infix:<..> (RT#127279)
02:27
synopsebot RT#127279 [new]: rt.perl.org/Ticket/Display.html?id=127279 [BUG] Infinite loop with |"a".."z" 02:28
Geth rakudo/corresponding-starter: 279bae08a5 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Grammar.nqp
Mention corresponding starter in <, <<, « subscripts

Resolves part of RT #125641.
Wordy descriptions were substituted with actual symbols to make it more consistent with other error messages. Also, this way the error message is much shorter.
02:35
synopsebot RT#125641 [new]: rt.perl.org/Ticket/Display.html?id=125641 [LTA] error messages when right bracket is missing: $<> , <>, () and so on
rakudo/corresponding-starter: 6542bb8032 | (Aleks-Daniel Jakimenko-Aleksejev)++ | 2 files
Mention corresponding starter for most of the things

This should handle quotes, parens, brackets, subscripts and many other things.
Resolves part of RT #125641.
Unfortunately I don't know how to print the actual starter, but as long as the line number is mentioned it's alright.
rakudo: AlexDaniel++ created pull request #1183:
“corresponding … was at” RT #125641
02:36
AlexDaniel_ that's depressing 02:41
MasterDuke AlexDaniel_: did you happen to try with passing :cache(1) to lineof()? 02:47
AlexDaniel_ MasterDuke: no, I don't know what cache does 02:48
MasterDuke: fwiw this commit has no :cache also github.com/rakudo/rakudo/commit/38...eb2d50fef6 02:49
MasterDuke sometimes the line info has already been calculated and cached, it's a bunch faster
yeah, i added it more recently than that, when i got backtraces to print the actual filename+linenumber instead of the generated file 02:50
Geth roast: 3014999fcd | (Daniel Green)++ | S32-exceptions/misc.t
Add test for suggesting enum values as types

For RT #123926.
02:52
synopsebot RT#123926 [new]: rt.perl.org/Ticket/Display.html?id=123926 [BUG] LTA error message without Levenshtein distance suggestion when mistyping enum value in signature in Rakudo
AlexDaniel_ MasterDuke: oh, it was you who did that 02:54
awesome
MasterDuke yeah, almost a year ago now 02:56
AlexDaniel_ MasterDuke: that seems to be working fine 02:58
MasterDuke cool
Geth rakudo/corresponding-starter: a462d0a26e | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Grammar.nqp
Cache some line numbers

  MasterDuke++
  irclog.perlgeek.de/perl6-dev/2017-...i_15269029
03:00
rakudo: MasterDuke17++ created pull request #1184:
Use cached line numbers for token label
03:23
samcv going to improve the error when you try and use whitespace with a combiner as a separator 05:33
to try and close this LTA RT
[Tux] This is Rakudo version 2017.09-230-gf62ae60c4 built on MoarVM version 2017.09.1-576-g64b5dc03 07:57
csv-ip5xs 1.257 - 1.360
test 9.982 - 10.344
test-t 3.219 - 3.236
csv-parser 0.883 - 0.901
smls Does anyone know how variables interpolated into regexes like / @foo / are parsed by Rakudo? 08:47
P6Regex'es `metachar:sym<var>` looks like it only catches backreferences, not external variables: github.com/perl6/nqp/blob/576d78ee...r.nqp#L334 08:48
ah, there's github.com/rakudo/rakudo/blob/f62a....nqp#L5314 08:52
buggable New CPAN upload: IRC-Client-Plugin-Github-0.1.2.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...1.2.tar.gz
New CPAN upload: Config/Config-1.2.1.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...2.1.tar.gz 09:02
New CPAN upload: Config-Parser-toml/Config-Parser-toml-1.0.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...0.0.tar.gz 09:12
New CPAN upload: Config-Parser-yaml/Config-Parser-yaml-1.0.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...0.0.tar.gz
New CPAN upload: Hash-Merge/Hash-Merge-0.1.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...1.0.tar.gz
New CPAN upload: MPD-Client/MPD-Client-0.1.2.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...1.2.tar.gz
Geth roast: Tyil++ created pull request #342:
WIP: Expand file tests
09:36
lizmat Files=1227, Tests=75600, 346 wallclock secs (14.70 usr 5.24 sys + 2419.35 cusr 218.79 csys = 2658.08 CPU) 09:44
tyil github.com/perl6/roast/pull/342/fi...d3a273fR54 this returns a Failure, not a Bool at this moment, should I adapt the test or is the Failure return incorrect? 09:46
Geth rakudo/nom: 26bdc95c78 | skids++ | 2 files
Warn on typical precedence errors with infix:<..> (RT#127279)

Alerts user to potential accidental use of prefix:<|> or
   prefix:<~> on a range start value, when they are usually intended
   to apply to the entire range, and encourages use of parens.
10:41
rakudo/nom: cb9d2e040d | lizmat++ (committed using GitHub Web editor) | 2 files
Merge pull request #1182 from skids/rt127279

Warn on typical precedence errors with infix:<..> (RT#127279)
synopsebot RT#127279 [open]: rt.perl.org/Ticket/Display.html?id=127279 [BUG] Infinite loop with |"a".."z"
rakudo/nom: 4868e9271a | (Zoffix Znet)++ | 2 files
Remove dead code for `use v5`

Dies while complaining about missing `Perl5` module. A thought occured to make it load Inline::Perl5, but felt like useless magic, so I removed the code entirely and just make it complain there's no compiler for v5, like it'd do for `use v6.z`
Fixes RT#130834: rt.perl.org/Ticket/Display.html?id=130834
11:50
synopsebot RT#130834 [open]: rt.perl.org/Ticket/Display.html?id=130834 Is `use v5` still a thing?
Geth rakudo/full-make-test: 50 commits pushed by (Zoffix Znet)++, skids++, (Elizabeth Mattijsen)++, (Jonathan Worthington)++, (Patrick Spek)++, (Samantha McVey)++, (Daniel Green)++
review: github.com/rakudo/rakudo/compare/c...54d5096732
12:20
rakudo/full-make-test: 6757f93473 | (Zoffix Znet)++ | t/02-rakudo/01-is_approx.t
Fix up t/02-rakudo/01-is_approx.t tests

The author of planck tests wrote them without any regard to `e-34` part of the number and adjusted the numbers in the decimal portion alone, expecting changes to be of 1e-5 magnitude when they were actually of 1e-39 magnitude.
Fix by shifting the original value by expected differences instead.
12:48
roast: 4f224ec562 | (Patrick Spek)++ (committed by Zoffix Znet) | S32-io/file-tests.t
Expand file tests (#342)

  * Use subtests
  * Add tests for .d
  * Add tests for IO.l
... (34 more lines)
12:51
Skarsnik Hello :) 12:59
Geth rakudo/full-make-test: 0d5bb9097a | (Zoffix Znet)++ | t/02-rakudo/dump.t
Fudge failing tests

Not sure there's any point in fixing this, since it's not part of the language and we already got `dd` and language-wise we got `.perl`
Skarsnik How the squashaton going? 13:00
Geth rakudo/full-make-test: 93c4555db3 | (Zoffix Znet)++ | t/02-rakudo/dump.t
Unfudge now-passing test
Skarsnik squashable6, status 13:01
squashable6 Skarsnik, 🍕🍕 SQUASHathon is in progress! The end of the event in ≈22 hours. See github.com/rakudo/rakudo/wiki/Mont...Squash-Day
Skarsnik, Log and stats: gist.github.com/276df459191e96cb5c...dcfdc60aef
Zoffix .tell bartolin I think Proc::Async is already implemented on JVM, so maybe this ticket can be closed and all the fudges unfudged? rt.perl.org/Ticket/Display.html?id=126524 13:06
yoleaux Zoffix: I'll pass your message to bartolin.
12:55Z <tyil> Zoffix: thanks for the tip :>
Zoffix Hm, looks like we're running with zero REPL tests whatsoever. 13:11
Because all the tests are run as non-TTY and it just evals the code instead of firing up REPL. 13:12
Do we have a way to make Proc pretend STDIN it as a tty? 13:13
Zoffix doesn't know anything about TTY stuff
Geth roast: c4d7af0c4d | (Zoffix Znet)++ | 2 files
Toss is_run_repl helper routine

It's not used anywhere and all but the basic sanity REPL tests have been moved to rakudo's test suite
13:15
rakudo/full-make-test: 78eca2141d | (Zoffix Znet)++ | 2 files
Start making repl tests work
13:25
Zoffix After a bit of googling don't see a sure-fire portable way to fake a TTY, so I'm gonna add a compiler command line flag to force interactive mode even when STDIN aint a TTY 13:27
dogbert17: doubt it. I think it was already fixed before. Bisectable can likely tell 13:32
"Stage parse : 94.820" man. And I used to feel sad about low 80s :( this box sucks
lizmat hmmm... --profile-compile on the setting compile does not appear to generate a profile :-( 13:39
timotimo does it finish? 13:44
dogbert2 Zoffix: can you do IO stuff with that bot? 13:46
bistect: my $f = open("test.txt", :rw); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode 13:47
bisect: my $f = open("test.txt", :rw); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode
bisectable6 dogbert2, Bisecting by output (old=2015.12 new=4868e92) because on both starting points the exit code is 1
dogbert2, bisect log: gist.github.com/3d98e8c1cd197ef3b3...a006db2cd7
dogbert2, (2015-12-25) github.com/rakudo/rakudo/commit/07...dc61f84053
dogbert2, The result looks a bit unrealistic, doesn't it? Most probably the output is different on every commit (e.g. 「bisect: say rand」)
Zoffix usually creates all the files in `/tmp` to not pollute dirs with test files
timotimo yeah, you have to do it in /tmp, otherwise no write access 13:48
Zoffix bisect: my $f = open("/tmp/dasdasdasdastest.txt", :rw); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode
bisectable6 Zoffix, On both starting points (old=2015.12 new=4868e92) the exit code is 0 and the output is identical as well
Zoffix, Output on both points: «hello»
Zoffix releases: my $f = open("/tmp/dasdasdasdastest.txt", :rw); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode
c: releases my $f = open("/tmp/dasdasdasdastest.txt", :rw); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode
committable6 Zoffix, ¦releases (23 commits): «hello»
Zoffix c: releases my $f = open("/tmp/dasdasdasdastest.txt", :r, :w); $f.write("hello".encode); $f.seek(0); say $f.read(5).decode
committable6 Zoffix, ¦releases (23 commits): «hello» 13:49
dogbert2 so it was fixed ages ago then
ok, if I find some tests in roast I'll close it 13:51
lizmat timotimo: yes, but no file :-( 13:54
timotimo huh.
Geth roast: skids++ created pull request #343:
Add tests for new range precedence worries (RT#127279)
14:02
synopsebot RT#127279 [open]: rt.perl.org/Ticket/Display.html?id=127279 [BUG] Infinite loop with |"a".."z"
Geth roast: 9d56ba1e58 | skids++ | S03-operators/range.t
Add tests for new range precedence worries (RT#127279)
roast: 6e9bab5de4 | skids++ (committed using GitHub Web editor) | S03-operators/range.t
Merge pull request #343 from skids/rt127279

Add tests for new range precedence worries (RT#127279)
Zoffix heh parse stage 96s now! :) It goes up by 2s every time I run it ^_^ 14:05
timotimo oops 14:06
Zoffix What's the standard practice to rebuild nqp when I make changes to it? 14:07
timotimo make install; cd ../rakudo; make clean; make install
Zoffix thanks
timotimo just what i use 14:08
timotimo is taking a heap snapshot of the heap snapshot analyzer analyzing a heap snapshot
Geth roast: e3e1f75cc5 | usev6++ | 6 files
[jvm] Remove fudges for RT #126524

Proc::Async has been implemented for the jvm backend; jnthn++.
Unfortunately, some of the fudged tests still fail. I updated the fudge reasons accordingly.
14:11
synopsebot RT#126524 [open]: rt.perl.org/Ticket/Display.html?id=126524 [JVM] Proc::Async NYI
Zoffix \o/
bartolin Zoffix++ I'm gonna close the issue
yoleaux 13:06Z <Zoffix> bartolin: I think Proc::Async is already implemented on JVM, so maybe this ticket can be closed and all the fudges unfudged? rt.perl.org/Ticket/Display.html?id=126524
bartolin sadly it's not possible to run spectest on the jvm backend atm due to a severe memory leak (RT #132104), but i looked at the aboved tests in isolation 14:13
synopsebot RT#132104 [open]: rt.perl.org/Ticket/Display.html?id=132104 [JVM] EvalServer seems to leak memory
timotimo i've already filtered to only major collections, but it's already past 4 gigs of heap snapshot data
Geth rakudo/nom: f59b2887ad | (Zoffix Znet)++ (committed using GitHub Web editor) | lib/NativeCall.pm6
Fix precedence error; RabidGravy++
14:14
lizmat afk for some 2049 &
timotimo cool
Zoffix bartolin: sucks about `doesn't-hang` errors. I'm trying to make repl tests in rakudo's test suite run and some of them use it :/ And there's no fudger for `make test` 14:15
bartolin Zoffix: I'll try to take a closer look at that error (haven't seen that specific error before). Please just go ahead and use it, anyway! 14:16
Zoffix OK
By the sounds of it. Proc::Async.kill is not implemented on JVM 14:17
(that's what doesn't-hang uses under the hood)
bartolin yeah, jnthn++ did map that op with nqp commit 76098315da, but it's NYI 14:20
timotimo oh, looks like i was running a moar without the filter functionality %) 14:23
Zoffix man, this is taking too long. Still have trouble rebuilding nqp :/ 14:35
"/home/zoffix/CPANPRC/rakudo/install/bin/nqp-m: 2: exec: ../install/bin/moar: not found"
$ type build-nqp
build-nqp is aliased to `cd nqp; perl Configure.pl --gen-moar --backend=moar --prefix=../install; make; make install; cd ..; make clean; make install'
timotimo perhaps the cd fails? :\ 14:38
Zoffix timotimo: do I need to manually build MoarVM too then? 14:41
timotimo shouldn't have to
as long as the moar binary is in that prefix it should be fine 14:42
Zoffix this is getting on my tits 14:43
Zoffix runs `rm -fr install; build-rakudo` 14:45
timotimo: how do you get it to the point that all you do is `make install; cd ../rakudo; make clean; make install` ? What's the setup with a fresh checkout of everythinng? 14:49
timotimo i git clone moar, nqp, and rakudo to be side-by-side rather than nested, and the install/ folder which is my prefix lives as another sibling 14:51
Geth nqp: c20643c97c | (Samantha McVey)++ | docs/ops.markdown
[docs] Document `ordbaseat` op in docs/ops.markdown
nqp: 0bed230bda | (Samantha McVey)++ | src/HLL/Grammar.nqp
Improve error message when space with combiners is encountered

New message: q ̌̌(hi) Whitespace character ‘SPACE’ (0x20) with 2 combining characters is not allowed as a delimiter at line 2, near " ̌̌(hi)\n"
Zoffix ok, I'll try that
Geth rakudo/nom: ba2d858fb2 | (Zoffix Znet)++ (committed using GitHub Web editor) | lib/NativeCall.pm6
Fix precedence error; RabidGravy++
Zoffix gah, my terminal's busted now due to that commit message 14:53
samcv oh?
Zoffix Yeah, crappy unicode support
samcv can't handle two combining carons or something. or maybe that it's on a space. it shows up in hexchat as on the parenthesis for me 14:54
Geth nqp: 108db21525 | (Zoffix Znet)++ | src/HLL/Compiler.nqp
Remove trailing whitespace
14:58
Zoffix ZofBot: two combining carrots! 15:03
ZofBot Zoffix, In the absence of that information, the tool can choose either to download and use the compunit directly, or the tool can proceed in ignorance
travis-ci NQP build failed. Samantha McVey 'Improve error message when space with combiners is encountered 15:06
travis-ci.org/perl6/nqp/builds/284691199 github.com/perl6/nqp/compare/576d7...ed230bdaa0
NQP build failed. Zoffix Znet 'Remove trailing whitespace' 15:12
travis-ci.org/perl6/nqp/builds/284693561 github.com/perl6/nqp/compare/0bed2...8db2152508
Zoffix `make clean; cd nqp; make install; cd ..; make install` does the trick with the standard nested installation 15:14
Geth nqp: c38cfe87f5 | (Zoffix Znet)++ | src/HLL/Compiler.nqp
Implement --force-stdin-eval-mode command line arg

  - Takes 'interactive' and 'non-interactive' as valid values
  - Provides the means to switch between REPL and plain eval STDIN
   without depending on whether STDIN is a TTY or not. At the
   very least, this lets us easily write REPL tests.
15:21
dogbert2 bisect: my @isprime = False,False; say (for 1 .. 10 -> $i { $i if @isprime[$i] //= so $i %% none 2 ...^ * > $i.sqrt.floor; }) 15:28
bisectable6 dogbert2, Bisecting by output (old=2015.12 new=f59b288) because on both starting points the exit code is 0
dogbert2, bisect log: gist.github.com/80f33c40afa832a376...f73a7626be
dogbert2, (2017-01-13) github.com/rakudo/rakudo/commit/d1...dcb4918120
Skarsnik hm 15:31
what that resolved at some point?
m: role A {method foo(){say "a"}}; class B does A { method foo(Str $a) {say "b"}}; my B $b .= new; $b.foo
camelia Too few positionals passed; expected 2 arguments but got 1
in method foo at <tmp> line 1
in block <unit> at <tmp> line 1
dogbert2 the bisect above is from RT #126415 15:35
synopsebot RT#126415 [new]: rt.perl.org/Ticket/Display.html?id=126415 [BUG] WhateverCode loses its mind inside //= 15:36
travis-ci NQP build failed. Zoffix Znet 'Implement --force-stdin-eval-mode command line arg 15:37
travis-ci.org/perl6/nqp/builds/284700960 github.com/perl6/nqp/compare/108db...8cfe87f54e
Geth rakudo/nom: 5f6896bde8 | (Timo Paulssen)++ | src/core/Str.pm
use nqp::codes op to speed up Int

not a vast improvement, but having one allocation less is a nice bonus.
16:09
rakudo: jonathanstowe++ created pull request #1185:
Add tests to cover a common NC usage
16:24
rakudo/nom: d8a20d1134 | (Jonathan Stowe)++ | 2 files
Add tests to cover a common NC usage

C functions that consistently take some struct or pointer as the first argument are declared as methods of that CStruct or CPointer
16:25
rakudo/nom: 64d41ffa11 | (Zoffix Znet)++ (committed using GitHub Web editor) | 2 files
Merge pull request #1185 from jonathanstowe/nom

Add tests to cover a common NC usage
roast: 5f64bc0c6e | (Zoffix Znet)++ | S03-operators/range.t
Fix missing `use lib`
16:36
rakudo/nom: 4 commits pushed by (Aleks-Daniel Jakimenko-Aleksejev)++ 16:39
roast: 85b9380acd | (Zoffix Znet)++ | S02-lexical-conventions/comments.t
Remove incorrect test

Per samcv++ from github.com/perl6/nqp/commit/02a426e0e6
Remove support for ornate parenthesis from quoting constructs Reasons:
  * These are not recognized by Unicode as being brackets
... (6 more lines)
16:43
roast/6.c-errata: d78ab87a65 | (Zoffix Znet)++ | S02-lexical-conventions/comments.t
Remove incorrect test

Per samcv++ from github.com/perl6/nqp/commit/02a426e0e6
Remove support for ornate parenthesis from quoting constructs Reasons:
  * These are not recognized by Unicode as being brackets
... (6 more lines)
rakudo/nom: 9ce896d8bf | (Zoffix Znet)++ | tools/build/NQP_REVISION
Bump NQP
16:46
rakudo/nom: version bump brought these changes: github.com/perl6/nqp/compare/2017....5-gc38cfe8
20518454ef | (Zoffix Znet)++ | src/Perl6/Compiler.nqp

Since recently-ish, if STDIN is not a TTY, we've treated STDIN as simply a file with code to eval, without starting REPL.
rakudo/nom: 5c7bbea0a9 | (Zoffix Znet)++ | src/Perl6/Compiler.nqp
Add --force-stdin-eval-mode opt to usage msg
17:04
RabidGravy Ah, that very thing there is probably why the fdreopen fd 0 to a listening socket isn't working 17:06
Zoffix doubts it 17:07
RabidGravy why?
Zoffix If you're talking about --force-stdin-eval-mode. That's only used when deciding whether to start the REPL or just eval code from STDIN without REPL machinery
(without the option, this is decided on whether STDIN is a TTY; starting REPL when it is) 17:08
RabidGravy no I'm talking about " if STDIN is not a TTY, we've treated STDIN as simply a file with code to eval"
Zoffix *based on
RabidGravy: that's only when otherwise we'd start REPL
RabidGravy: if you passed -e '' or gave some file to read code form, this codepath isn't used
RabidGravy Oh okay 17:09
misunderstanding
moritz --force-interactive? 17:12
sorry, I wasn't there for the bikeshedding, so I should shut up 17:13
Zoffix moritz: I thought of that originally, but figured if we're adding an option to force REPL on non-TTY displays, may as well add one to force non-interactive mode on TTY displyas
moritz: there wasn't any bikeshedding. I just added it, 'cause I need to fix REPL tests :) If there's a better name we can change it 17:14
Zoffix leaves for a quick jam session 17:15
RabidGravy Mmm jam 17:19
Zoffix "--repl-mode" ? The only thing is that suggests the non-interactive stuff still loads REPL.pm but it doesn't 17:30
Don't like `--force-stdin-eval-mode` no more 17:38
Gonna swap to `--repl-mode` later, unless someone suggests better name 17:39
Skarsnik hm thoughs on gist.github.com/Skarsnik/3fc513cbd...1a64286f51 18:15
AlexDaniel_ q.viva64.com/ # that's kinda a product ad, but fun anyway 18:18
travis-ci Rakudo build passed. Zoffix Znet 'Add --force-stdin-eval-mode opt to usage msg' 18:19
travis-ci.org/rakudo/rakudo/builds/284733899 github.com/rakudo/rakudo/compare/2...7bbea0a920
[Coke] waves. 18:26
Skarsnik the 5th one is dubious
A buffer containing private data should be erased before function returns. The memset() function is used for this task, but it is incorrect. As compiler is aware that after calling memset() the 'final' buffer will not be utilized anymore, it is possible that compiler will remove memset() call altogether. 18:27
what can lead a compilater to remove a function call, that make no sense
AlexDaniel_ shrugs 18:34
Skarsnik that the kind of reason I don't like coding in C++ 18:39
You need to much of compiler knowledge
timotimo C and C++ compilers have heavy optimizations regarding memset and friends 18:52
they will even recognize a for loop that is equivalent to memset and turn it into memset instead
Skarsnik I wonder what this tool could give on MoarVM Code 18:54
timotimo i think i've done this quiz before 18:55
AlexDaniel_ fwiw I gonna kill anybody who merges my last 5 tickets that I've intentionally submitted separately 19:04
Zoffix :o 19:06
Skarsnik it's fail.6lang.org now?
Zoffix Skarsnik: no, fail.rakudo.org 19:08
err
Skarsnik: no, fail.rakudo.party
ZofBot: too many domain names
ZofBot Zoffix, However, if it is completely empty or consists of a single list, the first element of which is either a hash or a pair, it is executed immediately to compose a "Hash" object
Skarsnik ZofBot, fail 19:11
ZofBot Skarsnik, It has been argued that "
Skarsnik ZofBot, RT 19:12
ZofBot Skarsnik, 2 # false, equivalent to 1 <= 2
Skarsnik what 19:13
AlexDaniel_ Skarsnik: it is retarded
Skarsnik how the gumbo crash perform with the nc-jit call now btw? x) 19:16
AlexDaniel_ I haven't tried it, should I? 19:17
Skarsnik probably change nothing 19:18
AlexDaniel_ Skarsnik: ok, that's bad 19:23
Skarsnik: it's no longer crashing
Skarsnik hm 19:24
AlexDaniel_ but I really don't think that the issue is resolved, so we just masqueraded it
Skarsnik: 50000 iterations and nothing 19:27
ok I can bisect it for fun 19:28
Geth roast: 34577134e0 | TimToady++ | S32-exceptions/misc.t
overspecific test disallows eject in correct spot
dogbert2 would anyone be terribly upset if I enabled the test mentioned in RT #122951 and then closed the issue with an appropriate comment? 19:29
synopsebot RT#122951 [open]: rt.perl.org/Ticket/Display.html?id=122951 [BUG] negative lookahead doesn't LTM properly
AlexDaniel_ dogbert2: does it pass on HEAD? 19:30
dogbert2 m: say "abcde" ~~ / ab <![e]> cde | ab.. /
camelia 「abcd」
dogbert2 wait a sec, it doesn't work does it 19:32
dogbert2 must learn the art of reading
Geth rakudo/nom: 5747bc7121 | TimToady++ | src/Perl6/Grammar.nqp
deconfuse statement modifier errors

Catch missing expressions on statement modifiers earlier so that the tests eat_terminator are only for statement controls. Also, move the cursor to the front of the cursor controls to indicate where the semicolon is actually perceived to be missing if we report that. Additionally, we change the kok check to use .panic instead of .sorry to avoid multiple errors resulting from repeated terminator checks that use statement modifier keywords for terminators.
Fixes #125596 and #125674.
19:34
synopsebot RT#125596 [new]: rt.perl.org/Ticket/Display.html?id=125596 [LTA] error message with if; and unless; without conditions
RT#125674 [new]: rt.perl.org/Ticket/Display.html?id=125674 [LTA] error message prints wrong eject position (if True if { };)
AlexDaniel_ dogbert2: once you learn how to read, please teach me too
dogbert2 :)
anyway, this is not something we can close atm 19:36
another one then RT #126415, according to bisectable it was fixed with github.com/rakudo/rakudo/commit/d1...dcb4918120 19:37
synopsebot RT#126415 [new]: rt.perl.org/Ticket/Display.html?id=126415 [BUG] WhateverCode loses its mind inside //= 19:38
dogbert2 jnthn: added tests for that commit here, github.com/perl6/roast/commit/e797...1a05fe8a46
but do they really cover the RT
Skarsnik btw last week I try to look at rt.perl.org/Ticket/Display.html?id=127345 but I was not sure where/how to add the code that annotate the instruction with the line (since MoarVM does not get it) 19:39
timotimo dogbert2: fwiw, the output abcd is the correct one for that alteration 19:40
dogbert2: the negative lookahead isn't declarative, so the longest declarative prefix is 2 letters on the LHS and 4 on the RHS 19:41
Geth roast: a0b678199c | (Moritz Lenz)++ | S32-num/rat.t
RT #126101: Another test for 0.9999...
synopsebot RT#126101 [open]: rt.perl.org/Ticket/Display.html?id=126101 [BUG] 0.lotsof9s badly approximated 19:42
dogbert2 timotimo: so the report and test is bogus then ?
or did things change between Parrot and Rakudo?
timotimo whoa 19:43
moritz there is no parrot.
and chromatic is its prophet.
timotimo there was a quote from the synopses
i did not know about that detail
.o( the regex optimizer could throw that match out )
(i mean the negative lookahead) 19:44
dogbert2 moritz: what do you think about RT #122951, do you cover this in yout new book?
synopsebot RT#122951 [open]: rt.perl.org/Ticket/Display.html?id=122951 [BUG] negative lookahead doesn't LTM properly
moritz dogbert2: it doesn't go into that much detail on LTM 19:45
dogbert2 it was worth a shot :)
moritz it's suprising how little fits into ~150 pages :/
but I trust pmichaud's judgement on this one
dogbert2 which means that the problem still exist even though Parrot does not 19:46
moritz it's not as if parrot dictated Rakudo regex semantics at the time this bug was submitted 19:47
dogbert2 interesting, I know to little about the history of these things 19:49
timotimo the regex impl was probably pretty much 1:1 ported to jvm and moar 19:50
AlexDaniel_ c: HEAD gist.githubusercontent.com/AlexDan...bocrash.p6
committable6 AlexDaniel_, Successfully fetched the code from the provided URL.
AlexDaniel_, ¦HEAD(5747bc7): «0␤MoarVM panic: Internal error: zeroed target thread ID in work pass «exit code = 17»»
AlexDaniel_ Skarsnik: ummmm
timotimo that's not nice 19:51
AlexDaniel_ Skarsnik: I don't know? How come it happens on the first iteration?
dogbert2 is that Gumbo thing still a problem
AlexDaniel_ dogbert2: not entirely sure. locally it works fine but blows up immediately on committable
Skarsnik does not crash on my vm 19:52
AlexDaniel_ actually, maybe the output is cached so we only see 0 but there was more stuff?
dogbert2 which P6 version does c: use?
Skarsnik but crash on my dedicated 64bits server
AlexDaniel_ dogbert2: HEAD(5747bc7)
Skarsnik AlexDaniel, did it fail on fetching the website? 19:53
AlexDaniel_ Skarsnik: no because it printed 0
hm or… I don't know
timotimo: RT #131003 19:54
synopsebot RT#131003 [open]: rt.perl.org/Ticket/Display.html?id=131003 [ANNOYING][SEGV] Heap corruption when using Gumbo
timotimo i know about this bug :(
hm, i wonder if nativecast is involved there
Skarsnik could be 19:55
timotimo i had some code where i nativecast one kind of CArray or something to another and it proceeded to double-free the backing array
Skarsnik I had issue trying to replace the nativecast with a typed pointed object and doing unref on it but it corrupted the addre 19:56
timotimo m: use NativeCall; sub doit { my $foo = CArray[int64].new(1, 2, 3, 4, 5, 6, 7, 8, 9, 10); my $bar = nativecast(CArray[int8], $foo); 1 }; for ^100 { doit }
camelia ( no output )
timotimo well, that's not how
m: use NativeCall; sub malloc(int32 $size) is native(Str) returns CArray[int8] {*}; sub doit { my $foo = malloc(128); my $bar = nativecast(CArray[int64], $foo); 1 }; for ^100 { doit } 19:57
camelia ( no output )
timotimo well, that's not how i guess?
m: use nqp; use NativeCall; sub malloc(int32 $size) is native(Str) returns CArray[int8] {*}; sub doit { my $foo = malloc(128); my $bar = nativecast(CArray[int64], $foo); 1 }; for ^100 { doit; nqp::force_gc }
camelia ( no output )
timotimo m: use nqp; use NativeCall; sub doit { my $foo = CArray[int64].new(1, 2, 3, 4, 5, 6, 7, 8, 9, 10); my $bar = nativecast(CArray[int8], $foo); 1 }; for ^100 { doit; nqp::force_gc }
camelia ( no output )
AlexDaniel_ hm but it doesn't crash here so I really don't know 19:58
AlexDaniel_ wonders if [ANNOYING] should be capitalized as [aNNOYING] 19:59
timotimo don't make it even harder to work on this kind of bug :) 20:01
Skarsnik I wonder if my typered branch of gptrixie still make MoarVM segfault at compile/init time xD 20:05
moritz ‘789’.comb(/ . {say $/} <!> /)'
m: ‘789’.comb(/ . {say $/} <!> /)'
camelia 5===SORRY!5=== Error while compiling <tmp>
Two terms in a row
at <tmp>:1
------> 3‘789’.comb(/ . {say $/} <!> /)7⏏5'
expecting any of:
infix
infix stopper
statement end
statement modifier…
AlexDaniel_ trailing '
moritz ‘789’.comb(/ . {say $/} <!> /)
m: ‘789’.comb(/ . {say $/} <!> /)
camelia 「7」
「8」
「9」
AlexDaniel_ oh, I remember that
moritz AlexDaniel_: thanks
yes, fun one
AlexDaniel_ wasn't it closed?
or is it TESTNEEDED? 20:06
moritz the latter
Geth roast: afa1f74636 | (Moritz Lenz)++ | S03-operators/repeat.t
RT #121327: tests for scoping bug in xx
20:08
synopsebot RT#121327 [open]: rt.perl.org/Ticket/Display.html?id=121327 [TESTNEEDED] Scoping bug with `xx`
AlexDaniel_ errr that's not the ticket I was thinking about 20:09
right, because it isn't. 20:11
nvm
Geth roast: 005b4f76f7 | (Moritz Lenz)++ | S32-str/comb.t
RT #130186: $/ inside comb
synopsebot RT#130186 [new]: rt.perl.org/Ticket/Display.html?id=130186 Match object inside .comb is weird (‘789’.comb(/ . {say $/} <!> /)) 20:12
Skarsnik releasable6, status
releasable6 Skarsnik, Next release in 13 days and ≈22 hours. 1 blocker. Changelog for this release was not started yet
Skarsnik, Details: gist.github.com/a2b5146055cf744685...4f181faf04
Geth rakudo/nom: 5144216f6e | (Elizabeth Mattijsen)++ | 2 files
Cache types we always need in the CompilerServices

  - this appears to make the setting compilation a bit faster
  - but getting timings on that is always difficult :-(
20:14
dogbert2 interesting, the Gumbo example seems to work on my system as well 20:22
Skarsnik test with a previous release maybe? 20:23
dogbert2 ok have 2017.07 available 20:24
Skarsnik I use linuxfr.org to test, it crash faster ^^ 20:25
AlexDaniel_ Skarsnik: still does not crash 20:27
Geth nqp/jvm_killprocasync: 11ef6ea37d | usev6++ | 10 files
[jvm] Update stage0 for modified op killprocasync
20:28
nqp/jvm_killprocasync: 7ca352fdf5 | usev6++ | 3 files
[jvm] Primitive implementation of killprocasync
Skarsnik it does stuff thou?
AlexDaniel_ facepalm 20:31
curl: command not found
bartolin ^^ it would be nice to get a feedback for that branch (jvm_killprocasync). it's rudimentary but it makes "doesn't-hang" usable -- though not with the default wait of 1.5 sec *g*
AlexDaniel_ Skarsnik: OK nevermind, no change, it still crashes 20:32
committable6 is right, I'm wrong
Skarsnik yay? 20:33
dogbert2 works with 2017.07 as well and also with linuxfr.org
AlexDaniel_ … yeeeeeeeeeeey… :'(
dogbert2: nope. Can't be. Do you have curl installed?
dogbert2 yes
AlexDaniel_ dogbert2: or do you have 32-bit system?
dogbert2 and yes :)
have you tried disabling the jit? 20:34
AlexDaniel_ ok, that we knew already… yeeeey… :'(
Geth rakudo/nom: a09f5f2194 | (Elizabeth Mattijsen)++ | 4 files
Set up Builtin encodings at setting compile time

  - so we don't need to do this each time on startup
  - only saves maybe 1 msec of startup
  - but it makes future optimizations opening STDIN/OUT/ERR easier
Skarsnik it's an old crash
bartolin .tell jnthn if you have a few minutes to look at my try to implement killprocasync on jvm: github.com/perl6/nqp/tree/jvm_killprocasync -- when you stubbed the op, you planned to return $RT_OBJ. do you remember what you expected the op to return?
yoleaux bartolin: I'll pass your message to jnthn.
AlexDaniel_ dogbert2: yea, doesn't help
dogbert2 annoying 20:35
Skarsnik I should have a closer look at the bt of other threads
for this
dogbert2 I can get it to fail, if run under valgrind with MVM_SPESH_BLOCKING=1 20:44
Skarsnik interesting
Geth roast: 31506095f1 | usev6++ | 2 files
[jvm] Update fudge messages: killprocasync NYI
20:45
dogbert2 but it won't fail outside valgrind with that same flag set, odd 20:46
Skarsnik for me it does not crash with efence before filling the whole memory x) 20:47
same with valgrind
AlexDaniel_ … and people argue whether [ANNOYING] tag is needed… 20:48
Skarsnik [Alexdanielable] 20:49
Geth rakudo/nom: c7a82d451d | usev6++ (committed by Aleks-Daniel Jakimenko-Aleksejev) | src/Perl6/Grammar.nqp
Try harder to diagnose problem

instead of reporting 'Malformed my'
Fixes RT #125902
20:53
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: ac97a40161 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Grammar.nqp
Fix complaints about existing types

Call typo_typename only if <typename> or <value> failed, not if something else went wrong.
Resolves ½ of RT #127100.
21:58
synopsebot RT#127100 [new]: rt.perl.org/Ticket/Display.html?id=127100 [LTA] error message when specifying return type before parameters ( --> Bool, Int $x, Int $y )
Geth rakudo/nom: bb5583aeb7 | (Elizabeth Mattijsen)++ | src/core/io_operators.pm
Streamline initialization of $*IN/$*OUT/$*ERR

  - do as much as possible during setting compilation
  - only plug in the handle/encoder/decoder at runtime
  - shaves off about 1.5% from bare startup time
22:12
rakudo: dwarring++ created pull request #1186:
implement typed pointer increment and array dereference salortiz++
22:36
travis-ci Rakudo build errored. Zoffix Znet 'Merge pull request #1185 from jonathanstowe/nom 22:41
travis-ci.org/rakudo/rakudo/builds/284721668 github.com/rakudo/rakudo/compare/5...d41ffa1104
buggable [travis build above] ✓ All failures are due to: missing build log (1 failure).
Geth rakudo/nom: 64b001a146 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Grammar.nqp
Awesome error message for RT #127100

If you can do this:
   sub foo($x, --> 42) {}
Then why not this? ... (6 more lines)
22:49
synopsebot RT#127100 [new]: rt.perl.org/Ticket/Display.html?id=127100 [LTA] error message when specifying return type before parameters ( --> Bool, Int $x, Int $y )
buggable New CPAN upload: Config/Perl6/Config-1.2.1.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...2.1.tar.gz 23:02
New CPAN upload: Config-Parser-toml/Perl6/Config-Parser-toml-1.0.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...0.0.tar.gz
New CPAN upload: Config-Parser-yaml/Perl6/Config-Parser-yaml-1.0.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...0.0.tar.gz
New CPAN upload: Hash-Merge/Perl6/Hash-Merge-0.1.0.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...1.0.tar.gz
New CPAN upload: MDP-Client/Perl6/MPD-Client-0.1.2.tar.gz by TYIL cpan.metacpan.org/authors/id/T/TY/...1.2.tar.gz
Geth nqp: 943f7f757c | (Zoffix Znet)++ | src/HLL/Compiler.nqp
Rename `--force-stdin-eval-mode` command line opt

To `--repl-mode`
23:43
rakudo/nom: 93e599dbf2 | (Zoffix Znet)++ | tools/build/NQP_REVISION
Bump NQP
23:44
rakudo/nom: version bump brought these changes: github.com/perl6/nqp/compare/2017....6-g943f7f7
de0533c4d4 | (Zoffix Znet)++ | src/Perl6/Compiler.nqp

To incorporate renaming of `--force-stdin-eval-mode` command line opt to `--repl-mode`
rakudo/full-make-test: 19 commits pushed by (Aleks-Daniel Jakimenko-Aleksejev)++, (Zoffix Znet)++, (Timo Paulssen)++, (Jonathan Stowe)++, TimToady++, (Elizabeth Mattijsen)++, usev6++
review: github.com/rakudo/rakudo/compare/7...87dc51f514
23:45
travis-ci NQP build failed. Zoffix Znet 'Rename `--force-stdin-eval-mode` command line opt 23:57
travis-ci.org/perl6/nqp/builds/284860386 github.com/perl6/nqp/compare/c38cf...3f7f757c9b