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
pythonPackages.mrbob: fix build #37950
Conversation
''; | ||
|
||
propagatedBuildInputs = [ argparse jinja2 six ] | ||
++ (lib.optionals isPy26 [ importlib ordereddict ]); |
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.
This line can be dropped as Python 2.6 is no longer supported.
sha256 = "6737eaf98aaeae85e07ebef844ee5156df2f06a8b28d7c3dcb056f811c588121"; | ||
}; | ||
|
||
buildInputs = [ glibcLocales mock ]; |
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.
checkInputs
++ (lib.optionals isPy26 [ importlib ordereddict ]); | ||
|
||
meta = with stdenv.lib; { | ||
homepage = https://github.com/domenkozar/mr.bob.git; |
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.
Drop the .git
suffix here.
Tests broke on Hydra as the `checkPhase` wasn't configured properly. By explicitly relying on `nosetests` and injecting `LC_ALL` into the `checkPhase` the tests work again. The license (bsd3) according to `LICENSE` distributed with the upstream package wasn't specified in the meta section which could've caused legal issues. The expression has been moved into its own file to reduce the length and complexity of `pkgs/top-level/python-packages.nix`. See https://hydra.nixos.org/build/70689499/log See NixOS#36453
@dotlambda fine now? |
@Ma27 Thanks for fixing so much stuff! |
Backported in cde1f31 |
Motivation for this change
Tests broke on Hydra as the
checkPhase
wasn't configured properly. Byexplicitly relying on
nosetests
and injectingLC_ALL
into thecheckPhase
the tests work again.The license (bsd3) according to
LICENSE
distributed with the upstreampackage wasn't specified in the meta section which could've caused legal
issues.
The expression has been moved into its own file to reduce the length and
complexity of
pkgs/top-level/python-packages.nix
.See https://hydra.nixos.org/build/70689499/log
See #36453
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)