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
rocksdb: package binary tools #38998
Conversation
@@ -1,4 +1,4 @@ | |||
{ stdenv | |||
{ 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.
Use stdenv.lib
instead.
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.
@yegortimoshenko I always write lib
as a formal argument in my expressions; have we made a formal decision somewhere to prefer stdenv.lib
over a separate lib
argument?
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.
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.
I'd wager that stdenv.lib
vastly outnumbers passing lib
separately (for package expressions). I'm unaware of any arguments beyond uniformity.
Whether it makes sense for stdenv
to provide lib
is debatable I suppose, but it does & is widely relied upon, so to me it makes more sense to just stick with convention for convention's sake.
@GrahamcOfBorg build rocksdb |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: rocksdb Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: rocksdb Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: rocksdb Partial log (click to expand)
|
Motivation for this change
rocksdb includes some binaries that can be useful for debugging (sst_dump, ldb, rocksdb_dump, rocksdb_undump, blob_dump).
Add a bin output and install them there.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)