--define string parsing error about uglifyjs

[mishoo]

The defines are evaluated, so it complains that a is not defined. Evaluating arguments is probably not a good idea.

Any case, global defines should be constant. What would you actually expect from your sample, something like console.log(a,b)?

[mal]

Not quite, I was hoping for console.log("a,b");

I use this as I have a dictionary in a text file which is easier to maintain using line breaks, so in my cake build script I read it, convert the newlines to commas, and use DICTIONARY.split(',') in my main code. For now I've worked around it by changing my delimiter, however it'd still be good to solve this problem when defining strings.

[mishoo]

Ah, I see. That almost works:

echo 'console.log(TEST)' | uglifyjs2 -cd TEST='"a;b"'
→ console.log("a;b")

There's a bug currently if you include a comma, but it's just the command-line parser, nothing deep. I'll fix that.

Also note that you need to double the quotes (the first level is dropped by your shell).

[mal]

Yup, that's the work around I came to too! Thanks!