00:56
frost-lab joined
02:12
kvw_5_ joined
02:15
kvw_5 left
03:48
Kaiepi left,
Kaiepi joined
04:46
Kaiepi left,
Kaiepi joined
05:09
maggotbrain joined
05:50
maggotbrain left
05:51
maggotbrain joined
05:55
Kaiepi left
05:56
Kaiepi joined
06:35
Kaiepi left,
Kaiepi joined
07:12
Kaiepi left,
Kaiepi joined
07:41
squashable6 left
07:44
squashable6 joined
08:20
kawaii_ joined
08:23
kawaii left,
kawaii_ is now known as kawaii
08:44
domidumont joined
|
|||||||||||||||||||||||||||||||||||||||
nine | github.com/rakudo/rakudo/pull/4256 "Make signal handler setup synchronous to avoid race conditions" On MoarVM setup of a signal handler is itself an asynchronous operation. Wait for it to complete before returning as not only our spec tests but probably every single user expects the signal handler to be in place at that point. This was already true for the JVM and JS backends. | 09:55 | |||||||||||||||||||||||||||||||||||||
Fixes a race condition in t/spec/S17-procasync/kill.t | |||||||||||||||||||||||||||||||||||||||
nwc10 | I wonder if it's the same race condition as I just hit in t/spec/S07-hyperrace/basics.rakudo.moar | 09:56 | |||||||||||||||||||||||||||||||||||||
(lots of threads wating on a futex, and one sleeping in code called from the JIT) | |||||||||||||||||||||||||||||||||||||||
nine | github.com/Raku/roast/commit/da922...f413b07d0f "Fix yet another race condition in kill.t" Most of the time the 'Output correct for Supply.merge on signals' test was not executed at all because the test program finished normally before it ever got the signal. Even the test plan did not expect that test to run. Fix by having the test program basically wait forever for that | ||||||||||||||||||||||||||||||||||||||
signal to arrive. | |||||||||||||||||||||||||||||||||||||||
nwc10 | not investigating this yet, as I have other things I can make progress on | ||||||||||||||||||||||||||||||||||||||
sena_kun | nine, this won't go into the release. | 10:03 | |||||||||||||||||||||||||||||||||||||
nine | sena_kun: I figured as much :) | 10:04 | |||||||||||||||||||||||||||||||||||||
But at least for the release after that kill.t won't be an issue anymore | |||||||||||||||||||||||||||||||||||||||
sena_kun | nine++ for caring | ||||||||||||||||||||||||||||||||||||||
10:36
frost-lab left
11:45
Kaiepi left
11:47
Kaiepi joined
11:59
Kaiepi left
12:00
Kaiepi joined
12:02
Kaeipi joined
12:03
Kaiepi left
12:04
Kaeipi left
12:05
Kaeipi joined
|
|||||||||||||||||||||||||||||||||||||||
nine | github.com/Raku/roast/commit/93676...8f0bc88c27 "Fix race condition in bug-coverage-stress.t signal test" The signal handler sometimes did not even get a chance to run as the test program finished right after signalling that it started. To make it reliable the only proper way to exit the test program must be from the signal handler.So give that handler a really long time to | 12:45 | |||||||||||||||||||||||||||||||||||||
fire and exit unsuccessfully if it doesn't. | |||||||||||||||||||||||||||||||||||||||
Sigh...there's yet another race condition in bug-coverage-stress.t. It contains a test that does almost the same as the one in kill.t but uses react/whenever instead of act on the signal supply. And with react/whenever we do not seem to always catch the signal . | 13:09 | ||||||||||||||||||||||||||||||||||||||
lizmat | that may point to a more general issue with react / whenever? | 13:14 | |||||||||||||||||||||||||||||||||||||
nine | no idea | 13:15 | |||||||||||||||||||||||||||||||||||||
I don't understand how that could make a difference | |||||||||||||||||||||||||||||||||||||||
13:18
b2gills left
13:44
b2gills joined
14:20
Kaeipi left
14:51
cog__ joined
14:54
cog_ left
15:11
Kaiepi joined
|
|||||||||||||||||||||||||||||||||||||||
sena_kun | weekly twitter.com/koto_san_kana/status/1...8622331910 | 16:03 | |||||||||||||||||||||||||||||||||||||
I bet this is not how it works. | |||||||||||||||||||||||||||||||||||||||
:S | |||||||||||||||||||||||||||||||||||||||
16:18
epony left
16:22
epony joined
|
|||||||||||||||||||||||||||||||||||||||
[Tux] |
|
16:50 | |||||||||||||||||||||||||||||||||||||
17:30
evalable6 left,
linkable6 left
17:32
linkable6 joined,
evalable6 joined
18:21
sena_kun left
18:26
sena_kun joined
18:35
domidumont left
18:38
domidumont joined
18:40
domidumont left
19:16
epony left
19:17
epony joined
19:22
Xliff joined
|
|||||||||||||||||||||||||||||||||||||||
lizmat | weekly: twitter.com/koto_san_kana/status/1...8622331910 | 19:52 | |||||||||||||||||||||||||||||||||||||
notable6 | lizmat, Noted! (weekly) | ||||||||||||||||||||||||||||||||||||||
lizmat | sena_kun: weekly: foo not weekly foo | ||||||||||||||||||||||||||||||||||||||
sena_kun | lizmat, next release is next month, so I'll forget till then. :] | 19:53 | |||||||||||||||||||||||||||||||||||||
lizmat, but thanks, and thank you very much for all your efforts. :) | |||||||||||||||||||||||||||||||||||||||
lizmat | you're welcome! | ||||||||||||||||||||||||||||||||||||||
btw, spectest was clean for me :-) | 19:54 | ||||||||||||||||||||||||||||||||||||||
19:57
HarmtH joined
20:03
HarmtH left,
HarmtH joined
20:16
HarmtH left
20:17
HarmtH joined
20:18
HarmtH left
20:19
HarmtH joined
20:22
HarmtH left
20:23
HarmtH joined
21:17
sena_kun left,
sena_kun joined
21:19
sena_kun left,
sena_kun joined
21:23
sena_kun left
21:25
sena_kun joined
21:28
softmoth joined
21:53
softmoth_ joined,
softmoth_ left
22:40
HarmtH left
22:41
HarmtH joined
22:42
HarmtH left,
HarmtH joined
22:43
HarmtH left,
HarmtH_ joined
22:46
HarmtH_ left,
HarmtH joined
22:51
HarmtH left,
HarmtH_ joined
23:37
[Coke]_ is now known as [Coke]
|