[PR] [IT] Refactor executor [maven]

["cstamas (via GitHub)" <gi...@apache.org>]

cstamas opened a new pull request, #2175:
URL: https://github.com/apache/maven/pull/2175

   There was overlapping concerns, so better to separate them. Also, this now allows to use tool with other maven that the one being tested, so result should not be affected.
   
   Also contains fix for ExecutorTool implementation to enforce that process do produce output, as under some circumstances (on Windows) the process may exit with 0 exit code (no error) but produce empty output
   on standard output, which should not be possible.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [IT] Refactor executor [maven]

["cstamas (via GitHub)" <gi...@apache.org>]

cstamas merged PR #2175:
URL: https://github.com/apache/maven/pull/2175


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org