-
Notifications
You must be signed in to change notification settings - Fork 32
runtime: Update to 48 #484
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
Conversation
|
Started test build 191899 |
|
@bbhtt libgphotos-lib32 with the 47 runtime did build, did something change in 24.08? |
I also had to add libusb for some reason. |
|
Build 191899 failed |
|
Started test build 191903 |
|
Build 191903 successful |
|
What's the error? |
|
|
Started test build 191964 |
|
Build 191964 successful |
|
Breaks Steam Deck input again. |
|
Thanks for working on this! As 46 is EOL, could we first update to 47 while we figure out the issues with 48? |
|
No, both 47 and 48 use the fdo 24.08 runtime which is where the issue starts. I have no idea what the issue is to be honest, the steam and bottles flatpak do not have this issue. I think it is very likely the issue is somewhere in the flatpak manifest and not in the runtime itself or the app itself. |
|
Perhaps additional permission are required to solve this issue, kinda like Heroic Games Launcher approached it? |
|
I do not see any permission related to input in the PR. Do you have anything specific in mind? |
No, not necessarily. It's kind of a long shot, but the permissions are the only change I see to this PR that seems to have issues with Steam Input, which I'd assume might be what's breaking the Steam Deck controller. According to the comment in the PR, it seems that they copied the permissions (at least in some part) from the Steam Flatpak, which also doesn't have issues with the 24.04 runtime. Sadly, a lot of ifs and buts, but I can't really help more than that since I don't own a Steam Deck. |
|
The good thing is that all of the permissions in the linked PR can be tested in the build generated by this PR by adding those flags to the |
Maybe you can copy-paste the permissions and then @pizzadude can retest? Doing it via CLI might be cumbersome on the Deck. It would be an easy revert anyway. |
I tested the latest Heroic flatpak on the deck, and it has the same issue for older wine/proton versions (8.x and below), however, Proton 9.x and GE-Proton 9.x work fine. (didn't test 10.x yet). Maybe going forward, if we can't find a solution to this issue (why does Bottles work?), since Proton 9.x versions work, maybe we could ignore this issue and just tell people to use Proton 9.x and above. Though we would have to test if that's the case on Lutris also. |
|
I locally built the Flatpak from this branch, installed it on the deck, and tested various configurations. Proton 9 and above and Wine 10 seem to work fine. Wine GE 8 does not work in game mode.
|
Can you test if you have this issue also? Open-Wine-Components/umu-launcher#470 |
|
I just randomly stumbled upon this PR and noticed that it adds libusb. I don't think this is necessary because org.winehq.Wine already has libusb in its manifest and Lutris utilizes it as a base app. libusb is also going to be included in runtime 25.08. |
|
Could we merge this? the only regression is on a incredibly old version of proton, of which not even on steam os I had working controls for the deck. |
|
I am marking as not-a-draft, but someone should mention what exactly has regressed and whether this should be merged as is. |
|
✅ Test build succeeded. To test this build, install it from the testing repository: |
|
the Heroic Launcher which faced the same limitations/issues has merged and released their 23.08->25.08 update today I can't say whether they implemented workarounds or accepted the breakage with fairly old runners. |
|
This fixes the file picker for me, and brings a more recent NVK version with significantly better performances. It may just be due to my fancy exotic setup, but I would rather have "a controller bug on a runner you shouldn't use" than "a bug on the 'select executable' button and be forced to install the nvidia proprietary drivers" I don't know if anyone is having the same issues, I just want to point out that the EOL Mesa and Gnome runtimes are likely to cause at least as much pain as the controller-on-old-runner situation |
|
bot, build |
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
✅ Test build succeeded. To test this build, install it from the testing repository: |
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
✅ Test build succeeded. To test this build, install it from the testing repository: |
|
bot, build |
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
❌ Test build failed. Help
|
|
bot, build |
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
Does not compile with the 48 runtime
This reverts commit 20977c0.
|
🚧 Test build enqueued. |
|
❌ Test build was cancelled. Help
|
|
🚧 Started test build. |
|
✅ Test build succeeded. To test this build, install it from the testing repository: Built for x86_64 architecture. |
Marked as WIP as libgphotos-lib32 is not compiling.