-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Use Ruby included with Sonic Pi for start-server #18
Comments
This is great, thanks Chris. Would you like to send over a PR with as far as you've got and then we can see if we can resolve the Rust tricky-ness. You've done work here, so it'd be nice for you to get a contribution in the repo :) Probably only need to prepend the sandboxed ruby path to the command and ignore the shebang? |
Great project, thank you for your work! Leaving this note in case anyone runs into a similar issue as I. I experienced something similar to what is described in this Issue and I resolved it by upgrading my Ruby system install to 2.5.x given how the Sonic Pi dependencies seem to resolve (it appears 2.4.x will work too). There wasn't support for my default macOS Mojave Ruby install of 2.3.7 or the most recent stable Ruby version (2.6.3 as of the time of this writing). |
Sonic Pi's
server/bin/sonic-pi-server.rb
has a shebang line for#!/usr/bin/env ruby
. When runningsonic-pi-tool start-server
from the command line, it uses whateverruby
it finds in yourPATH
. If it's not 2.3 (the version bundled with Sonic Pi), it fails:It's looking for files in
Sonic Pi.app/app/server/rb-native/osx/2.4.0
instead ofSonic Pi.app/app/server/rb-native/osx/2.3.0
.It would be super cool if the
start-server
command, after determining the correct path to the Sonic Pi installation, started the server using the Ruby included in the package (Sonic Pi.app/app/server/native/osx/ruby/bin
).Again, as mentioned in #17, I'd gladly implement this but I don't have the Rust knowledge to make it happen :(
The text was updated successfully, but these errors were encountered: