ascheman commented on PR #11983: URL: https://github.com/apache/maven/pull/11983#issuecomment-4341552409
> I see that the code was introduced in #2213 are you clear that the new approach does not break the intent in #2213 (I took time to search when the code was done, I have absolute no clue: that's why I'm asking) I was sure that my change didn't break the fix from #2213 wrt. the still running integration tests for [MNG-4559](https://issues.apache.org/jira/browse/MNG-4559). But I took the chance to introduce an IT for my own change. At least it proves that there is no shell replacement of injected '${...}' any longer. Testing correct replacement in a nested `mvn` call as surefire does it is way harder to prove with the given IT test capabilities of Maven core. However, I tested that with surefire on apache/maven-surefire#3345. -- 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]
