Perl 6 language and compiler development | Logs at colabti.org/irclogger/irclogger_log/perl6-dev | For toolchain/installation stuff see #perl6-toolchain | For MoarVM see #moarvm
Set by Zoffix on 27 July 2018.
00:27 cognominal-p6 joined 00:28 p6bannerbot sets mode: +v cognominal-p6 00:43 lizmat left 00:51 cognominal-p6 left 01:03 lizmat joined, p6bannerbot sets mode: +v lizmat 01:15 fake_space_whale joined, p6bannerbot sets mode: +v fake_space_whale 02:24 ufobat___ joined 02:25 p6bannerbot sets mode: +v ufobat___ 02:26 ufobat_ left 03:29 Ven`` joined 03:30 p6bannerbot sets mode: +v Ven`` 03:32 Ven`` left 03:42 Ven`` joined, p6bannerbot sets mode: +v Ven`` 03:46 Ven`` left, Ven`` joined 03:47 Ven`` left, cognominal-p6 joined 03:48 p6bannerbot sets mode: +v cognominal-p6 03:51 Ven`` joined, p6bannerbot sets mode: +v Ven`` 03:53 Ven`` left 03:56 Ven`` joined 03:57 p6bannerbot sets mode: +v Ven`` 04:00 cognominal-p6 left 04:01 cognominal-p6 joined, p6bannerbot sets mode: +v cognominal-p6 04:03 Ven`` left 04:11 cognominal-p6 left
AlexDaniel lizmat: related discussion: github.com/perl6/doc/issues/1425 05:14
05:39 AlexDaniel left 05:42 fake_space_whale left 06:20 ExtraCrispy joined, p6bannerbot sets mode: +v ExtraCrispy 07:07 cognominal-p6 joined 07:08 cognominal-p6 left, cognominal-p6 joined 07:09 p6bannerbot sets mode: +v cognominal-p6 07:31 cognominal-p6 left
[Tux] Rakudo version 2018.09-417-g2c43d2bf5 - MoarVM version 2018.09-124-g588873f7a
csv-ip5xs0.895 - 0.903
csv-ip5xs-207.559 - 7.663
csv-parser22.008 - 24.150
csv-test-xs-200.436 - 0.440
test8.266 - 8.698
test-t1.774 - 1.805
test-t --race0.833 - 0.870
test-t-2031.133 - 32.639
test-t-20 --race11.233 - 11.672
07:48
08:38 PufferBot left 08:40 PufferBot joined 08:41 p6bannerbot sets mode: +v PufferBot 09:04 patrickb joined 09:05 p6bannerbot sets mode: +v patrickb 10:15 Ven`` joined 10:16 p6bannerbot sets mode: +v Ven`` 10:27 cognominal-p6 joined 10:28 p6bannerbot sets mode: +v cognominal-p6 10:42 cognominal-p6 left, cognominal-p6 joined 10:43 p6bannerbot sets mode: +v cognominal-p6 10:50 Ven`` left 10:51 cognominal-p6 left
lizmat Files=1256, Tests=76252, 337 wallclock secs (15.92 usr 5.47 sys + 2368.35 cusr 233.54 csys = 2623.28 CPU) 10:56
11:17 Ven`` joined 11:18 p6bannerbot sets mode: +v Ven`` 11:42 Ven`` left 11:49 Kaiepi joined 11:50 p6bannerbot sets mode: +v Kaiepi
Geth rakudo: 266fcc7818 | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
add info
11:53
11:56 Ven`` joined 11:57 p6bannerbot sets mode: +v Ven``
Geth rakudo: bea099889e | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
add info
11:59
rakudo: 5b84611e72 | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
add to list
12:03
12:06 Ulti left
Geth rakudo: 9908c7ee8d | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
add note
12:08
12:11 Ulti joined
Geth rakudo: c0bb8c7ccc | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
order NYI items by my priority
12:12
12:12 p6bannerbot sets mode: +v Ulti
Geth rakudo: 9408080c96 | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
Update rakudo-nqp-notes.md
12:25
rakudo: 7668d8c760 | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
add info
12:31
rakudo: 64a777404e | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
update
12:32
12:38 robertle joined 12:39 p6bannerbot sets mode: +v robertle 12:40 ggoebel left 12:41 Ven`` left 12:44 Ven`` joined, p6bannerbot sets mode: +v Ven`` 13:01 Ven`` left, Ven`` joined 13:02 p6bannerbot sets mode: +v Ven`` 13:06 Ven`` left 13:15 Ven`` joined 13:16 p6bannerbot sets mode: +v Ven`` 13:31 Ven`` left 13:32 yoleaux left 13:33 Ven`` joined, p6bannerbot sets mode: +v Ven`` 13:38 Ven`` left 13:42 Ven`` joined 13:43 p6bannerbot sets mode: +v Ven`` 13:47 cognominal-p6 joined 13:48 p6bannerbot sets mode: +v cognominal-p6 13:49 travis-ci joined, p6bannerbot sets mode: +v travis-ci
travis-ci Rakudo build passed. Tom Browder 'update' 13:49
travis-ci.org/rakudo/rakudo/builds/440988239 github.com/rakudo/rakudo/compare/7...a777404e66
13:49 travis-ci left 13:58 Ven`` left
tbrowder___ hm, do we need a travis rebuild for doc changes? if not, can the tooling be changed to prevent it? 13:58
s/doc/all doc/ 13:59
14:01 patrickb left
Geth rakudo: 8e4d17a79b | (Tom Browder)++ (committed using GitHub Web editor) | docs/rakudo-nqp-notes.md
Update rakudo-nqp-notes.md
14:10
14:11 cognominal-p6 left 14:20 fake_space_whale joined, p6bannerbot sets mode: +v fake_space_whale 14:24 cognominal-p6 joined
tbrowder___ hereā€™s a question fo grammar experts: in Rakudo Grammar.nqp there are several instances of ā€œ<pod_content=.pod_textcontent>**0..1ā€ and TimToady said the **0..1 forces the object to be an array. Both ā€œpod_contentā€ and ā€œpod_textcontentā€ are tokens and both have methods in Actions.nqp. I think the ā€œ.tokenā€ construct means the match isnā€™t to be saved but does it still trigger the action method? 14:24
14:25 p6bannerbot sets mode: +v cognominal-p6
tbrowder___ my guess is no, but i have no proof yet 14:27
geekosaur mm, iirc it means it's not saved as both pod_content and pod_textcontent. I would thnk action methods still fire, it just doesn't get saved twice in the final Match object 14:30
14:33 Ven`` joined 14:34 p6bannerbot sets mode: +v Ven`` 14:38 Ven`` left 14:44 Ven`` joined, Ven`` left 14:45 Ven`` joined 14:46 Ven`` left 14:48 Ven`` joined 14:49 p6bannerbot sets mode: +v Ven`` 15:16 cognominal-p6 left
Geth roast: 2b3f279afb | (Zoffix Znet)++ | S32-io/open.t
[v6.d REVIEW] Don't create a monster filename in bug cover

