DavidoTek / ProtonUp-Qt

Install and manage GE-Proton, Luxtorpeda & more for Steam and Wine-GE & more for Lutris with this graphical user interface.
https://davidotek.github.io/protonup-qt
GNU General Public License v3.0
1.25k stars 40 forks source link

Wine-tkg: Clarify Wine version used for Wine-tkg variants #351

Closed sonic2kk closed 7 months ago

sonic2kk commented 8 months ago

This PR changes the description of the Valve Wine and Vanilla Wine -tkg builds to note which branch they use for clarity.

I think we had some discussion about which CI to use and how to name these at the time, but I think using these names is a bit clearer. Noting the CI used for Wine Master might seem a little strange though, but I think it's an important distinction as the Ubuntu CI has different dependencies and thus builds Wine differently. The main distinction as far as I know currently being the Ubuntu CI is not built with Wine 9.0's Wayland support enabled. However, the Arch CIs are built with Wayland support.


I learned this the hard way the other night when I couldn't figure out why I couldn't get any Wine 9.0 builds to properly enable the Wayland driver (Arch's Wine package is not built with --with-wayland for fear of introducing the Wayland dependencies unnecessarily for an experimental in-development feature that is off by default).

Even if/when the Ubuntu CI is updated, it might be good to note it in the description of this tool. I think we skipped putting this in the name before for concerns of confusing users thinking that the Ubuntu CI might only be for Ubuntu, but I think putting it in brackets should be fine.

The ctmod names were also updated to better reflect their branches. In hindsight, "Valve Wine" doesn't carry the implication that it's the latest bleeding_edge, and it's not clear that this is the latest Valve Wine patch. It's equivalent to selecting Proton Experimental in your library -> Properties -> Betas, and selecting the "bleeding_edge" beta (although this means every game using Proton Experimental will use the bleeding_edge branch, which is normally undesirable). We could update the description further to note this in some way.


This PR was motivated by my own misunderstandings, and I thought that it might be good to further clarify what the Wine-tkg builds are using.

With this PR, I essentially want to find a way to make it clearer to users what the Wine-tkg builds are using, because this can be important in some cases. I don't know if I've done it in the best way in this PR, so all feedback on phrasing welcome :slightly_smiling_face:

Thanks!

DavidoTek commented 7 months ago

Makes sense. Thanks!