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
Add pgweb, a light and fast web-based PostgreSQL database browser #104597
Conversation
My first real PR to nixpkgs so be gentle :). I'm not sure how to do the checklist items from the PR template, nor which even apply to such a PR. |
Would be nice to add a Welcome on board 👋 |
Add pgweb, a light and fast web-based PostgreSQL database browser
@@ -0,0 +1,26 @@ | |||
{ buildGoPackage, fetchFromGitHub, lib }: |
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.
A question: is lib
required here? If yes, is maintainers
required too? If not, why not?
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.
lib is exposed as a top-level "package" so injection will work for it, maintainers, platforms and so on are nested fields, so they can't be injected by callPackage.
So yes you inject lib, and then you use it to build the meta field (and potentially to call other functions)
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.
So it's good like it is now?
Motivation for this change
Pgweb is a light & fast browser-based PostgreSQL browser that I have been using for years. Recently I moved from homebrew to darwin-nixos and added the code from this PR into my
configuration.nix
file. Then came the realization that I could/should upstream it! So here we are.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)