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
nixos/graphite: fix + cleanups #83005
Conversation
This PR also replaces #82371 which should be closed if this one is accepted. |
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.
Thanks for bringing this module up to date!
Change LGTM, with some minor suggestions.
This module is python2 only. Upstream repo does not exist in github anymore (returns 404), so it is safer to drop it in nixpkgs.
This version is heavily outdated and is not required anymore. Closes NixOS#52679
LGTM! Very excited to close that longstanding CVE bug :) |
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.
diff LGTM
[13 built, 13 copied (51.2 MiB), 7.7 MiB DL]
https://github.com/NixOS/nixpkgs/pull/83005
1 package blacklisted:
tests.nixos-functions.nixos-test
10 package built:
python27Packages.carbon python27Packages.txamqp python37Packages.carbon python37Packages.graphite-web python37Packages.txamqp python37Packages.waitress-django python38Packages.carbon python38Packages.graphite-web python38Packages.txamqp python38Packages.waitress-django
trusting the other reviewers and @lsix that the service works as intended
Motivation for this change
Graphite was heavely broken, and did depend on django-1.8.
This PR revives it, and makes it migrate to python3.
This PR also closes #52679
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)cc @danderson