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

Please get the 9.0 Proton tree (and bleeding-edge) working #1175

Open
KyunLFA opened this issue May 15, 2024 · 5 comments
Open

Please get the 9.0 Proton tree (and bleeding-edge) working #1175

KyunLFA opened this issue May 15, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@KyunLFA
Copy link

KyunLFA commented May 15, 2024

Currently on my end (CachyOS), building with _proton_branch set to anything higher than the experimental_8.0 tree in proton-tkg (and vrclient also overrides to an even older commit) with proton flavor set to Wine master produces an obscene amount of errors I could not fix try as I may, unrelated to wine itself as it builds just fine, but instead occurring in the Proton bits like vrclient and lsteamclient (that aren't addressed by silencing the implicit declaration and incompatible pointer errors).

At this point I'm nudging towards the idea that it would be easier to just use the official proton building script (but somehow beating the requirement for containers) because so much has changed since 8.0-experimental.

Please consider fixing proton-tkg with 9.0 and bleeding-edge branches if you find that they are also broken for you.

Thanks!

@Tk-Glitch
Copy link
Member

That tree you mention is only used on our side for the steam-specific tooling (lsteamclient, vrclient and steam_helper) and indeed it's totally broken past a certain commit and not compatible with our build system. The wine part doesn't depend on it and isn't defined by that variable. Other external tooling is handled through their own respective trees, also not depending on those steam-specific libs/tools.
We are overriding the tooling tree commit to f5e9c76903e4e18e0416e719a6d42d0cb00998aa with https://github.com/Frogging-Family/wine-tkg-git/blob/master/proton-tkg/proton-tkg.sh#L968 as any newer commit depends on that different approach we do not support. However unless you nuked that part of the script it should use that commit and work just fine (as it does for me and the CI).
If you haven't fiddled with it, the error might be unrelated and just a dependency issue (in that case please attach a log or a copy of those errors you get).

@KyunLFA
Copy link
Author

KyunLFA commented May 16, 2024

Yes I have overridden to use the newer proton tree (deleted the override that vrclient does) to see if it would work, and it's severely broken when doing that.

My intention was to ask for newer Proton tree support when you find the time to add it.

Thanks for answering

@Tk-Glitch
Copy link
Member

There's actually no vrclient specific target commit, we're using what's in the tree on f5e9c769, while using openvr master branch (which is newer than what's used in bleeding edge currently).
So, new question from me. What's the reason for you to want/need that newer tooling? VR I assume (that would be the only point to it really)?

Anyway it's being worked on but I have no ETA yet.

@KyunLFA
Copy link
Author

KyunLFA commented May 17, 2024

I guess for same reason we all update, to be on the bleeding edge and the perks it gives. I would personally prefer if the whole Proton tree could be used on the latest (release) commit, as minor as that may be, so that it matches what's currently available with the official Proton 9.0. Otherwise it gives me the impression of proton-tkg falling slightly "behind" in that aspect (mostly the lsteamclient and steam_helper updates). Sorry if this seems stupid/weird to you.

No biggie with the ETA subject, I just wanted to know if it's being developed or not, no rushing intended.

Thanks!

@Tk-Glitch
Copy link
Member

Right now it's not effectively offering more "perks" and is more of an integration change, which lead us to this fallback situation that was deemed acceptable as a buffer. The steam.exe and lsteamclient triggers are happening on game launch basically and we're done. No expected user-side change.
There are other plans regarding future proton handling, too, which I'll hopefully get to a satisfying spot soon and will offer more options. I'll update this ticket when we have proper handling of the new compilation process for the steam bridge and friends.

@Tk-Glitch Tk-Glitch added the enhancement New feature or request label May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants