| IRC logs at
Set by AlexDaniel on 12 June 2018.
00:02 patrickz left 00:55 Altai-man_ joined 00:57 sena_kun left 02:55 sena_kun joined 02:57 Altai-man_ left 04:55 Altai-man_ joined 04:57 sena_kun left 06:56 sena_kun joined 06:57 Altai-man_ left 07:44 leont joined 08:55 Altai-man_ joined 08:57 sena_kun left 10:56 sena_kun joined 10:57 Altai-man_ left 11:36 synthmeat left, synthmeat joined
Geth MoarVM: 0racle++ created pull request #1317:
Pre-declare loop vars.
nwc10 # For some reason their new (or current) ftp server doesn't work with how curl does FTP URLs. (wget seems to be fine) 12:48
might actually be a curl bug. (IIRC there's some crazy thing about who paths you send to the FTP server should not be abssolute unless the FTP URL has a // after the hostname) 12:52
but you don't hit this if you ensure that your anon FTP user's home directory is / 12:53
and IIRC you definately do hit it if your daft enough to have your server permissions set up such that `chdir /` is not allowed
12:55 Altai-man_ joined 12:57 sena_kun left
nwc10 for the avoidance of doubt, I'm not planning to (try to) upgrade to Uncode 13.0 13:54
Geth MoarVM: 0c651c23bf | (Nicholas Clark)++ | tools/UCD-download.p6 no longer works with curl's default ftp method.

Adding --ftp-method nocwd seems to fix this.
Note, the script as-is will fail to get emoji-13.0/emoji-data.txt because
  (in their infinite wisdom) they have decided that it is now in a different
directory -- ... (6 more lines)
nwc10 I just wanted to avoid anyone else having to repeat my little fight 14:01
lizmat hopes samcv will find the tuits to do that 14:04
14:10 rbt joined 14:55 sena_kun joined 14:57 Altai-man_ left, patrickb joined
Geth MoarVM: 06d8cdd164 | (Daniel Green)++ | src/gc/debug.c
Fix compiler warning in GC debug code...

since `szc->num_pages` is an MVMuint32.
16:55 Altai-man_ joined 16:57 sena_kun left
Geth MoarVM: patrickbkr++ created pull request #1318:
Explicitly state we use gnu99 on GCC
18:09 vrurg left 18:11 zakharyas joined
Geth MoarVM/partial-heap-profiler-fix: 0107a64912 | (Nicholas Clark)++ | src/profiler/heapsnapshot.c
Correct 2 regressions in the binary heap profiler output.

I believe that it has been broken in master since early September 2019.
The first (the assignment to i) was broken by commit 505d0b53830e27c7:
   Revert "Revert "Merge branch 'master' of""
... (13 more lines)
¦ MoarVM: nwc10 assigned to timo Issue binary --profile dumps have been broken since Sept 2019 18:23
18:39 vrurg joined 18:55 sena_kun joined 18:57 Altai-man_ left
samcv nwc10, so you got it working? 19:00
nwc10 I got the download part (msotly) working
apart from the particular file in emjoi 13.0 which they've moved to a different directory 19:01
and I hacked the URLs (two different variants; thank you Unicode consortium) to get 12.1 locally
and could (almost perfectly) regenerate what we have checked into git
then I was singularly failing to get the binary format for the heap profiler working 19:03
19:10 rbt left 19:11 rbt joined
timotimo nwc10:what did you use to read the heap snapshot files? 19:47
nwc10 which I still thought was the correct tool 19:48
20:04 brrt joined
nwc10 good *, brrt 20:05
brrt ohai nwc10
good * to you
timotimo nwc10: sorry, you will want timo's fork with nqp-ops-make-stuff-faster branch 20:31
20:43 zakharyas left, brrt left 20:55 Altai-man_ joined 20:57 sena_kun left 22:55 sena_kun joined 22:57 Altai-man_ left 23:18 patrickb left 23:55 leont left