| 18 Mar 2025 |
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 |
| sinan changed their display name from sinan to sinan (we're so back). | 17:52:14 |
| sinan changed their profile picture. | 17:52:43 |
| sinan changed their profile picture. | 17:54:05 |
jade_ | FYI in re https://git.lix.systems/lix-project/lix/issues/750, it seems like nixos/nix fixed this by accident, but still has some amount of an underlying bug (or at least poor documentation of the treatment of the underlying bug): the info.path in this spot is the bad 9hagx value
https://github.com/NixOS/nix/blob/3032512425a09fc58f2d658442043894e0aab256/src/libfetchers/tarball.cc#L74-L83
| 19:00:58 |
jade_ | (and it does not seem like it can be trivially changed to recursive hash mode there, since that breaks a bunch of observable behaviour. i have no idea how this is supposed to behave, but it seems like a really quite large bug) | 19:01:44 |
jade_ | * (and it does not seem like it can be trivially changed to recursive hash mode there, since that breaks a bunch of observable behaviour. i have no idea how this is supposed to behave, but it seems like a really quite tricky bug) | 19:01:50 |