Tibor17 commented on PR #527: URL: https://github.com/apache/maven-surefire/pull/527#issuecomment-1109032624
The initial problem was with tabs in CLI. This was fixed in [SUREFIRE-2063](https://issues.apache.org/jira/browse/SUREFIRE-2063) but it was not a real fix because it was not a real bug. It was not a real bug, it was just an improvement because the CLI and the platform can see the tab as collision, in JPMS to be concrete, and hard to understand for most of the common users. Even Maarten did not understand the issue immediately and he found the cause finally. So we make these treatments in order to come over the user troubles. We can argue in a loop and everybody may have a different opinion about these details, how to espace white characters, and what characters should be considered, etc. I believe that having \t and maybe new line characters is the issue which origins in the POM itself and not in this code. So, if it is this place of the root cause then it should be fixed there as well. Maybe some warning would be helpful, and maybe enough, just my thoughts. -- 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-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
