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
cockroachdb: 1.1.5 -> 2.0.0 #38662
cockroachdb: 1.1.5 -> 2.0.0 #38662
Conversation
This still needs a little work: the Go version pin in all-packages.nix needs to be removed, and on darwin it requires libedit (this is an undocumented dependency because we didn't contemplate darwin build environments that don't include things that are present in the default xcode install) I'm working on an alternative PR that also refactors it to use stdenv.mkDerivation, since building cockroachdb from a source tarball is more like a standard tarball build that happens to depend on go than a typical go package. |
To be fair, I changed the minimum possible to update the package - I would consider any changes to how the package is built not the scope of my PR. |
@bdarnell I noticed in your PR you did not change |
Yeah, I was looking at an old version of all-packages.nix that still had the Go pin. Sorry for the confusion. |
@GrahamcOfBorg build cockroachdb |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: cockroachdb Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: cockroachdb Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: cockroachdb Partial log (click to expand)
|
Motivation for this change
Lots of shiny new features: https://www.cockroachlabs.com/docs/releases/v2.0.0.html#core-features
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)One of the dependencies failed to build without
-ltinfo
, so I've addedncurses6
to thebuildInputs
.