ugexe | from what i remember it was a good reason | 00:36 | |
yoleaux | 30 May 2017 14:31Z <Zoffix> ugexe: TAP.pm6 is gone from core now: github.com/rakudo/rakudo/commit/ae891f9367 | ||
Zoffix doubts that reason considered programmers typing that name | 00:40 | ||
ugexe | my general perl6 experience is typing out CompUnit::Repository::DependencySpecification.new(:short-name<...>, :auth-matcher<..>, :ver-matcher<...>) a lot, i have no sympathy | 00:45 | |
Zoffix | "<bioexpress> Hello, why was the module Term::ANSIColor renamed in Terminal::ANSIColor?" "<TimToady> because someone thought it might collide with the idea of "term" in parsing, but it's an unfounded fear" | 00:46 | |
Exactly what I thought. | |||
ugexe: that stuff has the same problem. Design of the names thought too much about meaning and too little about typing. CU::Repo::DepSpec is what I would've gone with | 00:47 | ||
Zoffix retires to bed | |||
\o | |||
ZofBot: good night, sweet prince! | |||
ZofBot | Zoffix, Oops? | ||
ugexe | its not bad, DependencySpecification( ... ) is brutal though | 00:50 | |
samcv | good * everyone | 01:33 | |
BenGoldberg | samcv, Is that a good Whatever star? | 01:39 | |
samcv | it is a * i dunno | ||
good ???? everyone | 01:40 | ||
BenGoldberg | That's a bit much, I'll settle for a ā . | 01:41 | |
Or, considering my timezone, a good ????. (wow that looks horrible in the font I'm using!). For samcv and others in the same timezone, good ???? or ????. | 01:46 | ||
samcv | it's actually 6:46pm here. but. might as well be morning idk. maybe afternoon in my workday | ||
[Tux] | This is Rakudo version 2017.05-303-g5e3393454 built on MoarVM version 2017.05-25-g62bc54e9 | 06:22 | |
csv-ip5xs 2.733 | |||
test 12.828 | |||
test-t 4.364 - 4.368 | |||
csv-parser 13.036 | |||
good * perl6 folk! | |||
nine | I wonder how we lost the 0.3 seconds in csv-ip5xs | 08:33 | |
Zoffix: code is read much more often than written. "CompUnit::DependencySpecification" (there's no "Repository" in that name) is fairly self documenting and unambiguous. | 08:45 | ||
Geth | rakudo/nom: 1ed284e295 | (Elizabeth Mattijsen)++ | src/Perl6/Metamodel/Naming.nqp Hacky fix for RT #131406 Assuming we will never get something more complicated than foo[bar] |
09:35 | |
rakudo/nom: cb82760625 | (Elizabeth Mattijsen)++ | lib/Test.pm6 Fix for RT #131408 Make sure $output is initialized so that we don't take the Any.put candidate. |
09:43 | ||
nqp/nqp-io: 42a77898ac | (Jonathan Worthington)++ | t/nqp/019-file-ops.t Toss tests that depend on no CRLF conversion. They were only run on JVM, and now they don't run there either. |
09:57 | ||
Zoffix | nine: long names negatively affect readability too. At 4-12char indent you'd typically have in some method, you're adding another 65 chars just to instantiate a single object (CompUnit::DependencySpecification.new(:short-name<Inline::Perl5>)) so you're forced to write your code in many lines, reducing its readability. | 10:04 | |
yoleaux | 03:56Z <MasterDuke> Zoffix: github.com/rakudo/rakudo/pull/1091 has been updated | ||
Zoffix | CompUnit::DepSpec is equally self-documenting and unambiguous | 10:05 | |
Think about it: everyone was calling it it "CUR" when it was being developed and people got around. | 10:08 | ||
We call pod stuff Pod::Block and Pod::Heading. No one seems to have issues without it being named Pod::Documentation::heading or Plain::Old::Documentation::Heading | 10:09 | ||
jnthn | Most people use them rarely enough in their day-to-day Perl 6 work I don't think it matters all that much. | 10:10 | |
It's just the huffman coding principle at work | |||
lizmat | also: everybody knows what Pod is, with the way we do precomp in Rakudo we are in undiscovered country | ||
Geth | nqp/nqp-io: 0d1aa3c1c9 | (Jonathan Worthington)++ | src/vm/jvm/runtime/org/perl6/nqp/sixmodel/reprs/DecoderInstance.java Fix buffer handling thinko in JVM decoder. Caught by the NQP tests, after switching to the new NQP file handle. |
10:12 | |
roast: fc90118495 | (Elizabeth Mattijsen)++ | S17-supply/watch-path.t Fix test if we're being fed an absolute path |
10:15 | ||
Zoffix | Everyone knows what a dependency is too :P | 10:16 | |
jnthn | Yay, seems my branches don't bust up JVM :) | 10:21 | |
That means...le merge | 10:22 | ||
Zoffix | \o/ | ||
lizmat | whhe! | 10:24 | |
whee! rather :-) | 10:25 | ||
jnthn | And then finally I've shaved all the yaks to let me re-work file handles in MoarVM to not use libuv... | ||
Of course, the one test file that would change in NQP while I have my branch would have to be the one I totally rewrote :) | 10:27 | ||
So, to summarize here what's happened | 10:29 | ||
Because it's a relatively large change | |||
The various char-level IO nqp:: ops are gone | |||
nqp::print and nqp::say live on because they're useful for debugging | |||
open in NQP now returns an NQPFileHandle object | 10:30 | ||
close works on it | |||
NQP ops do *not* work on it, instead you call methods | |||
The API matches the Perl 6 one pretty closely | |||
nqp::printfh(nqp::getdstderr(), ...) was pretty common | 10:31 | ||
There's a note in NQP if the thing as a newline at the end | |||
Otherwise, stderr() gets the handle, so it's stderr().print(...) | |||
*has a newline | 10:33 | ||
Geth | nqp/master: 37 commits pushed by (Jonathan Worthington)++ review: github.com/perl6/nqp/compare/fda38...05e5d75c72 |
10:34 | |
jnthn | That's the NQP merge | ||
Geth | rakudo/nom: 8 commits pushed by (Jonathan Worthington)++
|
10:37 | |
Ā¦ rakudo/nom: version bump brought these changes: github.com/perl6/nqp/compare/2017....8-g05e5d75 | |||
jnthn | And, now spectest is done, the Rakudo one. :) | ||
lizmat | jnthn: would this change affect github.com/edumentab/rakudo-and-nq...als-course ? | 10:38 | |
jnthn | Very possibly | ||
I suspect it does I/O somewhere :) | |||
lizmat | yeah... ok | ||
jnthn | Though probably not all that heavily | ||
lizmat | once this dust has settled, I'll look at creating some PR's :-) | 10:39 | |
jnthn | In that most I/O in compilers you'd write is done for you in HLL::Compiler | ||
So it'd only be if you have I/O bits in your language | |||
And for toying around, nqp::say and nqp::print are still there | |||
lizmat | ah, yes... :-) | ||
travis-ci | NQP build failed. Jonathan Worthington 'Merge branch 'nqp-io'' | 11:15 | |
travis-ci.org/perl6/nqp/builds/237874583 github.com/perl6/nqp/compare/fda38...e5d75c728d | |||
nine | t/nqp/019-file-ops.t (Wstat: 0 Tests: 105 Failed: 0) | 11:17 | |
Parse errors: Bad plan. You planned 106 tests but ran 105. | 11:18 | ||
Geth | nqp: 6007ead8f6 | (Stefan Seifert)++ | t/nqp/019-file-ops.t Fix skip counter in nqp/019-file-ops.t |
11:19 | |
travis-ci | NQP build passed. Stefan Seifert 'Fix skip counter in nqp/019-file-ops.t' | 11:53 | |
travis-ci.org/perl6/nqp/builds/237887892 github.com/perl6/nqp/compare/05e5d...07ead8f699 | |||
dogbert17 | .seen Zoffix | 11:58 | |
yoleaux | I saw Zoffix 11:00Z in #moarvm: <Zoffix> Well, the \o/ was about the general thing :D | ||
dogbert17 | .tell Zoffix the code example for method lock looks a bit broken in the browser. docs.perl6.org/type/IO::Handle#method_lock | 11:59 | |
yoleaux | dogbert17: I'll pass your message to Zoffix. | ||
Zoffix | dogbert17: Fixed. Thanks. | 12:04 | |
yoleaux | 11:59Z <dogbert17> Zoffix: the code example for method lock looks a bit broken in the browser. docs.perl6.org/type/IO::Handle#method_lock | ||
Geth | rakudo/nom: 7f84df1db3 | (Elizabeth Mattijsen)++ | src/core/Rakudo/QuantHash.pm Add R:Q.ADD-(MAP|OBJECTHASH)-TO-SET helper methods To be used in set operators and coercions to Set. |
12:11 | |
rakudo/nom: 160de7e6f0 | (Elizabeth Mattijsen)++ | 3 files Use new R:Q methods to generalize Map -> Set coercions This actually also fixes a hereto unseen bug in objecthash.Set coercion. |
|||
roast: 80f77dc88a | (Elizabeth Mattijsen)++ | S02-types/set.t Add some Map/object Hash -> Set coercion tests |
|||
roast: 009a59bf39 | (Elizabeth Mattijsen)++ | S02-types/sethash.t Add some Map/object Hash -> SetHash coercion tests |
12:14 | ||
roast: 3e5c2bd01a | (Elizabeth Mattijsen)++ | S03-operators/set.t Remove subset/superset tests They're more thoroughly tested in (proper-)subset.t |
12:38 | ||
roast: ac434e7a6c | (Elizabeth Mattijsen)++ | 2 files Also test R versions of subset/superset operators As these were previously tested in set.t, so we should at least keep that coverage. |
|||
lizmat | m: my ::foo $x # pretty strange code and error ? | 13:06 | |
camelia | No such method 'sink' for invocant of type 'foo' in block <unit> at <tmp> line 1 |
||
lizmat | MasterDuke: ^^^ | ||
hangs with PR #1091 applied :-( | |||
jnthn | m: $ = my ::foo $x | 13:07 | |
camelia | ( no output ) | ||
jnthn | m: $ = my ::foo $x; say $x.^name | ||
camelia | foo | ||
jnthn | m: $ = my ::foo $x; say $x.HOW.archetypes | ||
camelia | No such method 'gist' for invocant of type 'Perl6::Metamodel::Archetypes' in block <unit> at <tmp> line 1 |
||
jnthn | m: $ = my ::foo $x; say $x.HOW.archetypes.generic | ||
camelia | 1 | ||
jnthn | Ah, it compiles it with a type-var there | ||
lizmat | afk for a bit& | 13:11 | |
Geth | roast: e88e087289 | (Elizabeth Mattijsen)++ | S03-operators/elem.t Also test R(elem) family of set operators |
13:18 | |
lizmat | jnthn: fwiw, all green for me after the last NQP bump | ||
jnthn | \o/ | 13:22 | |
Zoffix | What a nice coincidence. A song I really liked before I started with Perl 6 is called "Six" :) www.youtube.com/watch?v=M0pYWGjECLE | 13:24 | |
Zoffix reads the lyrics... | |||
Well, glad I didn't say it was a song /about/ Perl 6 :D | |||
jnthn | Alright, lesson time :) | 13:26 | |
bbi1h | |||
b2gills | m: ''.length | 14:07 | |
m: ''.graphs | |||
camelia | No such method 'length' for invocant of type 'Str' Did you mean 'elems', 'chars', 'graphs' or 'codes'? in block <unit> at <tmp> line 1 |
||
No such method 'graphs' for invocant of type 'Str' in block <unit> at <tmp> line 1 |
|||
Zoffix | hehe | 14:12 | |
nine | Looks like graphs has never been implemented | ||
Zoffix | What were it meant to be? | 14:13 | |
Graphemes? Isn't it what .chars does now? | |||
nine | Could be. Could also be that .chars actually counts grapheme clusters | 14:14 | |
Zoffix | m: length '' | ||
camelia | 5===SORRY!5=== Error while compiling <tmp> Undeclared routine: length used at line 1. Did you mean 'elems', 'chars', 'graphs', 'codes'? |
||
Zoffix | github.com/rakudo/rakudo/blob/nom/....nqp#L4050 and github.com/rakudo/rakudo/blob/nom/...on.pm#L160 | 14:15 | |
jnthn | I think the idea was that at some time in the distant future there may somehow be a way for users to define what character means at some higher level that graphemes | 14:36 | |
At which point .graphs would unambiguously means graphemes | |||
It's not quite accurate to say that .chars counts graphemes; it doesn't have to count, because strings are stored as NFG :) | 14:37 | ||
pmurias | jnthn: the character oriented ops should be removed from the JVM backend too? | 15:24 | |
pmurias is taking care of update the js backend to the new ops now | |||
jnthn | pmurias: Yeah, it will need a rebootstrap first, however | 15:26 | |
For js backend, I tried to update a few places that obviously needed updating from grepping, but didn't try it out. Hopefully it was more helpful than doing nothing.:) | |||
pmurias: If you're motivated to the JVM rebootstrap and clearup, please feel free :) | 15:27 | ||
mr_ron | .tell Zoffix wondering about p6 equivalent of $. as part of work on IO. perl6/doc issue github.com/perl6/doc/issues/401 | 15:42 | |
yoleaux | mr_ron: I'll pass your message to Zoffix. | ||
timotimo | mr_ron: did you see the LN (or something?) module zoffix made for $.? | 15:43 | |
yoleaux | 03:57Z <MasterDuke> timotimo: github.com/rakudo/rakudo/pull/1091 has been updated | ||
Zoffix | I've not made it yet | 15:44 | |
yoleaux | 15:42Z <mr_ron> Zoffix: wondering about p6 equivalent of $. as part of work on IO. perl6/doc issue github.com/perl6/doc/issues/401 | ||
timotimo | oh, i didn't know | 15:45 | |
Zoffix | .in 3d Future-Zoffix: github.com/perl6/doc/issues/401 | ||
yoleaux | Zoffix: I'll remind you on 3 Jun 2017 15:45Z | ||
Zoffix | Well, the module is just sugar for 'my $ln; $*ARGFILES.on-switch = { $ln = 1; };' uuuhhhh | 15:46 | |
Zoffix tries to remember what the plan was for incrementing $*NL .. | |||
Maybe there wasn't :) | 15:47 | ||
jnthn | Mix in to the handle and override .get :) | ||
Zoffix | mr_ron: thanks. I'll update the docs and resolve that issue when I publish NL module; should be some time this weekend | ||
Yeah, that should probably be it. | 15:48 | ||
s:g/NL/LN/; | 15:49 | ||
jnthn | We even made it so .lines only does its fast path trick if you ain't in a subclass, but calls .get otherwise, so you only have to override method get and not method lines :) | ||
In many cases, the for $handle.lines.kv -> $line-no, $line { } is also sufficient (though $line-no will start from 0) | 15:50 | ||
Zoffix | Yeah, for a single file. | 15:51 | |
Geth | nqp: eaa6ee5710 | pmurias++ | 2 files [js] Support for the rakudo nqp::p6finddispatcher op |
||
nqp: 90c4a6f243 | pmurias++ | 4 files [js] Partially unbreak after IO changes |
|||
Zoffix | And for multi-files you can do these things: twitter.com/zoffix/status/868761719639871488 | 15:52 | |
And LN module will be sugar to do it for $*ARGFILES | |||
mr_ron | thanks | 15:58 | |
pmurias | jnthn: amusing fact, we use wtf8 in a test as an encoding that doesn't exist, while it in fact exists | 16:09 | |
jnthn | lol | 16:10 | |
Zoffix | hah | 16:11 | |
Zoffix reads ugexe.com/perl-toolchain-summit-201...and-perl6/ | 16:26 | ||
Cool! | |||
Wonder how Perl 6 reports looks like, given that most people seem to be using Perl 6 rakudobrewed off a random commit | 16:28 | ||
lizmat: so, yeah, 9 days ago you asked if my ecosystem smoker will report to CPAN testers; I said no, but I didn't realize reporting involved just installing amodule. So yeah, it'll report then. | 16:30 | ||
hm, doesn't behave for me like in that blog tho; just says "Reporting with plugin: Zef::CPANReporter+{<anon|80622192>}" without giving the URL of the report | 16:53 | ||
Zoffix looks at source of Zef::CPANReporter ... then uninstall it off work boxes | 16:54 | ||
On second thought... | 16:57 | ||
Zoffix uninstalls it entirely | 16:58 | ||
Reports way too much info. | |||
dogbert17 | IOninja: perhaps I'm stupid, but if I run the snippet 'mkdir "newdir"' twice, shouldn't I get an error the second time? | 17:32 | |
Zoffix | dogbert17: rt.perl.org/Ticket/Display.html?id...et-history | 17:33 | |
dogbert17 | docs.perl6.org/type/IO::Path#routine_mkdir : Creates a new directory. See mode for explanation and valid values for $mode. Returns the IO::Path object pointing to the newly created directory on success; fails with X::IO::Mkdir if directory cannot be created. | 17:35 | |
FWIW, trying the same in bash results in "mkdir: cannot create directory ānewdirā: File exists" | 17:36 | ||
Zoffix | dogbert17: well, yeah, see ticket. My understanding of it is mkdir erroneously always returns True on MoarVM< | 17:37 | |
j: with '/tmp/blah4343243'.IO { .mkdir.chmod: 000; .mkdir | 17:38 | ||
j: with '/tmp/blah4343243'.IO { .mkdir.chmod: 000; .mkdir } | |||
camelia | 5===SORRY!5=== Missing block at <tmp>:1 ------> 3h4343243'.IO { .mkdir.chmod: 000; .mkdir7ā5<EOL> expecting any of: statement end statement modifier statement modifier loop Other potential difficultiesā¦ |
||
Potential difficulties: Leading 0 has no meaning. If you meant to create an octal number, use '0o' prefix; like, '0o00'. If you meant to create a string, please add quotation marks. at <tmp>:1 ------> 3/tmp/blah4343243'.IO { .mkdir.chmoā¦ |
|||
Zoffix | gah | 17:39 | |
r: with '/tmp/blah4343243'.IO { .mkdir.chmod: 0o000; .mkdir } | |||
camelia | ( no output ) | ||
Zoffix | s: '.'.IO, 'mkdir', \() | ||
SourceBaby | Zoffix, Sauce is at github.com/rakudo/rakudo/blob/160d...th.pm#L525 | ||
Zoffix | Yeah. It's just an op call. I suppose we could do the checks in question in Rakudo land, but perhaps the op should be fixed instead | 17:40 | |
dogbert17 | Zoffix: so the docs are correct but reality is different :) | ||
Zoffix | ZOFVM: Files=1239, Tests=137928, 118 wallclock secs (21.92 usr 3.00 sys + 2371.78 cusr 140.06 csys = 2536.76 CPU) | 18:01 | |
Geth | rakudo/nom: fb7dd8a430 | (Zoffix Znet)++ | 3 files Improve errors for length routine - Do not suggest non-existent 'graphs' For method form: - Do not suggest 'chars' or 'codes' for List types - Do not suggest 'elems' for non-List Cool types |
18:03 | |
Zoffix | b2gills: that fixes the issue | ||
Geth | roast: 83e984a190 | (Zoffix Znet)++ | S24-testing/7-bail_out.t Test immediate bail-out does not crash RT#131408: rt.perl.org/Ticket/Display.html?id=131408 Rakudo fix: github.com/rakudo/rakudo/commit/cb...b8fec64a75 |
18:11 | |
Zoffix | .ask MasterDuke do you think it's possible to improve the "Too many positionals" error to include the subname/invocant+method; similar to how the methodcalls on :D/:U were improved recently? Part of rt.perl.org/Ticket/Display.html?id...xn-1461759 | 18:16 | |
yoleaux | Zoffix: I'll pass your message to MasterDuke. | ||
Zoffix | dogbert17: FWIW, I'll likely nuke doc/Language/io.pod6 when doc/Language/io-guide.pod6 | 18:19 | |
dogbert17: when The Definitive IO Guide is completed. | |||
/io.pod6 is a bit too heavy on the basics and too light on the heavier stuff. I also see it suggest dumb things, like parsing CSV without a module | 18:20 | ||
my $fh = open "testfile", :w; | 18:21 | ||
$fh.say("data and stuff"); | |||
$fh.close; | |||
You'd just write that as 'testfile'.IO.spurt: "data and stuff"; The example over-use `open` | 18:22 | ||
+ "\n" | |||
zm: sub (Int:D()) {}( Str ) | 18:34 | ||
dogbert17 | Zoffix: cool | ||
Zoffix | m: sub (Int:D()) {}( Str ) | ||
camelia | Type check failed in binding to parameter '<anon>'; expected Int but got Str (Str) in sub at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
Zoffix | m: sub (Int()) {}( Str ) | ||
camelia | Invocant of method 'Int' must be an object instance of type 'Str', not a type object of type 'Str'. Did you forget a '.new'? in sub at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
Zoffix | Filed as rt.perl.org/Ticket/Display.html?id=131413 | 18:37 | |
Geth | roast: b25a0b9045 | usev6++ | S15-string-types/Str.t [JVM] Unfudge passing test |
18:42 | |
Zoffix | m: class { has Int:D() $.x = '42'}.new.x | 18:43 | |
camelia | Type check failed in assignment to $!x; expected Int:D but got Str ("42") in block <unit> at <tmp> line 1 |
||
Zoffix | m: class { has Int() $.x = '42'}.new.x | ||
camelia | 5===SORRY!5=== Error while compiling <tmp> Coercion Int(Any) is insufficiently type-like to qualify a variable at <tmp>:1 ------> 3class { has Int() $.x7ā5 = '42'}.new.x expecting any of: constraint |
||
Zoffix | m: class { has Int:D() $.x = 42}.new.x | 18:45 | |
camelia | ( no output ) | ||
Zoffix | m: class { has Int:D() $.x = 42}.new(:x('42')) | ||
camelia | Type check failed in assignment to $!x; expected Int:D but got Str ("42") in block <unit> at <tmp> line 1 |
||
Zoffix | Filed as rt.perl.org/Ticket/Display.html?id=131414 | 18:47 | |
ZofBot: it's almost PI ticket time! | |||
ZofBot | Zoffix, No num shenanigans in there | ||
Zoffix | hehe | ||
Well, tried having a go at rt.perl.org/Ticket/Display.html?id...et-history | 18:48 | ||
But it's a can-o-warms. We have quite a lot of methods that'll happily accept a non-D type and fail in some weird ways | 18:49 | ||
m: say Date.new-from-daycount: Int | |||
camelia | Invocant of method 'Bridge' must be an object instance of type 'Int', not a type object of type 'Int'. Did you forget a '.new'? in block <unit> at <tmp> line 1 |
||
Zoffix | .oO( hyphens in only half the method name. New low for naming inconsincies ) |
18:50 | |
Geth | nqp: 84e460fb82 | raiph++ (committed using GitHub Web editor) | README.pod (c) 2017 and JVM backend is currently experimental |
19:09 | |
Zoffix | *sigh* the Internet is a terrible place to find medical info. Apparently more people like to play Internet doctors than Internet laywers. | 19:12 | |
I might have acute liver failure and be dead soon... so, if I stop committing... Flip the cushion :P | |||
travis-ci | Rakudo build passed. Zoffix Znet 'Improve errors for length routine | 19:53 | |
travis-ci.org/rakudo/rakudo/builds/238027089 github.com/rakudo/rakudo/compare/1...7dd8a4305c | |||
Zoffix | omg.. Can't figure out if I'm really that stupid or if google's hiding the way to authorize with their API really well. | 21:39 | |
"use a service account"... fine I got the key. Now what :| what URL do I send stuff to or what? And stop telling me to use a python library | 21:40 | ||
I think I finally found it. | 21:43 | ||
I think by far the most time I'll spend in this auto-ecosystem-smoker is figuring out how to make it power on a google VM instance -_- | 21:44 | ||
Geth | rakudo: MasterDuke17++ created pull request #1093: Add routine name to 'Too many positionals error' |
21:52 | |
yoleaux | Zoffix: Future-Zoffix: Address the .sink-all stuff: github.com/rakudo/rakudo/commit/0c...t-22320387 | 21:54 | |
Zoffix | yoleaux: yeah, I'm a bit busy and don't know how to test that change, so I'm just gonna ignore it for now. | 22:02 | |
Gonna write the smoker first, then figure out how to make it auto-start VM. With all the encoding/ArgFiles/IO changes this month, we need to start smoking ASAP | 22:03 | ||
NeuralAnomaly: status | |||
NeuralAnomaly | Zoffix, [ā] Next release will be in 2 weeks and 2 days. Since last release, there are 28 new still-open tickets (25 unreviewed and 0 blockers) and 301 unreviewed commits. See perl6.fail/release/stats for details | ||
Geth | rakudo/nom: 7783fcab24 | (Daniel Green)++ | 2 files Add routine name to 'Too many positionals error' Also add some tests. Fixes rt.perl.org/Ticket/Display.html?id=131408 |
22:05 | |
rakudo/nom: c30afd445f | (Zoffix Znet)++ (committed using GitHub Web editor) | 2 files Merge pull request #1093 from MasterDuke17/make_too_many_positionals_error_better_RT131408 Add routine name to 'Too many positionals error' |
|||
Zoffix | MasterDuke++ | ||
MasterDuke | heh, i couldn't for the life of me remember how i'd added the routine name to that other error, or even which error it was. had to go on github and look at my merged PRs to figure it out | 22:07 | |
any reason not to resolve the ticket now? | 22:08 | ||
Zoffix | Don't think so. | 22:10 | |
ugexe | bah, "Testo:ver(v1.*)" works but not anything with a + | 22:11 | |
Zoffix | Hah | 22:29 | |
japhb | Zoffix: Perhaps Inline::Perl5 and some of the Perl 5 Google API modules as a bandage until you can write your own? The supported API list from metacpan.org/pod/Moo::Google looks promising .... | 22:40 | |
Zoffix | japhb: don't want to use Perl 5 | 22:46 | |
Don't see compute engine in the list of APIs anyway | 22:47 | ||
ah, it's just listed as "compute" | |||
This is why our ecosystem is stunted. Everyone's just using Inline::Perl5 | 22:48 | ||
I don't want to contribute to the problem :) | |||
Though I wasn't gonna write a full google API anyway... so maybe, yeah | 22:49 | ||
japhb++ :) | |||
samcv | still haven't figured out why the rakudo appimages stopped building properly a week ago for nom | 23:09 | |
"Unhandled exception: Can't convert 'yes' to num: expecting a number" | 23:11 | ||
O.o travis-ci.org/samcv/rakudo-appimag...7836#L1234 | |||
Geth | rakudo/nom: 9bf3ea3a4a | (Samantha McVey)++ | src/core/REPL.pm REPL: Don't show message to install line editor if using rlwrap Detect if the user is using rlwrap, and if so, don't tell them to install it, because they are already using it. |
23:18 | |
samcv | yay. that will make me and hopefully others happy | 23:19 | |
Zoffix | \o/ | 23:30 | |
samcv | Zoffix, do you use rlwrap or the modules or none of the above? | 23:32 | |
also maybe we should have that message by to stderr? | 23:34 | ||
will make testing a lot easier | |||
or maybe there's a good reason to not use stderr and keep it all on stdout idk | 23:35 | ||
o i see why this repl test in rakudo source tree. (test is disableld btw) but relied on the message saying to use panda not zef | 23:38 | ||
Geth | rakudo/nom: 8ec1810198 | (Samantha McVey)++ | t/02-rakudo/repl.t test: fix a few REPL tests * REPL message now says to use zef, so filter out that message properly * Fix an incorrectly written test - FYI test is currently not enabled when doing `make test` |
23:48 | |
Zoffix | samcv: don't even know what rlwrap is | 23:50 | |
I don't use REPL though... It's just so... uncivilized | 23:51 | ||
samcv | oh. it's like installing Readline | ||
you just `alias perl6='rlwrap perl6'` | |||
sometimes for very simple things i'll use repl | |||
anything more i make a file and execute from atom | |||
or save etc | |||
and rlwrap similar to Readline module saves your previously entered lines | 23:52 | ||
allows you to delete things easily etc. and i don't have to worry about a module not being installed or waiting for it to precomp the Readline module again | 23:53 | ||
after rebuilding rakudo | |||
buggable | ???????????? It's time for the monthly Accidental /win Lottery ???????????? We have 14 ballots submitted by 10 users! DRUM ROLL PLEASE!... | 23:59 |