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
pipenv: 2018.5.18 -> 2018.7.1 #43823
Conversation
}; | ||
|
||
propagatedBuildInputs = [ six attrs arpeggio ]; | ||
buildInputs = [ pytest hypothesis pretend ]; |
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
for test dependencies
a2e5cf8
to
f6e57be
Compare
@FRidh should this go to staging? |
pname = "hypothesis"; | ||
|
||
# Upstream prefers github tarballs | ||
src = fetchFromGitHub { |
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 should remain fetchFromGitHub
because upstream doesn't include tests in their tarball
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.
They have completely reworked their testing infrastructure (there' no more setup.py even) so I haven't bothered reverse engineering it.
}; | ||
|
||
checkInputs = [ pytest pytest_xdist flaky mock ]; | ||
propagatedBuildInputs = [ attrs coverage ] ++ lib.optional (!isPy3k) [ enum34 ]; | ||
|
||
inherit doCheck; | ||
|
||
# https://github.com/DRMacIver/hypothesis/issues/300 | ||
checkPhase = '' |
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 likely should be kept as well, unless they've changed their tox.ini
to reduce the test scope.
Yep, staging as well. |
Success on x86_64-darwin (full log) Attempted: pipenv Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: pipenv Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pipenv Partial log (click to expand)
|
Seems they redid their repo layout. It is now in a subfolder, so you could use |
f6e57be
to
01ddbff
Compare
@FRidh switched to staging-next and fixed hypothesis to use sourceRoot subfolder. |
@domenkozar this goes via |
(cherry picked from commit 01ddbffce6b371741c471db6e5dc71cb6addc4a8) Signed-off-by: Domen Kožar <domen@dev.si>
01ddbff
to
e06ecf6
Compare
@FRidh done |
Failure on x86_64-darwin (full log) Attempted: pipenv Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: pipenv Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pipenv Partial log (click to expand)
|
Included in #43972 |
Things done
Checked that pipenv works.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)