Giters

bdraco / chacha20poly1305-reuseable

A reusable version of ChaCha20Poly1305 for use with cryptography and asyncio

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Fails to load DLL on windows

postlund opened this issue · comments

commented

Describe the bug
My pyatv GitHub actions jobs does not seem to work on Windows anymore. I get this error:

...
pyatv\support\chacha20.py:2: in <module>

    from chacha20poly1305_reuseable import ChaCha20Poly1305Reusable as ChaCha20Poly1305

E   ImportError: DLL load failed while importing chacha20poly1305_reuseable: %1 is not a valid Win32 application.

To Reproduce

In my case I just import the library on a Windows machine (windows-latest on GitHub in this case).
Additional context

https://github.com/postlund/pyatv/actions/runs/4728869785/jobs/8395410792?pr=1956

commented

It looks like ciwheelbuild has some fixes for windows builds https://github.com/pypa/cibuildwheel/releases

I'll bump the version and rebuild

commented

Opened #10 to test on windows and use the 2019 images + new ciwheelbuild

Lets see how it goes

commented

https://github.com/bdraco/chacha20poly1305-reuseable/actions/runs/4826157736

Its rebuilding now with the newer version + the base image downgraded to windows 2019

commented

Feel free to reach out on discard if there is still an issue. I am likely to miss any updates here

commented

windows-2019 has trouble with utf-8
so
one more attempt

https://github.com/bdraco/chacha20poly1305-reuseable/actions/runs/4826325094

commented

Looks like 3rd time was a charm. Wheels should be up on pypi in the next 30m

commented

I do see the same issue still, anything I need to do?

commented

Is it possible you have a 32 bit windows?

commented

We are building both 32bit and 64bit wheels.

Maybe the wrong one is getting installed somehow

commented 
13 wheels produced in 10 minutes:
  chacha20poly1305_reuseable-0.2.5-cp310-cp310-win32.whl            66 kB
  chacha20poly1305_reuseable-0.2.5-cp310-cp310-win_amd64.whl        66 kB
  chacha20poly1305_reuseable-0.2.5-cp311-cp311-win32.whl            66 kB
  chacha20poly1305_reuseable-0.2.5-cp311-cp311-win_amd64.whl        66 kB
  chacha20poly1305_reuseable-0.2.5-cp37-cp37m-win32.whl             66 kB
  chacha20poly1305_reuseable-0.2.5-cp37-cp37m-win_amd64.whl         66 kB
  chacha20poly1305_reuseable-0.2.5-cp38-cp38-win32.whl              66 kB
  chacha20poly1305_reuseable-0.2.5-cp38-cp38-win_amd64.whl          66 kB
  chacha20poly1305_reuseable-0.2.5-cp39-cp39-win32.whl              66 kB
  chacha20poly1305_reuseable-0.2.5-cp39-cp39-win_amd64.whl          66 kB
  chacha20poly1305_reuseable-0.2.5-pp37-pypy37_pp73-win_amd64.whl   66 kB
  chacha20poly1305_reuseable-0.2.5-pp38-pypy38_pp73-win_amd64.whl   66 kB
  chacha20poly1305_reuseable-0.2.5-pp39-pypy39_pp73-win_amd64.whl   66 kB
commented

I don’t think it’s a problem with the binary though since we are using pypa ciwheelbuild and there isn’t anything outlandish.

I looked through all the issues on ciwheelbuild and didn’t see anything related.

I don’t have a windows machine to test this locally so I’m at a loss on how to proceed

commented

Yeah, I'm not sure either. I did try it on my work laptop (Windows 10) and I didn't see the problem there. So I suspect it's something fishy with the GitHub runners.