Skip to content
This repository has been archived by the owner on Jun 6, 2022. It is now read-only.

Implement basic Linux support #43

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Newbytee
Copy link

This pull request adds very basic Linux support. Steam library shows up if Steam is installed in the default location ($HOME/.steam/steam/) and that's about it. It's able to find Steam libraries outside of the default location (found the one I have on my second hard drive mounted at /mnt/storage/), and you can scroll through the list. Finds both Linux native games and Windows ones (that run via Proton), and everything looks correct. Search also works.

That aside however, importing games from other launchers is broken. A significant amount of work would probably be required to get this part to work (I did not attempt to implement it for the time being). Currently, opening the Import Games tab results in a JavaScript error and an infinite loading screen. Accent colour is also currently hard-coded on platforms that are not Windows. May continue to work on these things, may leave it for someone else.

Regardless, these patches should not break Windows functionality whatsoever.

@markisherwood
Copy link

Just pulled and built this on Fedora 30. Working great for me with the noted exception of Import Games functionality.

@Newbytee
Copy link
Author

Just pulled and built this on Fedora 30. Working great for me with the noted exception of Import Games functionality.

Yeah, I didn't implement that part. I might work on doing it, but I'd have to discuss it with the maintainer first as this would require some UI/UX changes due to most of these services not having native clients, and only being available through Wine. You'd have to add some sort of directory picker (or Wineprefix picker) for this to be possible on Linux since you can't just grab their paths from the registry (...which does not exist on Linux).

@parkerlreed
Copy link

Any update on this?

@Newbytee
Copy link
Author

Newbytee commented Dec 3, 2019

Any update on this?

I have not done any work since sending in this pull request.

@parkerlreed
Copy link

Ahh thanks. I may try to take a stab at it, but no promises. Not very skilled in JS/Electron, heh.

@doZennn
Copy link
Member

doZennn commented Dec 4, 2019

This will need to be reworked after the latest changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants