Skip to content
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.imageio: 1.6 -> 2.4.1 #49147

Merged
merged 1 commit into from Nov 4, 2018

Conversation

pmiddend
Copy link
Contributor

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 when doCheck is false.

I've tested the library itself.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

@pmiddend pmiddend requested a review from FRidh as a code owner October 26, 2018 11:47
@pmiddend
Copy link
Contributor Author

@GrahamcOfBorg build python36Packages.imageio

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1/lib  /nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1/bin
patching script interpreter paths in /nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1
checking for references to /build in /nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1...
wrapping `/nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1/bin/imageio_remove_bin'...
wrapping `/nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1/bin/imageio_download_bin'...
pytestcachePhase
/nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1/lib  /nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1/bin
patching script interpreter paths in /nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1
checking for references to /build in /nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1...
wrapping `/nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1/bin/imageio_download_bin'...
wrapping `/nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1/bin/imageio_remove_bin'...
pytestcachePhase
/nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

/nix/store/sfnlxf8bk5m15pzd1qhy5zvhzcd0wbk2-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

/nix/store/jlqlmj58m55vm9c8jpmr7zjjq7dasm1f-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1/lib  /nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1/bin
patching script interpreter paths in /nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1
checking for references to /build in /nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1...
wrapping `/nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1/bin/imageio_download_bin'...
wrapping `/nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1/bin/imageio_remove_bin'...
pytestcachePhase
/nix/store/1kb1ihsbddffnrzqi6fg1ar1f6ix5rg6-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1/lib  /nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1/bin
patching script interpreter paths in /nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1
checking for references to /build in /nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1...
wrapping `/nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1/bin/imageio_remove_bin'...
wrapping `/nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1/bin/imageio_download_bin'...
pytestcachePhase
/nix/store/gyykd3cznrflddljqyqhkgglaaxxh10b-python3.6-imageio-2.4.1

@dotlambda
Copy link
Member

# Tries to write in /var/tmp/.imageio
doCheck = false;

Maybe we can patch the tests to make them run? Or maybe it's sufficient to set some environment variable like $TMP.

@pmiddend
Copy link
Contributor Author

@dotlambda It's a good point. I found out that setting IMAGEIO_USERDIR="$TMP" solves the problem, but then the tests begin downloading images from the web. Now, one might be able to download the images in the nix derivation, but...that seems a bit overkill, in my opinion.

@pmiddend
Copy link
Contributor Author

I'm conversing with the author, let's keep the PR open until that's cleared up.

@pmiddend
Copy link
Contributor Author

Okay, I've updated the build so the tests work. This can be merged now.

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/rbm37a199q06x786xvjh0kp2lid0vjcm-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 37.30 seconds =========
pytestcachePhase
/nix/store/rbm37a199q06x786xvjh0kp2lid0vjcm-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/98kbqyhh0f1wrg48lkras7fpncvfkjaw-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 41.71 seconds =========
pytestcachePhase
/nix/store/98kbqyhh0f1wrg48lkras7fpncvfkjaw-python3.6-imageio-2.4.1

@pmiddend
Copy link
Contributor Author

pmiddend commented Nov 2, 2018

@dotlambda I removed the patch and replaced with with a postPatch phase.

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 37.29 seconds =========
pytestcachePhase
/nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 25.96 seconds =========
pytestcachePhase
/nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1

@pmiddend
Copy link
Contributor Author

pmiddend commented Nov 3, 2018

@dotlambda Fixed the doCheck

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

/nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 37.31 seconds =========
pytestcachePhase
/nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1

@dotlambda
Copy link
Member

@GrahamcOfBorg build python2.pkgs.imageio python3.pkgs.imageio

@GrahamcOfBorg
Copy link

Failure on x86_64-darwin (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/077iha76h2x7xxyzw2anm2fcyvbgikpk-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 19.98 seconds =========
pytestcachePhase
error: build of '/nix/store/kq1rx2miv93b8x3n1d2fiachj7rxbcp3-python2.7-imageio-2.4.1.drv' failed

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 37.25 seconds =========
pytestcachePhase
error: build of '/nix/store/wl2y9g8vv65c2mkp0cdkyhq7qq4k4bhx-python2.7-imageio-2.4.1.drv' failed

@GrahamcOfBorg
Copy link

Failure on x86_64-linux (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)

    warnings.warn("%s%s" % (e, warn))

tests/test_pillow.py::test_png
  /nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 26.34 seconds =========
pytestcachePhase
error: build of '/nix/store/97mni974n1qa17y0rvznnv33wwn9pakx-python2.7-imageio-2.4.1.drv' failed

@dotlambda dotlambda changed the title python36Packages.imageio: 1.6 -> 2.4.1 pythonPackages.imageio: 1.6 -> 2.4.1 Nov 4, 2018
@pmiddend
Copy link
Contributor Author

pmiddend commented Nov 4, 2018

@dotlambda I fixed the failures with Python 2. Two more dependencies were needed.

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

/nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python36Packages.imageio

Partial log (click to expand)

/nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1

@dotlambda
Copy link
Member

@GrahamcOfBorg build python2.pkgs.imageio python3.pkgs.imageio

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)


tests/test_pillow.py::test_png
  /nix/store/hgv4vl4l2m81d4wx1pw7jcrmfjx0h3jx-python2.7-imageio-2.4.1/lib/python2.7/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 35.68 seconds =========
pytestcachePhase
/nix/store/hgv4vl4l2m81d4wx1pw7jcrmfjx0h3jx-python2.7-imageio-2.4.1
/nix/store/5w6p7s9xvrd8akk3z40bbqwasfhqamv2-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-darwin (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)


tests/test_pillow.py::test_png
  /nix/store/077iha76h2x7xxyzw2anm2fcyvbgikpk-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 25.71 seconds =========
pytestcachePhase
/nix/store/q41fa6vb5kmsi7sn7kjnsmq142nf513y-python2.7-imageio-2.4.1
/nix/store/077iha76h2x7xxyzw2anm2fcyvbgikpk-python3.6-imageio-2.4.1

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: python2.pkgs.imageio, python3.pkgs.imageio

Partial log (click to expand)


tests/test_pillow.py::test_png
  /nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1/lib/python3.6/site-packages/imageio/plugins/pillow.py:317: UserWarning: PIL PNG writer cannot produce interlaced images.
    warn("PIL PNG writer cannot produce interlaced images.")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 56 passed, 67 skipped, 3 xfailed, 5 warnings in 41.79 seconds =========
pytestcachePhase
/nix/store/mf4m78grgdf9ciz751i3x4h3giv8g4b7-python2.7-imageio-2.4.1
/nix/store/m019mqrfs0xhbrrr5dpscm32gdk2fcdf-python3.6-imageio-2.4.1

@dotlambda dotlambda merged commit ef9b17b into NixOS:master Nov 4, 2018
@pmiddend pmiddend deleted the imageio-1.6_2.4.1 branch November 4, 2018 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants