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

Bump bindgen version for asio-sys #577

Closed
wants to merge 1 commit into from

Conversation

toastmod
Copy link

So that #383 doesn't happen again :)

@est31
Copy link
Member

est31 commented May 22, 2021

Runs into the same CI failure as #568 . Do you have a windows computer to debug it? Also why not update to 0.58?

@toastmod
Copy link
Author

Yes I can debug it. I was able to compile it and run it. Must just be the tests to figure out.

@ottworks
Copy link

ottworks commented Jun 6, 2021

This is preventing me from using cpal!

@est31
Copy link
Member

est31 commented Jun 7, 2021

FTR it's ok if the emscripten tests fail, as the error seems unrelated. But the windows 32 bit CI failure is quite related I think. It should be fixed before this is merged.

@toastmod
Copy link
Author

toastmod commented Jun 8, 2021

Thanks, I honestly totally forgot to go back and fix this up sorry I've been doing lots of school. I'll try taking a look right now.

@toastmod
Copy link
Author

toastmod commented Jun 8, 2021

Okay sorry for the triple edit on this comment, but it looks like the problem could be that libclang is missing in the environment for i686? Not sure how these github environments work so ??? :(

Here is the error in the windows test:

  --- stderr
  thread 'main' panicked at 'Unable to find libclang: "the `libclang` shared library at C:\\msys64\\mingw64\\bin\\libclang.dll could not be opened: LoadLibraryExW failed"', C:\Users\runneradmin\.cargo\registry\src\github.com-1ecc6299db9ec823\bindgen-0.56.0\src/lib.rs:1922:31
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Error: Process completed with exit code 1.

@est31
Copy link
Member

est31 commented Jun 8, 2021

@toastmod the installation step of libclang is here:

run: choco install llvm

The CI does work for i386 windows without this PR applied, see #581.

@est31
Copy link
Member

est31 commented Jan 14, 2022

Merged #620 instead, although that one too triggered the CI failure. It would be great to have that one resolved.

@est31 est31 closed this Jan 14, 2022
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.

3 participants