Skip to content

update binaries #21

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

Closed
wants to merge 1 commit into from
Closed

update binaries #21

wants to merge 1 commit into from

Conversation

bicycle1885
Copy link
Member

No description provided.

@bicycle1885
Copy link
Member Author

Windows... 😭

@visr
Copy link

visr commented Jul 8, 2018

FWIW, this works for me on Win64 0.7. Maybe restart the build?

Julia Version 0.7.0-beta.108
Commit 3ac99765de (2018-07-01 22:57 UTC)
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i7-4710MQ CPU @ 2.50GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.0 (ORCJIT, haswell)
(v0.7) pkg> test CodecZlib
   Testing CodecZlib
 Resolving package versions...
Test Summary: | Pass  Total
Gzip Codec    |  538    538
Test Summary: | Pass  Total
Zlib Codec    |  529    529
Test Summary: | Pass  Total
Deflate Codec |  501    501
┌ Warning: `mapfoldl(f, op, v0, itr)` is deprecated, use `mapfoldl(f, op, itr; init=v0)` instead.
│   caller = test_chunked_read(::Type, ::Type) at testtools.jl:86
└ @ TranscodingStreams testtools.jl:86
Test Summary:      | Pass  Total
TranscodingStreams | 1028   1028
   Testing CodecZlib tests passed

visr added a commit to JuliaGeo/GDAL.jl that referenced this pull request Jul 8, 2018
For zlib this does not use the modified build.jl in the CodecZlib repo, but the original bicycle1885/ZlibBuilder version.
We don't support the other platforms yet anyway.

If we run into issues like JuliaIO/CodecZlib.jl#21, perhaps we should use an older build.
@bicycle1885
Copy link
Member Author

I tried it again but still see the same error. This may be due to lack of memory on AppVeyor (https://stackoverflow.com/a/23444925/1243761).

@visr
Copy link

visr commented Jul 8, 2018

Oh that is odd, I don't see anything large being processed in the tests...

visr added a commit to JuliaGeo/GDAL.jl that referenced this pull request Jul 8, 2018
For zlib this does not use the modified build.jl in the CodecZlib repo, but the original bicycle1885/ZlibBuilder version.
We don't support the other platforms yet anyway.

If we run into issues like JuliaIO/CodecZlib.jl#21, perhaps we should use an older build.
@bicycle1885
Copy link
Member Author

Superseded by #30.

@bicycle1885 bicycle1885 deleted the update-zlib-binaries branch August 13, 2018 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants