| 5 Jan 2024 |
infinisil | This PR is ready to be reviewed! | 00:55:59 |
infinisil | It doesn't change any functionality though, and the tests still pass, so I'm fairly confident and will likely self-merge rather soon in order to unblock the main PR that actually does the enforcing of pkgs/by-name | 00:58:11 |
profpatsch | infinisil: so is the system in place now? i.e. any new package should use the new style? | 01:20:01 |
profpatsch | Ah I see a bunch of packages have already been migrated | 01:21:17 |
infinisil | Profpatsch: It's been usable every since Nixcon, and a lot of people are already using it. After the above PR (and one more), it's going to be enforced for new packages | 01:22:19 |
infinisil | * Profpatsch: It's been usable ever since Nixcon, and a lot of people are already using it. After the above PR (and one more), it's going to be enforced for new packages | 01:22:27 |
profpatsch | infinisil: can’t there be a pkgs/by-name/fo/foo/overrides.nix? | 01:22:43 |
profpatsch | Instead of putting overrides in all-packages.nix | 01:22:50 |
profpatsch | feels like a weird step back if all-packages.nix is still needed for that | 01:23:06 |
infinisil | It's not a step back, all-packages.nix has been required so far, now it's only required less | 01:23:35 |
infinisil | In reply to @profpatsch:augsburg.one infinisil: can’t there be a pkgs/by-name/fo/foo/overrides.nix? Profpatsch: It's a future change that we could do, but for now nothing has been decided, it's not in RFC 140 | 01:24:17 |
profpatsch | ok | 01:24:34 |
infinisil | Profpatsch: https://github.com/NixOS/rfcs/blob/master/rfcs/0140-simple-package-paths.md#allow-callpackage-arguments-to-be-specified-in-argsnix | 01:25:03 |
profpatsch | what should I do for the skawarePackages namespace? | 01:25:11 |
profpatsch | I only alias packages on the toplevel from that namespace | 01:25:27 |
infinisil | Yeah that continues working no problem | 01:25:43 |
infinisil | Profpatsch: CI will complain and tell you what to do if something isn't allowed :) | 01:26:23 |
profpatsch | but I guess it’s hard to transfer that into the new style then | 01:26:26 |
profpatsch | well, I’ll see | 01:26:42 |
infinisil | Future work, lots of future work! Small steps :) | 01:26:47 |
profpatsch | I don’t maintain that many packages anymore | 01:26:49 |
profpatsch | I hope at one point somebody does the work so much of the tranfer can be automated :) | 01:27:14 |
infinisil | Future work | 01:27:38 |
infinisil | Profpatsch: Automating the migration is also part of the RFC, that's gonna be next after enforcing it for new packages | 01:28:05 |
infinisil | Profpatsch: https://github.com/NixOS/nixpkgs/pull/211832 | 01:28:31 |
profpatsch | ah, nice | 01:30:02 |
| 9999years set a profile picture. | 21:01:26 |
| 9999years changed their profile picture. | 21:01:48 |
infinisil | @room Because it's a significant milestone, I'd like to schedule a meet for the final review and ideally merge of https://github.com/NixOS/nixpkgs/pull/275539 in ~2 weeks! For anybody wanting to join (but especially NAT members), please fill out https://crab.fit/making-pkgsbyname-mandatory-for-new-packages-908672 with your availability | 22:21:54 |
| @emilytrau:matrix.org left the room. | 22:32:09 |