| 27 Aug 2023 |
 infinisil | fzakaria: There's already a proposal for that :D https://github.com/NixOS/rfcs/pull/146 | 10:08:19 |
 Robert Hensing (roberth) | I guess to frame this differently, unused options may not have to be loaded. For instance we should have a module for publishing and checking pkg-config modules (involving meta and more, so not just hooks). Supporting such a feature would require unconditional code in the mkDerivation hot path, whereas with modules, you'd only import that logic in a package that uses it. | 19:49:12 |
| infinisil | I updated https://github.com/NixOS/nixpkgs/pull/250885 (RFC 140) twice based on the feedback now. I planned to have it be merged this Tuesday 15:30 CEST together with the NAT (and anybody else who wants to join the call). Of course only if it's considered ready by the others, so please take a look at the code ideally this Monday and accept the PR if it looks good! | 23:52:27 |
| 28 Aug 2023 |
|  @cf11:0x2c.org joined the room. | 17:07:40 |
| @cf11:0x2c.org | 17:09:11 |
| 29 Aug 2023 |
| infinisil | We're meeting in https://meet.jit.si/nixpkgs-architecture in 50 minutes (15:30 CEST) for https://github.com/NixOS/nixpkgs/pull/250885! | 12:40:14 |
| infinisil | Robert Hensing (roberth): John Ericson: DavHau: growpotkin ( Alex Ameen ): Who of you is also joining? | 13:32:39 |
 Growpotkin | I am | 13:34:52 |
 yorik.sar | ---- tests::test_cases stdout ----
Given Nixpkgs path does not contain a pkgs/by-name subdirectory, no check necessary.
thread 'tests::test_cases' panicked at 'Failed test case case-sensitive-duplicate-package, expected these errors:
pkgs/by-name/fo: Duplicate case-sensitive package directories "foO" and "foo".
but got these:
', src/main.rs:118:17
| 13:55:22 |
| infinisil | Thanks for joining and helping everyone! I'll work on the next PR now, trying to get it done asap so it can be reviewed until next Tuesday when we have another meeting scheduled :D | 15:00:26 |
 K900 | lmfak | 18:45:19 |
| K900 | * lmfao | 18:45:24 |
| K900 | infinisil: https://hydra.nixos.org/build/233371356/nixlog/1 | 18:45:32 |
| infinisil | Lol | 18:46:19 |
| infinisil | That's the point | 18:46:29 |
| K900 | I know | 18:46:41 |
| K900 | That just makes it funnier | 18:46:48 |
| infinisil | Yeah xD | 18:47:03 |
| infinisil | Looking into it.. | 18:47:12 |
| 30 Aug 2023 |
| Robert Hensing (roberth) | sorry, didn't get a notification while I was recovering my system from a fried ssd 😞 | 08:40:16 |
| Robert Hensing (roberth) | In reply to @k900:0upti.me
infinisil: https://hydra.nixos.org/build/233371356/nixlog/1 just wrap it in testBuildFailure and grep the output to make sure it still fails | 08:42:02 |
| Robert Hensing (roberth) | (I know, I know. I'm just plugging that function because why not. Test functions are cool) | 08:42:37 |
| infinisil | Robert Hensing (roberth): I don't think that would work well here, it's the channel tarball build that's failing, and it should definitely keep on failing when such files are present | 10:44:45 |
| infinisil | No problem ;) | 10:47:02 |
| infinisil | Nice, the pkgs/by-name CI check runs in about 30 seconds! https://github.com/NixOS/nixpkgs/actions/runs/6026577029/job/16349848006?pr=237439 | 15:00:39 |
| 31 Aug 2023 |
| infinisil | Question for https://github.com/NixOS/nixpkgs/pull/237439:
We can almost declare all top-level packages in pkgs/by-name, except ones that should be declared using e.g. python3Packages.callPackage, libsForQt5.callPackage, etc. | 18:28:28 |
| infinisil | Now there's two options, either: | 18:29:05 |
| infinisil |
- Keep the category hierarchy alive only for new packages of that kind
| 18:29:39 |
| infinisil |
- Deprecate the category hierarchy and introduce something like
pkgs/alt-callPackage, where you can put all packages of that kind
| 18:30:39 |
| K900 | We can absolutely get rid of libsForQt5.callPackage and qt6Packages.callPackage | 18:31:04 |
