Skip to content

lib/minver: bump to nix 2.18.0 #398444

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

lib/minver: bump to nix 2.18.0 #398444

wants to merge 1 commit into from

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Apr 13, 2025

Improved sentence:

This version was released in September 2023, which should provide users with sufficient time to upgrade.

Bumping to this version allows us to use unsafeDiscardReferences for NixOS images. Additionally, the current minimum version (2.3.17) contains several unpatched security vulnerabilities as documented in the Nix security advisories. While not all vulnerabilities apply to version 2.3, maintaining an older version requirement would force us to run that same version for regression testing, which is problematic from a security perspective.

Improved sentence:

**This version was released in September 2023**, which should provide **users with sufficient time to upgrade**.

Bumping to this version allows us to use unsafeDiscardReferences for NixOS images.
Additionally, the current minimum version (2.3.17) contains **several unpatched security vulnerabilities** as documented in the Nix security advisories.
While not all vulnerabilities apply to version 2.3, maintaining an older version requirement would force us to run that same version for regression testing,
which is problematic from a security perspective.
@github-actions github-actions bot added the 6.topic: lib The Nixpkgs function library label Apr 13, 2025
@nix-owners nix-owners bot requested review from hsjobeki and infinisil April 13, 2025 20:44
@github-actions github-actions bot added 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Apr 13, 2025
@@ -1,2 +1,2 @@
# Expose the minimum required version for evaluating Nixpkgs
"2.3.17"
Copy link
Member Author

@Mic92 Mic92 Apr 13, 2025

Choose a reason for hiding this comment

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

Why does this version actually needs to be packaged in Nix? We should be just able to use a different version of Nix for our tarball check, no?

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: lib The Nixpkgs function library 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant