| 29 May 2023 |
@piegames:matrix.org | I thought you meant the <nixpkgs> path, not the value of the import | 13:19:40 |
infinisil | 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 it's trivially resolveable | 13:21:06 |
infinisil | * 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 resolveable | 13:23:35 |
infinisil | The file structure is not an API | 13:23:58 |
@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 | 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 | * 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 | 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 | 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 | 13:28:36 |
@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 |
Robert 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 |
Robert Hensing (roberth) | also fyi inputs.nixpkgs.lib.nixos == import ./nixos/lib | 13:38:34 |
Robert Hensing (roberth) | it's not a complete representation of nixos entrypoints yet though | 13:38:50 |
@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 |
Robert Hensing (roberth) | I've followed along, and I don't feel strongly about the points raised | 13:40:03 |
nbp | well … 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 |
Robert Hensing (roberth) | Or I guess you're asking me to add to the alternatives text? | 13:40:32 |
nbp | slectively update … like suggested in S.O.S. | 13:40:35 |
@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 | * 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 |
Robert 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 | No the question is just whether you accept the state of the current RFC | 13:41:47 |
Robert Hensing (roberth) | Oh, right hadn't read up on the messages before that. I'll have a final look | 13:42:28 |
infinisil | If so, approving on GitHub would be great, and then we'd have everything for FCP | 13:43:09 |
Robert Hensing (roberth) | The unit base directory must only contain subdirectories of the form pkgs/unit/${shard}/${name}, | 13:44:12 |
Robert Hensing (roberth) | technically you're duplicating pkgs/unit, as in pkgs/unit/pkgs/unit, but everyone understand what you mean | 13:44:30 |
infinisil | Hehe good catch | 13:45:07 |
Robert Hensing (roberth) | nix-build -E is --expr? | 13:48:12 |
infinisil | Yeah | 13:48:21 |
Robert Hensing (roberth) | hm ok | 13:48:25 |