[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

Re: [VOTE] Release Apache Maven Invoker Plugin version 3.0.1 / Apache Maven Script Interpreter 1.2

2017-07-15 Thread Olivier Lamy
Hi Sounds to be a legitimate complain :-). My goal was to have something to be sure running after tests wether were failing or not. (my use case stop a running Jetty instance). But doesn't work for this case. Ok I cancel the vote and will fix that! Thanks! Olivier On 15 July 2017 at 20:59,

Re: [VOTE] Release Apache Maven Invoker Plugin version 3.0.1 / Apache Maven Script Interpreter 1.2

2017-07-15 Thread Guillaume Boué
Hi, I have a question about the post-build invocation change in MINVOKER-223. With 3.0.0, the post-build script is executed only once, after all Maven invocations (defined for example with invoker.goals.1, invoker.goals.2, etc.) are performed successfully. Now, with the fix made in