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
wine: update to 8.13. #45202
wine: update to 8.13. #45202
Conversation
Works fine for me on x86_64-glibc. |
cc @Hoshpak |
It fails to build on x86_64-musl: This needs to be solved before the package can be updated. |
@Hoshpak Reading the logs I suspect this is a case of a missing include. An undefined macro next to a variable declaration with a forward-declared struct. I will look into it. |
After some digging I found the commit that broke musl support
the problem with it is at the following code:
previously I will consider reverting the commit to upstream wine. |
Update: The pull request was approved, wine on musl is back! Probably will be included with the next wine version, 8.14 @Hoshpak Until then, do we apply the patch to the package and ship it, or do we wait until wine does? |
Thanks for the great work! Feel free to include the patch and let's get this update merged. |
also includes a patch to fix compatibility issues with musl.
@Hoshpak Tested it on musl, works as expected, feel free to merge. Gotta remember to remove the patch when they merge upstream! |
Testing the changes
Local build testing