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

XAudio2 Wine DLLs #11

Merged
merged 40 commits into from May 17, 2018
Merged

XAudio2 Wine DLLs #11

merged 40 commits into from May 17, 2018

Conversation

JohanSmet
Copy link
Contributor

@flibitijibibo
Copy link
Member

Finally got to try this out in depth and everything works incredibly well! This is really really good work, thanks for taking this on.

The only two things I noticed are based on one common thing: SDL2.dll doesn't get copied into the output folder, so regsvr32 fails silently because of the classic "module could not be found" message, which is Windows trying to tell us SDL2 is what's actually missing...

It may be worth documenting that in the README for those who intend to take the DLLs out of the VC output folder and ship them somehow, and for RegisterDLLs maybe we can add a line to copy from the SDL2-devel folder to do that part automatically for Windows. Not sure how much it matters for the average Wine hacker but if we can do that there too, could be helpful!

Once that's fixed that'll be it, I think. I could probably dig super deep into style things but you've spent enough time on this and I can always take a personal day to do silly things like that.

@JohanSmet
Copy link
Contributor Author

Ah, that's a good remark. I had SDL2.dll in the %PATH% somewhere and didn't think about it anymore.

The VC RegisterDLLs project and the mingw Makefile both copy SDL2 to the output directory now. And the wine setup script also links it into the wine-prefix.

The README has been extended with a small note about redistributing the DLLs.

@flibitijibibo
Copy link
Member

Tried it out, works great, another job well done!

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.

None yet

2 participants