| 1 Aug 2022 |
 Winter (she/her) | In reply to @elvishjerricco:matrix.org
Is there a way to get the rust dependencies to build in a separate derivation from make-initrd-ng? It's really annoying that what used to be like a 1s build now takes closer to 10s when trying to iterate on it * Yes, but our infrastructure doesn't support it. | 16:50:14 |
| Winter (she/her) | (External projects do, though.) | 16:50:54 |
 @elvishjerricco:matrix.org | It'd be really nice. haskellPackages has spoiled me :P | 16:52:36 |
| Winter (she/her) | K900: Why would the Goblin changes potentially fix cross? Like, were we not able to pass in the proper patchelf binary paths? | 16:53:04 |
| @elvishjerricco:matrix.org | Winter (she/her): It looks like patchelf would have done the trick | 16:53:29 |
| @elvishjerricco:matrix.org | But there's a lot to like about using goblin for other reasons | 16:53:42 |
| Winter (she/her) | Yeah, definitely. | 16:53:49 |
 K900 | patchelf does seem to work, but ldd doesn't, and neither does strip | 16:53:53 |
| Winter (she/her) | But was just curious about that aspect. | 16:53:53 |
| K900 | Linus fixed it to use the right strip | 16:54:06 |
| K900 | And I honestly wasn't 100% sure patchelf would have worked | 16:54:17 |
| K900 | And also I knew goblin was great | 16:54:23 |
| Winter (she/her) | Yeah Goblin is definitely better. | 16:54:35 |
| Winter (she/her) | Was ZFS support added since the initial PR, btw? | 16:55:11 |
| @elvishjerricco:matrix.org | Yes | 16:55:16 |
| Winter (she/her) | I didn't see any commits but I may be looking in the wrong place. | 16:55:21 |
| Winter (she/her) | Oh, cool. | 16:55:26 |
| @elvishjerricco:matrix.org | AFAIK zfs support is now on par with scripted initrd | 16:55:28 |
| Winter (she/her) | What commit added that, what should I be looking for, out of curiosity? | 16:55:46 |
| @elvishjerricco:matrix.org | https://github.com/NixOS/nixpkgs/pull/168269 | 16:56:10 |
| @elvishjerricco:matrix.org | Redacted or Malformed Event | 17:25:09 |
| K900 | Applied all the suggestions on the goblin PR | 19:57:56 |
| @elvishjerricco:matrix.org | Cool. I'd still like someone's answer to this comment, but I'm inclined to merge if we get no such answer in the next day or two | 20:08:27 |
| @elvishjerricco:matrix.org | Oh, and apparently we're getting an eval error because there's no lib.maintainers.dasJ | 20:11:10 |
| @elvishjerricco:matrix.org | Ah, lib.maintainers.das_j | 20:11:40 |
| @elvishjerricco:matrix.org | Also, Janne Heß did you ever consent to being added to that maintainer list? I gave a thumbs up for myself; I apologize if I accidentally indicated you were cool with it too | 20:12:35 |
| Winter (she/her) | So I'm considering giving systemd initrd a whirl on my machines. The one requirement I have is that I'm able to wipe my ZFS root dataset on startup. Would this be accomplished by defining a unit that requires and runs after zfs-import-<pool name>? I think that's... all I really have to do? | 21:49:27 |
| Winter (she/her) | (I do this with postDeviceCommands now.) | 21:49:43 |
| @elvishjerricco:matrix.org | That sounds right to me. | 21:50:13 |
| Winter (she/her) | Also, semi off-topic but I think someone here would know: why does the ZFS module explicitly use ${cfgZfs.package}/sbin/zfs in its scripts, when this is already added to initrd's packages and the system packages, so it really should just be able to call zfs, unless there's a reason this is explicitly done that I'm missing? | 21:53:02 |
