| 14 Jul 2025 |
QuadRadical (Ping) | okay thanks | 14:27:23 |
QuadRadical (Ping) | its taking ages to build for some reason | 14:27:27 |
QuadRadical (Ping) | perl and gettext is taking so long | 14:27:46 |
Toma | By the way, John Ericson do you remember why the python package-set's spliced scope has splicing disabled for hooks?
The original PR: https://github.com/NixOS/nixpkgs/pull/104201 Currently: https://github.com/NixOS/nixpkgs/blob/b125d87d5b6473acc534c46f06cc0c835c595452/pkgs/development/interpreters/python/passthrufun.nix#L78 Usage of keep: https://github.com/NixOS/nixpkgs/blob/e6d2b439c23fa01b18ae6f677f5e5aba3be48518/lib/customisation.nix#L688-L691 (the comment was added by someone else in https://github.com/NixOS/nixpkgs/commit/dd48cb0b3479912a9c6adfaa7279dbadabaeae2b)
Recent discussion: https://github.com/NixOS/nixpkgs/pull/424913#discussion_r2204760754
| 14:31:52 |
Alyssa Ross | Just checking which one of the many Rust target variables is the right one here, since we've learned a lot about them since 2022 | 14:32:40 |
Alyssa Ross | Yeah it's cargoShortTarget | 14:35:18 |
dramforever | looks away whistling | 14:36:05 |
emily | it's not worth testing until it's rebased | 14:40:09 |
emily | but yes, staging means rebuilding the world | 14:40:13 |
emily | you will want to cherry-pick the rebased form to test instead | 14:40:19 |
emily | though since it touches pkg-config it will probably still rebuild a lot regardless | 14:40:31 |
QuadRadical (Ping) | makes sense | 14:41:10 |
Alyssa Ross | I'll be rebasing onto the master/staging merge base | 14:43:12 |
Alyssa Ross | So not quite so much rebuilding | 14:43:15 |
Alyssa Ross | Oh right but it rebuilds the hook 😅 | 14:48:47 |
QuadRadical (Ping) | it actually did end up building | 14:56:06 |
QuadRadical (Ping) | so i can confirm it works | 14:56:09 |
emily | uh, what were you building exactly? | 14:56:22 |
emily | because the PR is sufficiently old that the package would not have been broken in the first place on its HEAD | 14:56:35 |
emily | if you tested the merge then that means something | 14:56:42 |
QuadRadical (Ping) | `nix build github:alyssais/nixpkgs/cargo-cross#pkgsCross.aarch64-multiplatform.thin-pro visioning-tools` | 14:56:52 |
QuadRadical (Ping) | nix build github:alyssais/nixpkgs/cargo-cross#pkgsCross.aarch64-multiplatform.thin-pro
visioning-tools
| 14:57:14 |
emily | yeah, that is testing the package as it existed in like 2023 when cross wasn't broken in the first place | 14:57:20 |
QuadRadical (Ping) | nix build github:alyssais/nixpkgs/cargo-cross#pkgsCross.aarch64-multiplatform.thin-provisioning-tools | 14:57:28 |
QuadRadical (Ping) | ohh, the rust cross breaking is a recent thing? | 14:57:39 |
emily | it was probably caused by https://github.com/NixOS/nixpkgs/pull/419746. | 14:58:27 |
emily | not sure it was even Rust in 2022. | 14:58:34 |
QuadRadical (Ping) | oh, i see | 14:58:47 |
QuadRadical (Ping) | ill retest as soon as its rebased so let me know please :) | 14:59:00 |
QuadRadical (Ping) | and thank you emily and alyssa | 14:59:06 |