| 7 Jun 2024 |
jade_ | oh it's breaking in the daemon | 07:23:53 |
jade_ | great, time to delete some exception handling from that too | 07:24:01 |
jade_ | i want to do so many very mean things to the daemon | 07:28:53 |
jade_ | really really need to make daemon exceptions obvious they were thrown on the daemon | 07:30:48 |
jade_ | ... what does the daemon have its stderr attached to? can it even log to system log? gods. this is so annoying | 07:58:53 |
jade_ |  Download image.png | 08:06:22 |
jade_ | alright i concede, the nix daemon's exception handling wins today | 08:06:28 |
jade_ | this is a json decode error btw | 08:06:39 |
jade_ | error: (daemon error)[json.exception.type_error.302] type must be string, but is array | 08:06:56 |
puck | hrm | 08:30:26 |
puck | there's only like three json decodes in the daemon, right | 08:30:42 |
jade_ | https://git.lix.systems/jade/lix/src/branch/jade/json-bugs nix build .#dockerImage here's your repro | 08:36:17 |
puck | okay wow | 09:16:10 |
puck | that's. not the one i was expecting | 09:16:15 |
puck | turns out exportReferencesGraph being invalidly shaped is handled very badly | 09:29:43 |
delroth | and the bug in your nix code is that "deps = [ layer2Pkgs ];" should likely be "deps = layer2Pkgs;" (same with other layers)
| 09:32:40 |
puck | yup | 09:40:27 |
jade_ | yup | 21:48:49 |
jade_ | that's quite a bug | 21:48:57 |
jade_ | we should probably minimize it and file it on the issue tracker, and then someone can possibly do something about it at a later date | 21:49:30 |
puck | i started writing a fix and then got lighly distracted | 21:50:04 |
jade_ | happens | 21:50:28 |
puck | In reply to @jade_:matrix.org
error: (daemon error)[json.exception.type_error.302] type must be string, but is array issue is the patch needs validation both where ^ is generated, and inside derivationStrict | 21:51:39 |
puck | at least, i think it's worth "breaking langver" to disallow derivations that will instantly error if built to be crafted | 21:52:01 |
jade_ | i concur | 21:52:26 |
jade_ | these should not be made meme, etc etc | 21:52:38 |
puck | i mean, if you want to, you can just add __json as an attribute to a drv and do the same | 21:52:41 |
jade_ | is that how we store those? holy fucik | 21:52:58 |
jade_ | * is that how we store those? holy fuck | 21:53:02 |
jade_ | just shove some json string in there? ??? | 21:53:13 |