dsubelman commented on PR #523:
URL: https://github.com/apache/maven-surefire/pull/523#issuecomment-1104461256
> @dsubelman `argLine.replaceAll("\\s+", " ");` is nice. The code we have
now still cannot trust the values which have a new line in system properties,
e.g. `-DmyProp=my \n prop` but we cannot do anyting about it because we do not
have smart mechanism to understand the structure of `argLine`. Till now nobody
had any objections. The solution that we have is a kind of "too technical" but
maybe this would stay untill the people would shout too much.
When you say '_maybe this would stay until the people would shout too
much_', do you mean to merge this pull request or not?
--
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]
