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
redis: 4.0.11 -> 5.0.0 #48783
redis: 4.0.11 -> 5.0.0 #48783
Conversation
@GrahamcOfBorg build redis |
Success on x86_64-darwin (full log) Attempted: redis Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: redis Partial log (click to expand)
|
@GrahamcOfBorg build redis |
Success on aarch64-linux (full log) Attempted: redis Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: redis Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: redis Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: redis Partial log (click to expand)
|
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.
Are there any major changes we should document in the 19.03 release notes?
@srhb afaik redis doesn't use semantic versioning, so this upgrade doesn't have major breaking changes. I've found the following https://aws.amazon.com/redis/Whats_New_Redis5/#Upgrade_considerations but I'm not sure if we should add it to the release notes |
@marsam Thank you for that. If I read this correctly, it's entirely forwards-compatible, so no release notes should be necessary. |
Tested with every test that incidentally uses redis in nixos/tests. Looks good. Should this be backported to 18.09? |
cc @berdario |
Motivation for this change
https://raw.githubusercontent.com/antirez/redis/5.0/00-RELEASENOTES
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)