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
patroni: init at 1.6.4 #80696
patroni: init at 1.6.4 #80696
Conversation
pkgs/servers/sql/patroni/default.nix
Outdated
preCheck = "export HOME=$(mktemp -d)"; | ||
|
||
meta = with lib; { | ||
homepage = https://patroni.readthedocs.io/en/latest/; |
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.
homepage = https://patroni.readthedocs.io/en/latest/; | |
homepage = "https://patroni.readthedocs.io/en/latest/"; |
ydiff | ||
]; | ||
|
||
preCheck = "export HOME=$(mktemp -d)"; |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
pkgs/servers/sql/patroni/default.nix
Outdated
}; | ||
|
||
postPatch = '' | ||
# cdiff renamed to ydiff; remove when patroni source reflects this. |
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.
Please put the comments outside the hook.
pkgs/servers/sql/patroni/default.nix
Outdated
name = "patroni-${version}"; | ||
version = "1.6.4"; | ||
|
||
src = fetchurl { |
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.
Use fetchFromGithub
.
pkgs/servers/sql/patroni/default.nix
Outdated
{ lib, pythonPackages, fetchurl }: | ||
|
||
pythonPackages.buildPythonApplication rec { | ||
name = "patroni-${version}"; |
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.
The name
attr will be applied automatically.
name = "patroni-${version}"; | |
pname = "patroni"; |
pkgs/servers/sql/patroni/default.nix
Outdated
done | ||
''; | ||
|
||
nativeBuildInputs = with pythonPackages; [ |
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.
Not a big deal, but I'd use checkInputs
instead:
nativeBuildInputs = with pythonPackages; [ | |
checkInputs = with pythonPackages; [ |
fd55d09
to
e6c42d3
Compare
Thanks for reviewing. I amended and force-pushed. :) |
patroni: init at 1.6.4 (cherry picked from commit b6f8e00)
Motivation for this change
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)