| releasable6 | Next release in ≈6 days and ≈15 hours. 2 blockers. Please log your changes in the ChangeLog: github.com/rakudo/rakudo/wiki/ChangeLog-Draft | 03:00 | |
|
03:20
evalable6 left,
linkable6 left
03:22
linkable6 joined
03:23
evalable6 joined
04:17
maggotbrain joined
06:32
frost-lab joined
07:53
evalable6 left,
linkable6 left
07:55
linkable6 joined
07:56
evalable6 joined
08:08
Altai-man joined
|
|||
| Geth | roast: 44685318d2 | (Christian Bartolomäus)++ | S10-packages/precompilation.t [JVM] Skip test that started to fail recently |
08:35 | |
| rakudo: usev6++ created pull request #4110: [JVM] Make optional coercive parameter work again |
08:43 | ||
|
08:53
stoned75 joined
09:09
sena_kun joined
09:10
Altai-man left
|
|||
| Geth | rakudo: 1c43c46efb | (Elizabeth Mattijsen)++ | 7 files Remove return Sig from .elems methods We can be pretty sure that these will either return an Int or a Failure. So there's no real need to check here. This appears to have a positive effect, typically on multi-dimensional shaped array accesses, hence the reason for doing this (typically from 15% of total to 15% of total effort). |
13:05 | |
|
13:08
Altai-man joined
13:10
sena_kun left
|
|||
| Geth | nqp: f7be5c5932 | (Nicholas Clark)++ | t/nqp/019-file-ops.t floating point atime should be compared with integer atime, not mtime. Similarly ctime with ctime, not mtime. This appears to be a copy-paste thinko that dates back to when the test was originally added in 2016 by commit 37862cd52d63eb61: Add subsecond file time ops stat_time and lstat_time |
13:36 | |
| nqp: a083615656 | (Jonathan Worthington)++ (committed using GitHub Web editor) | t/nqp/019-file-ops.t Merge pull request #688 from Raku/atime-with-atime floating point atime should be compared with integer atime, not mtime. |
|||
|
13:51
MasterDuke left,
lucasb joined
14:20
MasterDuke joined
14:40
sxmx left
14:44
sxmx joined
|
|||
| Geth | roast: 9c443ac27d | (Christian Bartolomäus)++ | S17-supply/syntax.t [JVM] Unfudge tests (no UnwindException anymore) It's not clear what made the UnwindException go away, but the tests seem to run fine now. |
16:13 | |
|
16:49
stoned75 left
17:09
sena_kun joined
17:10
Altai-man left
17:35
leont joined
|
|||
| Geth | rakudo: nwc10++ created pull request #4111: Use `exec` instead of `system` and then exiting with the return code. |
18:30 | |
| nwc10 | that totally doesn't fit the PR template :-) | ||
| or :-( | |||
| Geth | rakudo: 856dfb2d2e | (Christian Bartolomäus)++ | src/vm/jvm/runtime/org/raku/rakudo/Binder.java [JVM] Make optional coercive parameter work again Fixes github.com/rakudo/rakudo/issues/4092 for the JVM backend. Inspired by github.com/rakudo/rakudo/commit/44cc88b54c. |
18:47 | |
| rakudo: 5e791d7995 | (Christian Bartolomäus)++ (committed using GitHub Web editor) | src/vm/jvm/runtime/org/raku/rakudo/Binder.java Merge pull request #4110 from usev6/jvm_coercive_optional [JVM] Make optional coercive parameter work again |
|||
| nqp: b12517b062 | (Christian Bartolomäus)++ | src/vm/jvm/QAST/Compiler.nqp [JVM] Make gather work with last/next/redo in loop This helps with a long standing bug on the JVM backend: github.com/rakudo/rakudo/issues/3634 If last/next/redo were called within an extra block (marked as 'immediate' or 'immediate_static'), that call was translated to ... (8 more lines) |
18:56 | ||
|
19:45
Xliff joined
|
|||
| Xliff | Oh my.... | 19:45 | |
| IRC looks brilliant on my new 86" TV | 19:46 | ||
| An actual on-topic question: Has anyone run into a "too many open files" error when precompiling nested modules? | 19:47 | ||
| In my larger work, I am trying to run a script, go 7 extra levels down (modules requiring other modules...) and can trigger this error. | 19:48 | ||
| If not, will try to golf and reproduce. | |||
| Bumping ulimit fixed it. Is raku leaking file handles? | 19:50 | ||
| MasterDuke | that's not a tv, that's a small wall | 20:28 | |
| lizmat | Xliff: possibly | 20:33 | |
| Xliff | MasterDuke: :) | ||
| lizmat: I've been looking at my system. I think it MIGHT not be rakudo | 20:34 | ||
| My browsers are memory and file hungry beasts. | |||
| If I see the problem again, I will golf and bug. | 20:35 | ||
| MasterDuke | Xliff: maybe check lsof close to the end of your precompilation? see if raku/moar have a lot of files open | 20:38 | |
| Xliff | That's pretty much what I did. Except I counted files without raku running. | 20:43 | |
| Previous ulimit was 1024 files | |||
| Now it's 2048. | |||
| Browsers had close to 1000 files open! | 20:44 | ||
| MasterDuke | ha | ||
| Xliff | Exactly! | 20:45 | |
|
20:50
stoned75 joined
|
|||
| stoned75 | Hi | 20:52 | |
| I'm looking github.com/rakudo/rakudo/blob/mast...re.pm6#L21 | |||
| and I'm wondering if the final comma is a typo or if it serves a specific purpose ? | |||
| MasterDuke | looks like a typo | 20:57 | |
| Xliff | Yeah. A typo that stumbles into the right behavior because the value is sunk? | 21:00 | |
|
21:08
Altai-man joined
|
|||
| Xliff | Noticing a definite slide in rakudo compile times over the last 4 weeks | 21:08 | |
|
21:11
sena_kun left
|
|||
| lizmat | stoned75: yup, a harmless typo | 21:11 | |
| Geth | rakudo: 6843f80330 | (Elizabeth Mattijsen)++ | src/core.c/Signature.pm6 Fix harmless typo Spotted by stoned75++ |
21:12 | |
| Xliff | docs.google.com/spreadsheets/d/1M3...sp=sharing | 21:13 | |
| Note the 2 timings tabs | |||
| lizmat | Xliff: sadly not visible for the Google login impaired | ||
| stoned75 | lizmat: yeah, I had to recompile rakudo a few times to convince myself it was harmless, ... as I've been mislead by Signature.new's caveat wrt to its count parameter's type :-} | 21:14 | |
| Xliff | lizmat: I'll see what I can do. | ||
| lismat: Does this work? docs.google.com/spreadsheets/d/1M3...sp=sharing | 21:15 | ||
| lizmat ^^ | 21:19 | ||
| lizmat | yup | 21:20 | |
| Xliff: O and P ? | 21:21 | ||
| Xliff | lizmat: where? | 21:22 | |
| lizmat | looking at docs.google.com/spreadsheets/d/1M3...2044890947 I'm not sure what to look for ? | ||
| ah, wrong tab | 21:23 | ||
| Xliff | Yah. tabs ending in "timings" | ||
| Even with GIO and ATK removed from 11/16 numbers that still would be the fastest timing in 4 weeks | 21:25 | ||
| Geth | rakudo: e1f09cfaf8 | (Elizabeth Mattijsen)++ | src/core.c/Parameter.pm6 Fix copy-pasto, also spotted by stoned75++ |
21:30 | |
| lizmat | Xliff: you *are* saying that compilation is taking *longer*, right ? | 21:31 | |
| stoned75 | lizmat: I wonder if I should ask you to describe Parameter.usage-name in one sentence or if I go on trying to double check my understanding of it and spot other typos and pastos ;-) | 21:37 | |
| lizmat | m: sub a(:a(:$b)) { }; dd &a.signature.params.head.usage-name | 21:41 | |
| camelia | "b" | ||
| lizmat | so I guess it's the name of the parameter inside the scope? Without its sigil | 21:43 | |
| Xliff | lizmat: Yes. | ||
| m: sub a(:$a) { }; dd &a.signature.params.head.usage-name | 21:44 | ||
| camelia | "a" | ||
| Xliff | m: sub a($b, :$a, :xliff(:$c)) { }; dd &a.signature.params.map( *.usage-name) | ||
| camelia | ("b", "a", "c").Seq | ||
| lizmat | stoned75: looks like it really is an implementation detail | 21:45 | |
| Xliff | lizmat: Ah! Cool. | ||
| lizmat | Xliff: not sure where that was an answer to | 21:46 | |
| Xliff | <lizmat> Xliff: you *are* saying that compilation is taking *longer*, right ? | ||
| lizmat | well... it taking longer I wouldn't describe as "Cool" :-( | 21:47 | |
| Xliff | lizmat: Oh. No. The "Cool" was for .usage-name | 21:48 | |
| lizmat | ah, right, ok :-) | ||
| stoned75 | lizmat: ok. but it's only used once in rakudo's sources, and btw it was added by you ;-) | ||
| lizmat | could well be | 21:49 | |
| hmmm... my git blame says Ben Davies ? | 21:50 | ||
| stoned75 | github.com/rakudo/rakudo/commit/67...f90616d8c0 | ||
| lizmat | ah, ok, :-) almost 5 years ago :) | 21:51 | |
|
21:51
patrickb joined
|
|||
| lizmat | ok, in that light, for people wanting to generate their own USAGE message, it's probably wise to document this as user-facing | 21:53 | |
| stoned75 | that's what I'm doing. but I wanted to understand its implementation and well... I got carried away :-} | 21:54 | |
| as now I'm wondering how a parameter could have a * or ! or . twigil | 21:58 | ||
|
22:02
maggotbrain left
|
|||
| lizmat | m: sub a(*@a) { dd @a }; a 1,2,3, <a b c> | 22:07 | |
| camelia | Array element = [1, 2, 3, "a", "b", "c"] | ||
| lizmat | m: sub a($*FOO) { dd $*FOO }; a 42 | ||
| camelia | 42 | ||
| stoned75 | m: say Parameter.new(name => '$^a').usage-name | 22:17 | |
| camelia | ^a | ||
| stoned75 | hum so now this seems wrong | ||
| lizmat | yup that's wrong | ||
| m: otoh, I don't think you can actually create such a parameter in plain Raku code | 22:18 | ||
| camelia | 5===SORRY!5=== Error while compiling <tmp> Two terms in a row at <tmp>:1 ------> 3ly create such a parameter in plain Raku7⏏5 code expecting any of: infix infix stopper postfix statement en… |
||
| lizmat | m: my $b = { $^a }; dd $b.signature.params.head.usage-name | ||
| camelia | "a" | ||
| lizmat | { $^a } is syntactic sugar for -> $a { $a } | 22:20 | |
| stoned75 | sure | 22:21 | |
| Xliff | lizmat: Did you ever get a chance to comment on: gist.github.com/Xliff/f4af67ed380d...975e4ed759 | 22:22 | |
| lizmat | I think I said a PR would be nice? "Any thoughts? If you'd rather this be a PR, I'll have to work more on it." | 22:23 | |
| Xliff | Ah. OK. | 22:24 | |
| I need to finish writing the tests for it, anyway. | |||
| lizmat | ++Xliff | 22:25 | |
|
22:41
Altai-man left
22:42
patrickb left
|
|||
| releasable6 | Next release in ≈5 days and ≈19 hours. 2 blockers. Please log your changes in the ChangeLog: github.com/rakudo/rakudo/wiki/ChangeLog-Draft | 23:00 | |