slawekjaranowski commented on PR #29: URL: https://github.com/apache/maven-executor/pull/29#issuecomment-5006003680
> Unsure why surefire change is needed? In fact, this library wants to improve isolation, so MAVEN_ARGS are picked up by forked one? > > As in _that case_ this is proper solution, same thing as here: https://github.com/apache/maven-executor/blob/main/maven-executor/src/main/java/org/apache/maven/executor/ExecutorRequest.java#L138 right, we already process commands from MAVEN_ARGS and we also try to remove it ... we also need to remove it from ProcessBuilder evn -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
