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

Add Arch Linux support (and some auto code reformatting) #22

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

Conversation

mathgeniuszach
Copy link

@mathgeniuszach mathgeniuszach commented Oct 20, 2022

This pull request fundamentally changes how paths are created to make it possible to work with Arch Linux out of the box without any configuration. On Linux, it's not guaranteed that all the sonic-pi content will be in a single app folder, so I separated paths to make it slightly more flexible to configure where certain files are, as absolute or relative paths.

I've only tested this with Arch with the community/sonic-pi package installed. If someone with a debian based machine can figure out where the sonic-pi bin folder is (it has the sonic-pi-server.rb file in it), where the theme folder is (it has a file called app.qss in it), and where the samples folder is (filled with .flac files), then feel free to mention it on issue #5 and I'll add it in or feel free to pull request it yourself.

I also did some formatting changes and made it so the sonic pi server will stop when you close/reload vscode.

@llloret
Copy link
Owner

llloret commented Oct 20, 2022

Hey, @XMgzx, thank you for the PR. As you say, let's see if someone can check on another distro. Sadly, at the moment, I do not have the bandwidth to test it myself :(

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