| 30 Jul 2022 |
K900 | Both are possible | 18:52:36 |
@janne.hess:helsinki-systems.de | In reply to @elvishjerricco:matrix.org It would surprise me if patchelf even had any cross-dependent behavior You can ask @Mic92 about that, he started maintaining that | 18:52:44 |
K900 | Oh actually wait | 18:53:16 |
K900 | It does have some endianness checks | 18:53:22 |
@elvishjerricco:matrix.org | $ nix build --no-link --json nixpkgs#patchelf
[{"drvPath":"/nix/store/kshhm1fpfdv5a1qkd7sf18plxh7ddlg4-patchelf-0.14.5.drv","outputs":{"out":"/nix/store/96q22iw1i2m960x9y3v6fsj55691pzdl-patchelf-0.14.5"}}]
$ nix build --no-link --json nixpkgs#pkgsCross.aarch64-multiplatform.buildPackages.patchelf
[{"drvPath":"/nix/store/kshhm1fpfdv5a1qkd7sf18plxh7ddlg4-patchelf-0.14.5.drv","outputs":{"out":"/nix/store/96q22iw1i2m960x9y3v6fsj55691pzdl-patchelf-0.14.5"}}]
| 18:53:27 |
@elvishjerricco:matrix.org | Looks the same to me | 18:53:33 |
K900 | Yeah OK it's doing very cursed things | 18:53:39 |
K900 | https://github.com/NixOS/patchelf/blob/master/src/patchelf.cc#L239 | 18:53:40 |
@linus:schreibt.jetzt | In reply to @janne.hess:helsinki-systems.de Who updates them? r-ryantm once we've added an update script that runs cargo update | 18:54:05 |
K900 | This is freaking scary | 18:55:14 |
K900 | It's basically just bytemucking all the values in-place at read time | 18:55:32 |
K900 | RIIR patchelf when | 18:55:47 |
@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 |