ijse / freemarker.js

make freemarker available for node.js with FMPP.
http://freemarker.js.org/
MIT License
111 stars 41 forks source link

fix bug of data model contains spaces #1

Closed SimonDolph closed 9 years ago

ijse commented 9 years ago

Thanks for your pr. But it fail on Windows due to the quote problem.

SimonDolph commented 9 years ago

Sorry for that, I'm using Mac, and I don't test it on Windows. But if no quotes, it fails on my Mac.

ijse commented 9 years ago

I know that. This is okay for Linux and Mac but Windows. I have another plan and am working on it.

ijse commented 9 years ago

@SimonDolph checkout planB. https://github.com/ijse/freemarker.js/tree/planB

SimonDolph commented 9 years ago

It works after I change 1 line in index.js from

var args = [tplFile, '-C', cfgFile, '-o=' + tmpFile];

to

var args = [tplFile, '-C', cfgFile, '-o', tmpFile];

Am I on the right way?

ijse commented 9 years ago

@SimonDolph Try 35f34d669a31846596fc65eafbae828fd1d7c963

SimonDolph commented 9 years ago

It works, nice job!