| 18 Mar 2025 |
 John Ericson | oh it was never called from a destructure | 18:43:58 |
| John Ericson | Las: not sure if this is necessary, but if you split the coroutine part at the end of amDone | 18:45:29 |
| John Ericson | make two functions on Goal and amDone calls both | 18:45:37 |
| John Ericson | then this change can be 100% mechanical | 18:45:47 |
 Las | In reply to @Ericson2314:matrix.org
Las: not sure if this is necessary, but if you split the coroutine part at the end of amDone You mean the bug workaround? | 18:46:09 |
| John Ericson | well the large point is I think we should live review both PRs commit by commit, take commits from each and land them | 18:46:53 |
| John Ericson | so interview as necessary | 18:47:00 |
| John Ericson | I mean split amDone where cleanup used to be called | 18:47:24 |
| John Ericson | amDonePart1 (returns void), amDonePart2 (co_return Done {}) | 18:48:10 |
| Las | But why | 18:48:20 |
| John Ericson | it might not be necessary, but see it could be important to call worker.removeGoal(shared_from_this()) before putting the goal in an invalid state | 18:49:08 |
| John Ericson | it is unclear to me whether it is possible for two threads to be messing with a goal at once | 18:49:24 |
| John Ericson | but if the goal is removed from worker, and from waitees/waiters, then I don't think another thread can find the goal very easily | 18:49:50 |
| John Ericson | If Robert is busy today, well we at least have the Nix meeting tomorrow | 18:50:11 |
| Las | You can’t move around the worker interactions freely because you might mutate the data structures that are being iterated over at the same time | 18:50:38 |
| Las | Maybe not an issue there though | 18:50:48 |
| Las | Goals are single threaded | 18:51:12 |
| Las | Maybe in 2035 Mobile Element will support threads so I can respond it a thread | 18:52:03 |
| Las | * | 18:52:12 |
 emily | (Element X does) | 18:54:10 |
 K900 | Normal Element does too | 18:54:27 |
| K900 | Though we generally try to not use threads in public rooms because there's still clients that don't display them properly or at all | 18:54:45 |
| Las | I can’t make one on iOS I think? | 19:03:25 |
| John Ericson | I never saw any mutex or something that made it obviously single threaded, but yes I believe that is the intent | 19:08:15 |
| Las | Sealed state was a mistake | 19:14:37 |
| Las | * | 21:56:23 |
| Las | * | 21:56:29 |
| 19 Mar 2025 |
|  Sergei Zimmerman (xokdvium) changed their display name from xokdvium to Sergei Zimmerman (xokdvium). | 21:12:20 |
| John Ericson | +(shell.sh:21) nix shell -f shell-hello.nix hello-symlink -c hello
+(shell.sh:21) grep 'Hello World'
+(/home/jcericson/src/nix/master/tests/functional/common/functions.sh:341) checkGrepArgs 'Hello World'
+(/home/jcericson/src/nix/master/tests/functional/common/functions.sh:290) local arg
+(/home/jcericson/src/nix/master/tests/functional/common/functions.sh:291) for arg in "$@"
+(/home/jcericson/src/nix/master/tests/functional/common/functions.sh:292) [[ Hello World != \H\e\l\l\o\ \W\o\r\l\d ]]
+(/home/jcericson/src/nix/master/tests/functional/common/functions.sh:342) command grep 'Hello World'
this derivation will be built:
/tmp/nix-shell.FPcQP5/nix-test/main/shell/store/bbqd8bbcr4i32nahm7ygqhwpwj3xfidm-hello-symlink.drv
building '/tmp/nix-shell.FPcQP5/nix-test/main/shell/store/bbqd8bbcr4i32nahm7ygqhwpwj3xfidm-hello-symlink.drv'...
error: 'tmp' is too short to be a valid store path
| 21:41:30 |
| 20 Mar 2025 |
 Robert Hensing (roberth) | Martin Schwaighofer: https://github.com/NixOS/nix/issues/12361 | 13:40:36 |