975-char path still reproes the issue
Orig: github.com/perl6/roast/commit/11fb4415f
15:18
dogbert11 .seen lizmat 15:28
hmm
15:29 pmurias joined, p6bannerbot sets mode: +v pmurias
timotimo about 5 hours ago in here 15:33
4 hours ago in #perl6
dogbert11 timotimo: do we have a dead bot? 15:35
.seen timotimo 15:36
timotimo dunno
yoleaux isn't here, maybe has never been here?
pmurias Zoffix: nothing obvious comes to my bind regarding the blocker 15:38
dogbert11 timotimo: colabti.org/irclogger/irclogger_log...10-03#l166 15:39
15:39 yoleaux joined
dogbert11 yay 15:39
<someone>++
15:40 p6bannerbot sets mode: +v yoleaux
timotimo cool 15:40
dogbert11 indeed
15:40 Ven`` left
pmurias Zoffix: I fixed :U :D in 4b60df8f84fdf2ed8f5eab1426247fbc649d062c but I think that was put in master before the merge 15:42
yoleaux 12 Oct 2018 14:56Z <Zoffix> pmurias: would you have any guesses on this release blocker github.com/rakudo/rakudo/issues/23...-427702563 ? It bisects to js branch merge, yet is fine on both parents. Seems to be something with precomp.
16:04 Ven`` joined, p6bannerbot sets mode: +v Ven`` 16:48 cognominal-p6 joined, p6bannerbot sets mode: +v cognominal-p6 16:53 cognominal-p6 left 16:54 cognominal-p6 joined, p6bannerbot sets mode: +v cognominal-p6
Geth roast: cf93c5a351 | (Zoffix Znet)++ | S32-io/io-handle.t
[v6.d REVIEW] Fix up fh-close-on-exit tests

