| 29 Aug 2023 |
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 |
infinisil | Both kind of suck but also aren't very significant, but I think in order to keep the number of changes to a minimum I'll go with the first one | 18:31:07 |
K900 | At least | 18:31:08 |
K900 | But we can do that in a follow-up | 18:31:17 |
infinisil | It's a bit tricky because there seems to be some cross compilation shenanigans sometimes | 18:31:46 |
K900 | In fact I'd really like to get rid of those and maybe get smaller better scoped package sets for KF6/Plasma6/KDE Gear stuff | 18:31:50 |
K900 | qt6Packages isn't even spliced | 18:32:02 |
K900 | And qt5 cross was broken until like two days ago | 18:32:20 |
infinisil | See also my little rant from yesterday: https://matrix.to/#/!kjdutkOsheZdjqYmqp:nixos.org/$U5DUxbsottP-50Q5d0PQYHnX0xZdNpJDJ4hXiNkBdfs?via=nixos.org&via=matrix.org&via=nixos.dev | 18:32:24 |