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
mydumper: init at 0.9.3 #32385
mydumper: init at 0.9.3 #32385
Conversation
sha256 = "139v6707sxyslg7n1fii8b1ybdq50hbqhc8zf6p1cr3h2hhl6ns9"; | ||
}; | ||
|
||
buildInputs = [ cmake glib mariadb.client.dev libressl pcre pkgconfig zlib ]; |
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.
cmake
and pkgconfig
should be under nativeBuildInputs
instead
|
||
buildInputs = [ cmake glib mariadb.client.dev libressl pcre pkgconfig zlib ]; | ||
|
||
meta = with stdenv.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.
Are you willing to maintain this package?
If so please add yourself as a maintainer in https://github.com/NixOS/nixpkgs/blob/master/lib/maintainers.nix
Personally I'm very reluctant to merge new packages without a maintainer.
src = fetchFromGitHub { | ||
owner = "maxbube"; | ||
repo = "mydumper"; | ||
rev = "eef4d8291803e9ba57b4e58458cec3f4f7f0b203"; |
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 to rev = "v${version}";
.
I have rebased and squashed your commits (usually a new package would be one commit, any fixup commits should be squashed). Thank you! |
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)