The first write to a handle is unbuffered, so the original test likely doesn't cover the bug.
Orig: github.com/perl6/roast/commit/08ac054de
17:03
roast: 75d4e63be9 | (Zoffix Znet)++ | S32-io/out-buffering.t
[v6.d REVIEW] Remove dupe close-on-exit test

First write to a handle is unbuffered, so the test doesn't cover the bug. Instead of fixing it, remove it, as there's already another test covering the same ticket on S32-io/io-handle.t:214
Orig: github.com/perl6/roast/commit/40ed...d7b1482a8a
17:04
17:12 AlexDaniel joined 17:14 AlexDaniel left, AlexDaniel joined, p6bannerbot sets mode: +v AlexDaniel
AlexDaniel oh! It's possible to have multiple issue templates on github 17:14
17:14 cognominal-p6 left, cognominal-p6 joined 17:15 p6bannerbot sets mode: +v cognominal-p6 17:39 cognominal-p6 left
Geth roast: 635d660465 | (Zoffix Znet)++ | S17-promise/basic.t
Remove trailing whitespace
17:43
roast: 60344f5892 | (Zoffix Znet)++ | S17-promise/basic.t
[v6.d REVIEW] Test ex. payload instead of .message

Orig: github.com/perl6/roast/commit/6eb5a91c9
17:53 Ven`` left
Geth roast: 8d35f3f525 | (Zoffix Znet)++ | S13-overloading/operators.t
[v6.d REVIEW] Reword test desc to avoid impl details

Orig: github.com/perl6/roast/commit/730b5c823
18:03
roast: 19723fa1e0 | (Zoffix Znet)++ | 3 files
[v6.d REVIEW] Remove Bag/Set/Mix.new !=:= tests

In earlier discussions[^1] we agreed that it's sensible to allow implementations to intern objects that have the same .WHICH and these tests would block such interning.
Orig: github.com/perl6/roast/commit/cbfc93971
  [1] github.com/rakudo/rakudo/issues/19...-400833085
18:08
18:22 Zoffix joined, p6bannerbot sets mode: +v Zoffix
Zoffix .tell timotimo IIRC you worked on async socks in the past. Would you be able to review and merge github.com/perl6/nqp/pull/505 ? 18:22
yoleaux Zoffix: I'll pass your message to timotimo.
18:22 Zoffix left 18:25 undersightable6 left, undersightable6 joined, ChanServ sets mode: +v undersightable6 18:26 p6bannerbot sets mode: +v undersightable6
Geth 6.d-prep: d92a18d4cc | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/README.md
Add Pod to non-in-depth review status of Zofreviews
18:30
Kaiepi .tell timotimo don't merge the asyncsocket op pullreqs yet, i need to tweak it a bit so it'll actually be possible to get the fd of sockets in the jvm later on
yoleaux Kaiepi: I'll pass your message to timotimo.
18:55 Zoffix joined, p6bannerbot sets mode: +v Zoffix
Zoffix .tell samcv would you double-check this commit? github.com/perl6/roast/commit/6c5aaa7e7 `man sprintf` suggests it's right and output should have a plus, but Perl 5 `perl -e 'printf("%+ 10b\n", 9)'` does not produce a `+`. Wondering if P5 just gets it wrong or if there's something missing 18:57
yoleaux Zoffix: I'll pass your message to samcv.
18:57 Zoffix left
Geth roast: bf3d4c484a | (Zoffix Znet)++ | S02-types/native.t
[v6.d REVIEW] Fix incorrect skip count

And considering dogbert never noticed this, suggests $?BITS isn't actually 32 on 32-bit platforms
19:00
roast: c5f03163d1 | (Zoffix Znet)++ | 2 files
[v6.d REVIEW] Appendicize questionable `int` limits test

