| 18 May 2024 |
 puck | and honestly isn't a great name for what it does (it still requires a valid derivation, just it may not be registered with the store properly; which is, tbh, not something that should be able to happen) | 14:21:48 |
| puck | e.g. the easiest way to make this happen is builtins.toFile "hi.drv" "meow" | 14:22:42 |
| puck | it writes to /nix/store/233k2fcz02xfpi48ildccq1740gh5mcs-hi.drv but also errors out saying error: expected string 'D'; but claims the path is invalid simultaneously | 14:23:46 |
| *  K900 flips the counter | 14:28:07 |
 John Ericson | puck: yeah that reflects my memory | 14:30:45 |
| John Ericson | I moved the .drv validation code out of LocalStore so all implementations can use it | 14:31:07 |
| John Ericson | but it not make it be called more eagerly for things like that | 14:31:16 |
| John Ericson | so the file is removed if the validations fails | 14:32:15 |
| John Ericson | (could also assueme drvs are small and buffer in memory first) | 14:32:32 |
| John Ericson | you all will probably want to take my code unifying the other add to store with the add text to store, so there are fewer versions of this shit to play wack-a-mole with | 14:33:04 |
 mei 🌒& | hi it seems like lix repl doesn't detect cycles and print them correctly w :p ? | 19:58:59 |
| puck | it does here; are you thinking about cycles or infinitely recursive non-cycle structures | 20:00:10 |
| mei 🌒& | uh i'm not sure, just OOM'd after writing that msg | 20:01:46 |
| mei 🌒& | luckily i made it recover nicely so im back here already. :p mapAttrsRecursiveCond (as: true) id nodes.cookiemonster.options | 20:02:18 |
| puck | ah, that is in fact infinitely recursive non-cycle | 20:02:28 |
| mei 🌒& | eats all the memory and a few cpu cores and yeah | 20:02:31 |
| mei 🌒& | not good | 20:02:36 |
| mei 🌒& | In reply to @puck:puck.moe
ah, that is in fact infinitely recursive non-cycle bad ux,, | 20:02:45 |
| mei 🌒& | lost my big repl thing cuz i printed a recursive thing | 20:03:08 |
| puck | yeahhhhh we should have an issue for repl history imploding | 20:03:21 |
| mei 🌒& | and recurse limit 🥺 | 20:03:32 |
| mei 🌒& | maybe even configurable interactively-only in the repl | 20:03:47 |
| mei 🌒& | silly code should not hang the machine | 20:04:18 |
| mei 🌒& | ill make the issues | 20:05:01 |
| mei 🌒& | REPL history should be saved immediately #328 | 20:07:35 |
 raitobezarius (DECT: 7248) | if i remember, there was a crude recurse limit detector | 20:07:56 |
| raitobezarius (DECT: 7248) | but i think eelco wanted to remove it? | 20:08:01 |
| puck | i think we added one in, yeah | 20:08:37 |
| puck | but that doesn't help wider recursion issues, which i think this was | 20:08:47 |
| mei 🌒& | not making the other ticket since idk enough | 20:10:46 |
