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
postgresql: enable pre-install checks #61395
postgresql: enable pre-install checks #61395
Conversation
You should set checkTarget instead i think. |
33efe9a
to
3c33718
Compare
Well whaddya know... Done. |
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 change the base of your change to staging
. This is mass rebuilding change according to the automatic labels.
3c33718
to
d7f36ec
Compare
Cool - just wanted to start off with something that's quick for people to build & verify (now I know I can switch a PR's target...) |
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.
LGTM.
Motivation for this change
Postgres' pre-install checks are quite minimal but could catch basic problems early. For some reason the make target detection thinks
make check
doesn't exist, so I need to set it explicitly.Tests pass reliably for me on non-nixos linux x86_64 & macos 10.13, from postgres 11 all the way back to 9.4.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)