| 25 Aug 2023 |
 infinisil | @room https://github.com/NixOS/nixpkgs/pull/250885 (RFC 140 Part 1a) is ready, please review, it's 99% Rust, and it would be great to have it reviewed and merged next week so we can get Part 1b done before Nixcon too, which would then allow people to actually use the new directory for packages! | 07:09:19 |
| 26 Aug 2023 |
 fzakaria | i watched it on Nix Hour | 22:11:34 |
| fzakaria | pretty cool; would be neat to still offer some symlink to top-level directories like "tools" etc.. so you can find something other than name | 22:12:03 |
| fzakaria | or maybe it should be in the meta or passthru | 22:12:14 |
| 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 |
