| 12 Aug 2025 |
Sergei Zimmerman (xokdvium) | That'd be great | 20:08:48 |
Sergei Zimmerman (xokdvium) | The fact that old tests didn't cover the most fucked semantics is infuriating | 20:09:25 |
raitobezarius | Yeahh, I suppose I was too optimistic about Tom's small language being well behaving | 20:10:09 |
raitobezarius | All these fromXYZ file format deserializers are ticking bombs :( | 20:10:34 |
emily | yes, fromTOML by default rejects timestamps | 20:12:14 |
emily | and I cannot find any evidence of anyone relying on this feature | 20:12:24 |
emily | it was introduced for https://github.com/input-output-hk/foliage/issues/46 which didn't seem to ever happen | 20:12:31 |
emily | (TOML is definitely not well-behaved. the top-level value can only be one type…) | 20:12:48 |
Sergei Zimmerman (xokdvium) | https://toml.io/en/v1.0.0#local-time
Millisecond precision is required. Further precision of fractional seconds is implementation-specific.
FML
| 20:13:15 |
emily | I think that the semantics that fromTOML exposes stably should be relatively safe; it's just that experimentally extending it to timestamps turns out to be more complicated than expected | 20:13:17 |
emily | that fromTOML cannot handle all of TOML… is suboptimal, but oh well | 20:13:32 |
emily | easier to expand than shrink :) | 20:13:47 |
emily | I do think that the patch should be applied to stable releases, because breaking eval semantics even for experimental features in stable releases seems bad, and the Nixpkgs bump is blocked right now | 20:14:27 |
emily | but +2 for dropping for next release | 20:14:40 |
raitobezarius | Somehow obvious amirite | 20:15:06 |
raitobezarius | Alright | 20:15:39 |
raitobezarius | Taking away exp features from stable releases is a bit meh tho | 20:15:57 |
emily | Sergei Zimmerman (xokdvium)'s patch doesn't take anything away | 20:16:16 |
emily | (it computes the previous behaviour back on top of the new toml11) | 20:16:27 |
emily | it's just an unfortunate amount of complexity for a broken feature that's clearly an eval stability liability, so I'd prefer to see a drop on HEAD | 20:17:01 |
emily | doing it only for stable releases bounds the lifetime of the complexity | 20:17:21 |
raitobezarius | Sorry, you're absolutely right | 20:17:58 |
raitobezarius | That sounds like a good plan, patch for releases and drop for HEAD | 20:18:13 |
raitobezarius | Thank you xokdvium for this! | 20:18:46 |
Sergei Zimmerman (xokdvium) | Sure. I'll bring this issue up at the nix team meeting as well. I'd be in favor of dropping this feature on master. Digging through toml11 def wasn't a great experience. | 20:20:09 |
raitobezarius | (expectations for the work above: I can probably review some CLs around that but I'm on holidays for the next 5 days, so I won't take care of it immediately) | 21:13:26 |
| 13 Aug 2025 |
niko ⚡️ | oh no I was wondering why nix build on lix main wouldn't reproduce them and turns out the other failing install checks only error on new nixpkgs unstable that's not a good sign | 09:53:56 |
| Picnoir changed their display name from Picnoir DECT 7426 to Picnoir. | 13:24:51 |
emily | has anyone run into this on macOS error: failed to extract archive (Cannot extract through symlink /tmp/nix-50z48nbfhpxw5h6rvfcpq4ibip/nix-darwin-e04a388232d9a6ba56967ce5b53a8a6f713cdfcf)
it might be related to me trying to recover my busted system right now but I usually have /nix/tmp as my temp-dir and I'm wondering if there is actually some broken behaviour around /tmp -> /private/tmp when using --store local with the default config
| 15:35:17 |
emily | --temp-dir /private/tmp fixes it | 15:35:35 |