Space in Ruby Path causes ml.bat to fail

[rjkennedy98]

After doing Roxy upgrade to 1.7.3 master the ml.bat script fails in windows.

C:\EDH_ML_Path4_3\eie_edh_framework\EDH_ML>C:\Program Files (x86)\Ruby21\bin\rub
y.exe -Ideploy -Ideploy\lib deploy\lib\ml.rb
'C:\Program' is not recognized as an internal or external command,
operable program or batch file.

[dmcassel]

I don't have a Windows setup handy. @rjkennedy98 could you run a quick test? In ml.bat, line 12, add quotes around the value for RUBYFOUND, so that it looks like this:

set RUBYFOUND="%%~$PATH:X"

If that works, we can update the script. If not, I invite input from someone with more Windows scripting fu than I have.

[rjkennedy98]

That gave me:
C:\EDH_ML_Path4_3\eie_edh_framework\EDH_ML>ml local deploy modules
'""' is not recognized as an internal or external command,

I have made a change ml.bat, line 158
"%RUBYFOUND%" -Ideploy -Ideploy\lib -Ideploy\test deploy\test\test_main.rb

and ml.bat, line 167
"%RUBYFOUND%" -Ideploy -Ideploy\lib deploy\lib\ml.rb %*

that seemed to resolve issue

[rlouapre]

I am not sure we should have provided a fix for it...
Screen from Ruby installer:

It clearly says "please avoid any folder name that contains spaces"

[dmcassel]

@rlouapre interesting find. While I agree with you -- the instructions are pretty clear -- it's a pretty simple fix for Roxy. Might as well.

[rjkennedy98]

The client I am working with has managed VMs which have a space in the ruby install path. Otherwise, yes, it could be easily avoided by following directions.

[grtjn]

Fixed in dev..