| 1 Oct 2025 |
 lovesegfault | o/ | 22:40:39 |
| lovesegfault | I think https://github.com/NixOS/nix/pull/14135 is good to merge pending Sergei Zimmerman (xokdvium)'s approval. The last thing I wanted to look at was ReleaseStaticDefaultClientBootstrap, but that's private and already called by the ApiHandle destructor | 22:43:07 |
 jade_ | oh yeah i found this bug in lix the other day, i don't have any idea why it's like that, it sucks! | 22:57:36 |
 John Ericson | oh sheesh that sounds like some code dedup that I wanted to do that we never quite got to | 22:58:15 |
| John Ericson | shouldn't be too hard though | 22:58:24 |
 Sergei Zimmerman (xokdvium) | lassulus: Maybe this is the issue I was thinking about https://github.com/NixOS/nix/issues/8059 | 23:02:25 |
| 2 Oct 2025 |
 fzakaria | Maybe you are thinking of https://github.com/NixOS/nix/issues/13727 Sergei Zimmerman (xokdvium) | 00:40:36 |
| fzakaria | nix edit vs :edit; Looks like the repl might allow file paths vs. "installables" in a few more cases | 00:40:58 |
| lovesegfault | okay, I have a fix for https://github.com/NixOS/nix/issues/4313 | 03:43:47 |
| lovesegfault | we'll need to bikeshed on naming a little bit but I think it's good | 03:44:00 |
| lovesegfault | just need to get the tests looking a little nicer | 03:44:16 |
| lovesegfault | https://github.com/NixOS/nix/pull/14138 | 03:52:59 |
| lovesegfault | the bikeshedding needed is on that isArchiveBased() name, idk what else to call it or if that's good | 03:54:37 |
 getchoo | opened https://github.com/NixOS/experimental-nix-installer/pull/49 so we can use builds with the previous installer fix in nix's main ci | 05:13:58 |
| getchoo | it does this by using action artifacts instead of the cache, and in the process of doing that, i ended up de-duplicating the small mess of workflow files | 05:17:21 |
| Sergei Zimmerman (xokdvium) | In reply to @lovesegfault:matrix.org
the bikeshedding needed is on that isArchiveBased() name, idk what else to call it or if that's good substitutatableFromNarHash? | 06:03:46 |
 Mic92 | getchoo: where you able to run https://github.com/NixOS/experimental-nix-installer/blob/main/assemble_installer.py btw? | 10:59:48 |
| lovesegfault | Nice | 18:56:51 |
| lovesegfault | why do we have this rev and lastModified stuff in fetchTree? | 19:15:53 |
| lovesegfault | it feels to me (naively) really weird that narHash is not sufficient to say "I already have this" reproducibly | 19:16:36 |
| Sergei Zimmerman (xokdvium) | I lack the full context why it's there in the first place. As Eelco mentions here https://github.com/NixOS/nix/pull/14138#pullrequestreview-3293258649 we can probably treat the fetchTarball as final at least. | 19:29:05 |
| lovesegfault | yeah, I'm working on that right now | 19:29:26 |
| lovesegfault | it's also weird to me that I can't explicitly set lastModified for a fetchTree { type = "tarball"; } and make it final | 19:30:00 |
| Sergei Zimmerman (xokdvium) | lovesegfault, Mic92 https://github.com/NixOS/nix/pull/14142. I think that was just an small oversight, right? | 20:14:42 |
| lovesegfault | btw Sergei Zimmerman (xokdvium) are we clear to merge the aws-crt-cpp changes? I've rebased the other work on top of it and made sure it all still works | 20:16:09 |
| lovesegfault | whoops, yes! | 20:16:54 |
| lovesegfault | John Ericson: you're currently the assigned reviewer | 20:33:24 |
| John Ericson | @lovesegfault:matrix.org: I am on a walk right now, but I should do it when I get back, and if you don't mind, make sure I don't forget, I got a lot on my mind right now :) | 21:43:54 |
| 3 Oct 2025 |
|  sinan changed their profile picture. | 02:43:04 |
| lovesegfault | next little bit of the curl-based-s3 work: https://github.com/NixOS/nix/pull/14144 | 03:47:54 |
