Skip to content

vmtools.diskImageFuns: drop EOL distributions#409108

Merged
philiptaron merged 1 commit intoNixOS:masterfrom
emilazy:push-qwtvnplsmkmn
May 20, 2025
Merged

vmtools.diskImageFuns: drop EOL distributions#409108
philiptaron merged 1 commit intoNixOS:masterfrom
emilazy:push-qwtvnplsmkmn

Conversation

@emilazy
Copy link
Member

@emilazy emilazy commented May 20, 2025

These went back as far as 2011, and none of them are in active security support. We shouldn’t ship them without a warning and I doubt anyone is using them with a recent Nixpkgs.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

These went back as far as 2011, and none of them are in active security
support. We shouldn’t ship them without a warning and I doubt anyone
is using them with a recent Nixpkgs.
@emilazy emilazy added the 1.severity: security Issues which raise a security issue, or PRs that fix one label May 20, 2025
@nix-owners nix-owners bot requested a review from philiptaron May 20, 2025 15:57
@emilazy emilazy mentioned this pull request May 20, 2025
13 tasks
Copy link
Contributor

@philiptaron philiptaron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a little sticky note on my desk (which sadly has collected a bit of dust) to update all of these to recent versions. This is a fine standin.

I suspect you can yank further and remove now-unused helpers, etc.

I do have one request: can you leave a breadcrumb comment referring to this PR so that there's a pathway for the person who comes along and tries to resurrect things?

@mweinelt mweinelt requested a review from a team May 20, 2025 16:01
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels May 20, 2025
@emilazy
Copy link
Member Author

emilazy commented May 20, 2025

I suspect you can yank further and remove now-unused helpers, etc.

Eh, maybe they’ll be useful in future. I mostly wanted to get rid of the last users of some of these mirrors.

I am not sure that the CentOS stuff will be useful to anyone in future though since that distribution is dead. The Ubuntu and Debian ones follow the same pattern as the versions that remain. So I’m not sure what pointer you want to see left here.

@philiptaron philiptaron merged commit 41aa41e into NixOS:master May 20, 2025
19 of 22 checks passed
@philiptaron
Copy link
Contributor

I was imagining a # See https://github.com/NixOS/nixpkgs/pull/409108 for the now-removed RPM distributions. comment. But it's low value so I hit that merge button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.severity: security Issues which raise a security issue, or PRs that fix one 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants