| 30 Jul 2022 |
@janne.hess:helsinki-systems.de | Mic92: do you have more insight on that? | 18:55:55 |
@elvishjerricco:matrix.org | In reply to @k900:0upti.me RIIR patchelf when If we did that we wouldn't need goblin :P | 18:56:46 |
K900 | I mean if I were to attempt it, goblin is what I'd build off of | 18:56:58 |
@elvishjerricco:matrix.org | Anyway, the point stands that we don't need to add deps if we're willing to use patchelf instead of ldd to detect dynamic libs | 18:58:06 |
@elvishjerricco:matrix.org | but that comes at the cost of it being... maybe a lil wonky. And we'd lose the justification for using json for the input | 18:58:26 |
K900 | Actually kinda hot take | 18:58:44 |
K900 | We might just want to extract make-initrd-ng into its own repo | 18:59:02 |
K900 | So we could have dependabot and CI and all the other good stuff | 18:59:12 |
K900 | Maybe not right now but in a few months when it's more mature | 18:59:29 |
@elvishjerricco:matrix.org | That's not crazy. I mean patchelf is another repo, right? | 18:59:31 |
@linus:schreibt.jetzt | yeah that take is lukewarm at best :p | 18:59:46 |
@elvishjerricco:matrix.org | Shoot I don't even see a reason not to do it immediately | 19:00:21 |
K900 | I actually don't think doing it now is a great idea because there will probably be one-liner fixes and having to do the whole release dance is awkard | 19:00:28 |
@linus:schreibt.jetzt | idk, having it in nixpkgs has its advantages too | 19:00:31 |
K900 | * I actually don't think doing it now is a great idea because there will probably be one-liner fixes and having to do the whole release dance is awkawrd | 19:00:31 |
K900 | * I actually don't think doing it now is a great idea because there will probably be one-liner fixes and having to do the whole release dance is awkward | 19:00:34 |
@linus:schreibt.jetzt | yeah that | 19:00:36 |
@elvishjerricco:matrix.org | ah fair enough | 19:00:42 |
@linus:schreibt.jetzt | and also running tests is really really easy currently | 19:00:59 |
@linus:schreibt.jetzt | change the code, nix-build nixos/tests/... | 19:01:08 |
K900 | That can be fixed if we expose make-initrd-ng as a top level package in nixpkgs | 19:01:46 |
K900 | Then we can overlay that from the make-initrd-ng repo | 19:01:55 |
K900 | And just run the same tests | 19:02:00 |
@elvishjerricco:matrix.org | yea. And it already is a top level package | 19:02:13 |
@elvishjerricco:matrix.org | Do we have any NixOS tests for building cross things? Or is cross considered not officially supported? | 19:08:27 |
@elvishjerricco:matrix.org | On an unrelated note, I think networkctl doesn't work if you don't have dbus, which makes messing with networking in initrd kind of annoying... Probably not worth including dbus over, but noteworthy | 19:25:40 |
K900 | There's something built on Hydra for mobile | 19:28:17 |
K900 | But I'm not sure how much or if there's any actual VM tests there | 19:28:30 |
| 31 Jul 2022 |
Mic92 | In reply to @janne.hess:helsinki-systems.de Mic92: do you have more insight on that? K900: It's reading the whole file into memory. | 05:41:17 |
Mic92 | See this: https://github.com/NixOS/patchelf/blob/master/src/patchelf.cc#L230 | 05:41:34 |