[Coke] | (google code in, summer of code) they are good for building community, but take a lot of effort, and SOC is dependant on the org getting accepted, which we have no control over. | 04:29 | |
.sen Zoffix | 04:40 | ||
.seen Zoffix | |||
yoleaux | I saw Zoffix 13 May 2017 19:55Z in #perl6-dev: <Zoffix> What for? | ||
[Coke] | m: sub eof(IO::Handle:D: --> Bool:Ds) {}; | 04:45 | |
camelia | 5===SORRY!5=== Error while compiling <tmp> Invalid type smiley 'Ds' used in type name at <tmp>:1 ------> 3sub eof(IO::Handle:D: --> Bool:Ds7⏏5) {}; |
||
Zoffix | Ugh. Perl 6 dreams suck because you start to forget which bugs are real and which are you dreamed of | 04:48 | |
s: "", 'Str', \() | |||
SourceBaby | Zoffix, Sauce is at github.com/rakudo/rakudo/blob/4298...Str.pm#L43 | ||
Zoffix | I guess we don't *actually* have any cases where Str.Str would throw ^_^ | ||
[Coke] | Zoffix: I have a bunch of fixes for xtest in the pipe. | 04:55 | |
Zoffix | ok | 04:56 | |
[Coke] | looks like you got one of my 5 commits with that last one. | 04:59 | |
Geth | rakudo/nom: 12c50b6397 | (Daniel Green)++ | 2 files Throw X::NYI when trying to subset a native type Native type subsets can be created currently, but don't work, always saying the type check failed (see RT #131310). |
05:13 | |
rakudo/nom: 80100aed90 | (Zoffix Znet)++ (committed using GitHub Web editor) | 2 files Merge pull request #1082 from MasterDuke17/native_types_subset_are_NYI Throw X::NYI when trying to subset a native type |
|||
synopsebot6 | Link: rt.perl.org/rt3/Public/Bug/Display...?id=131310 | ||
Zoffix | Any idea how to get an IO::Handle that has .t ("is tty?") returning true? | 05:25 | |
Seems the only case we test is it giving False | |||
d'oh... $*OUT.t :) | 05:33 | ||
wonder how to have it still True in a test :/ | 05:34 | ||
my $tt = shell :out, "tty"; $tt.out.slurp(:close).trim.IO.open.t.say | 05:37 | ||
This will do in a pinch, I guess | |||
Geth | roast: 60f5a6d9e7 | (Zoffix Znet)++ | S32-io/io-handle.t [io grant] Test IO::Handle.t when handle is a TTY |
05:46 | |
roast: a3410d1143 | (Zoffix Znet)++ | S32-io/io-handle.t Ensure we close the handle when done with the test |
05:47 | ||
rakudo/nom: c3b47280c6 | (Elizabeth Mattijsen)++ | src/core/Compiler.pm Compiler.verbose-config stringifies to -V output - my $a = Compiler.verbose-config.Str # perl -V output in var - Compiler.verbose-config(:say) # say perl -V output - say Compiler.verbose-config # say perl -V output - Compiler.verbose-config<moar><version> # index into it as hash |
06:03 | ||
rakudo/nom: d18a3332e6 | (Zoffix Znet)++ | src/core/io_operators.pm Add :D to IO::Handle arg in IO routines To maintain consistency with methods |
07:15 | ||
Zoffix | ZofBot: happy IO::Handles only! | ||
ZofBot | Zoffix, Soon we plan to fill in the gaps with the Perl 5 details though | ||
Zoffix | ZofBot: eww no! | ||
ZofBot | Zoffix, 01 | ||
Zoffix | yeah... searching the logs for discussion of ".tell" method is rather a futile excersise :| | 07:55 | |
[TuxCM] | This is Rakudo version 2017.04.3-249-gd18a3332e built on MoarVM version 2017.04-64-g6d5ea042 | 07:59 | |
csv-ip5xs 2.516 | |||
test 12.549 | |||
test-t 4.194 - 4.258 | |||
csv-parser 12.809 | |||
Zoffix | ZOFVM: Files=1242, Tests=135560, 114 wallclock secs (21.92 usr 3.48 sys + 2338.84 cusr 156.00 csys = 2520.24 CPU) | 08:07 | |
Geth | rakudo/nom: 276d4a7e7e | (Zoffix Znet)++ | src/core/IO/Handle.pm [io grant] Remove .tell info in IO::Handle.gist .tell doesn't always give right info for handles in non-binary mode and per discussion[^1], this is because .tell is for binary mode only. [1] irclog.perlgeek.de/perl6-dev/2017-...i_14477943 |
08:08 | |
[TuxCM] | buggable speed | 08:11 | |
Zoffix | buggable: speed | 08:13 | |
buggable | Zoffix, ▇▆▆▆▆▆█▇▇▇▇██▆▂▂▂▃▄▂▂▃▃▂▂▂▁▂▂▂▂▂▂▁▁▁▁▁▁▁▁↑▁▁▁▂▁▁▁▁ data for 2017-04-25–2017-05-14; range: 4.194s–6.181s; 23% faster | ||
Zoffix | buggable: speed 20 | ||
buggable | Zoffix, ▄▃▃▂▂▂▂▂▂▂▂↑▂▁▂▃▁▁▂▁ data for 2017-05-06–2017-05-14; range: 4.194s–6.181s; 4% faster | ||
Geth | roast: 31e39932d8 | (Zoffix Znet)++ | S32-io/io-path.t [io grant] Test IO::Path*.gist |
08:27 | |
roast: e2a3a25751 | (Zoffix Znet)++ | S32-io/io-path.t Add mixed slashes to IO::Path.gist tests |
08:34 | ||
rakudo/nom: 7c8f8d3eed | (Elizabeth Mattijsen)++ | src/core/VM.pm Simplify getting osname from VM.config We shouldn't need to know the backend to find out the OS name. |
09:00 | ||
Zoffix | "Perl5 to Java compiler - first release": blogs.perl.org/users/flavio_s_glock...lease.html | ||
ZOFFLOP: t/spec/S32-io/IO-Socket-INET.t # again starts taking ages to run; until you reboot the box | 09:14 | ||
Seems something is left over listening: gist.github.com/zoffixznet/75784f6...804bdd16bd | 09:15 | ||
tho killing that one moar process doesn't fix the issue | |||
and I notice shutting off the google VM with that issue takes waaay longer than normal :/ | 09:17 | ||
Geth | rakudo/nom: 48cf0e670b | (Zoffix Znet)++ | src/core/IO/Spec/Cygwin.pm [io grant] Make IO::Spec::Cygwin.is-absolute 21x faster |
09:24 | |
lizmat started working on a Perl 6 version of metacpan.org/pod/System::Info | 09:26 | ||
progress can be seen at github.com/lizmat/System-Info | |||
Geth | rakudo/nom: e79d749864 | (Elizabeth Mattijsen)++ | src/core/VM.pm Introducing VM.osname Quick and dirty way to get at what is $^O in Perl 5. Works both as an instance method, as well as a class method (the latter case will just look at the information of your OS, rather than what is livin inside the object). |
09:29 | |
Zoffix | m: say IO::Spec::Unix.is-absolute: "/\x[308]" | ||
camelia | False | ||
Zoffix | m: say IO::Spec::Win32.is-absolute: "/\x[308]" | ||
camelia | True | ||
Zoffix | Oh, the irony... | ||
m: use nqp; my $path := "/\[308]"; dd nqp::eqat($path, '/', 0); dd nqp::ord($path); dd nqp::iseq_i(($_ := nqp::ord($path)), 92) || nqp::iseq_i($_, 47) | 09:30 | ||
camelia | 1 47 1 |
||
Zoffix | I don't get how it ends up False in Unix tho :/ | ||
s: IO::Spec::Unix, 'is-absolute', \("/\x[308]") | |||
SourceBaby | Zoffix, Sauce is at github.com/rakudo/rakudo/blob/7c8f...nix.pm#L96 | ||
Zoffix | yeah, so wtf... | ||
m: use nqp; class Foo { method foo (Str() \p) { dd nqp::p6bool(nqp::eqat(p, '/', 0)); } }.foo: "/\[308]" | 09:32 | ||
camelia | Bool::True | ||
Zoffix | that's IO::Spec::Unix.is-absolute :/ gives true here | 09:33 | |
ZofBot: what sorcery is this!? | |||
ZofBot | Zoffix, For example: 1, 2, :a<b> The list represented here has 3 positional items | ||
Zoffix | I don't get it | 09:34 | |
OMG | 09:36 | ||
I missed the damn "x" in the escape -_- | 09:37 | ||
m: use nqp; class Foo { method foo (Str() \p) { dd nqp::p6bool(nqp::eqat(p, '/', 0)); } }.foo: "/\x[308]" | |||
camelia | Bool::False | ||
Zoffix | m: use nqp; my $path := "/\x[308]"; dd nqp::eqat($path, '/', 0); dd nqp::ord($path); dd nqp::iseq_i(($_ := nqp::ord($path)), 92) || nqp::iseq_i($_, 47) | ||
camelia | 0 47 1 |
||
Zoffix | there we go | ||
Geth | rakudo/nom: 1870685276 | (Elizabeth Mattijsen)++ | src/core/VM.pm Partially revert 7c8f8d3eed7fc42b925c629 There could be code out there looking for .config<os.name> |
09:40 | |
rakudo/nom: 505ee33def | (Elizabeth Mattijsen)++ | src/core/Exception.pm Recommend VM.osname rather than Distro.name VM.osname is basically what $^O is in Perl 5 |
09:51 | ||
Zoffix | And apparently "/\x[308]" is absolute on Unix, but not Win32; "/\x[308]".IO.open: :w; creates file "\x[308]" in root dir on Unix but in current dir on Win32 | 09:57 | |
Perl 5 dies with "The filename, directory name, or volume label syntax is incorrect." for that path | 10:00 | ||
At least with `perl -wlE "open my $fh, ">", qq|e\x{308}| or die; close $fh"` which I don't get | 10:01 | ||
ah, `""` around `>` | 10:04 | ||
And Perl 5 makes them in root on Windows too, so I guess this is a bug | 10:06 | ||
in our impl | |||
Geth | rakudo/nom: 09506dc80f | (Elizabeth Mattijsen)++ | src/core/IO/Spec.pm Streamline IO::Spec.select Use the new and shiny VM.osname |
10:22 | |
Zoffix | PSA: don't use `skip-rest` in roast. Use `skip` however many tests need to be skipped. | 10:29 | |
cause when people later add more tests to the file they have to puzzle over why the tests they added getting skipped... | |||
Zoffix greps for `skip-rest` and sees a ton of usage :( | 10:31 | ||
ugexe | then you'll just get people who wonder why their test doesn't get run because they put them within the skip range | 10:32 | |
Zoffix | Nope, you won't. | 10:33 | |
ugexe | i guess if you assume that all skipped tests would fail | 10:36 | |
Zoffix | all the skipped tests won't get run at all :/ | 10:37 | |
skip "...", 42; prints 42 `ok`s with # SKIP marker | |||
Geth | rakudo/nom: f4309de9f3 | (Zoffix Znet)++ | src/core/IO/Spec/Unix.pm [io grant] Fix IO::Spec::Unix.is-absolute for combiners on / |
10:42 | |
roast: c481433260 | (Zoffix Znet)++ | 5 files [io grant] Test .is-absolute method for / with combiners Rakudo fix: github.com/rakudo/rakudo/commit/f4309de9f3 |
10:47 | ||
Zoffix | Actually the "/\x[308]" thing opening wrong file on Windows is due to Win32 SPEC's rel2abs; nqp::open works fine. | 11:21 | |
And I have its perf-boosted version laying around somewhere that should already have this issue fixed IIRC | 11:22 | ||
Zoffix will fix after a nap | 11:23 | ||
tadzik | github.com/tadzik/rakudobrew/issues/102 | 14:01 | |
Zoffix | I don't have such issue with that module; installed with zef.. "Aug 2016".. yeah, that's a pretty stale Issue | 14:44 | |
geth ded eh | 14:48 | ||
tadzik | but muh karma | ||
Zoffix | well, adding a log was useless :/ this is all I see at the end of geth's screen log; no errors or anthing: gist.github.com/zoffixznet/5bb174e...cf5c74a11e | 14:50 | |
Second time a charm maybe.... | |||
Will wait for another mystery crash | |||
tadzik | would you like to take a look at modules.perl6.org/problems in the meantime? :) | 14:54 | |
Zoffix | 404 | ||
tadzik | I did a bit of cargo-culting especially in the DBIC parts | ||
impossibru | |||
Zoffix is busy with IO | |||
tadzik | alrighty then | ||
lizmat | m: my %h = (a => 42, b => 666) # works as expected | 15:07 | |
camelia | ( no output ) | ||
lizmat | m: my constant %h = (a => 42, b => 666) # WAT? | 15:08 | |
camelia | 5===SORRY!5=== Error while compiling <tmp> Type check failed in constant declaration of %h; expected Associative but got List (List) at <tmp>:1 ------> 3nstant %h = (a => 42, b => 666) # WAT?7⏏5<EOL> |
||
lizmat | feels to me this is a bug, no ? | ||
Zoffix | lizmat: yeah, that's a known issue | ||
lizmat | ok | ||
:-( | |||
Zoffix | lizmat: the bug being it doesn't auto-convert stuff, but if you do it yourself you can have a constant hash | 15:09 | |
m: my constant %h = %(a => 42, b => 666) | |||
camelia | ( no output ) | ||
Zoffix | This is the only ticket I see ATM: rt.perl.org/Ticket/Display.html?id...et-history | 15:11 | |
But I recall filing one as well. | |||
Geth | rakudo/nom: c96727aa55 | (Zoffix Znet)++ | src/core/IO/Spec/Win32.pm [io grant] Fix combiners on SPEC::Win32.rel2abs; make 6% faster - When the path starts with a slash that has a combiner on it, we fail to corretly make an absolute path. Fix by avoiding regex and using nqp string operations instead. - Possible other (unfixed) cases still exist due to splitpath using a regex, but will be fixed when splitpath is fixed - Make 6% faster; much more boostage will happen after after splitpath is fixed/improved |
16:26 | |
Zoffix | ugh | 16:28 | |
GitHub is at it again with the dumb styling "improvements" | |||
This is what happens when too many voices ask for features and they're implemented without careful thought. | 16:31 | ||
Geth | roast: 8ee0a0a6a5 | (Zoffix Znet)++ | S32-io/io-spec-win.t [io grant] Test IO::Spec::Win32.rel2abs with combiners Rakudo fix: github.com/rakudo/rakudo/commit/c96727aa55 |
16:32 | |
Zoffix | .tell samcv sent you a pr: github.com/samcv/IRC-TextColor/pull/2 | 16:34 | |
yoleaux | Zoffix: I'll pass your message to samcv. | ||
Zoffix | m: say IO::Spec::Cygwin.is-absolute: 「C:\foo」; | 16:59 | |
camelia | True | ||
Zoffix | But it's not, is it? | 17:00 | |
'cause C:\ on my cygwin is at /cygdrive/c | |||
Ah, nevermind. I was testing wrong | 17:01 | ||
or not.. | 17:02 | ||
`cd C:\` works and gets me to /cygdrive/c, but `less C:\eula.1028.txt` gives "C:eula.1028.txt: No such file or directory" | 17:03 | ||
note the lack of slash... and if I use C:/eula... slash is there but still can't open file | 17:04 | ||
and `less /cygdrive/c/eula.1028.txt` reads the file. | |||
eater | Zoffix: what does `ls C:\` give? | 17:05 | |
Zoffix | waits for more input | 17:06 | |
ls C:/ gives contents of my C drive | 17:07 | ||
:S | |||
less C:/CPANPRC/rakudo/README.md works fine | 17:08 | ||
stupid system | |||
eater | ah! | ||
permissions | |||
Zoffix | Ah | ||
eater | try running it as admin | ||
and see if it works then | |||
Zoffix | But how? I can `less /cygdrive/c/eula.1028.txt` but not `less C:/eula.1028.txt` | 17:09 | |
Screw it. Will poke at it some time later. | 17:10 | ||
Geth | rakudo/nom: 5feb3906d7 | (Elizabeth Mattijsen)++ | src/core/Compiler.pm Show System::Info information with -V if available If System::Info is installed, then show that info in the -V output as well. |
17:16 | |
Zoffix | NeuralAnomaly: status | 17:23 | |
NeuralAnomaly | Zoffix, [✘] Next release will be in 5 days and 11 hours. Since last release, there are 37 new still-open tickets (13 unreviewed and 1 blockers) and 284 unreviewed commits. See perl6.fail/release/stats for details | ||
Zoffix | ZofBot: lotsacommits! | ||
ZofBot | Zoffix, subst(/pat/, "replacement"); $str | ||
dogbert17 | Zoffix: do you want to proofread some new docs? | 17:53 | |
eater | Zoffix: for me in Cygwin, `less C:\help.txt` will give me an: No such file found, but `less C:/help.txt` works | 17:54 | |
dogbert17 | or anyone else for that matter: gist.github.com/dogbert17/d76977c9...4ea308ea74 | ||
eater | it will translate C:\help.txt to C:help.txt so it didn't work | ||
Zoffix | nope; busy | 17:55 | |
dogbert17 | caught up in IO madness I presume :) | 17:56 | |
eater | dogbert17: if start is 2, wouldn't it result in an index of 3 | 17:57 | |
m: say "banana".indices("ana", 2); | |||
camelia | (3) | ||
dogbert17 | eater++, indeed (a typo) | 17:58 | |
gist has been updated | |||
eater | dogbert17: left a comment, LGTM tho :) | 18:04 | |
Zoffix | m: use Test; for (('C:', './', 'ber'), ('C:', './', '/ber/'), ('C:', './', 'ber'), ('', './', 'ber'), ('', './', ''), ('', '', ''),) -> $p { for IO::Spec::Win32, IO::Spec::Unix { is .join(|$p), .catpath(|$p); } } | 18:05 | |
camelia | ok 1 - ok 2 - ok 3 - ok 4 - ok 5 - ok 6 - ok 7 - ok 8 - ok 9 - ok 10 - ok 11 - ok 12 - |
||
Zoffix | So what exactly is the differente between $*SPEC.join and $*SPEC.catpath? :/ | ||
Seems the only case they differ at is when both dirname and filename are strings "/" | 18:14 | ||
m: with ("/", "/") { without IO::Spec::Unix -> $s { $s.join(|$_).say; $s.catpath($_).say } } | |||
camelia | Too few positionals passed; expected 4 arguments but got 3 in block at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
Zoffix | m: with ("/", "/") { without IO::Spec::Unix -> $s { $s.join(|$_).say; $s.catpath(|$_).say } } | ||
camelia | Too few positionals passed; expected 4 arguments but got 3 in block at <tmp> line 1 in block <unit> at <tmp> line 1 |
||
Zoffix | m: with ("", "/", "/") { without IO::Spec::Unix -> $s { $s.join(|$_).say; $s.catpath(|$_).say } } | 18:15 | |
camelia | / // |
||
Zoffix | But that seems more of a bug than anything -_- | ||
m: with ("", "/", "/f/") { without IO::Spec::Unix -> $s { $s.join(|$_).say; $s.catpath(|$_).say } } | |||
camelia | //f/ //f/ |
||
Zoffix | m: with ("", "/f/", "/f/") { without IO::Spec::Unix -> $s { $s.join(|$_).say; $s.catpath(|$_).say } } | ||
camelia | /f//f/ /f//f/ |
||
Zoffix | 'cause all these cases are the same... | ||
eater | github.com/rakudo/rakudo/blob/48cf...32.pm#L114 | 18:16 | |
Join has a bit more checking?? | |||
Zoffix | It has more code... but what sort of checking? It just changes file/dirname to something other and then passes the stuff to catpath | 18:17 | |
eater | yeah... | ||
Unix has the exact same code | 18:20 | ||
Zoffix | .ask labster what's the difference between IO::Spec::*.join and .catpath? Seems the only dif is .join has an extra check that dir and file are both not "/"; but what usecase does that cover that there are two methods for it? | ||
yoleaux | Zoffix: I'll pass your message to labster. | ||
eater | only indenting is different :') | ||
Zoffix sets $*SPEC on fire | |||
eater | :') | ||
Zoffix | eater: you're looking at too old a version then | ||
s: IO::Spec::Unix, 'join', \() | 18:21 | ||
SourceBaby | Zoffix, Something's wrong: ERR: Type check failed in binding to parameter '&code'; expected Callable but got Method+{<anon|63132592>} (Method+{<anon|6313259...) in sub do-sourcery at /home/zoffix/services/lib/CoreHackers-Sourcery/lib/CoreHackers/Sourcery.pm6 (CoreHackers::Sourcery) line 42 in sub sourcery at /home/zoffix/services/lib/CoreHackers-Sourcery/lib/CoreHackers/Sourcery.pm6 (CoreHackers::Sourcery) line 29 in block < | ||
Zoffix | *sigh* | ||
left is unix and center is win32; completely different codes: i.imgur.com/AfeyHpC.png | 18:23 | ||
But behaviour is the same; it's just win32 has to care for volume too | |||
eater | is using nqp functions faster? | 18:24 | |
Zoffix | Yeah, much faster | ||
At least for now; I think the end-goal is for Perl 6 code to be automagically equally fast | |||
eater | :m say $*SPEC.join("", ".", "ha") ~ " <=> " ~ $*SPEC.catpath("", ".", "ha"); | 18:27 | |
bleh | |||
m: say $*SPEC.join("", ".", "ha") ~ " <=> " ~ $*SPEC.catpath("", ".", "ha"); | |||
camelia | ha <=> ./ha | ||
eater | there is your difference | 18:28 | |
Zoffix | Thanks. | 18:30 | |
Though I still don't get why that difference exists. | 18:31 | ||
eater | can't really find a reason when this is useful tho | ||
except when `run`'ing things | |||
Zoffix | As in: if you documented .join/.catpath, what would you write? | ||
eater has no clue | |||
I think this would be better as flag than 2 seperate functions | 18:32 | ||
if there is a use-case | |||
m: say IO::Spec::Win32.join("C:", "/", "/") ~ " <=> " ~ IO::Spec::Win32.catpath("C:", "/", "/"); | 18:33 | ||
camelia | C:/ <=> C:// | ||
eater | also this | ||
Zoffix | .tell labster and another special case in join is with `dirname` = '.' and `file` = 'anything'; with .catpath putting a space between, but .join just returning the filename. How should this stuff be documented?? | ||
yoleaux | Zoffix: I'll pass your message to labster. | ||
eater | when would that extra slash matter? every system just ignores it | 18:34 | |
but the weirdest thing is | 18:35 | ||
if you do | |||
say IO::Spec::Win32.join("C:", "/", "/ha") ~ " <=> " ~ IO::Spec::Win32.catpath("C:", "/", "/ha"); | |||
evalable6 | C://ha <=> C://ha | ||
AlexDaniel | \o/ | 18:38 | |
Zoffix | ? | 18:39 | |
Ah :) autodetection :) | |||
AlexDaniel | Zoffix: by the way “your new diff highlights make it much harder to view large diffs”, what changed? And also, why not view it side by side instead? | 18:44 | |
Zoffix | $ perl6 -e 'IO::Handle.new.nl-in = 42' | ||
set separator requires an object with REPR MVMOSHandle (got Scalar with REPR P6opaque) | |||
boo | |||
AlexDaniel | or is it highlighting parts of the line even if that line is completely new? | ||
Zoffix | AlexDaniel: yes, they now highlight each like like a peacock, making it harder to read | 18:45 | |
AlexDaniel | ah ok, I see | ||
Zoffix | s/like/line/; | ||
ZOFFLOP: t/spec/S11-modules/nested.t | 19:05 | ||
Geth | rakudo/nom: 06d8800e2b | (Zoffix Znet)++ | src/core/IO/Handle.pm [io grant] Fix crash when setting .nl-in ... ... on unopened IO::Handle |
19:09 | |
Zoffix grumbles | 19:34 | ||
IO::Handle.nl-in = [<foo bar>] but IO::Handle.encoding('utf8') -_- | |||
heh | 19:57 | ||
eater: in this community we usually don't thank/praise sarcastically :) | 19:58 | ||
unless you're a bot... | 20:03 | ||
ZofBot: thank you for your ears of service! | |||
ZofBot | Zoffix, c#L240 at least doesn't show it on coverage reports | ||
eater | :') | ||
Geth | rakudo/nom: 7e9496dd19 | (Zoffix Znet)++ | src/core/IO/Handle.pm [io grant] Make IO::Handle.encoding settable via .new Just like all the other publicly-accessible attributes. |
20:14 | |
rakudo/nom: 7003885575 | (Zoffix Znet)++ | src/core/IO/Handle.pm [io gran] Fix crash when setting IO::Handle.encoding ... on an unopened handle |
20:15 | ||
rakudo/nom: 95e49dcbe2 | (Zoffix Znet)++ | src/core/IO/Handle.pm [io grant] Make IO::Handle.open respect attribute values Default to the values of attributes instead of hardcoding own defaults in the `method open`. |
20:16 | ||
roast: a41027f9fb | (Zoffix Znet)++ | S32-io/io-handle.t [io grant] Test IO::Handle.nl-in can be set |
20:17 | ||
roast: bfca66bee0 | (Zoffix Znet)++ | S32-io/open.t Mark Failure as handled after test So it doesn't spam "failure in destroy" warnings in output |
|||
timotimo | oh crap | ||
Zoffix | ? | 20:18 | |
timotimo | for some reason my browser splits a line "$proc.stdout.tap(-" and "> $v { ... });" | ||
docs.perl6.org/type/Proc::Async - do you see that? it's in the first code block | |||
Zoffix | 1 sec | ||
Geth | roast: e82b7984a0 | (Zoffix Znet)++ | S32-io/open.t [io grant] Test IO::Handle.open respects attributes Rakudo fix: github.com/rakudo/rakudo/commit/95e49dcbe2 |
20:19 | |
roast: 2c291509b9 | (Zoffix Znet)++ | S32-io/io-handle.t [io grant] Test IO::Handle.nl-in attribute |
|||
timotimo | "you split words at dashes, right?" - yeah, browser, definitely. | 20:20 | |
AlexDaniel | u: no break dash | 20:22 | |
unicodable6 | AlexDaniel, Found nothing! | ||
AlexDaniel | u: no break | ||
unicodable6 | AlexDaniel, U+006E LATIN SMALL LETTER N [Ll] (n) | ||
AlexDaniel, U+006F LATIN SMALL LETTER O [Ll] (o) | |||
AlexDaniel, 9 characters in total: gist.github.com/509fa06681ed1660a2...15cbb302dd | |||
AlexDaniel | u: no break | ||
unicodable6 | AlexDaniel, U+00A0 NO-BREAK SPACE [Zs] ( ) | ||
AlexDaniel, U+2011 NON-BREAKING HYPHEN [Pd] (‑) | |||
AlexDaniel, 4 characters in total: gist.github.com/9fd357c35129472a5d...93f75ea137 | |||
Zoffix | Doesn't wrap for me. | ||
AlexDaniel | hyphen, right | ||
timotimo | ah, it's because i'm always zoomed in at least a bit | ||
Zoffix | Well, unless I make my browser super tiny | 20:23 | |
timotimo | it's ok since it's my fault | ||
Zoffix | timotimo: you can fix that by changing this `pre-wrap` to just `pre`, but I think the reason it's a pre-wrap in the first place is because with `pre` you'd have to always scroll most of the codeblocks on tiny screens. So it's a choice between two evils and the lesser one might be wrapping at weird places: github.com/perl6/doc/blob/master/a...e.scss#L39 | 20:26 | |
like wrapping after a `}` likely also lead to broken code | |||
timotimo | that's true | 20:28 | |
hm, right | |||
Zoffix | bisect: '.'.IO.^lookup('add').say | 20:40 | |
bisectable6 | Zoffix, Bisecting by output (old=2015.12 new=95e49dc) because on both starting points the exit code is 0 | ||
Zoffix, bisect log: gist.github.com/4fdf21d0798f97c6af...fc62d76c2f | |||
Zoffix, (2017-04-16) github.com/rakudo/rakudo/commit/9d...696b244786 | |||
timotimo | it me \o/ | ||
Zoffix | lulzwut. Wrong commit, bisectable6 | ||
eater: but yeah, .add is new | 20:41 | ||
eater | ah | ||
it was called concat-with | |||
thanks Zoffix :) | |||
Zoffix | briefly | ||
eater | well guess my project will just depend on 2017.04 then | 20:42 | |
Zoffix | buggable: eco RakudoPrereq | 20:43 | |
buggable | Zoffix, RakudoPrereq 'Require specific versions of Rakudo': github.com/zoffixznet/perl6-RakudoPrereq | ||
Zoffix | You can use that | ||
eater: or you can use .child method. It's the same as .add, but will be slightly changed in 6.d | |||
eater | oh yes! | ||
thanks Zoffix :) | 20:46 | ||
Zoffix | AlexDaniel: seems bisectable is off-by-one above. The actual commit one later after what it gave: github.com/rakudo/rakudo/commits/0...7d020de665 | ||
AlexDaniel | that sounds pretty bad | ||
Zoffix | c: 0b5a41b '.'.IO.^lookup('add').say | ||
committable6 | Zoffix, ¦0b5a41b: «add» | ||
Zoffix | c: 0b5a41b~1 '.'.IO.^lookup('add').say | ||
committable6 | Zoffix, gist.github.com/feca134d6dcb0a882b...7a9ae46656 | ||
AlexDaniel | except that it seems to be right? | ||
Zoffix | c: 9d8e391f3b45 '.'.IO.^lookup('add').say | ||
committable6 | Zoffix, gist.github.com/e681999bb0fa34c60b...2b6040ef88 | ||
AlexDaniel | c: 0b5a41b^ '.'.IO.^lookup('add').say | 20:47 | |
committable6 | AlexDaniel, gist.github.com/036013b812603f850c...888ace3c30 | ||
AlexDaniel | ah wait | ||
that's just broken build | |||
Zoffix | oh ok | ||
AlexDaniel | I'm really sorry about this, but I don't know yet why this happens | 20:49 | |
I know that redoing it will help, sometimes… sometimes I had to delete it a couple of times | |||
I have a script that attempts to detect builds like this, but it doesn't work for some reason | 20:50 | ||
or rather, it checks for /perl6 executable, and it seems to be there | |||
but in this case the issue is with nqpmo.moarvm | 20:51 | ||
but still, I'm not sure why this happens in the first place… something is wrong when it creates the archive | |||
here github.com/perl6/whateverable/blob...#L172-L173 | 20:52 | ||
a clever mind will already notice an unrelated error there… | |||
bisect: '.'.IO.^lookup('add').say | 20:53 | ||
bisectable6 | AlexDaniel, Bisecting by output (old=2015.12 new=95e49dc) because on both starting points the exit code is 0 | ||
Zoffix | You're not closing .out? :) | ||
bisectable6 | AlexDaniel, bisect log: gist.github.com/8ae1bf550d28900731...b1ae700784 | ||
AlexDaniel, (2017-04-16) github.com/rakudo/rakudo/commit/0b...7d020de665 | |||
AlexDaniel | yes | ||
Zoffix | \o/ | ||
AlexDaniel | but this should simply leave the socket open, I don't think it would cause any problems? | 20:54 | |
or maybe it will | |||
Zoffix | no idea | ||
AlexDaniel | I should close it anyway | ||
Geth | roast: 03ce93b7d6 | (Zoffix Znet)++ | S32-io/io-handle.t [io grant] Test IO::Handle.encoding can be set Rakudo fix: github.com/rakudo/rakudo/commit/7003885575 |
||
Zoffix calls it a day | |||
ZofBot: good night! | |||
ZofBot | Zoffix, However, we're rather liberal in what we consider a name | 20:55 | |
Zoffix | ZofBot: I knew that the minute you've entered this channel! | ||
ZofBot | Zoffix, (Sigspace after the entire construct matches once after the all repetitions are found | ||
AlexDaniel | ZofBot: sometimes you are not making sense | ||
ZofBot | AlexDaniel, do we want to restrict the actual github ids that can even push to 6 | ||
MasterDuke_ | .ask jnthn do you have an opinion re rt.perl.org/rt3/Public/Bug/Display...?id=131310 ? should subsets of native types be allowed? | 21:24 | |
yoleaux | MasterDuke_: I'll pass your message to jnthn. |