-
Notifications
You must be signed in to change notification settings - Fork 10
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
Causes LoadError when executing on Windows #36
Comments
Sorry I forgot to tell OS version. It was Windows 7 (x64) |
I have thesame problem. How to corrent it? |
Thanks for the reports. I don't have a windows install readily available, I'll happily merge any pull requests that get it working on all platforms.
|
FFIW i went in the compiled .js and changed both """ (escaped ) to "" (empty strings). "ruby instarepl_compiled.js":925 |
It looks like proc/exec ultimately hands things over to node's ChildProcess.spawn. That handles putting quotes around arguments with spaces for us, so we probably don't need the escape-spaces function at all. I fixed this by editing .\src\lt\plugins\ruby.cljs (in lighttable so it would recompile the js file for me) and updating the args var in the run-rb function as follows: original: modified: |
Hey Windows folks-- I got around to pushing a new version that implements @lmtthws 's suggestion. Let me know how it works. |
When executing Ruby-Instarepl on Windows, it causes "Invalid argument" error as below
ruby: Invalid argument -- "C:/Users/yutaka/AppData/Local/LightTable/plugins/Ruby
_Instarepl/rb-src/lt_client.rb" (LoadError)
After investigating, I found that lt-ruby invoked ruby with following argument, and it causes the error above.
ruby ""C:\Users\yutaka\AppData\Local\LightTable\plugins\Ruby_Instarepl\rb-src\lt_client.rb"" 63336 985
It looks like lt_client.rb is over-quoted and function escape-spaces needs to be revised...
Versions:
Light Table 0.6.7
ruby 2.0.0p481 (2014-05-08) [x64-mingw32]
Ruby-instarepl 0.0.15
The text was updated successfully, but these errors were encountered: