-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
[backport 24.11] treewide: remove deprecations up until 24.11 #358798
[backport 24.11] treewide: remove deprecations up until 24.11 #358798
Conversation
Following through on message itself. (cherry picked from commit 73df63f)
Follow through on NixOS#296384. (cherry picked from commit de69ff5)
This was planned for 21.11, so let's follow through. (cherry picked from commit 749a6fe)
Those were scheduled to be removed in 24.11, so let's do that. Added in NixOS#311893. (cherry picked from commit 9966353)
Scheduled for 24.11. (cherry picked from commit 8bdfc5e)
This is also true in 24.11 and up, so no need to state the version number explicitly. (cherry picked from commit e1e2193)
Scheduled for 24.11. Added in NixOS#272147. (cherry picked from commit 4255d7a)
This had been scheduled for removal in 23.11. (cherry picked from commit 6646eeb)
Those were scheduled for removal in 24.11. (cherry picked from commit 89981f1)
Scheduled for removal for 24.11. (cherry picked from commit 352f462)
This was scheduled for 24.11. (cherry picked from commit 97c3fa4)
This was scheduled for 24.11. (cherry picked from commit b6c3e15)
Scheduled for 24.05. (cherry picked from commit 6c258ee)
I think we ought to backport the rest of 51da8b6 since it’s in the 24.11 release notes and all. Seems harmless to leave the paragraph in the manual for now though. |
Had been deprecated and scheduled for removal in 24.11.
Ah, true. Release notes win. Pushed. |
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.
RM approved backport, so will merge pending ofborg results.
@emilazy ofborg done |
Backport of #356732
I left out the conflicting commit 51da8b6. The conflict was because the redirects.json file doesn't exist in 24.11, yet. Thus I decided not to backport the removal of
writeReferencesToFile
, because that would leave us in the odd situation, that the hash for#writeReferencesToFile
would lead nowhere for the 24.11 manual, but then be redirected properly for the 25.05 version again.@emilazy
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.