Problem with paths includeing whitespace

[aslakjo]

Hi

We are using cuke4nuke, which is using systemu. Today we moved to a new server. On this new server our teamcity installation is placed under program files. This fact seemed to make some trouble for systemu. And the rest of the CI(contious integration)-server.

After some debugging i found that this is the parameter to IO.popen in the systemu method:
"C:/Program Files/TeamCity/buildAgent/temp/buildTmp/systemu_OSLHK-I050_0_4452_0.0249346645890749_1/program"

Which is the source of the following stack trace.
Error:
C:/Ruby187/bin/ruby.exe: No such file or directory -- C:/Program (LoadError)C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:85: wtf? (RuntimeError)
from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:72:in popen' from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:72 from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:160:inquietly'
from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:70
from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:224:in call' from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:224:intmpdir'
from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:209:in loop' from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:209:intmpdir'
from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:63:in systemu' from C:/Ruby187/lib/ruby/gems/1.8/gems/systemu-1.2.0/lib/systemu.rb:10:insystemu'
from C:/Ruby187/lib/ruby/gems/1.8/gems/cuke4nuke-aslakjo-0.3.5/bin/../lib/cuke4nuke/main.rb:53:in launch_cucumber' from C:/Ruby187/lib/ruby/gems/1.8/gems/cuke4nuke-aslakjo-0.3.5/bin/../lib/cuke4nuke/main.rb:25:inrun'
from C:/Ruby187/lib/ruby/gems/1.8/gems/cuke4nuke-aslakjo-0.3.5/bin/cuke4nuke:6
from C:/Ruby187/bin/cuke4nuke:19:in `load' from C:/Ruby187/bin/cuke4nuke:19

I fixed this by altering:
IO.popen "#{ @ruby } #{ c['program'] }", 'r+' do |pipe|
To:
IO.popen "#{ @ruby } "#{ c['program'] }"", 'r+' do |pipe|

This solved our issue.

[aslakjo]

I have commited the patch on my fork : http://github.com/aslakjo/systemu/commit/aaac375e3daba66869e88175e089368136da0d6a

But since I am in a windows only environment at the moment :( I am not able to test this patch on all platforms. Pull and use the patch if it meets your requirements.

[ahoward]

see if the latest commit fixes your problem please. i cannot test on windoze.

[aslakjo]

Thanks for the quick reply!

The patch looks good. I'll see if it works I practise tomorrow.