| 15 Dec 2023 |
infinisil | Btw the RFC specifies to do it in a different way, but the basic idea is the same:
All satisfying definitions that can't be automatically migrated due to the above restrictions will be added to a CI exclusion list. CI is added to ensure that all satisfying definitions except the CI exclusion list must be using the new directory structure. This means that the new directory structure becomes mandatory for new satisfying definitions after this PR. The CI exclusion list should be removed eventually once the non-automatically-migratable satisfying definitions have been manually migrated. Only in very limited circumstances is it allowed to add new entries to the CI exclusion list.
| 00:27:33 |
Philip Taron (UTC-8) | I'm all about the spirit dominating the letter. | 00:28:27 |
infinisil | I'm planning to merge https://github.com/NixOS/nixpkgs/pull/272395 once CI passes so I can move onto the follow-up PR | 16:48:29 |
Philip Taron (UTC-8) | Sounds good to me. | 16:50:25 |
| nbp changed their display name from nbp to nbp {pto}. | 19:40:55 |
| nbp changed their display name from nbp {pto} to nbp. | 19:41:16 |
problems | https://github.com/NixOS/nixpkgs/issues/273972 this seems like a good architecture issue | 21:53:58 |
| 16 Dec 2023 |
infinisil | Not sure, it's an interesting idea, but I don't think it's an overarching issue with Nixpkgs architecture | 02:31:44 |
| 19 Dec 2023 |
infinisil | Oh nice, https://github.com/NixOS/nixpkgs/pull/255023 added an unnecessary callPackage ../by-name/... { }, which won't be possible anymore after https://github.com/NixOS/nixpkgs/pull/274591, but the existing definition won't give an error! | 02:38:34 |
infinisil | That second PR is almost ready btw, after that's merged I can work on making pkgs/by-name mandatory for new packages! | 02:39:32 |
infinisil | PS: I moved the 4-weekly meeting by 2 weeks from 2023-12-26 to 2024-01-09 to avoid the holidays :) | 16:43:56 |
infinisil | This PR is ready now, reviews (by anybody) appreciated: https://github.com/NixOS/nixpkgs/pull/274591 I'd like to merge it as soon as possible, ideally this week Ping the NAT Robert Hensing (roberth), John Ericson, Growpotkin, DavHau, tomberek | 22:38:32 |
| 20 Dec 2023 |
infinisil | And I also went ahead with a draft of the next step after the above, enforcing new packages to use pkgs/by-name: https://github.com/NixOS/nixpkgs/pull/275539 | 01:17:15 |
Robert Hensing (roberth) | In reply to @infinisil:matrix.org And I also went ahead with a draft of the next step after the above, enforcing new packages to use pkgs/by-name: https://github.com/NixOS/nixpkgs/pull/275539 It says: This PR is a draft and isn't ready to be reviewed yet | 16:33:11 |
infinisil | Robert Hensing (roberth): This one is ready: https://github.com/NixOS/nixpkgs/pull/274591 | 16:34:36 |
Robert Hensing (roberth) | Just read that one | 16:34:58 |
infinisil | Robert Hensing (roberth): Ah yes, thanks, just addressed that | 16:39:08 |
| 21 Dec 2023 |
infinisil | Unless there's opposition, I'll merge https://github.com/NixOS/nixpkgs/pull/274591 later today | 19:01:05 |
| 23 Dec 2023 |
| raitobezarius changed their display name from raitobezarius to raitobezarius (DECT 2128). | 22:22:09 |