treewide: migrate packages to by-name hierarchy#254417
treewide: migrate packages to by-name hierarchy#254417AndersonTorres merged 20 commits intoNixOS:masterfrom atorres1985-contrib:migrate-to-by-name
Conversation
|
I think it was discussed at NixCon that we should move to the new schema using automation (where possible) to avoid the overhead of many PRs. You also did some cleanup so in this case this is probably fine. |
|
This is an opportunity to review my own code and not lose some tracks. I have found some packages that should be removed and others needing more attention, like TLOE. Also there is no problem in some manual intervention. |
|
@ofborg build pkgsLLVM.mksh mksh |
|
Yeah so I will work on updating #211832 next, which is the automated mass migration. I wouldn't want people to have to do this manually to save on both reviewer and contributor time. I think it's fine if you happen to clean up the code or update the versions anyways though. Since you did the work here already I guess it's fine, but others should definitely not be encouraged to do this manually until the automated migration is done (it doesn't work automatically for all packages) 🙂 |
|
I opened #254632 to have this written down |
|
OK, I will stop this thread for now and concentrate in the version bumps. |
|
@ofborg build xorriso dosbox-staging |
|
Are you intending on having this reviewed and merged after all? Or are you using this as a testing ground before splitting it into smaller cleanup PRs? If so this should probably be turned into a draft. |
|
Both |
- multiple outputs - add meta.mainProgram - migrate to by-name
- add meta.mainProgram - migrate to by-name
- rec-less
- finalAttrs - no nested with
- factor out tests
Description of changes
The ones maintained by @AndersonTorres
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)