-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
pythonPackages.imageio: 1.6 -> 2.4.1 #49147
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
Conversation
@GrahamcOfBorg build python36Packages.imageio |
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
bb2ded7
to
7816838
Compare
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Maybe we can patch the tests to make them run? Or maybe it's sufficient to set some environment variable like |
@dotlambda It's a good point. I found out that setting |
I'm conversing with the author, let's keep the PR open until that's cleared up. |
7816838
to
07a4beb
Compare
Okay, I've updated the build so the tests work. This can be merged now. |
07a4beb
to
dc320e9
Compare
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
dc320e9
to
319d801
Compare
@dotlambda I removed the patch and replaced with with a |
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
319d801
to
d77ca36
Compare
@dotlambda Fixed the |
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
@GrahamcOfBorg build python2.pkgs.imageio python3.pkgs.imageio |
Failure on x86_64-darwin (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
d77ca36
to
48daa12
Compare
@dotlambda I fixed the failures with Python 2. Two more dependencies were needed. |
Success on aarch64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python36Packages.imageio Partial log (click to expand)
|
@GrahamcOfBorg build python2.pkgs.imageio python3.pkgs.imageio |
Success on aarch64-linux (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2.pkgs.imageio, python3.pkgs.imageio Partial log (click to expand)
|
Motivation for this change
A normal update. I tried to keep the nix expression like it was before, although I'm not sure why there's a
checkPhase
and even check dependencies whendoCheck
is false.I've tested the library itself.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)