|
01:18
nativecallable6 joined,
quotable6 joined,
committable6 joined,
bloatable6 joined,
bisectable6 joined,
greppable6 joined,
releasable6 joined,
benchable6 joined,
evalable6 joined,
coverable6 joined,
unicodable6 joined,
statisfiable6 joined,
squashable6 joined
01:20
lizmat joined
01:56
ilbot3 joined
02:16
evalable6 joined
04:02
evalable6 joined
06:02
domidumont joined
06:38
nativecallable6 joined,
greppable6 joined,
evalable6 joined,
committable6 joined,
quotable6 joined,
coverable6 joined,
unicodable6 joined,
bisectable6 joined,
bloatable6 joined,
releasable6 joined,
benchable6 joined,
squashable6 joined,
statisfiable6 joined
06:52
Ven joined
07:10
domidumont joined
07:27
Ven_ joined
07:57
domidumont joined
08:03
Ven joined,
rba joined
08:13
zakharyas joined
08:15
robertle joined
08:34
rba_ joined
09:00
rba joined
|
|||
| rba | timotimo: I have now build MoarVM on Solaris with --debug=3 and --optimize=0. Got todays MoarVM and nqp from git and still get same test errors: | 09:01 | |
| timotimo: t/nqp/059-nqpop.t and t/nqp/104-method-cache.t failed | 09:26 | ||
| not ok 92 - nqp::pow_n(1, nqp::nan) | |||
| not ok 93 - nqp::pow_n(1, nqp::inf) | |||
| not ok 94 - nqp::pow_n(1, nqp::neginf) | |||
| # ./nqp-m t/nqp/104-method-cache.t | 09:27 | ||
| 1..16 | |||
| ok 1 - calling a method found in cache | |||
| ok 2 - nqp::can on a method found in cache | |||
| ok 3 - nqp::can on a method not in cache | |||
| ok 4 - ...find_method is not called when the cache is authoritative | |||
| Segmentation Fault (core dumped) | |||
| bash-4.3# ./nqp-m -e 'say(nqp::pow_n(1, nqp::nan))' | 09:31 | ||
| NaN | |||
| bash-4.3# ./nqp-m -e 'say(nqp::pow_n(1, nqp::inf))' | |||
| NaN | |||
| bash-4.3# ./nqp-m -e 'say(nqp::pow_n(1, nqp::neginf))' | |||
| NaN | |||
| About the segmentation fault: | 09:44 | ||
| gist.github.com/b83f789c58dcfda29a...82e2f4edd0 | |||
| If someone could adivise me how to "cook" the gdb to get valuable information to dig to the root cause, let me know. dbg session still open... | 09:46 | ||
| gist.github.com/21673ab1df34016a06...8910b67203 | 09:49 | ||
|
10:01
Ven joined
10:23
Ven_ joined
11:01
Ven joined
11:16
rba joined
11:42
patrickz joined
|
|||
| timotimo | rba: if the debug session for the seg fault is still open, please "frame 5" and "print c_name" | 11:49 | |
| samcv | yhello guys | 11:57 | |
| yoleaux | 19 Oct 2017 20:53Z <AlexDaniel`> samcv: irclog.perlgeek.de/moarvm/2017-10-19#i_15327049 | ||
| samcv | so i volunteered for something :) ok | ||
| oh yes i did | |||
| AlexDaniel` | hehe :D | ||
| samcv | i've been needing something to do anyway | 11:58 | |
| releasable6, status | |||
| releasable6 | samcv, Next release in 1 day and ā7 hours. 1 blocker. Changelog for this release was not started yet | ||
| samcv, Details: gist.github.com/37d496766d04eaa162...138697e98a | |||
| samcv | ok cool | ||
| AlexDaniel`, there's a list right? of things to be done before release? | |||
| AlexDaniel` | samcv: github.com/MoarVM/MoarVM/blob/mast...e_guide.md | 11:59 | |
| samcv | ok cool | ||
| AlexDaniel` | samcv: arguably the most time consuming part is the changelog, so that's probably something you want to start with early | ||
| samcv | that's what I was thinking too | 12:00 | |
| AlexDaniel` | samcv: if you have any extra resources, there's one thing that can be improved in the process (and in that document) ā signing | 12:01 | |
| I've noticed that your commits are signed so it shouldn't be too hard for you | |||
| samcv: rakudo release guide talks about signing (e.g. step 16) github.com/rakudo/rakudo/blob/nom/..._guide.pod | 12:02 | ||
| but depending on how you have your git set up it may actually happen automatically | 12:03 | ||
| then you'll just need to sign the tarball (like in step 17), and that's basically it | |||
| samcv | i sign all my commits | 12:05 | |
| only things i don't sign is doing merges using github | 12:06 | ||
| AlexDaniel` | yeah, me too | ||
|
12:14
Ven joined
12:47
zakharyas joined
13:33
rba joined
|
|||
| rba | timotimo: output from "frame 5" and "print c_name": gist.github.com/38c6b0898d6dd2fd40...0fda2ab330 | 13:36 | |
| timotimo | ok, what's STABLE(obj)? and what's STABLE(obj)->debug_name? | 13:37 | |
| (again with print) | |||
| i wonder if debug_name is a null pointer and that simply crashes on freebsd whereas it just prints an empty string on other systems? | 13:38 | ||
| ah, yes, passing NULL there is "officially undefined" | 13:39 | ||
| i'll build something | 13:42 | ||
| rba | timotimo: gist.github.com/f9738f74c0358fc45b...2f6d22a26b | ||
|
13:42
Ven joined
|
|||
| timotimo | yup, that's it | 13:43 | |
| might be time to run moarvm with ubsan at one point | |||
| rba | timotimo: what can I do now? | 13:44 | |
| timotimo | wait for my patch | ||
| rba | timotimo: nice! | ||
|
13:50
arnsholt joined
|
|||
| timotimo | so, so, so many usages of debug_name everywhere | 13:55 | |
|
14:09
rba joined
|
|||
| Geth | MoarVM/debug_name_safe_null: 51158fa1b3 | (Timo Paulssen)++ | 21 files don't printf NULL via an unset debug_name it's undefined behaviour and actually segfaults under solaris. |
14:12 | |
| timotimo | rba: try this branch please | ||
| rba | timotimo: building, stand by | 14:15 | |
| jnthn | timotimo++ | 14:18 | |
| timotimo | i wonder if we should probe for "printf null is dangerous" during configure and make this function cheaper otherwise ... | 14:19 | |
| jnthn | Not really; we only use it in error reporting | ||
| timotimo | true | ||
| and spesh dump :P | 14:20 | ||
| jnthn | And the compiler knows its builtins, it can strip the check out anyway | ||
| Oh, maybe not | |||
| But still... | |||
| timotimo | it's not out of the question that gcc might be smart enough for this | ||
| MSVC, though ... nope :) | |||
|
14:21
domidumont joined
|
|||
| timotimo | it took me a minute or two to notice sed without /g at the end will only replace stuff once per file :) | 14:21 | |
| ilmari | once per line, no? | 14:22 | |
| timotimo | it seemed to be once per file | ||
| but it could have been the other thing | |||
|
14:23
Ven_ joined
|
|||
| rba | timotimo++ | 14:23 | |
| timotimo: Confirmed. Error not schowing up any more on Solaris build. | 14:24 | ||
| timotimo | cool. now to build rakudo | ||
| not sure how to handle the mismatch of NaN semantics | |||
| rba | Still have the pow_n thing... | ||
| timotimo | it's practically harmless | ||
| only for apps that actually do floating point arithmetic and throw nans around | 14:25 | ||
| there'll be that kind of program, of course | |||
| lizmat | .oO( hmmmm , nans ) |
||
| timotimo | but we can at least ignore that until rakudo successfully compiles | ||
| rba | Ok. will go ahaed... | ||
| timotimo | oh, compiler warnings about missing casts | 14:29 | |
| Geth | MoarVM/debug_name_safe_null: 4faa2ba88a | (Timo Paulssen)++ | 3 files add a few missing casts to MVMObject * |
14:31 | |
|
14:32
domidumont joined
|
|||
| timotimo | no compiler warnings any more | 14:32 | |
| jnthn | .tell samcv To do the final release step (uploading the tarball), commit the .tar.gz into github.com/MoarVM/moarvm.org/tree/...r/releases (I gave you permissions on it). The change will be pulled within a minute. | 14:34 | |
| yoleaux | jnthn: I'll pass your message to samcv. | ||
| jnthn | That also means changes to the website will appear automatically within a minute too :) | 14:35 | |
| timotimo | oh wow i get a crash immediately with this moarvm | 14:36 | |
| in nqp's configure.pl | |||
| oh, got it | 14:42 | ||
| Geth | MoarVM/debug_name_safe_null: cdd0fb45b6 | (Timo Paulssen)++ | src/core/interp.c setdebugtypename must use debug_name directly |
14:44 | |
| timotimo | rba: without this commit moarvm will most probably immediately crash for any invocation | 14:57 | |
| Geth | MoarVM: 1a9be0ad48 | (Stefan Seifert)++ | src/core/nativecall.c Revert "JIT compile native calls with at most 1 string argument" This reverts commit 4eadf94599cc021ec7a9e0e49e198f5861468dc1 which is not yet ready for prime time. Seems to cause RT #132328 |
15:05 | |
| synopsebot | RT#132328 [new]: rt.perl.org/Ticket/Display.html?id=132328 [SEGV][REGRESSION] DBIish tests are failing spectacularly (JIT compilation of native calls) | ||
| rba | timotimo: rakudo (latest git) whent fine, test has small errors: gist.github.com/42a73d311c589991c5...8d151b1e29 | 15:07 | |
| timotimo | oh? solaris doesn't have ssize_t? | 15:16 | |
| oh, interesting, that's not a standard type | |||
| jnthn: i ran spectest and stresstest and nqp test with debug_name_safe_null; i'm not sure how well the calls are covered, but i assume it's safe to merge now | 15:31 | ||
| jnthn | timotimo: Yeah, it's probably pretty safe | 15:32 | |
| And fixes a blow-up on solaris, so it's nice to have in | |||
| rba | timotimo: may I send you a patch for memmem.h? gist.github.com/29520a3c47d7b3d1d1...db415de9c6 | 15:33 | |
| timotimo | want to put a mention of solaris into the comment above, too? | 15:34 | |
| rba | timotimo: yes please | ||
| jnthn | .tell samcv I'll be around a bit tonight/tomorrow, if you've any questions or need help on the release stuff | 15:35 | |
| yoleaux | jnthn: I'll pass your message to samcv. | ||
| Geth | MoarVM/master: 4 commits pushed by (Timo Paulssen)++ | ||
| jnthn | timotimo: You also have access to moarvm.org, fwiw :) | ||
| timotimo | rba: i was hoping *you* would add a mention :D | ||
| rba | timotimo: will do, give me a sec | ||
| timotimo | jnthn: cool :) | ||
| with my new-found love for javascript, i'll put in some snowflakes if you view the site on christmas or something | 15:36 | ||
| (surely you can build something like that in pure css) | |||
| jnthn | Well, I was more thinking that you can put stuff in the releases/ directory, but sure :P :P | ||
| Alright, home time :) bbl | |||
| timotimo | o/ | 15:38 | |
| rba | timotimo: memmem.h patch gist.github.com/fc0372dedcc2b3e966...82528fb4a3 | 15:41 | |
| timotimo | want to tell me what the --author for the commit should be? | 15:43 | |
|
16:05
quotable6 joined
16:09
xi- joined
16:10
robertle joined
16:11
domidumont1 joined
16:41
domidumont joined
16:45
ZofBot joined
|
|||
| Geth | MoarVM: 1952356854 | (Timo Paulssen)++ | src/platform/memmem.h solaris doesn't come with memmem, so use freebsd one. Thanks rba++ |
16:48 | |
|
16:50
bisectable6 joined
|
|||
| stmuk | does moarvm work on solaris? | 16:54 | |
| timotimo | not sure, it does get through nqp's test suite at least, and build rakudo | 16:56 | |
| rba asked about how to run spec test but didn't report any results yet | |||
| AlexDaniel` | .tell samcv fwiw please take your time. Release happens when it's ready, there's no rush | 17:07 | |
| yoleaux | AlexDaniel`: I'll pass your message to samcv. | ||
|
17:11
zakharyas joined
19:17
zakharyas joined
19:26
buggable joined
19:46
xi- joined
|
|||
| bartolin | wrt failures with nqp::pow_n(1, nqp::nan) on Solaris: we have RT #124147 for similar failures on NetBSD. looks like OpenSolaris and older OpenBSD versions had that problem, too. (just for the records) | 20:07 | |
| synopsebot | RT#124147 [open]: rt.perl.org/Ticket/Display.html?id=124147 [MATH] pow edge cases with rakudo-moar on NetBSD | ||
| timotimo | bartolin: good catch :) | 20:18 | |
| samcv | AlexDaniel`, thanks! | 20:37 | |
| yoleaux | 14:34Z <jnthn> samcv: To do the final release step (uploading the tarball), commit the .tar.gz into github.com/MoarVM/moarvm.org/tree/...r/releases (I gave you permissions on it). The change will be pulled within a minute. | ||
| 15:35Z <jnthn> samcv: I'll be around a bit tonight/tomorrow, if you've any questions or need help on the release stuff | |||
| 17:07Z <AlexDaniel`> samcv: fwiw please take your time. Release happens when it's ready, there's no rush | |||
| samcv | gonna eat somethnig then will start doing work towards it | 20:39 | |
|
21:42
rba joined
22:35
evalable6 joined
23:40
evalable6 joined
|
|||