-
-
Notifications
You must be signed in to change notification settings - Fork 480
[Linux] Added option to prefer system libs #1564
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In terms of code, looks good to me. Will give this a test on a fresh VM without dependencies tomorrow
Co-authored-by: Mathis Dröge <[email protected]>
Co-authored-by: Mathis Dröge <[email protected]>
Co-authored-by: Mathis Dröge <[email protected]>
Co-authored-by: Mathis Dröge <[email protected]>
Nice thank you. What do you think about moving |
Note that since you ran |
Yeah I was wondering why you did it in |
I wanted to place it as near as possible to spawn for testing. |
You need to switch wine version once, else the lib and lib32 are not stored correctly. |
Yup, looks even better now. |
Hm, the "common" |
@CommandMC maybe we need to add wine bin folder aswell? |
Hm, I don't think that would help anything Controversial take here: Now this doesn't mean this PR is irrelevant. I'd say it doesn't hurt to prefer these shipped libraries, and maybe we can one day figure out why this error happens and add something that works around it. For now though, doing this shot-in-the-dark fixing when there's a clear solution just doesn't make sense IMO |
I thought wine-ge ships all necessary libs to run it. I don't understand why we need runtimes if wine-ge can include all of them. Is it useful to add wine as a dependency for none flatpak (deb, rmp, etc.) ? |
This is a completely uneducated guess here, but it might be the case that just having the files isn't enough. I don't really know how all of the Wine dependencies interact with the host system, but there's a similar thing where I do know how it works: GameMode. |
GE suggest to use the system libs. Strange. But we could add wine as a dependecy for deb, pacman and rpm in the package.json. This will make sure people install wine, if they wanna install heroic via this three options. |
In the summary section he suggest system libs over runtime libs: That also explains, why sometimes the steam runtime break things in heroic. |
I believe that guide is mostly unmaintained by now. Judging by the discord messages, his stance on the matter has changed. The Steam Runtime not working correctly is most likely an issue in our implementation, maybe we're not setting a variable or not considering an edge case. I don't really bother with it anymore since Wine-GE is just the better option |
@CommandMC i tested the app image of this PR on my steam deck in desktop mode. I could run WheelsOfAurelia with wine-ge. |
Before we merge this with wine as dependency for deb, pacman and rpm, i wanna know if @flavioislima has a argument against it? |
It's definitely not game-specific, the command that's failing is the Although it is good to know that it isn't failing on the Deck. Maybe I'll try to find out which exact dependency it is that makes it work later |
I tested on my PC with just setting
These libs are not in the wine lib folders. |
There's the next thing I'm wondering about: On the VM, it just says this:
Everything behaves like the file doesn't even exist, but it's there:
|
Do you have a 64 bit VM? I don't know if 32 and 64Bit makes a difference here. |
I don't think you can just make a 32-bit VM, it's up to your host CPU which architecture your VM is
|
Modern CPU's like Ryzen's support 64 Bit and 32Bit VM's. I think 64bit support exist for a long time now. |
@CommandMC have the same problem like you. Question is what we need to add as dependecy for the user? For Ubuntu i can't just install libelf. Needed to install: |
Tried this out on a fresh Ubuntu VM. Like CommandMC i had problems with the wine executable. If this lib is installed, everything works as expected and no system wine is needed. I don't know if it is possible to add this libs as dependencies for deb, pacman and rpm, because they are 32bit libs? Note:
|
As far as I know, you can depend on 32-bit packages just fine in the AUR (and in official Arch repos). A more hacky solution would be to invoke the package manager and see if the package is installed:
|
You can add every dependency to electron-builder in the package.json. This will install the package alongside heroic. But i wonder what happens if i386 is not enabled. |
LD_LIBRARY_PATH
for wine-ge, wine-lutrisLD_PRELOAD
if undefined.Use the following Checklist if you have changed something on the Backend or Frontend: