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

Update windows-rs dependency #791

Merged
merged 3 commits into from
Jul 25, 2023
Merged

Update windows-rs dependency #791

merged 3 commits into from
Jul 25, 2023

Conversation

paul-hansen
Copy link
Contributor

Just attempting to get Bevy's dependencies to all use the same windows crate version again. Lmk if you want any changes. Thanks!

@est31
Copy link
Member

est31 commented Jul 20, 2023

Hmm the choco install asio4all is broken in CI. @joachimschmidt557 do you have an idea what could be the cause?

 Downloading asio4all 
  from 'https://www.asio4all.org/downloads_11/ASIO4ALL_2_15_English.exe'
WARNING: C:\Users\runneradmin\AppData\Local\Temp\chocolatey\asio4all\2.15.0\ASIO4ALL_2_15_English.exe is of content type text/html; charset=UTF-8

Download of ASIO4ALL_2_15_English.exe (-1 B) completed.
Error - hashes do not match. Actual value was '215D115A3B5A0D3FD634BC995703216510638D910FC8670D8163EBCA20A455BB'.
ERROR: Checksum for 'C:\Users\runneradmin\AppData\Local\Temp\chocolatey\asio4all\2.15.0\ASIO4ALL_2_15_English.exe' did not meet '74dfe576c6f16e362ff2d44c313947aad9d63ed839b4d3ffac88a1320a486b90' for checksum type 'sha256'. Consider passing the actual checksums through with --checksum --checksum64 once you validate the checksums are appropriate. A less secure option is to pass --ignore-checksums if necessary.
The install of asio4all was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\asio4all\tools\chocolateyinstall.ps1'.
 See log for details.

Copy link
Member

@est31 est31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, but waiting for CI to work. Also maybe it might make sense to directly make a release from this PR.

@paul-hansen
Copy link
Contributor Author

I get a similar Installer hash does not match. error downloading it from winget too, tried multiple previous versions of asio4all as well. Not seeing a place to contact asio4all to report the issue.

@est31
Copy link
Member

est31 commented Jul 20, 2023

@paul-hansen the github account I pinged above belongs to the maintainer of the package on chocolatey, they hopefully know more.

@joachimschmidt557
Copy link

Hi, maintainer of the asio4all chocolatey package here 👋 . It seems that the download URL and checksum of ASIO4ALL has changed. I submitted an updated version of the package. It should take some days until the new package version is approved.

Thanks for notifying me of the issue with the package!

@joachimschmidt557
Copy link

The updated version of the chocolatey package has been approved and will be the default version when installed. Feel free to let me know if any further issues with the package arise in future 👍

@est31
Copy link
Member

est31 commented Jul 25, 2023

@joachimschmidt557 thanks for the quick help! Now I can merge this.

@est31 est31 merged commit 17d2cc2 into RustAudio:master Jul 25, 2023
16 checks passed
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