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

Fix Linux support #4

Merged
merged 3 commits into from
Dec 29, 2021
Merged

Fix Linux support #4

merged 3 commits into from
Dec 29, 2021

Conversation

Menci
Copy link
Contributor

@Menci Menci commented Jun 18, 2021

The Linux support in the latest release is broken. This repo contains a old version of SFB without Linux support. Running the Linux build of the latest release v0.3.10 and clicking the song cover results in this error:

image

It's because the bundled version of SFB doesn't have Linux support.

The latest version 1.2 of SFB has Linux support but it's broken (gkngkc/UnityStandaloneFileBrowser#43). I fixed it in my fork.

Note that I have NOT tested it because I failed to to build Arcade-plus. So it shouldn't be merged right now. But I have tested it in Arcade Zero (another fork of Arcade). I'll test it ASAP after I manage to build this project. I have tested it on my Linux PC and it works.

@Menci Menci changed the title [WIP] Fix Linux support Fix Linux support Jun 18, 2021
@Menci
Copy link
Contributor Author

Menci commented Jun 19, 2021

Here is a preview video recorded on Linux:

CN (P2): https://www.bilibili.com/video/BV19o4y1y7ki/
EN: https://www.youtube.com/watch?v=ZW1f3YCYYk4&t=13s

@yojohanshinwataikei
Copy link
Owner

yojohanshinwataikei commented Jun 19, 2021

I didn't has a Mac or Linux machine so I can't test those thing. The implementation of MacOS support was provided by another contributor (see #2). I thought that also provide support for Linux but It may be wrong since I didn't really tested it.
Thanks for your PR anyway, I will take a look on it when I start working on the next release(0.4.0).

@yojohanshinwataikei
Copy link
Owner

yojohanshinwataikei commented Jun 19, 2021

@yojohanshinwataikei yojohanshinwataikei merged commit 6a125fd into yojohanshinwataikei:master Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants