-
Notifications
You must be signed in to change notification settings - Fork 164
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
WIP: Linux support #191
WIP: Linux support #191
Conversation
This is janky, but does not require AppHandle.
On Linux URLs to app resources start with tauri://localhost instead of https://tauri.localhost
Also disabled install_ca_files since this causes crashes when not run as root
Rebased on |
What sorts of uncertainties do you have regarding reshade and Akebi/Acrepi/etc.? Are you unable to test them, or do you know they don't work? |
I'm not sure if they work. |
Okay, that's fine. I would feel fine merging without knowing if they work or not, I'll give a full code review when I have some free time 👍 |
Reshade does not work. Video: https://youtu.be/hYzUbkH8R14 |
I think it should be kept, no way to know if a new Wine version would just fix it or whatever. I would imagine it to be a similar situation with Akebi-likes, so we can just assume they would work the same. |
Quick note: Patching works, but the launcher attempts to unpatch the game two times.
This might not work on macOS
Windows's implementation of enable_grasscutter_watcher works just fine. systemd is pretty much standart for handling services. No "sudo" or "pkexec" is needed, systemd does that automatically, but since the user needs to authenticate, we can’t await service_status.
Kinda everts some changes of "Implement MongoDB autostart and GC watching". Only the "Capabilities" method is implemented right now. This commit makes launching MongoDB async
Remade since the previous version prevented the game from launching when offline.
It's for Reshade Injector, in case it starts working on Linux in the future.
Moved terminal detection to |
Oops. I was cleaning up branches and did not expect renaming the |
This PR adds Linux support.
Here's what currently works:
Here's what I'm unsure about: