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
salt: 2017.7.2 -> 2017.7.4, fix msgpack dependency #35891
salt: 2017.7.2 -> 2017.7.4, fix msgpack dependency #35891
Conversation
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
pkgs/tools/admin/salt/default.nix
Outdated
@@ -8,19 +8,19 @@ | |||
|
|||
python2Packages.buildPythonApplication rec { | |||
pname = "salt"; | |||
version = "2017.7.2"; | |||
version = "2017.7.3"; | |||
name = "${pname}-${version}"; |
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 remove name
, it is not needed for buildPython*
There is a 2017.7.4 version: https://pypi.org/project/salt/ |
Success on aarch64-linux (full log) Partial log (click to expand)
|
9e90c9b
to
e797e00
Compare
Success on x86_64-darwin (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Thanks for the review @dotlambda, updated the PR. (I did build locally but still won't be able to test for a while.) |
@aneeshusa Did you test yet? |
salt is still broken on master due to saltstack/salt#45790 |
Motivation for this change
Fix Salt build. Bump Salt to the latest release.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)It looks like this package was missed in #34809; the latest (2017.7.3) still seems to require
msgpack-python
.cc @danbst and @PierreR as Salt users: I won't be able to test this thoroughly for a bit, could you give this branch a spin and report how it goes?