| 15 Jan 2023 |
 @nickcao:nichi.co | We should patch that check out, It's not happy with absolute filenames. | 12:10:01 |
| @nickcao:nichi.co | Or shall we instead setting these dlopened libs to RPATH or whatever search pathes. | 12:11:16 |
 K900 | Wait what | 12:11:55 |
| K900 | How does it get an absolute path from fn = strjoina("libtss2-tcti-", driver, ".so.0"); | 12:12:28 |
| @nickcao:nichi.co | That line is patched | 12:12:39 |
| K900 | Oof | 12:12:47 |
 @elvishjerricco:matrix.org | Nick Cao: there's intentional code in the nix expression for replacing these dlopen expressions with ones that use absolute paths | 12:12:52 |
| @elvishjerricco:matrix.org | IIRC it's very much not easy to get it to work any other way | 12:13:20 |
| K900 | So I guess that line should be path_is_valid | 12:13:36 |
| @nickcao:nichi.co | In reply to @k900:0upti.me
So I guess that line should be path_is_valid I doubt whether upstream would accept this. | 12:14:08 |
| K900 | We can just patch it more | 12:14:17 |
| K900 | Since we're doing that anyway | 12:14:23 |
| @elvishjerricco:matrix.org | oh wait ok, sorry I didn't realize you were saying the following check was the problem. That's what I get for reading these messages on my phone without reading the links :P | 12:17:34 |
| @elvishjerricco:matrix.org | we probably need to come up with some better agreement with the systemd folks for how to live with all this dlopen crap | 12:19:48 |
| @elvishjerricco:matrix.org | * we probably need to eventually come up with some better agreement with the systemd folks for how to live with all this dlopen crap | 12:19:56 |
| K900 | Why can't we just add those to rpath/ | 12:21:00 |
| K900 | * Why can't we just add those to rpath? | 12:21:02 |
 Arian | flokli: didn't u find this dlopen thingy that valve uses? | 12:21:04 |
| Arian | In reply to @k900:0upti.me
Why can't we just add those to rpath? We could actually | 12:21:49 |
| @elvishjerricco:matrix.org | In reply to @k900:0upti.me
Why can't we just add those to rpath? IIRC that didn't work for some reason? Either that or we were being picky and wanted to make systemd use absolute paths so it would be harder to trick it | 12:21:52 |
| Arian | We did this trick because it makes the build fail if you forget any deps | 12:22:09 |
| @nickcao:nichi.co | Maybe just paying more attention to the failing tests is enough. | 12:22:14 |
| @elvishjerricco:matrix.org | ah | 12:22:15 |
| @nickcao:nichi.co | They ought to have caught this. | 12:22:20 |
| @nickcao:nichi.co | https://github.com/NixOS/nixpkgs/pull/210896 | 12:23:24 |
