search

laktek / punch

A fun and easy way to build modern websites
http://laktek.github.com/punch
MIT License
1.17k stars 107 forks source link

Copy command fails with paths that contain spaces #123

Open axelboc opened 8 years ago

axelboc commented 8 years ago

The setup command fails on Windows when dealing with paths that contain spaces. In lib/project_creator.js, line 32, the paths should be quoted:

// Current version that leads to an error:
copy_command = "ROBOCOPY " + source_path + " " + destination_path + " *.* /E";

// Adding double quotes around the two paths solves the issue:
copy_command = "ROBOCOPY \"" + source_path + "\" \"" + destination_path + "\" *.* /E";

The cp command on line 35 is probably also affected. The corresponding spec file, project_creator.spec.js, would need to be updated as well.

richard-edwards commented 8 years ago

Same issue with me, Windows 10 x64. Would love to try this out but failing on initial setup.