| 4 Aug 2025 |
 dramforever | what's the version of ring used here? it should say it earlier in the log | 16:40:26 |
| dramforever | actually maybe it will be more helpful if you just tell me the "cc" version in Cargo.lock that it uses | 16:49:14 |
 kjeremy | ring 0.17.8, cc 1.1.20 | 16:53:41 |
| dramforever | i just noticed this | 17:16:51 |
| dramforever | i don't think this is right | 17:17:05 |
| dramforever | rustPlatform.cargoSetupHook should be correct | 17:17:24 |
 Alyssa Ross | No? Usually the hook goes in nativeBuildInputs, and when they weren't doing that it used rustc built for aarch64 | 17:17:54 |
| dramforever | rustPlatform is like stdenv, i think | 17:18:39 |
| dramforever | rustPlatform.rust.rustc == buildPackages.rustc | 17:18:42 |
 emily | it's correct in nativeBuildInputs | 17:19:16 |
| emily | perhaps it is not going in nativeBuildInputs | 17:19:20 |
| emily | the hooks go in nativeBuildInputs though right? | 17:19:42 |
| dramforever | it should definitely be in nativeBuildInputs | 17:19:43 |
| emily | so rustPlatform.cargoSetupHook cannot be buildPackages.* w/o splicing | 17:19:50 |
| emily | otherwise nativeBuildInputs would splice it up "one too high" | 17:19:56 |
| emily | maybe? | 17:20:03 |
| emily | wait does that stuff even splice | 17:20:05 |
| emily | I hate splicing I hate splicing I hate splicing I hate splicing I hate splicing I hate splicing I hate splicing | 17:20:08 |
| dramforever | rustPlatform.cargoSetupHook is not spliced | 17:20:17 |
| dramforever | it's like stdenv.cc, right? | 17:20:49 |
| dramforever | * it's like stdenv.cc, right? | 17:20:53 |
| dramforever | that one goes in nativeBuildInputs, and is not spliced | 17:21:04 |
| dramforever | that's... that's not what i'm confused about though, what i'm confused about is why on earth is it building ring for vendor? | 17:21:56 |
| dramforever | oh nevermind i think the package literally is just called zenoh-cpp-vendor | 17:22:38 |
| Alyssa Ross | In reply to @dramforever:matrix.org
rustPlatform.cargoSetupHook is not spliced I thought we fixed the splicing | 17:23:45 |
| dramforever | nevermind i forgot to check in __splicedPackages | 17:25:11 |
| dramforever | it is spliced | 17:25:13 |
| emily | it's so good that we have splicing to make cross easier and less confusing | 17:25:34 |
| emily | and it achieves this goal all day every day every time anyone uses it | 17:25:43 |
| dramforever | kjeremy: so without the actual code we can't know for sure but if you put rustPlatform.cargoSetupHook in nativeBuildInputs it should work | 17:27:27 |
