Escaping $ in *nix platforms

[lekkas]

I noticed that $ is not escaped as it probably should in Linux/OS X. For instance, this:

Rsync = require('rsync')
var rsync = new Rsync()
  .flags('az')
  .source('/source')
  .destination('server:/destination$test')
  .command()

will yield

"rsync -az /source server:/destination$test"

So when the rsync command is run, $test will be interpreted as an environment variable.

[mattijs]

I wanted to keep this as a feature but I'm starting to think that it doesn't make sense. Would you be willing to open a PR for this?

[lekkas]

Sure.