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

wine 2.0 #9238

Closed
wants to merge 1 commit into from
Closed

wine 2.0 #9238

wants to merge 1 commit into from

Conversation

uyjulian
Copy link
Contributor

No description provided.

# https://bugs.winehq.org/show_bug.cgi?id=34166
patch do
url "https://bugs.winehq.org/attachment.cgi?id=52485"
sha256 "59f1831a1b49c1b7a4c6e6af7e3f89f0bc60bec0bead645a615b251d37d232ac"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this patch actually be removed, or just replaced with https://bugs.winehq.org/attachment.cgi?id=55968 ?

@apetresc
Copy link
Contributor

The -devel target needs to be updated so it's not behind the new -stable. Though there isn't a devel release yet, so maybe just remove it for now.

patch do
url "https://bugs.winehq.org/attachment.cgi?id=55968"
sha256 "1b5086798ce6dc959b3cbb8f343ee236ae06c7910e4bbae7d9fde3f162f03a79"
end
Copy link
Member

Choose a reason for hiding this comment

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

If these patches aren't getting merged upstream we should strongly consider removing them. Any information on why upstream won't merge them?

@apetresc
Copy link
Contributor

@MikeMcQuaid This comment explains it in some detail. Basically, they consider it a bug in Apple's OpenGL implementation, so they view hacking around it in Wine itself as some sort of moral compromise they're not willing to make, even though it does resolve the issue.

@MikeMcQuaid
Copy link
Member

@apetresc We need to get some resolution here, we can't just carry these unmerged patches forever. I've commented in https://bugs.winehq.org/show_bug.cgi?id=14939 and https://bugs.winehq.org/show_bug.cgi?id=34166 and let's hold off on this merge until there's more news.

@xrisk
Copy link
Contributor

xrisk commented Feb 1, 2017

Hi, any updates on this? Wine 2.0 patch is kinda important...

@apetresc
Copy link
Contributor

apetresc commented Feb 1, 2017

@xrisk: It looks like Mike is still going back-and-forth with the Wine developers in those two tickets, with responses as recently as this morning. So, it's still being figured out. As expected, they're resisting merging the patches upstream (for some seemingly good reasons, too).

@MikeMcQuaid MikeMcQuaid mentioned this pull request Feb 8, 2017
@MikeMcQuaid
Copy link
Member

@uyjulian Can you rebase this on master? Thanks!

@MikeMcQuaid
Copy link
Member

Thanks for your contribution to Homebrew (and patience)! Without people like you submitting PRs we couldn't run this project. You rock!

@alexchandel
Copy link
Contributor

alexchandel commented Feb 11, 2017

Just a heads up, it looks like there are newer versions of both patches than the ones we currently apply.

Also, the patch for https://bugs.winehq.org/show_bug.cgi?id=14939 was purportedly committed, so we may need to stop applying it to HEAD and devel builds soon.

As for https://bugs.winehq.org/show_bug.cgi?id=34166, it looks like the maintainer doesn't want to fix what he views as a macOS bug (although it may not be). Apple has engineers, and it's beyond me why he doesn't describe the bug to them at bugreport.apple.com, but we should test using the more recent patch in the meantime.

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants