!djTaTBQyWEPRQxrPTb:nixos.org

Nixpkgs Architecture

216 Members
Discussions about Nixpkgs' architecture - https://github.com/NixOS/nixpkgs/labels/architecture47 Servers

Load older messages


SenderMessageTime
15 Dec 2023
@philiptaron:matrix.orgPhilip Taron (UTC-8)Sounds good to me.16:50:25
@nbp:mozilla.orgnbp changed their display name from nbp to nbp {pto}.19:40:55
@nbp:mozilla.orgnbp changed their display name from nbp {pto} to nbp.19:41:16
@kity:kity.wtfproblemshttps://github.com/NixOS/nixpkgs/issues/273972 this seems like a good architecture issue21:53:58
16 Dec 2023
@infinisil:matrix.orginfinisilNot sure, it's an interesting idea, but I don't think it's an overarching issue with Nixpkgs architecture02:31:44
19 Dec 2023
@infinisil:matrix.orginfinisil Oh nice, https://github.com/NixOS/nixpkgs/pull/255023 added an unnecessary callPackage ../by-name/... { }, which won't be possible anymore after https://github.com/NixOS/nixpkgs/pull/274591, but the existing definition won't give an error! 02:38:34
@infinisil:matrix.orginfinisil That second PR is almost ready btw, after that's merged I can work on making pkgs/by-name mandatory for new packages! 02:39:32
@infinisil:matrix.orginfinisilPS: I moved the 4-weekly meeting by 2 weeks from 2023-12-26 to 2024-01-09 to avoid the holidays :)16:43:56
@infinisil:matrix.orginfinisil This PR is ready now, reviews (by anybody) appreciated: https://github.com/NixOS/nixpkgs/pull/274591
I'd like to merge it as soon as possible, ideally this week
Ping the NAT Robert Hensing (roberth), John Ericson, Growpotkin, DavHau, tomberek
22:38:32
20 Dec 2023
@infinisil:matrix.orginfinisil And I also went ahead with a draft of the next step after the above, enforcing new packages to use pkgs/by-name: https://github.com/NixOS/nixpkgs/pull/275539 01:17:15
@roberthensing:matrix.orgRobert Hensing (roberth)
In reply to @infinisil:matrix.org
And I also went ahead with a draft of the next step after the above, enforcing new packages to use pkgs/by-name: https://github.com/NixOS/nixpkgs/pull/275539
It says: This PR is a draft and isn't ready to be reviewed yet
16:33:11
@infinisil:matrix.orginfinisil Robert Hensing (roberth): This one is ready: https://github.com/NixOS/nixpkgs/pull/274591 16:34:36
@roberthensing:matrix.orgRobert Hensing (roberth)Just read that one16:34:58
@infinisil:matrix.orginfinisil Robert Hensing (roberth): Ah yes, thanks, just addressed that 16:39:08
21 Dec 2023
@infinisil:matrix.orginfinisilUnless there's opposition, I'll merge https://github.com/NixOS/nixpkgs/pull/274591 later today19:01:05
23 Dec 2023
@raitobezarius:matrix.orgraitobezarius changed their display name from raitobezarius to raitobezarius (DECT 2128).22:22:09
25 Dec 2023
@lassulus:lassul.uslassulus changed their display name from lassulus to lassulus (8587).22:41:42
28 Dec 2023
@piegames:matrix.org@piegames:matrix.org changed their display name from piegames to piegames ☎ 9712.01:33:05
@piegames:matrix.org@piegames:matrix.org changed their display name from piegames ☎ 9712 to piegames [☎ 9712].01:33:13
30 Dec 2023
@roberthensing:matrix.orgRobert Hensing (roberth)An instance of package metadata that really shouldn't be on a package attrset (passthru etc). https://github.com/NixOS/nixpkgs/pull/277221/files#r143856656811:35:03
@roberthensing:matrix.orgRobert Hensing (roberth)We should have a place for this kind of thing that isn't a random file somewhere11:36:06
@lassulus:lassul.uslassulus changed their display name from lassulus (8587) to lassulus.19:30:23
@raitobezarius:matrix.orgraitobezarius changed their display name from raitobezarius (DECT 2128) to raitobezarius.19:52:43
31 Dec 2023
@piegames:matrix.org@piegames:matrix.org changed their display name from piegames [☎ 9712] to piegames.12:40:25
@philiptaron:matrix.orgPhilip Taron (UTC-8) changed their display name from Philip Taron to Philip Taron (UTC-8).18:10:46
@kaction:envs.netkaction joined the room.23:36:39
5 Jan 2024
@infinisil:matrix.orginfinisil In order to enforce pkgs/by-name for new packages, I split off a bunch of internal code changes into a separate PR: https://github.com/NixOS/nixpkgs/pull/278805 00:55:36
@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.one@profpatsch:augsburg.oneinfinisil: so is the system in place now? i.e. any new package should use the new style?01:20:01

Show newer messages


Back to Room ListRoom Version: 9