!djTaTBQyWEPRQxrPTb:nixos.org

Nixpkgs Architecture Team

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

Load older messages


SenderMessageTime
29 May 2023
@piegames:matrix.org@piegames:matrix.org I thought you meant the <nixpkgs> path, not the value of the import 13:19:40
@infinisil:matrix.orginfinisilI'd say that we shouldn't be too afraid to pick the wrong name, because we can always rename it. Yes it will cause merge conflicts, but it's trivially resolveable13:21:06
@infinisil:matrix.orginfinisil * I'd say that we shouldn't be too afraid to pick the wrong name, because we can always rename it. Yes it will cause merge conflicts, but they're trivially resolveable13:23:35
@infinisil:matrix.orginfinisilThe file structure is not an API13:23:58
@piegames:matrix.org@piegames:matrix.org Totally agreed. And I'm fine with ending up on unit, just does this not mean that I already give up the search 13:24:03
@infinisil:matrix.orginfinisil Hehe yeah. At this point I feel like just calling it unit with all the arguments for it and only reconsidering when strong new arguments against up against it. And considering that a lot of people already looked at it and this hasn't happened, I don't think it will ever happen 13:25:05
@infinisil:matrix.orginfinisil * Hehe yeah. At this point I feel like just calling it unit with all the arguments for it and only reconsidering when strong new arguments against up against it. And since a lot of people already looked at it and this hasn't happened, I don't think it will ever happen 13:25:20
@infinisil:matrix.orginfinisil But, FCP is there for even more people to look at it, so it could happen. But if we as the authors and shepherds are okay with it, it's our job to be decisive about this, we shouldn't rely on FCP to make decisions for us. 13:27:13
@infinisil:matrix.orginfinisilOh and also, this is a rather minor aspect of the RFC, I'd rather people focus on reviewing the more important parts of it13:28:36
@piegames:matrix.org@piegames:matrix.org
In reply to @infinisil:matrix.org
Oh and also, this is a rather minor aspect of the RFC, I'd rather people focus on reviewing the more important parts of it
If a discussion gets stuck on names like this, then this usually means that the rest is generally approved
13:30:44
@roberthensing:matrix.orgRobert Hensing (roberth)
In reply to @infinisil:matrix.org
I'd be interested in more cleanly separating the top-level nixpkgs attribute into e.g. import nixpkgs {} == { pkgs = { ... }; lib = { ... }; nixos = ...; }
like the flake, but cleaner
13:37:45
@roberthensing:matrix.orgRobert Hensing (roberth) also fyi inputs.nixpkgs.lib.nixos == import ./nixos/lib 13:38:34
@roberthensing:matrix.orgRobert Hensing (roberth)it's not a complete representation of nixos entrypoints yet though13:38:50
@piegames:matrix.org@piegames:matrix.org
In reply to @infinisil:matrix.org
All good to me, I haven't pushed anything to the RFC I wouldn't be okay with, would like to hear from Robert Hensing (roberth) as well though. I think niksnut's arguments have been properly addressed here, but it might be proper to add the new arguments to the RFC as well.
Hi Robert Hensing (roberth), we've had quite a bit of backlog, but please have a look at
13:39:20
@roberthensing:matrix.orgRobert Hensing (roberth)I've followed along, and I don't feel strongly about the points raised13:40:03
@nbp:mozilla.orgnbpwell … in principle I would agree, but as long as we do not have a clean way to selectively "update" without replacing all attributes in an attribute set, this would increase the complexity for overlays which are trying to both library and packages.13:40:17
@roberthensing:matrix.orgRobert Hensing (roberth)Or I guess you're asking me to add to the alternatives text?13:40:32
@nbp:mozilla.orgnbpslectively update … like suggested in S.O.S.13:40:35
@piegames:matrix.org@piegames:matrix.org
In reply to @roberthensing:matrix.org
I've followed along, and I don't feel strongly about the points raised
So, go for FCP?
13:40:55
@nbp:mozilla.orgnbp * well … in principle I would agree (cleanly separating the top-level of nixpkgs), but as long as we do not have a clean way to selectively "update" without replacing all attributes in an attribute set, this would increase the complexity for overlays which are trying to both library and packages.13:41:07
@roberthensing:matrix.orgRobert Hensing (roberth)
In reply to @roberthensing:matrix.org
Or I guess you're asking me to add to the alternatives text?
infinisil has been taking care of that, so I'd have to sync up with him first; not sure if that's productive
13:41:25
@infinisil:matrix.orginfinisilNo the question is just whether you accept the state of the current RFC13:41:47
@roberthensing:matrix.orgRobert Hensing (roberth)Oh, right hadn't read up on the messages before that. I'll have a final look13:42:28
@infinisil:matrix.orginfinisilIf so, approving on GitHub would be great, and then we'd have everything for FCP13:43:09
@roberthensing:matrix.orgRobert Hensing (roberth)The unit base directory must only contain subdirectories of the form pkgs/unit/${shard}/${name},13:44:12
@roberthensing:matrix.orgRobert Hensing (roberth) technically you're duplicating pkgs/unit, as in pkgs/unit/pkgs/unit, but everyone understand what you mean 13:44:30
@infinisil:matrix.orginfinisilHehe good catch13:45:07
@roberthensing:matrix.orgRobert Hensing (roberth) nix-build -E is --expr? 13:48:12
@infinisil:matrix.orginfinisilYeah13:48:21
@roberthensing:matrix.orgRobert Hensing (roberth)hm ok13:48:25

Show newer messages


Back to Room ListRoom Version: 9