-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
treewide: remove orphaned packages #110179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to add throw aliases for all of them and remove them from top level to make ofborg happy.
961f231
to
5ecf2eb
Compare
5ecf2eb
to
8ecf10b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mic92 I know you asked Kampka to do this, but IMO this is a bad idea as long as we don't have popcon-like information about which packages are used. For instance, I know that mdadm
was maintainer-less when I noticed it had last been updated since a quite long time ago when looking into a bug, and at that time I “took over” maintainership (though in practice it's not like mdadm is very active a project anyway so I haven't been doing that much).
All that to say, IMO even orphaned packages have some potentially significant value, at least so long as there are no known security vulnerabilities in them — removing them when there's no longer any maintainer listed basically means that we decide that nixpkgs is only a distribution for people who do maintain packages.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
They also create work if we do treewide refactoring i.e. updating a central library that might break some other packages i.e. #109113 I asked on discourse if anyone wants to keep any of these packages. Also they make nixpkgs-review slower. |
I marked this as stale due to inactivity. → More info |
I agree. I think we should remove them if no one is stepping up. The packages will still live on in Git and are trivial to add back if someone wants to step up at some point in the future. |
mdadm is by default installed on other Distros and it is reasonable to assume that it is an important package unlike the ones that are removed here.
If you are using a package which lags a maintainer and it is to significant value to you than you should step up as a maintainer otherwise the package will degrade over time. We can't put the maintenance burden of maintainerless packages onto the community and fix time along side when doing major updates of other packages. Maintainers can take away some of the burden and fix packages quickly after they broke in nixpkgs-unstable and it is very important that people actually use those packages in their day to day live to find more indepth bugs which someone just checking if the program starts would have never found. |
I always thought an Ubuntu style "multiverse" repo would be a good solution to this problem. See this discourse thread for additional discussion on the topic. |
Motivation for this change
Requested by @Mic92 in #109813 (comment)