!djTaTBQyWEPRQxrPTb:nixos.org

Nixpkgs Architecture Team

227 Members
https://github.com/nixpkgs-architecture, weekly public meetings on Wednesday 15:00-16:00 UTC at https://meet.jit.si/nixpkgs-architecture53 Servers

Load older messages


SenderMessageTime
5 Jan 2024
@infinisil:matrix.orginfinisilThis PR is ready to be reviewed!00:55:59
@infinisil:matrix.orginfinisil 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:augsburg.oneprofpatschinfinisil: so is the system in place now? i.e. any new package should use the new style?01:20:01
@profpatsch:augsburg.oneprofpatschAh I see a bunch of packages have already been migrated01:21:17
@infinisil:matrix.orginfinisil 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:matrix.orginfinisil * 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:augsburg.oneprofpatschinfinisil: can’t there be a pkgs/by-name/fo/foo/overrides.nix?01:22:43
@profpatsch:augsburg.oneprofpatschInstead of putting overrides in all-packages.nix01:22:50
@profpatsch:augsburg.oneprofpatschfeels like a weird step back if all-packages.nix is still needed for that01:23:06
@infinisil:matrix.orginfinisil It's not a step back, all-packages.nix has been required so far, now it's only required less 01:23:35
@infinisil:matrix.orginfinisil
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:augsburg.oneprofpatschok01:24:34
@infinisil:matrix.orginfinisil 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:augsburg.oneprofpatschwhat should I do for the skawarePackages namespace?01:25:11
@profpatsch:augsburg.oneprofpatschI only alias packages on the toplevel from that namespace01:25:27
@infinisil:matrix.orginfinisilYeah that continues working no problem01:25:43
@infinisil:matrix.orginfinisil Profpatsch: CI will complain and tell you what to do if something isn't allowed :) 01:26:23
@profpatsch:augsburg.oneprofpatschbut I guess it’s hard to transfer that into the new style then01:26:26
@profpatsch:augsburg.oneprofpatschwell, I’ll see01:26:42
@infinisil:matrix.orginfinisilFuture work, lots of future work! Small steps :)01:26:47
@profpatsch:augsburg.oneprofpatschI don’t maintain that many packages anymore01:26:49
@profpatsch:augsburg.oneprofpatschI hope at one point somebody does the work so much of the tranfer can be automated :)01:27:14
@infinisil:matrix.orginfinisil Future work 01:27:38
@infinisil:matrix.orginfinisil 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:matrix.orginfinisil Profpatsch: https://github.com/NixOS/nixpkgs/pull/211832 01:28:31
@profpatsch:augsburg.oneprofpatschah, nice01:30:02
@9999years:matrix.org9999years set a profile picture.21:01:26
@9999years:matrix.org9999years changed their profile picture.21:01:48
@infinisil:matrix.orginfinisil@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 availability22:21:54
@emilytrau:matrix.org@emilytrau:matrix.org left the room.22:32:09

Show newer messages


Back to Room ListRoom Version: 9