[GitHub] maven issue #128: [MNG-6256] Surround variables in echo command with double ...

2017-07-17 Thread etzelc
Github user etzelc commented on the issue: https://github.com/apache/maven/pull/128 Now in `mvn.cmd` the string parameter of the echo command is surrounded with double quotes, like in `mvn`. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] maven issue #128: [MNG-6256] Surround variables in echo command with double ...

2017-07-15 Thread etzelc
Github user etzelc commented on the issue: https://github.com/apache/maven/pull/128 I will have a closer look next week, right now only tested with Windows (since only cmd script was modified). In the {{mvn}} script the string of the {{echo}} is already completely in double quotes,

[GitHub] maven issue #128: [MNG-6256] Surround variables in echo command with double ...

2017-07-15 Thread Tunaki
Github user Tunaki commented on the issue: https://github.com/apache/maven/pull/128 Shouldn't there be the same fix for `mvn`, and not just `mvn.cmd`? [`${basedir}`](https://github.com/apache/maven/blob/707cff6ffdecbfb0e0356c70e3a7b490535e7ce4/apache-maven/src/bin/mvn#L150) and