It might be fine, but I'm unsure whether it's accurate to say
  `int` can be 32 bits on 32-bit platforms (and it isn't on MoarVM),
which the text of the test suggests.
Shove it into appendices, until our natives are more solidly implemented.
Orig: github.com/perl6/roast/commit/f0dbe1c97
19:03
dogbert11 Zoffix: you're right. 'say $?BITS' returns 64 on 32 bit systems
19:04 Zoffix joined, p6bannerbot sets mode: +v Zoffix
Zoffix m: dd Kernel.bits 19:04
camelia Cannot look up attributes in a Kernel type object
in block <unit> at <tmp> line 1
Zoffix m: dd $*KERNEL.bits 19:05
camelia 64
Zoffix dogbert11: what about that one ^
dogbert11 Zoffix: 32
Zoffix Thanks. 19:06
R#2367 19:08
synopsebot R#2367 [open]: github.com/rakudo/rakudo/issues/2367 [6.d review][@LARRY] What's the deal with $?BITS ?
19:09 Zoffix left
samcv .tell Zoffix I think that is right. all our other tests are that way where we print the + 19:27
yoleaux 18:57Z <Zoffix> samcv: would you double-check this commit? github.com/perl6/roast/commit/6c5aaa7e7 `man sprintf` suggests it's right and output should have a plus, but Perl 5 `perl -e 'printf("%+ 10b\n", 9)'` does not produce a `+`. Wondering if P5 just gets it wrong or if there's something missing
samcv: I'll pass your message to Zoffix.
samcv so i'd think we'd have to change the spec otherwise
.tell Zoffix ok this is why: sprintf('%b', -1) #> 1111111111111111111111111111111111111111111111111111111111111111 19:32
yoleaux samcv: I'll pass your message to Zoffix.
samcv that's on Perl 5. so it prints it out the bits stored for a 64 bit integer 19:33
so basically our %b is 100% broken because it doesn't print things out as the exact 64 bit bits the computer stores them as. or our %b implementation works like our other integer format prints and is just a plain number and not two's complement notation 19:35
timotimo is sprintf able to differentiate between an int and an Int being passed? 19:36
yoleaux 18:22Z <Zoffix> timotimo: IIRC you worked on async socks in the past. Would you be able to review and merge github.com/perl6/nqp/pull/505 ?
18:30Z <Kaiepi> timotimo: don't merge the asyncsocket op pullreqs yet, i need to tweak it a bit so it'll actually be possible to get the fd of sockets in the jvm later on
AlexDaniel hah
Kaiepi timotimo: should be good for review now 19:45
19:49 ggoebel joined 19:50 p6bannerbot sets mode: +v ggoebel 19:59 fake_space_whale left
Geth roast: c820202a23 | (Zoffix Znet)++ | S32-str/utf8-c8.t
[v6.d REVIEW] Check we actually match right stuff

Orig: github.com/perl6/roast/commit/1ba17759b
20:17
20:35 cognominal-p6 joined 20:36 p6bannerbot sets mode: +v cognominal-p6 20:50 cognominal-p6 left, cognominal-p6 joined 20:51 p6bannerbot sets mode: +v cognominal-p6 20:59 cognominal-p6 left
Geth roast: 0e059d4078 | (Zoffix Znet)++ | S12-enums/basic.t
[v6.d REVIEW] Test actual type of enum.enums

Orig: github.com/perl6/roast/commit/1fbabe9b5
21:06
21:10 Kaiepi left, Kaiepi joined 21:11 p6bannerbot sets mode: +v Kaiepi 21:14 Kaiepi left 21:15 Kaiepi joined 21:16 p6bannerbot sets mode: +v Kaiepi 21:19 cognominal-p6 joined 21:20 p6bannerbot sets mode: +v cognominal-p6 21:22 cognominal-p6 left 21:23 cognominal-p6 joined 21:24 p6bannerbot sets mode: +v cognominal-p6
Geth roast: 4a740af6f1 | (Zoffix Znet)++ | S02-types/range.t
[v6.d REVIEW] Remove pointless comment
21:39
21:57 pmurias left 22:25 Kaypie joined, Kaiepi left 22:26 p6bannerbot sets mode: +v Kaypie 22:30 Kaypie is now known as Kaiepi 22:36 AlexDaniel left
Geth 6.d-prep: 45532479ca | (Zoffix Znet)++ (committed using GitHub Web editor) | TODO/README.md
Update zofcommit in roast review
23:19
23:33 fake_space_whale joined, p6bannerbot sets mode: +v fake_space_whale
Geth roast: 6279673fa1 | (Zoffix Znet)++ | S32-num/is-prime.t
[v6.d REVIEW] Toss .is-prime tests on unreduced rats

It's now a requirement for Rationals to be reduced on creation, so these tests don't make sense and .is-prime is well-covered already
Orig: github.com/perl6/roast/commit/a71839ea2
23:41
23:42 MasterDuke left